[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190521003709.GA21811@tower.DHCP.thefacebook.com>
Date: Tue, 21 May 2019 00:37:16 +0000
From: Roman Gushchin <guro@...com>
To: "Tobin C. Harding" <tobin@...nel.org>
CC: Andrew Morton <akpm@...ux-foundation.org>,
Matthew Wilcox <willy@...radead.org>,
Alexander Viro <viro@....linux.org.uk>,
"Christoph Hellwig" <hch@...radead.org>,
Pekka Enberg <penberg@...helsinki.fi>,
"David Rientjes" <rientjes@...gle.com>,
Joonsoo Kim <iamjoonsoo.kim@....com>,
Christopher Lameter <cl@...ux.com>,
Miklos Szeredi <mszeredi@...hat.com>,
Andreas Dilger <adilger@...ger.ca>,
Waiman Long <longman@...hat.com>,
"Tycho Andersen" <tycho@...ho.ws>, Theodore Ts'o <tytso@....edu>,
Andi Kleen <ak@...ux.intel.com>,
David Chinner <david@...morbit.com>,
Nick Piggin <npiggin@...il.com>,
Rik van Riel <riel@...hat.com>,
Hugh Dickins <hughd@...gle.com>,
Jonathan Corbet <corbet@....net>,
"linux-mm@...ck.org" <linux-mm@...ck.org>,
"linux-fsdevel@...r.kernel.org" <linux-fsdevel@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [RFC PATCH v5 01/16] slub: Add isolate() and migrate() methods
On Mon, May 20, 2019 at 03:40:02PM +1000, Tobin C. Harding wrote:
> Add the two methods needed for moving objects and enable the display of
> the callbacks via the /sys/kernel/slab interface.
>
> Add documentation explaining the use of these methods and the prototypes
> for slab.h. Add functions to setup the callbacks method for a slab
> cache.
>
> Add empty functions for SLAB/SLOB. The API is generic so it could be
> theoretically implemented for these allocators as well.
>
> Change sysfs 'ctor' field to be 'ops' to contain all the callback
> operations defined for a slab cache. Display the existing 'ctor'
> callback in the ops fields contents along with 'isolate' and 'migrate'
> callbacks.
>
> Co-developed-by: Christoph Lameter <cl@...ux.com>
> Signed-off-by: Tobin C. Harding <tobin@...nel.org>
> ---
> include/linux/slab.h | 70 ++++++++++++++++++++++++++++++++++++++++
> include/linux/slub_def.h | 3 ++
> mm/slub.c | 59 +++++++++++++++++++++++++++++----
> 3 files changed, 126 insertions(+), 6 deletions(-)
Reviewed-by: Roman Gushchin <guro@...com>
Thanks!
Powered by blists - more mailing lists