Thread (11 messages) 11 messages, 2 authors, 2021-09-01

Re: [PATCH 3/4] scsi: pm80xx: Corrected Inbound and Outbound queue logging

From: Jinpu Wang <jinpu.wang@ionos.com>
Date: 2021-08-30 08:24:01

On Mon, Aug 23, 2021 at 9:28 AM Ajish Koshy [off-list ref] wrote:
From: Viswas G <redacted>

Corrected inbound queue and outbound queue size in 'ib_log'
and 'ob_log' sysfs entries.

Signed-off-by: Viswas G <redacted>
Signed-off-by: Ajish Koshy <redacted>
Acked-by: Jack Wang <jinpu.wang@ionos.com>
Thanks
quoted hunk ↗ jump to hunk
---
 drivers/scsi/pm8001/pm8001_ctl.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/pm8001/pm8001_ctl.c b/drivers/scsi/pm8001/pm8001_ctl.c
index ec05c42e8ee6..b25e447aa3bd 100644
--- a/drivers/scsi/pm8001/pm8001_ctl.c
+++ b/drivers/scsi/pm8001/pm8001_ctl.c
@@ -409,6 +409,7 @@ static ssize_t pm8001_ctl_ib_queue_log_show(struct device *cdev,
        char *str = buf;
        int start = 0;
        u32 ib_offset = pm8001_ha->ib_offset;
+       u32 queue_size = pm8001_ha->max_q_num * PM8001_MPI_QUEUE * 128;
 #define IB_MEMMAP(c)   \
                (*(u32 *)((u8 *)pm8001_ha->     \
                memoryMap.region[ib_offset].virt_ptr +  \
@@ -419,7 +420,7 @@ static ssize_t pm8001_ctl_ib_queue_log_show(struct device *cdev,
                start = start + 4;
        }
        pm8001_ha->evtlog_ib_offset += SYSFS_OFFSET;
-       if (((pm8001_ha->evtlog_ib_offset) % (PM80XX_IB_OB_QUEUE_SIZE)) == 0)
+       if (((pm8001_ha->evtlog_ib_offset) % queue_size) == 0)
                pm8001_ha->evtlog_ib_offset = 0;

        return str - buf;
@@ -445,6 +446,7 @@ static ssize_t pm8001_ctl_ob_queue_log_show(struct device *cdev,
        char *str = buf;
        int start = 0;
        u32 ob_offset = pm8001_ha->ob_offset;
+       u32 queue_size = pm8001_ha->max_q_num * PM8001_MPI_QUEUE * 128;
 #define OB_MEMMAP(c)   \
                (*(u32 *)((u8 *)pm8001_ha->     \
                memoryMap.region[ob_offset].virt_ptr +  \
@@ -455,7 +457,7 @@ static ssize_t pm8001_ctl_ob_queue_log_show(struct device *cdev,
                start = start + 4;
        }
        pm8001_ha->evtlog_ob_offset += SYSFS_OFFSET;
-       if (((pm8001_ha->evtlog_ob_offset) % (PM80XX_IB_OB_QUEUE_SIZE)) == 0)
+       if (((pm8001_ha->evtlog_ob_offset) % queue_size) == 0)
                pm8001_ha->evtlog_ob_offset = 0;

        return str - buf;
--
2.27.0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help