Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
On 10/28/2016 03:03 PM, Arnd Bergmann wrote: > On Friday, October 28, 2016 2:58:33 PM CEST Vineet Gupta wrote: >> On 10/28/2016 02:52 PM, Vineet Gupta wrote: >>> On 10/28/2016 02:44 PM, Vineet Gupta wrote: This is configuration specific, and something caused your compiler to > be built

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
On 10/28/2016 03:03 PM, Arnd Bergmann wrote: > On Friday, October 28, 2016 2:58:33 PM CEST Vineet Gupta wrote: >> On 10/28/2016 02:52 PM, Vineet Gupta wrote: >>> On 10/28/2016 02:44 PM, Vineet Gupta wrote: This is configuration specific, and something caused your compiler to > be built

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Arnd Bergmann
On Friday, October 28, 2016 2:58:33 PM CEST Vineet Gupta wrote: > On 10/28/2016 02:52 PM, Vineet Gupta wrote: > > On 10/28/2016 02:44 PM, Vineet Gupta wrote: > >> This is configuration specific, and something caused your compiler to > >>> be built assuming that size_t is unsigned long, while the

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Arnd Bergmann
On Friday, October 28, 2016 2:58:33 PM CEST Vineet Gupta wrote: > On 10/28/2016 02:52 PM, Vineet Gupta wrote: > > On 10/28/2016 02:44 PM, Vineet Gupta wrote: > >> This is configuration specific, and something caused your compiler to > >>> be built assuming that size_t is unsigned long, while the

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
On 10/28/2016 02:52 PM, Vineet Gupta wrote: > On 10/28/2016 02:44 PM, Vineet Gupta wrote: >> This is configuration specific, and something caused your compiler to >>> be built assuming that size_t is unsigned long, while the kernel >>> headers are assuming it should be unsigned int. > > So yes

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
On 10/28/2016 02:52 PM, Vineet Gupta wrote: > On 10/28/2016 02:44 PM, Vineet Gupta wrote: >> This is configuration specific, and something caused your compiler to >>> be built assuming that size_t is unsigned long, while the kernel >>> headers are assuming it should be unsigned int. > > So yes

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
On 10/28/2016 02:44 PM, Vineet Gupta wrote: > This is configuration specific, and something caused your compiler to >> be built assuming that size_t is unsigned long, while the kernel >> headers are assuming it should be unsigned int. So yes this seems to be target specific gcc thing for ARC 4.8

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
On 10/28/2016 02:44 PM, Vineet Gupta wrote: > This is configuration specific, and something caused your compiler to >> be built assuming that size_t is unsigned long, while the kernel >> headers are assuming it should be unsigned int. So yes this seems to be target specific gcc thing for ARC 4.8

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Arnd Bergmann
On Friday, October 28, 2016 2:44:13 PM CEST Vineet Gupta wrote: > > Indeed if I hack include/linux/types.h > > -typedef __kernel_size_tsize_t; > +typedef unsigned long size_t; > > then the warning goes away, so gcc is indeed assuming size_t to be unsigned > long > and

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Arnd Bergmann
On Friday, October 28, 2016 2:44:13 PM CEST Vineet Gupta wrote: > > Indeed if I hack include/linux/types.h > > -typedef __kernel_size_tsize_t; > +typedef unsigned long size_t; > > then the warning goes away, so gcc is indeed assuming size_t to be unsigned > long > and

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
On 10/28/2016 02:33 PM, Arnd Bergmann wrote: > On Friday, October 28, 2016 2:03:21 PM CEST Vineet Gupta wrote: >> >> I'm trying to use about to be released ARC gcc 6.x with current kernels and >> see a >> flood of warnings due to these legit fixes - i.e.g arc gcc 6.2 complains >> when it >> sees

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
On 10/28/2016 02:33 PM, Arnd Bergmann wrote: > On Friday, October 28, 2016 2:03:21 PM CEST Vineet Gupta wrote: >> >> I'm trying to use about to be released ARC gcc 6.x with current kernels and >> see a >> flood of warnings due to these legit fixes - i.e.g arc gcc 6.2 complains >> when it >> sees

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Arnd Bergmann
On Friday, October 28, 2016 2:03:21 PM CEST Vineet Gupta wrote: > > I'm trying to use about to be released ARC gcc 6.x with current kernels and > see a > flood of warnings due to these legit fixes - i.e.g arc gcc 6.2 complains when > it > sees -zx formats. > > CC mm/percpu.o >

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Arnd Bergmann
On Friday, October 28, 2016 2:03:21 PM CEST Vineet Gupta wrote: > > I'm trying to use about to be released ARC gcc 6.x with current kernels and > see a > flood of warnings due to these legit fixes - i.e.g arc gcc 6.2 complains when > it > sees -zx formats. > > CC mm/percpu.o >

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
Hi Arnd, On 10/17/2016 05:35 AM, Arnd Bergmann wrote: > A recent bugfix introduced a harmless warning in the lpfc driver: > > drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': > drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects argument > of type 'long int', but

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-28 Thread Vineet Gupta
Hi Arnd, On 10/17/2016 05:35 AM, Arnd Bergmann wrote: > A recent bugfix introduced a harmless warning in the lpfc driver: > > drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': > drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects argument > of type 'long int', but

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread Martin K. Petersen
> "Arnd" == Arnd Bergmann writes: Arnd> A recent bugfix introduced a harmless warning in the lpfc driver: Arnd> drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': Arnd> drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects Arnd> argument of type

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread Martin K. Petersen
> "Arnd" == Arnd Bergmann writes: Arnd> A recent bugfix introduced a harmless warning in the lpfc driver: Arnd> drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': Arnd> drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects Arnd> argument of type 'long int', but

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread James Smart
Thanks Signed-off-by: James Smart -- james On 10/17/2016 5:35 AM, Arnd Bergmann wrote: A recent bugfix introduced a harmless warning in the lpfc driver: drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error:

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread James Smart
Thanks Signed-off-by: James Smart -- james On 10/17/2016 5:35 AM, Arnd Bergmann wrote: A recent bugfix introduced a harmless warning in the lpfc driver: drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread Johannes Thumshirn
On Mon, Oct 17, 2016 at 02:35:46PM +0200, Arnd Bergmann wrote: > A recent bugfix introduced a harmless warning in the lpfc driver: > > drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': > drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects argument > of type 'long

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread Johannes Thumshirn
On Mon, Oct 17, 2016 at 02:35:46PM +0200, Arnd Bergmann wrote: > A recent bugfix introduced a harmless warning in the lpfc driver: > > drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': > drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects argument > of type 'long

[PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread Arnd Bergmann
A recent bugfix introduced a harmless warning in the lpfc driver: drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects argument of type 'long int', but argument 9 has type 'size_t {aka const unsigned int}'

[PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread Arnd Bergmann
A recent bugfix introduced a harmless warning in the lpfc driver: drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects argument of type 'long int', but argument 9 has type 'size_t {aka const unsigned int}'