[REGRESSION] Re: [PATCH v6 05/56] scsi: qla2xxx: Add FC operational firmware load for 29xx

Uwe Kleine-König <[email protected]> Wed, 12 Aug 2026 08:06:28 +0200

Newsgroups	gmane.linux.scsi
Message-ID	<anwLLKJRVgL6a-aQ@monoceros>

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

signature.asc (application/pgp-signature, 488 B)