Re: [PATCH] memcg: add BUILD_BUG_ON() for string tables

2015-01-13 Thread Michal Hocko
On Mon 12-01-15 10:54:23, Greg Thelen wrote: > Use BUILD_BUG_ON() to compile assert that memcg string tables are in > sync with corresponding enums. There aren't currently any issues with > these tables. This is just defensive. > > Signed-off-by: Greg Thelen Acked-by: Michal Hocko > --- >

Re: [PATCH] memcg: add BUILD_BUG_ON() for string tables

2015-01-13 Thread Michal Hocko
On Mon 12-01-15 10:54:23, Greg Thelen wrote: Use BUILD_BUG_ON() to compile assert that memcg string tables are in sync with corresponding enums. There aren't currently any issues with these tables. This is just defensive. Signed-off-by: Greg Thelen gthe...@google.com Acked-by: Michal

Re: [PATCH] memcg: add BUILD_BUG_ON() for string tables

2015-01-12 Thread Johannes Weiner
On Mon, Jan 12, 2015 at 10:54:23AM -0800, Greg Thelen wrote: > Use BUILD_BUG_ON() to compile assert that memcg string tables are in > sync with corresponding enums. There aren't currently any issues with > these tables. This is just defensive. > > Signed-off-by: Greg Thelen Looks good to me,

[PATCH] memcg: add BUILD_BUG_ON() for string tables

2015-01-12 Thread Greg Thelen
Use BUILD_BUG_ON() to compile assert that memcg string tables are in sync with corresponding enums. There aren't currently any issues with these tables. This is just defensive. Signed-off-by: Greg Thelen --- mm/memcontrol.c | 4 1 file changed, 4 insertions(+) diff --git

Re: [PATCH] memcg: add BUILD_BUG_ON() for string tables

2015-01-12 Thread Johannes Weiner
On Mon, Jan 12, 2015 at 10:54:23AM -0800, Greg Thelen wrote: Use BUILD_BUG_ON() to compile assert that memcg string tables are in sync with corresponding enums. There aren't currently any issues with these tables. This is just defensive. Signed-off-by: Greg Thelen gthe...@google.com Looks

[PATCH] memcg: add BUILD_BUG_ON() for string tables

2015-01-12 Thread Greg Thelen
Use BUILD_BUG_ON() to compile assert that memcg string tables are in sync with corresponding enums. There aren't currently any issues with these tables. This is just defensive. Signed-off-by: Greg Thelen gthe...@google.com --- mm/memcontrol.c | 4 1 file changed, 4 insertions(+) diff