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

Mikael Starvik
2011-06-30T07:23:50Z

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 2537 bytes --]

>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@gmail.com [mailto:geert.uytterhoeven@gmail.com] 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@xenotime.net> 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@linux-m68k.org

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
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éÝ¶\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dÊ‡Ú™ë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