From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lb0-f177.google.com (mail-lb0-f177.google.com [209.85.217.177]) by kanga.kvack.org (Postfix) with ESMTP id 93DC36B0035 for ; Fri, 25 Apr 2014 08:33:21 -0400 (EDT) Received: by mail-lb0-f177.google.com with SMTP id z11so2978351lbi.36 for ; Fri, 25 Apr 2014 05:33:20 -0700 (PDT) Received: from relay.parallels.com (relay.parallels.com. [195.214.232.42]) by mx.google.com with ESMTPS id iz10si5584464lbc.207.2014.04.25.05.33.19 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 25 Apr 2014 05:33:19 -0700 (PDT) From: Vladimir Davydov Subject: [PATCH -mm 0/6] memcg/kmem: cleanup naming and callflows Date: Fri, 25 Apr 2014 16:33:06 +0400 Message-ID: MIME-Version: 1.0 Content-Type: text/plain Sender: owner-linux-mm@kvack.org List-ID: To: akpm@linux-foundation.org Cc: mhocko@suse.cz, hannes@cmpxchg.org, glommer@gmail.com, cl@linux-foundation.org, penberg@kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, devel@openvz.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 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org 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