RE: Build regressions/improvements in v3.0-rc5 (ssize_t ???)

From: Mikael Starvik
Date: Thu Jun 30 2011 - 03:23:54 EST


>The "long" should be "int".
>But probably it should just include <asm-generic/posix_types.h> instead.

Agreed. We wonÂt be able to test and push something like that before vacation but feel free to test
that change and commit it if it works. Otherwise we'll fix it after vacations.

/Mikael

-----Original Message-----
From: geert.uytterhoeven@xxxxxxxxx [mailto:geert.uytterhoeven@xxxxxxxxx] On Behalf Of Geert Uytterhoeven
Sent: den 30 juni 2011 08:59
To: Randy Dunlap
Cc: Linux Kernel Development; Mikael Starvik; Jesper Nilsson; linux-cris-kernel
Subject: Re: Build regressions/improvements in v3.0-rc5 (ssize_t ???)

On Wed, Jun 29, 2011 at 23:06, Randy Dunlap <rdunlap@xxxxxxxxxxxx> wrote:
> On Wed, 29 Jun 2011 10:13:17 +0200 (CEST) Geert Uytterhoeven wrote:
>
>> *** WARNINGS ***
>>
>> 358 regressions:
>
>> Â + drivers/net/wimax/i2400m/usb-fw.c: warning: format '%zd' expects type 'signed size_t', but argument 4 has type 'ssize_t': Â=> 210
>> Â + drivers/net/wimax/i2400m/usb-fw.c: warning: format '%zd' expects type 'signed size_t', but argument 8 has type 'ssize_t': Â=> 222
>
>> Â + drivers/staging/bcm/Bcmchar.c: warning: format '%zd' expects type 'signed size_t', but argument 2 has type 'ssize_t': Â=> 146
>> Â + drivers/staging/bcm/Bcmchar.c: warning: format '%zd' expects type 'signed size_t', but argument 3 has type 'ssize_t': Â=> 146
>
>> Â + net/sunrpc/auth_gss/auth_gss.c: warning: format '%Zd' expects type 'signed size_t', but argument 2 has type 'ssize_t': Â=> 708
>> Â + net/sunrpc/auth_gss/auth_gss.c: warning: format '%zd' expects type 'signed size_t', but argument 3 has type 'ssize_t': Â=> 690
>
>
> Can someone explain these warnings, please? Â(signed size_t vs. ssize_t)
>
> Is gcc confused?
> Looks like a compiler issue to me.

Nope. Also only happens on cris, which has in
arch/cris/include/asm/posix_types.h:

typedef long __kernel_ssize_t;

The "long" should be "int".
But probably it should just include <asm-generic/posix_types.h> instead.

Gr{oetje,eeting}s,

            Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
             Â Â -- Linus Torvalds
¢éì®&Þ~º&¶¬–+-±éÝ¥Šw®žË±Êâmébžìdz¹Þ)í…æèw*jg¬±¨¶‰šŽŠÝj/êäz¹ÞŠà2ŠÞ¨è­Ú&¢)ß«a¶Úþø®G«éh®æj:+v‰¨Šwè†Ù>Wš±êÞiÛaxPjØm¶Ÿÿà -»+ƒùdš_