[PULL,37/88] esp.c: introduce esp_get_phase() function

Message ID	20240213194052.1162753-38-mark.cave-ayland@ilande.co.uk (mailing list archive)
State	New, archived
Headers	show Return-Path: <qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org> From: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk> To: peter.maydell@linaro.org, qemu-devel@nongnu.org Date: Tue, 13 Feb 2024 19:40:01 +0000 Message-Id: <20240213194052.1162753-38-mark.cave-ayland@ilande.co.uk> In-Reply-To: <20240213194052.1162753-1-mark.cave-ayland@ilande.co.uk> References: <20240213194052.1162753-1-mark.cave-ayland@ilande.co.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [PULL 37/88] esp.c: introduce esp_get_phase() function Received-SPF: pass client-ip=2001:41c9:1:41f::167; envelope-from=mark.cave-ayland@ilande.co.uk; helo=mail.ilande.co.uk X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action Precedence: list Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org
Series	[PULL,01/88] esp.c: don't clear cmdfifo when esp_select() fails in get_cmd() \| expand [PULL,01/88] esp.c: don't clear cmdfifo when esp_select() fails in get_cmd() [PULL,02/88] esp.c: move existing request cancel check into esp_select() [PULL,03/88] esp.c: add FIFO wraparound support to esp_fifo_pop_buf() [PULL,04/88] esp.c: remove FIFO clear from esp_select() [PULL,05/88] esp.c: move esp_select() to ESP selection commands from get_cmd() [PULL,06/88] esp.c: update esp_set_tc() to set STAT_TC flag [PULL,07/88] esp.c: start removal of manual STAT_TC setting when transfer counter reaches zero [PULL,08/88] esp.c: move command execution logic to new esp_run_cmd() function [PULL,09/88] esp.c: update TC check logic in do_dma_pdma_cb() to check for TC == 0 [PULL,10/88] esp.c: move buffer and TC logic into separate to/from device paths in esp_do_dma() [PULL,11/88] esp.c: remove unused case from esp_pdma_read() [PULL,12/88] esp.c: don't accumulate directly into cmdfifo [PULL,13/88] esp.c: decrement the TC during MESSAGE OUT and COMMAND phases [PULL,14/88] esp.c: introduce esp_set_phase() helper function [PULL,15/88] esp.c: remove another set of manual STAT_TC updates [PULL,16/88] esp.c: remove MacOS TI workaround that pads FIFO transfers to ESP_FIFO_SZ [PULL,17/88] esp.c: don't reset the TC and ESP_RSEQ state when executing a SCSI command [PULL,18/88] esp.c: don't clear RFLAGS register when DMA is complete [PULL,19/88] esp.c: remove zero transfer size check from esp_do_dma() [PULL,20/88] esp.c: update condition for esp_dma_done() in esp_do_dma() from device path [PULL,21/88] esp.c: update condition for esp_dma_done() in esp_do_dma() to device path [PULL,22/88] esp.c: ensure that the PDMA callback is called for every device read [PULL,23/88] esp.c: don't immediately raise INTR_BS if SCSI data needed in esp_do_dma() [PULL,24/88] esp.c: remove TC adjustment in esp_do_dma() from device path [PULL,25/88] esp.c: remove unaligned adjustment in do_dma_pdma_cb() to device path [PULL,26/88] esp.c: remove unneeded if() check in esp_transfer_data() [PULL,27/88] esp.c: update end of transfer logic at the end of esp_transfer_data() [PULL,28/88] esp.c: consolidate async_len and TC == 0 checks in do_dma_pdma_cb() and esp_do_dma() [PULL,29/88] esp.c: fix premature end of phase logic esp_command_complete [PULL,30/88] esp.c: move TC and FIFO check logic into esp_dma_done() [PULL,31/88] esp.c: rename esp_dma_done() to esp_dma_ti_check() [PULL,32/88] esp.c: copy PDMA logic for transfers to device from do_dma_pdma_cb() to esp_do_dma() [PULL,33/88] esp.c: copy logic for do_cmd transfers from do_dma_pdma_cb() to esp_do_dma() [PULL,34/88] esp.c: update esp_do_dma() bypass if async_len is zero to include non-zero transfer ch… [PULL,35/88] esp.c: move end of SCSI transfer check after TC adjustment in do_dma_pdma_cb() [PULL,36/88] esp.c: remove s_without_satn_pdma_cb() PDMA callback [PULL,37/88] esp.c: introduce esp_get_phase() function [PULL,38/88] esp.c: convert esp_do_dma() to switch statement based upon SCSI phase [PULL,39/88] esp.c: convert do_dma_pdma_db() to switch statement based upon SCSI phase [PULL,40/88] esp.c: convert esp_do_nodma() to switch statement based upon SCSI phase [PULL,41/88] esp.c: convert esp_do_dma() do_cmd path to check for SCSI phase instead [PULL,42/88] esp.c: convert do_dma_pdma_cb() do_cmd path to check for SCSI phase instead [PULL,43/88] esp.c: convert esp_do_nodma() do_cmd path to check for SCSI phase instead [PULL,44/88] esp.c: convert esp_reg_write() do_cmd path to check for SCSI phase instead [PULL,45/88] esp.c: remove do_cmd from ESPState [PULL,46/88] esp.c: untangle MESSAGE OUT and COMMAND phase logic in esp_do_dma() [PULL,47/88] esp.c: untangle MESSAGE OUT and COMMAND phase logic in do_dma_pdma_cb() [PULL,48/88] esp.c: untangle MESSAGE OUT and COMMAND phase logic in esp_do_nodma() [PULL,49/88] esp.c: move CMD_SELATN end of message phase detection to esp_do_dma() and do_dma_pdma_… [PULL,50/88] esp.c: move CMD_TI end of message phase detection to esp_do_dma() and do_dma_pdma_cb() [PULL,51/88] esp.c: don't use get_cmd() for CMD_SEL DMA commands [PULL,52/88] esp.c: move CMD_SELATNS end of command logic to esp_do_dma() and do_dma_pdma_cb() [PULL,53/88] esp.c: replace do_dma_pdma_cb() with esp_do_dma() [PULL,54/88] esp.c: move CMD_ICCS command logic to esp_do_dma() [PULL,55/88] esp.c: always use esp_do_dma() in pdma_cb() [PULL,56/88] esp.c: remove unused PDMA callback implementation [PULL,57/88] esp.c: rename data_in_ready to to data_ready [PULL,58/88] esp.c: separate logic based upon ESP command in esp_command_complete() [PULL,59/88] esp.c: separate logic based upon ESP command in esp_transfer_data() [PULL,60/88] esp.c: use deferred interrupts for both DATA IN and DATA OUT phases [PULL,61/88] esp.c: remove DATA IN phase logic when reading from FIFO [PULL,62/88] esp.c: zero command register when TI command terminates due to phase change [PULL,63/88] esp.c: remove unneeded ti_cmd field [PULL,64/88] esp.c: don't raise INTR_BS interrupt in DATA IN phase until TI command issued [PULL,65/88] esp.c: move non-DMA TI logic to separate esp_nodma_ti_dataout() function [PULL,66/88] esp.c: process non-DMA FIFO writes in esp_do_nodma() [PULL,67/88] esp.c: replace get_cmd() with esp_do_nodma() [PULL,68/88] esp.c: move write_response() non-DMA logic to esp_do_nodma() [PULL,69/88] esp.c: consolidate end of command sequence after ICCS command [PULL,70/88] esp.c: ensure that STAT_INT is cleared when reading ESP_RINTR [PULL,71/88] esp.c: don't clear the SCSI phase when reading ESP_RINTR [PULL,72/88] esp.c: handle TC underflow for DMA SCSI requests [PULL,73/88] esp.c: remove restriction on FIFO read access when DMA memory routines defined [PULL,74/88] esp.c: handle non-DMA FIFO writes used to terminate DMA commands [PULL,75/88] esp.c: improve ESP_RSEQ logic consolidation [PULL,76/88] esp.c: only transfer non-DMA COMMAND phase data for specific commands [PULL,77/88] esp.c: only transfer non-DMA MESSAGE OUT phase data for specific commands [PULL,78/88] esp.c: consolidate DMA and PDMA logic in DATA OUT phase [PULL,79/88] esp.c: consolidate DMA and PDMA logic in DATA IN phase [PULL,80/88] esp.c: consolidate DMA and PDMA logic in MESSAGE OUT phase [PULL,81/88] esp.c: remove redundant n variable in PDMA COMMAND phase [PULL,82/88] esp.c: consolidate DMA and PDMA logic in STATUS and MESSAGE IN phases [PULL,83/88] esp.c: replace n variable with len in esp_do_nodma() [PULL,84/88] esp.c: implement DMA Transfer Pad command for DATA phases [PULL,85/88] esp.c: rename irq_data IRQ to drq_irq [PULL,86/88] esp.c: keep track of the DRQ state during DMA [PULL,87/88] esp.c: switch TypeInfo registration to use DEFINE_TYPES() macro [PULL,88/88] esp.c: add my copyright to the file

