Re: [PATCH] mm: sparse: use __aligned() instead of manual padding in mem_section

2013-05-30 Thread Dave Hansen
On 05/29/2013 04:14 PM, Cody P Schafer wrote: > Instead of leaving a trap for the next person who comes along and wants > to add something to mem_section, add an __aligned() and remove the > manual padding added for MEMCG. It doesn't need to be aligned technically. It needs to be a power-of-2: h

Re: [PATCH] mm: sparse: use __aligned() instead of manual padding in mem_section

2013-05-30 Thread Cody P Schafer
On 05/29/2013 05:54 PM, Jiang Liu wrote: On Thu 30 May 2013 07:14:39 AM CST, Cody P Schafer wrote: Also, does anyone know what causes this alignment to be required here? I found this was breaking things in a patchset I'm working on (WARNs in sysfs code about duplicate filenames when initing mem_

Re: [PATCH] mm: sparse: use __aligned() instead of manual padding in mem_section

2013-05-29 Thread Jiang Liu
On Thu 30 May 2013 07:14:39 AM CST, Cody P Schafer wrote: > Instead of leaving a trap for the next person who comes along and wants > to add something to mem_section, add an __aligned() and remove the > manual padding added for MEMCG. > > Signed-off-by: Cody P Schafer > --- > include/linux/mmzone