Hello, The 'break' is unnecessary at mem_cgroup_read() routine if the MEMFILE_TYPE is invalid IMHO. Signed-off-by: Jie Liu <jeff.liu@xxxxxxxxxx> --- mm/memcontrol.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/mm/memcontrol.c b/mm/memcontrol.c index 6aff93c..105972c 100644 --- a/mm/memcontrol.c +++ b/mm/memcontrol.c @@ -3886,7 +3886,6 @@ static u64 mem_cgroup_read(struct cgroup *cont, struct cftype *cft) break; default: BUG(); - break; } return val; } -- 1.7.9 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>