* [REGRESSION] Re: [PATCH v6 05/56] scsi: qla2xxx: Add FC operational firmware load for 29xx
[not found] ` <c324172a-a754-4194-a841-ec1d813dfde5@oracle.com>
@ 2026-08-12 6:06 ` Uwe Kleine-König
0 siblings, 0 replies; only message in thread
From: Uwe Kleine-König @ 2026-08-12 6:06 UTC (permalink / raw)
To: Martin K. Petersen
Cc: John Garry, Nilesh Javali, martin.petersen, linux-scsi,
GR-FC-Storage-Upstream, agurumurthy, emilne, jmeneghi, hare,
regressions
[-- Attachment #1: Type: text/plain, Size: 1710 bytes --]
On Mon, Aug 10, 2026 at 03:27:01PM +0100, John Garry wrote:
> On 23/07/2026 06:03, Nilesh Javali wrote:
> > +
> > + if (section == TIM) {
> > + if (section_size > req->length * qla_req_entry_size(ha)) {
> > + ql_log(ql_log_fatal, vha, 0x0098,
> > + "TIM section too large (0x%x bytes, ring 0x%lx bytes).\n",
> > + section_size,
> > + req->length * qla_req_entry_size(ha));
> > + return QLA_FUNCTION_FAILED;
> > + }
>
> Not that this is generating a warning for i386 builds on the mkp-scsi 7.3
> staging branch:
>
> drivers/scsi/qla2xxx/qla_init.c: In function ‘qla29xx_process_rd_image’:
> drivers/scsi/qla2xxx/qla_init.c:9272:53: error: format ‘%lx’ expects
> argument of type ‘long unsigned int’, but argument 6 has type ‘size_t’ {aka
> ‘unsigned int’} [-Werror=format=]
> 9272 | "TIM section too large (0x%x bytes, ring 0x%lx bytes).\n",
> | ~~^
> | |
> | long unsigned
> int
> | %x
> 9273 | section_size,
> 9274 | req->length * qla_req_entry_size(ha));
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> | |
> | size_t {aka unsigned int}
> cc1: all warnings being treated as errors
This patch became commit c3930ec119cb ("scsi: qla2xxx: Add FC
operational firmware load for 29xx") now and is included in
next-20260810 and later.
#regzbot introduced: c3930ec119cb916be879c017ea1e389e05dbe1aa ^
Best regards
Uwe
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] only message in thread