Subject: + memcg-remove-redundant-code-in-mem_cgroup_force_empty_write.patch added to -mm tree To: lizefan@xxxxxxxxxx,hannes@xxxxxxxxxxx,kamezawa.hiroyu@xxxxxxxxxxxxxx,mhocko@xxxxxxx,tj@xxxxxxxxxx From: akpm@xxxxxxxxxxxxxxxxxxxx Date: Tue, 30 Jul 2013 13:34:55 -0700 The patch titled Subject: memcg: remove redundant code in mem_cgroup_force_empty_write() has been added to the -mm tree. Its filename is memcg-remove-redundant-code-in-mem_cgroup_force_empty_write.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/memcg-remove-redundant-code-in-mem_cgroup_force_empty_write.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/memcg-remove-redundant-code-in-mem_cgroup_force_empty_write.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Li Zefan <lizefan@xxxxxxxxxx> Subject: memcg: remove redundant code in mem_cgroup_force_empty_write() vfs guarantees the cgroup won't be destroyed, so it's redundant to get a css reference. Signed-off-by: Li Zefan <lizefan@xxxxxxxxxx> Acked-by: Michal Hocko <mhocko@xxxxxxx> Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> Cc: Johannes Weiner <hannes@xxxxxxxxxxx> Cc: Tejun Heo <tj@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- mm/memcontrol.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff -puN mm/memcontrol.c~memcg-remove-redundant-code-in-mem_cgroup_force_empty_write mm/memcontrol.c --- a/mm/memcontrol.c~memcg-remove-redundant-code-in-mem_cgroup_force_empty_write +++ a/mm/memcontrol.c @@ -5005,15 +5005,10 @@ static int mem_cgroup_force_empty(struct static int mem_cgroup_force_empty_write(struct cgroup *cont, unsigned int event) { struct mem_cgroup *memcg = mem_cgroup_from_cont(cont); - int ret; if (mem_cgroup_is_root(memcg)) return -EINVAL; - css_get(&memcg->css); - ret = mem_cgroup_force_empty(memcg); - css_put(&memcg->css); - - return ret; + return mem_cgroup_force_empty(memcg); } _ Patches currently in -mm which might be from lizefan@xxxxxxxxxx are vmpressure-change-vmpressure-sr_lock-to-spinlock.patch vmpressure-do-not-check-for-pending-work-to-prevent-from-new-work.patch vmpressure-make-sure-there-are-no-events-queued-after-memcg-is-offlined.patch memcg-remove-redundant-code-in-mem_cgroup_force_empty_write.patch linux-next.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html