Message ID

20240213194052.1162753-38-mark.cave-ayland@ilande.co.uk (mailing list archive)

State

New, archived

Headers

From: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
To: peter.maydell@linaro.org,
	qemu-devel@nongnu.org
Date: Tue, 13 Feb 2024 19:40:01 +0000
Message-Id: <20240213194052.1162753-38-mark.cave-ayland@ilande.co.uk>
In-Reply-To: <20240213194052.1162753-1-mark.cave-ayland@ilande.co.uk>
References: <20240213194052.1162753-1-mark.cave-ayland@ilande.co.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Subject: [PULL 37/88] esp.c: introduce esp_get_phase() function
Received-SPF: pass client-ip=2001:41c9:1:41f::167;
 envelope-from=mark.cave-ayland@ilande.co.uk; helo=mail.ilande.co.uk
X-Spam_score_int: -20
X-Spam_score: -2.1
X-Spam_bar: --
X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,
 DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_NONE=0.001,
 SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no
X-Spam_action: no action
X-BeenThere: qemu-devel@nongnu.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
 <mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <https://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
 <mailto:qemu-devel-request@nongnu.org?subject=subscribe>
Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org
Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org

Series

[PULL,01/88] esp.c: don't clear cmdfifo when esp_select() fails in get_cmd() | expand

