lkml.org 
[lkml]   [2009]   [Feb]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [xfs-masters] [PATCH] fs: Add new pre-allocation ioctls to vfs for compatibility with legacy xfs ioctls
Christoph Hellwig wrote:
> The structures have been defined exactly like that in XFS (and ocfs2)
> before, and there are similar cases in other ioctls handlers.
>
> If anyone feels like changing this in some way feel free to wade through
> the endless discussions about the pros and cons for it, but I think
> doing it in context of this patch is not helpful.

OK so ia64 gcc is broken in regard to __attribute__((packed(1))),
and it should not be used.
But clearly the programmer, Like in this patch exactly, should spell
out the hole created by padding and spell that hole out and call it
__Padding. The porgrammer thought about it, identified there is an hole,
please don't drop this information on the floor. Put it in the code so I
don't have to break my head on it.

> Arnd Bergmann wrote:
>> > +struct space_resv {
>> > + __s16 l_type;
>> > + __s16 l_whence;
>> > + __s64 l_start;
>> > + __s64 l_len; /* len == 0 means until end of file */
>> > + __s32 l_sysid;
>> > + __u32 l_pid;
>> > + __s32 l_pad[4]; /* reserve area */
>> > +};
>
> What about telling the compiler exactly what you said above, just
> to be sure we all mean the same thing. (And as documentation for new
> comers):
>
> +struct space_resv_64 {
> + __s16 l_type;
> + __s16 l_whence;
> + __u32 reserved;
change that to
__u32 __padding_hole;
> + __s64 l_start;
> + __s64 l_len; /* len == 0 means until end of file */
> + __s32 l_sysid;
> + __u32 l_pid;
> + __s32 l_pad[4]; /* reserve area */
> +} __packed;
Drop the evil __packed but show me the padding

Thanks
Boaz


\
 
 \ /
  Last update: 2009-02-01 18:01    [from the cache]
©2003-2011 Jasper Spaans. Advertise on this site