* [PATCH v2 0/2] scsi: qedi: Couple of warning fixes
@ 2021-11-26 20:17 Florian Fainelli
2021-11-26 20:17 ` [PATCH v2 1/2] scsi: qedi: Remove set but unused 'page' variable Florian Fainelli
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Florian Fainelli @ 2021-11-26 20:17 UTC (permalink / raw)
To: linux-kernel
Cc: Florian Fainelli, Nilesh Javali, Manish Rangankar,
supporter:QLOGIC QL41xxx ISCSI DRIVER, James E.J. Bottomley,
Martin K. Petersen, open list:QLOGIC QL41xxx ISCSI DRIVER
These warnings started to show up after enabling PCI on BMIPS (32-bit
MIPS architecture) and were reported by the kbuild robot.
I don't know whether they are technically correct, in particular the
unused 'page' variable might be unveiling a genuine bug whereby it
should have been used. Please review.
Changes in v2:
- added Acked-by to patch #1
- changed SYSFS_FLAG_FW_SEL_BOOT to contain the typecasting and not
change the way it is formatted before sysfs printing
Florian Fainelli (2):
scsi: qedi: Remove set but unused 'page' variable
scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting
drivers/scsi/qedi/qedi.h | 2 +-
drivers/scsi/qedi/qedi_main.c | 3 ---
2 files changed, 1 insertion(+), 4 deletions(-)
--
2.25.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2 1/2] scsi: qedi: Remove set but unused 'page' variable
2021-11-26 20:17 [PATCH v2 0/2] scsi: qedi: Couple of warning fixes Florian Fainelli
@ 2021-11-26 20:17 ` Florian Fainelli
2021-11-30 4:00 ` Martin K. Petersen
2021-11-26 20:17 ` [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting Florian Fainelli
2021-12-03 3:03 ` [PATCH v2 0/2] scsi: qedi: Couple of warning fixes Martin K. Petersen
2 siblings, 1 reply; 9+ messages in thread
From: Florian Fainelli @ 2021-11-26 20:17 UTC (permalink / raw)
To: linux-kernel
Cc: Florian Fainelli, kernel test robot, Manish Rangankar,
Nilesh Javali, supporter:QLOGIC QL41xxx ISCSI DRIVER,
James E.J. Bottomley, Martin K. Petersen,
open list:QLOGIC QL41xxx ISCSI DRIVER
The variable page is set but never used throughout qedi_alloc_bdq()
therefore remove it.
Reported-by: kernel test robot <lkp@intel.com>
Acked-by: Manish Rangankar <mrangankar@marvell.com>
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
drivers/scsi/qedi/qedi_main.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/scsi/qedi/qedi_main.c b/drivers/scsi/qedi/qedi_main.c
index 1dec814d8788..f1c933070884 100644
--- a/drivers/scsi/qedi/qedi_main.c
+++ b/drivers/scsi/qedi/qedi_main.c
@@ -1538,7 +1538,6 @@ static int qedi_alloc_bdq(struct qedi_ctx *qedi)
int i;
struct scsi_bd *pbl;
u64 *list;
- dma_addr_t page;
/* Alloc dma memory for BDQ buffers */
for (i = 0; i < QEDI_BDQ_NUM; i++) {
@@ -1608,11 +1607,9 @@ static int qedi_alloc_bdq(struct qedi_ctx *qedi)
qedi->bdq_pbl_list_num_entries = qedi->bdq_pbl_mem_size /
QEDI_PAGE_SIZE;
list = (u64 *)qedi->bdq_pbl_list;
- page = qedi->bdq_pbl_list_dma;
for (i = 0; i < qedi->bdq_pbl_list_num_entries; i++) {
*list = qedi->bdq_pbl_dma;
list++;
- page += QEDI_PAGE_SIZE;
}
return 0;
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting
2021-11-26 20:17 [PATCH v2 0/2] scsi: qedi: Couple of warning fixes Florian Fainelli
2021-11-26 20:17 ` [PATCH v2 1/2] scsi: qedi: Remove set but unused 'page' variable Florian Fainelli
@ 2021-11-26 20:17 ` Florian Fainelli
2021-11-29 7:29 ` [EXT] " Manish Rangankar
2021-11-30 3:59 ` Martin K. Petersen
2021-12-03 3:03 ` [PATCH v2 0/2] scsi: qedi: Couple of warning fixes Martin K. Petersen
2 siblings, 2 replies; 9+ messages in thread
From: Florian Fainelli @ 2021-11-26 20:17 UTC (permalink / raw)
To: linux-kernel
Cc: Florian Fainelli, kernel test robot, Nilesh Javali,
Manish Rangankar, supporter:QLOGIC QL41xxx ISCSI DRIVER,
James E.J. Bottomley, Martin K. Petersen,
open list:QLOGIC QL41xxx ISCSI DRIVER
The format used for formatting SYSFS_FLAG_FW_SEL_BOOT creates the
following warning:
drivers/scsi/qedi/qedi_main.c:2259:35: warning: format specifies type
'char' but the argument has type 'int' [-Wformat]
rc = snprintf(buf, 3, "%hhd\n",
SYSFS_FLAG_FW_SEL_BOOT);
Fix this to cast the constant as an u8 since the intention is to print
it via sysfs as a byte.
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
drivers/scsi/qedi/qedi.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/scsi/qedi/qedi.h b/drivers/scsi/qedi/qedi.h
index ce199a7a16b8..421b3a69fd37 100644
--- a/drivers/scsi/qedi/qedi.h
+++ b/drivers/scsi/qedi/qedi.h
@@ -358,7 +358,7 @@ struct qedi_ctx {
bool use_fast_sge;
atomic_t num_offloads;
-#define SYSFS_FLAG_FW_SEL_BOOT 2
+#define SYSFS_FLAG_FW_SEL_BOOT (u8)2
#define IPV6_LEN 41
#define IPV4_LEN 17
struct iscsi_boot_kset *boot_kset;
--
2.25.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* RE: [EXT] [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting
2021-11-26 20:17 ` [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting Florian Fainelli
@ 2021-11-29 7:29 ` Manish Rangankar
2021-11-30 3:59 ` Martin K. Petersen
1 sibling, 0 replies; 9+ messages in thread
From: Manish Rangankar @ 2021-11-29 7:29 UTC (permalink / raw)
To: Florian Fainelli, linux-kernel
Cc: kernel test robot, Nilesh Javali, GR-QLogic-Storage-Upstream,
James E.J. Bottomley, Martin K. Petersen,
open list:QLOGIC QL41xxx ISCSI DRIVER
> -----Original Message-----
> From: Florian Fainelli <f.fainelli@gmail.com>
> Sent: Saturday, November 27, 2021 1:47 AM
> To: linux-kernel@vger.kernel.org
> Cc: Florian Fainelli <f.fainelli@gmail.com>; kernel test robot <lkp@intel.com>;
> Nilesh Javali <njavali@marvell.com>; Manish Rangankar
> <mrangankar@marvell.com>; GR-QLogic-Storage-Upstream <GR-QLogic-
> Storage-Upstream@marvell.com>; James E.J. Bottomley <jejb@linux.ibm.com>;
> Martin K. Petersen <martin.petersen@oracle.com>; open list:QLOGIC QL41xxx
> ISCSI DRIVER <linux-scsi@vger.kernel.org>
> Subject: [EXT] [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT
> formatting
>
> External Email
>
> ----------------------------------------------------------------------
> The format used for formatting SYSFS_FLAG_FW_SEL_BOOT creates the
> following warning:
>
> drivers/scsi/qedi/qedi_main.c:2259:35: warning: format specifies type 'char' but
> the argument has type 'int' [-Wformat]
> rc = snprintf(buf, 3, "%hhd\n", SYSFS_FLAG_FW_SEL_BOOT);
>
> Fix this to cast the constant as an u8 since the intention is to print it via sysfs as a
> byte.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> ---
> drivers/scsi/qedi/qedi.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/qedi/qedi.h b/drivers/scsi/qedi/qedi.h index
> ce199a7a16b8..421b3a69fd37 100644
> --- a/drivers/scsi/qedi/qedi.h
> +++ b/drivers/scsi/qedi/qedi.h
> @@ -358,7 +358,7 @@ struct qedi_ctx {
> bool use_fast_sge;
>
> atomic_t num_offloads;
> -#define SYSFS_FLAG_FW_SEL_BOOT 2
> +#define SYSFS_FLAG_FW_SEL_BOOT (u8)2
> #define IPV6_LEN 41
> #define IPV4_LEN 17
> struct iscsi_boot_kset *boot_kset;
> --
> 2.25.1
Acked-by: Manish Rangankar <mrangankar@marvell.com>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting
2021-11-26 20:17 ` [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting Florian Fainelli
2021-11-29 7:29 ` [EXT] " Manish Rangankar
@ 2021-11-30 3:59 ` Martin K. Petersen
2021-11-30 4:09 ` Florian Fainelli
1 sibling, 1 reply; 9+ messages in thread
From: Martin K. Petersen @ 2021-11-30 3:59 UTC (permalink / raw)
To: Florian Fainelli
Cc: linux-kernel, kernel test robot, Nilesh Javali, Manish Rangankar,
supporter:QLOGIC QL41xxx ISCSI DRIVER, James E.J. Bottomley,
Martin K. Petersen, open list:QLOGIC QL41xxx ISCSI DRIVER
Florian,
> The format used for formatting SYSFS_FLAG_FW_SEL_BOOT creates the
> following warning:
>
> drivers/scsi/qedi/qedi_main.c:2259:35: warning: format specifies type
> 'char' but the argument has type 'int' [-Wformat]
> rc = snprintf(buf, 3, "%hhd\n",
> SYSFS_FLAG_FW_SEL_BOOT);
>
> Fix this to cast the constant as an u8 since the intention is to print
> it via sysfs as a byte.
The other occurrences of SYSFS_FLAG_FW_SEL_BOOT use "%d". Since %hh is
deprecated I suggest you just fix the snprintf().
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] scsi: qedi: Remove set but unused 'page' variable
2021-11-26 20:17 ` [PATCH v2 1/2] scsi: qedi: Remove set but unused 'page' variable Florian Fainelli
@ 2021-11-30 4:00 ` Martin K. Petersen
0 siblings, 0 replies; 9+ messages in thread
From: Martin K. Petersen @ 2021-11-30 4:00 UTC (permalink / raw)
To: Florian Fainelli
Cc: linux-kernel, kernel test robot, Manish Rangankar, Nilesh Javali,
supporter:QLOGIC QL41xxx ISCSI DRIVER, James E.J. Bottomley,
Martin K. Petersen, open list:QLOGIC QL41xxx ISCSI DRIVER
Florian,
> The variable page is set but never used throughout qedi_alloc_bdq()
> therefore remove it.
Applied to 5.17/scsi-staging, thanks!
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting
2021-11-30 3:59 ` Martin K. Petersen
@ 2021-11-30 4:09 ` Florian Fainelli
2021-11-30 4:25 ` Martin K. Petersen
0 siblings, 1 reply; 9+ messages in thread
From: Florian Fainelli @ 2021-11-30 4:09 UTC (permalink / raw)
To: Martin K. Petersen
Cc: linux-kernel, kernel test robot, Nilesh Javali, Manish Rangankar,
supporter:QLOGIC QL41xxx ISCSI DRIVER, James E.J. Bottomley,
open list:QLOGIC QL41xxx ISCSI DRIVER
On 11/29/2021 7:59 PM, Martin K. Petersen wrote:
>
> Florian,
>
>> The format used for formatting SYSFS_FLAG_FW_SEL_BOOT creates the
>> following warning:
>>
>> drivers/scsi/qedi/qedi_main.c:2259:35: warning: format specifies type
>> 'char' but the argument has type 'int' [-Wformat]
>> rc = snprintf(buf, 3, "%hhd\n",
>> SYSFS_FLAG_FW_SEL_BOOT);
>>
>> Fix this to cast the constant as an u8 since the intention is to print
>> it via sysfs as a byte.
>
> The other occurrences of SYSFS_FLAG_FW_SEL_BOOT use "%d". Since %hh is
> deprecated I suggest you just fix the snprintf().
That was what v1 did here:
https://lkml.org/lkml/2021/11/26/9
however Manish seemed to want that flag to be printed as a byte I am
fine either way.
--
Florian
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting
2021-11-30 4:09 ` Florian Fainelli
@ 2021-11-30 4:25 ` Martin K. Petersen
0 siblings, 0 replies; 9+ messages in thread
From: Martin K. Petersen @ 2021-11-30 4:25 UTC (permalink / raw)
To: Florian Fainelli
Cc: Martin K. Petersen, linux-kernel, kernel test robot,
Nilesh Javali, Manish Rangankar,
supporter:QLOGIC QL41xxx ISCSI DRIVER, James E.J. Bottomley,
open list:QLOGIC QL41xxx ISCSI DRIVER
Florian,
>> The other occurrences of SYSFS_FLAG_FW_SEL_BOOT use "%d". Since %hh
>> is deprecated I suggest you just fix the snprintf().
>
> That was what v1 did here:
>
> https://lkml.org/lkml/2021/11/26/9
>
> however Manish seemed to want that flag to be printed as a byte I am
> fine either way.
Not sure I understand the concern since this is a constant which will
always be "2".
But if you must cast, do it in snprintf() and not in the macro
definition. checkpatch also complains about the cast.
I would prefer for all instances of this to be consistent, though. So
whatever you do, please also fix qla4xxx.
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 0/2] scsi: qedi: Couple of warning fixes
2021-11-26 20:17 [PATCH v2 0/2] scsi: qedi: Couple of warning fixes Florian Fainelli
2021-11-26 20:17 ` [PATCH v2 1/2] scsi: qedi: Remove set but unused 'page' variable Florian Fainelli
2021-11-26 20:17 ` [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting Florian Fainelli
@ 2021-12-03 3:03 ` Martin K. Petersen
2 siblings, 0 replies; 9+ messages in thread
From: Martin K. Petersen @ 2021-12-03 3:03 UTC (permalink / raw)
To: Florian Fainelli, linux-kernel
Cc: Martin K . Petersen, James E.J. Bottomley,
supporter:QLOGIC QL41xxx ISCSI DRIVER,
open list:QLOGIC QL41xxx ISCSI DRIVER, Manish Rangankar,
Nilesh Javali
On Fri, 26 Nov 2021 12:17:06 -0800, Florian Fainelli wrote:
> These warnings started to show up after enabling PCI on BMIPS (32-bit
> MIPS architecture) and were reported by the kbuild robot.
>
> I don't know whether they are technically correct, in particular the
> unused 'page' variable might be unveiling a genuine bug whereby it
> should have been used. Please review.
>
> [...]
Applied to 5.17/scsi-queue, thanks!
[1/2] scsi: qedi: Remove set but unused 'page' variable
https://git.kernel.org/mkp/scsi/c/6d8619f034f0
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2021-12-03 3:04 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-26 20:17 [PATCH v2 0/2] scsi: qedi: Couple of warning fixes Florian Fainelli
2021-11-26 20:17 ` [PATCH v2 1/2] scsi: qedi: Remove set but unused 'page' variable Florian Fainelli
2021-11-30 4:00 ` Martin K. Petersen
2021-11-26 20:17 ` [PATCH v2 2/2] scsi: qedi: Fix SYSFS_FLAG_FW_SEL_BOOT formatting Florian Fainelli
2021-11-29 7:29 ` [EXT] " Manish Rangankar
2021-11-30 3:59 ` Martin K. Petersen
2021-11-30 4:09 ` Florian Fainelli
2021-11-30 4:25 ` Martin K. Petersen
2021-12-03 3:03 ` [PATCH v2 0/2] scsi: qedi: Couple of warning fixes Martin K. Petersen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).