Commit Message

Mark Cave-Ayland Feb. 13, 2024, 7:40 p.m. UTC

Make use of this new function in all places where the SCSI phase bits are
manually masked from the ESP_RSTAT register.

Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Tested-by: Helge Deller <deller@gmx.de>
Tested-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20240112125420.514425-38-mark.cave-ayland@ilande.co.uk>
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
---
 hw/scsi/esp.c | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

diff --git a/hw/scsi/esp.c b/hw/scsi/esp.c
index 16cb6c72fd..de8d98082a 100644
--- a/hw/scsi/esp.c
+++ b/hw/scsi/esp.c
@@ -196,6 +196,11 @@  static void esp_set_phase(ESPState *s, uint8_t phase)
     trace_esp_set_phase(esp_phase_names[phase]);
 }
 
+static uint8_t esp_get_phase(ESPState *s)
+{
+    return s->rregs[ESP_RSTAT] & 7;
+}
+
 static uint8_t esp_pdma_read(ESPState *s)
 {
     uint8_t val;
@@ -537,7 +542,7 @@  static void esp_dma_ti_check(ESPState *s)
 
 static void do_dma_pdma_cb(ESPState *s)
 {
-    int to_device = ((s->rregs[ESP_RSTAT] & 7) == STAT_DO);
+    int to_device = (esp_get_phase(s) == STAT_DO);
     uint8_t buf[ESP_CMDFIFO_SZ];
     int len;
     uint32_t n;
@@ -554,7 +559,7 @@  static void do_dma_pdma_cb(ESPState *s)
         }
 
         s->ti_size = 0;
-        if ((s->rregs[ESP_RSTAT] & 7) == STAT_CD) {
+        if (esp_get_phase(s) == STAT_CD) {
             /* No command received */
             if (s->cmdfifo_cdb_offset == fifo8_num_used(&s->cmdfifo)) {
                 return;
@@ -621,7 +626,7 @@  static void do_dma_pdma_cb(ESPState *s)
 static void esp_do_dma(ESPState *s)
 {
     uint32_t len, cmdlen;
-    int to_device = ((s->rregs[ESP_RSTAT] & 7) == STAT_DO);
+    int to_device = (esp_get_phase(s) == STAT_DO);
     uint8_t buf[ESP_CMDFIFO_SZ];
     int n;
 
@@ -654,7 +659,7 @@  static void esp_do_dma(ESPState *s)
         }
         trace_esp_handle_ti_cmd(cmdlen);
         s->ti_size = 0;
-        if ((s->rregs[ESP_RSTAT] & 7) == STAT_CD) {
+        if (esp_get_phase(s) == STAT_CD) {
             /* No command received */
             if (s->cmdfifo_cdb_offset == fifo8_num_used(&s->cmdfifo)) {
                 return;
@@ -762,7 +767,7 @@  static void esp_do_dma(ESPState *s)
 
 static void esp_do_nodma(ESPState *s)
 {
-    int to_device = ((s->rregs[ESP_RSTAT] & 7) == STAT_DO);
+    int to_device = (esp_get_phase(s) == STAT_DO);
     uint8_t buf[ESP_FIFO_SZ];
     uint32_t cmdlen;
     int len, n;
@@ -776,7 +781,7 @@  static void esp_do_nodma(ESPState *s)
         cmdlen = fifo8_num_used(&s->cmdfifo);
         trace_esp_handle_ti_cmd(cmdlen);
         s->ti_size = 0;
-        if ((s->rregs[ESP_RSTAT] & 7) == STAT_CD) {
+        if (esp_get_phase(s) == STAT_CD) {
             /* No command received */
             if (s->cmdfifo_cdb_offset == fifo8_num_used(&s->cmdfifo)) {
                 return;
@@ -856,7 +861,7 @@  static void esp_pdma_cb(ESPState *s)
 void esp_command_complete(SCSIRequest *req, size_t resid)
 {
     ESPState *s = req->hba_private;
-    int to_device = ((s->rregs[ESP_RSTAT] & 7) == STAT_DO);
+    int to_device = (esp_get_phase(s) == STAT_DO);
 
     trace_esp_command_complete();
 
@@ -909,7 +914,7 @@  void esp_command_complete(SCSIRequest *req, size_t resid)
 void esp_transfer_data(SCSIRequest *req, uint32_t len)
 {
     ESPState *s = req->hba_private;
-    int to_device = ((s->rregs[ESP_RSTAT] & 7) == STAT_DO);
+    int to_device = (esp_get_phase(s) == STAT_DO);
     uint32_t dmalen = esp_get_tc(s);
 
     assert(!s->do_cmd);
@@ -1103,7 +1108,7 @@  uint64_t esp_reg_read(ESPState *s, uint32_t saddr)
             qemu_log_mask(LOG_UNIMP, "esp: PIO data read not implemented\n");
             s->rregs[ESP_FIFO] = 0;
         } else {
-            if ((s->rregs[ESP_RSTAT] & 0x7) == STAT_DI) {
+            if (esp_get_phase(s) == STAT_DI) {
                 if (s->ti_size) {
                     esp_do_nodma(s);
                 } else {

[PULL,37/88] esp.c: introduce esp_get_phase() function

Commit Message

Patch