Re: [PATCH 12/35] workqueue: update cwq alignement
From: Frederic Weisbecker
Date: Tue Jun 29 2010 - 12:00:56 EST
On Tue, Jun 29, 2010 at 05:51:14PM +0200, Tejun Heo wrote:
> Hello, Frederic.
> On 06/29/2010 05:47 PM, Frederic Weisbecker wrote:
> > It means the size of the struct itself doesn't need to aligned to anything.
> Yeah, it could be that I'm kinda dense today but your mode of
> communication is a bit too implicit for me. So, can you please
> elaborate what problem you see in the patch and how you want it to be
So, imagine you allocate your struct with alloc_percpu(align).
The per cpu pointer is 0x400 (purely imagination).
Now you have two cpus and they have the following base offsets for
per cpu allocations:
CPU 0 = 0xf1000000
CPU 1 = 0xf2000000
So, the true pointers for your cpu workqueue structs will be:
CPU 0 = 0xf1000400
CPU 1 = 0xf2000400
These addresses are aligned like you wanted to, and it seems it is what
matters, to store these addresses in the work flags.
So why does the size of the struct need to be aligned too? All you want
is that the two above addresses are aligned. Now why the size of the struct
itself needs this alignment too. That's the obscure point for me. If it's
useless, this could avoid all this alignment maintainance, except during
the allocation itself.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/