Re: [patch] generic rwsems

Christoph Hellwig
2007-04-13T12:13:48Z

On Fri, Apr 13, 2007 at 12:44:50PM +0100, David Howells wrote:
> Nick Piggin <npiggin@suse.de> wrote:
>
> > I think I should put wait_lock after wait_list, so as to get a better
> > packing on most 64-bit architectures.
>
> It makes no difference. struct lockdep_map contains at least one pointer and
> so is going to be 8-byte aligned (assuming it's there at all). struct
> rw_semaphore contains at least one pointer/long, so it will be padded out to
> 8-byte size.

I hope people are not going to enabled lockdep on their production systems :)