| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20250922152609.827311-1-cyan@cyano.uk>
Date: Mon, 22 Sep 2025 23:26:08 +0800
From: Xinhui Yang <cyan@...no.uk>
To: linux-scsi@...r.kernel.org
Cc: stable@...r.kernel.org,
Mingcong Bai <jeffbai@...c.io>,
Kexy Biscuit <kexybiscuit@...c.io>,
Xinhui Yang <cyan@...no.uk>,
Oliver Neukum <oliver@...kum.org>,
Ali Akcaagac <aliakc@....de>,
Jamie Lenehan <lenehan@...bble.org>,
"James E.J. Bottomley" <James.Bottomley@...senPartnership.com>,
"Martin K. Petersen" <martin.petersen@...cle.com>,
linux-kernel@...r.kernel.org (open list)
Subject: [PATCH v2] scsi: dc395x: correctly discard the return value in certain reads
There are certain read operations performed in this code which doesn't
really don't need its return value. Those read operations either clears
the FIFO buffer, or clears the interruption status. However, unused read
triggers compiler warnings. With CONFIG_WERROR on, these warnings get
converted into errors:
drivers/scsi/dc395x.c: In function ‘__dc395x_eh_bus_reset’:
drivers/scsi/dc395x.c:97:49: error: value computed is not used [-Werror=unused-value]
97 | #define DC395x_read8(acb,address) (u8)(inb(acb->io_port_base + (address)))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/scsi/dc395x.c:1003:9: note: in expansion of macro ‘DC395x_read8’
1003 | DC395x_read8(acb, TRM_S1040_SCSI_INTSTATUS);
| ^~~~~~~~~~~~
drivers/scsi/dc395x.c: In function ‘data_io_transfer’:
drivers/scsi/dc395x.c:97:49: error: value computed is not used [-Werror=unused-value]
97 | #define DC395x_read8(acb,address) (u8)(inb(acb->io_port_base + (address)))
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/scsi/dc395x.c:2032:33: note: in expansion of macro ‘DC395x_read8’
2032 | DC395x_read8(acb, TRM_S1040_SCSI_FIFO);
Create a new macro DC395x_peek8() to deliberately cast the return value
to void, which tells the compiler we really don't need the return value
of such read operations.
Other changes:
* Also slightly modify the formatting of the DC395x_* macros.
Cc: stable@...r.kernel.org
Signed-off-by: Xinhui Yang <cyan@...no.uk>
---
Changes since v1 [1]:
- Add Cc: tag to include this patch to the stable tree.
- Add additional description about the formatting changes.
[1]: https://lore.kernel.org/linux-scsi/20250922143619.824129-1-cyan@cyano.uk
---
drivers/scsi/dc395x.c | 34 ++++++++++++++++++++--------------
1 file changed, 20 insertions(+), 14 deletions(-)
diff --git a/drivers/scsi/dc395x.c b/drivers/scsi/dc395x.c
index 386c8359e1cc..013d0d5277d6 100644
--- a/drivers/scsi/dc395x.c
+++ b/drivers/scsi/dc395x.c
@@ -91,15 +91,21 @@
#endif
-#define DC395x_LOCK_IO(dev,flags) spin_lock_irqsave(((struct Scsi_Host *)dev)->host_lock, flags)
-#define DC395x_UNLOCK_IO(dev,flags) spin_unlock_irqrestore(((struct Scsi_Host *)dev)->host_lock, flags)
+#define DC395x_LOCK_IO(dev, flags) spin_lock_irqsave(((struct Scsi_Host *)dev)->host_lock, flags)
+#define DC395x_UNLOCK_IO(dev, flags) spin_unlock_irqrestore(((struct Scsi_Host *)dev)->host_lock, flags)
-#define DC395x_read8(acb,address) (u8)(inb(acb->io_port_base + (address)))
-#define DC395x_read16(acb,address) (u16)(inw(acb->io_port_base + (address)))
-#define DC395x_read32(acb,address) (u32)(inl(acb->io_port_base + (address)))
-#define DC395x_write8(acb,address,value) outb((value), acb->io_port_base + (address))
-#define DC395x_write16(acb,address,value) outw((value), acb->io_port_base + (address))
-#define DC395x_write32(acb,address,value) outl((value), acb->io_port_base + (address))
+/*
+ * read operations that may trigger side effects in the hardware,
+ * but the value can or should be discarded.
+ */
+#define DC395x_peek8(acb, address) ((void)(inb(acb->io_port_base + (address))))
+/* Normal read write operations goes here. */
+#define DC395x_read8(acb, address) ((u8) (inb(acb->io_port_base + (address))))
+#define DC395x_read16(acb, address) ((u16) (inw(acb->io_port_base + (address))))
+#define DC395x_read32(acb, address) ((u32) (inl(acb->io_port_base + (address))))
+#define DC395x_write8(acb, address, value) (outb((value), acb->io_port_base + (address)))
+#define DC395x_write16(acb, address, value) (outw((value), acb->io_port_base + (address)))
+#define DC395x_write32(acb, address, value) (outl((value), acb->io_port_base + (address)))
#define TAG_NONE 255
@@ -1000,7 +1006,7 @@ static int __dc395x_eh_bus_reset(struct scsi_cmnd *cmd)
DC395x_write8(acb, TRM_S1040_DMA_CONTROL, CLRXFIFO);
clear_fifo(acb, "eh_bus_reset");
/* Delete pending IRQ */
- DC395x_read8(acb, TRM_S1040_SCSI_INTSTATUS);
+ DC395x_peek8(acb, TRM_S1040_SCSI_INTSTATUS);
set_basic_config(acb);
reset_dev_param(acb);
@@ -2029,8 +2035,8 @@ static void data_io_transfer(struct AdapterCtlBlk *acb,
DC395x_write8(acb, TRM_S1040_SCSI_CONFIG2,
CFG2_WIDEFIFO);
if (io_dir & DMACMD_DIR) {
- DC395x_read8(acb, TRM_S1040_SCSI_FIFO);
- DC395x_read8(acb, TRM_S1040_SCSI_FIFO);
+ DC395x_peek8(acb, TRM_S1040_SCSI_FIFO);
+ DC395x_peek8(acb, TRM_S1040_SCSI_FIFO);
} else {
/* Danger, Robinson: If you find KGs
* scattered over the wide disk, the driver
@@ -2044,7 +2050,7 @@ static void data_io_transfer(struct AdapterCtlBlk *acb,
/* Danger, Robinson: If you find a collection of Ks on your disk
* something broke :-( */
if (io_dir & DMACMD_DIR)
- DC395x_read8(acb, TRM_S1040_SCSI_FIFO);
+ DC395x_peek8(acb, TRM_S1040_SCSI_FIFO);
else
DC395x_write8(acb, TRM_S1040_SCSI_FIFO, 'K');
}
@@ -2892,7 +2898,7 @@ static void set_basic_config(struct AdapterCtlBlk *acb)
DMA_FIFO_HALF_HALF | DMA_ENHANCE /*| DMA_MEM_MULTI_READ */ ;
DC395x_write16(acb, TRM_S1040_DMA_CONFIG, wval);
/* Clear pending interrupt status */
- DC395x_read8(acb, TRM_S1040_SCSI_INTSTATUS);
+ DC395x_peek8(acb, TRM_S1040_SCSI_INTSTATUS);
/* Enable SCSI interrupt */
DC395x_write8(acb, TRM_S1040_SCSI_INTEN, 0x7F);
DC395x_write8(acb, TRM_S1040_DMA_INTEN, EN_SCSIINTR | EN_DMAXFERERROR
@@ -3799,7 +3805,7 @@ static void adapter_uninit_chip(struct AdapterCtlBlk *acb)
reset_scsi_bus(acb);
/* clear any pending interrupt state */
- DC395x_read8(acb, TRM_S1040_SCSI_INTSTATUS);
+ DC395x_peek8(acb, TRM_S1040_SCSI_INTSTATUS);
}
--
2.51.0
Powered by blists - more mailing lists