rawhide report: 20080220 changes

Dave Anderson anderson at redhat.com
Thu Feb 21 12:56:57 UTC 2008


David Woodhouse wrote:
> On Wed, 2008-02-20 at 15:14 -0500, Jesse Keating wrote:
>> * Wed Feb 20 2008 Dave Anderson <anderson at redhat.com> - 4.0-6.0.4
>> - First attempt at addressing the GCC 4.3 build, which failed on x86_64
>>   because ptrace-abi.h (included by ptrace.h) uses the "u32" typedef,
>>   which relies on <asm/types.h>, and include/asm-x86_64/types.h
>>   does not not typedef u32 as done in include/asm-x86/types.h.
> 
> Hm. Did you send a patch to the upstream kernel to fix that? We
> shouldn't be using that 'u32' abomination in user-visible headers.

No.

> 
> If it's something which libc might include for itself, we have to avoid
> standard C types and use '__u32' to avoid namespace 'pollution'. 
> 
> Otherwise, as seems to be the case here, we can just require <stdint.h>
> and use the proper 'uint32_t' type.
> 

Right, since ptrace-abi.h explicitly #include's asm/types.h just
prior to the structure declaration, probably __u32 is more appropriate?

Dave





More information about the devel mailing list