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

Uwe Kleine-König <[email protected]>
Newsgroups dev.linux.lists.regressions,org.kernel.vger.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)
-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmp8DWAACgkQj4D7WH0S
/k6BvggAsG7SGFLcknG+/aqI/u1jGCPHEq5z8Jk49ehBP0rOAmmU27VdhIxUL7HJ
rK6HFRcjx4KfzjAO/NHRP+aOB9Rxluaf/enIVaGJBVQxx5RXmyAFxEM6XZKc9Emf
UykEio89rzrgOvDUWRxq8PRdAfDCL7frOZEyC2BPanyo6oe/lPrF44qC0CyBE6nl
+lQ6uKMqMnmvqsyXBQ9SKSRsaX8fKfNT1RYmRk9DyMvSVG6BZNpP2OXlwkmOrEPL
SIpi4P36GRqOwkrCHq9N1OJ4kZkuAE0r5KxKWbQSteKL00mehRgui0bN+pxvZqeg
HcpQfUtsre4sbXVojxGWaKtgnvhXxQ==
=0nEt
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.