From: Andrew Morton ERROR: switch and case should be at the same indent #59: FILE: mm/memcontrol.c:878: + switch (event) { + case RES_MAX_USAGE: [...] + case RES_FAILCNT: total: 1 errors, 0 warnings, 48 lines checked ./patches/memcgroup-implement-failcounter-reset.patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Please run checkpatch prior to sending patches Cc: Balbir Singh Cc: KAMEZAWA Hiroyuki Cc: Paul Menage Cc: Pavel Emelyanov Signed-off-by: Andrew Morton --- mm/memcontrol.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff -puN mm/memcontrol.c~memcgroup-implement-failcounter-reset-checkpatch-fixes mm/memcontrol.c --- a/mm/memcontrol.c~memcgroup-implement-failcounter-reset-checkpatch-fixes +++ a/mm/memcontrol.c @@ -876,12 +876,12 @@ static int mem_cgroup_reset(struct cgrou mem = mem_cgroup_from_cont(cont); switch (event) { - case RES_MAX_USAGE: - res_counter_reset_max(&mem->res); - break; - case RES_FAILCNT: - res_counter_reset_failcnt(&mem->res); - break; + case RES_MAX_USAGE: + res_counter_reset_max(&mem->res); + break; + case RES_FAILCNT: + res_counter_reset_failcnt(&mem->res); + break; } return 0; } _