From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753927AbaDYMd1 (ORCPT ); Fri, 25 Apr 2014 08:33:27 -0400 Received: from relay.parallels.com ([195.214.232.42]:33593 "EHLO relay.parallels.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753849AbaDYMdR (ORCPT ); Fri, 25 Apr 2014 08:33:17 -0400 From: Vladimir Davydov To: CC: , , , , , , , Subject: [PATCH -mm 0/6] memcg/kmem: cleanup naming and callflows Date: Fri, 25 Apr 2014 16:33:06 +0400 Message-ID: X-Mailer: git-send-email 1.7.10.4 MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [10.30.16.96] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, In reply to "[PATCH RFC -mm v2 3/3] memcg, slab: simplify synchronization scheme" Johannes wrote: > I like this patch, but the API names are confusing. Could we fix up > that whole thing by any chance? (see https://lkml.org/lkml/2014/4/18/317) So this patch set is about cleaning up memcg/kmem naming. While preparing it I found that some of the ugly-named functions constituting interface between memcontrol.c and slab_common.c can be neatly got rid of w/o complicating the code. Quite the contrary, w/o them call-flows look much simpler, IMO. So the first four patches do not rename anything actually - they just rework call-flows in kmem cache creation/destruction and memcg_caches arrays relocations paths. Finally, patches 5 and 6 clean up the naming. Reviews are appreciated. Thanks, Vladimir Davydov (6): memcg: get rid of memcg_create_cache_name memcg: allocate memcg_caches array on first per memcg cache creation memcg: cleanup memcg_caches arrays relocation path memcg: get rid of memcg_{alloc,free}_cache_params memcg: cleanup kmem cache creation/destruction functions naming memcg: cleanup kmem_id-related naming include/linux/memcontrol.h | 40 +---- include/linux/slab.h | 12 +- mm/memcontrol.c | 420 ++++++++++++++++++++------------------------ mm/slab.c | 4 +- mm/slab.h | 24 ++- mm/slab_common.c | 127 ++++++++++---- mm/slub.c | 10 +- 7 files changed, 320 insertions(+), 317 deletions(-) -- 1.7.10.4