Messages in this thread | | | Date | Tue, 29 Jun 2010 18:01:02 +0200 | From | Frederic Weisbecker <> | Subject | Re: [PATCH 12/35] workqueue: update cwq alignement |
| |
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 > changed? > > Thanks.
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.
| |