Date: Tue, 19 Dec 2017 18:56:51 -0600 (CST)
From: Christopher Lameter <cl@linux.com>
To: Rao Shoaib <rao.shoaib@oracle.com>
cc: linux-kernel@vger.kernel.org, paulmck@linux.vnet.ibm.com,
        brouer@redhat.com, linux-mm@kvack.org
Subject: Re: [PATCH] kfree_rcu() should use the new kfree_bulk() interface
 for freeing rcu structures
In-Reply-To: <b38f36d7-be4f-8cc4-208e-f0778077a063@oracle.com>
Message-ID: <alpine.DEB.2.20.1712191855060.24885@nuc-kabylake>
References: <rao.shoaib@oracle.com> <1513705948-31072-1-git-send-email-rao.shoaib@oracle.com> <alpine.DEB.2.20.1712191332090.7876@nuc-kabylake> <b38f36d7-be4f-8cc4-208e-f0778077a063@oracle.com>
User-Agent: Alpine 2.20 (DEB 67 2015-01-07)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 661
Lines: 21

On Tue, 19 Dec 2017, Rao Shoaib wrote:

> > > mm/slab_common.c
> > It would be great to have separate patches so that we can review it
> > properly:
> >
> > 1. Move the code into slab_common.c
> > 2. The actual code changes to the kfree rcu mechanism
> > 3. The whitespace changes

> I can certainly break down the patch and submit smaller patches as you have
> suggested.
>
> BTW -- This is my first ever patch to Linux, so I am still learning the
> etiquette.

You are doing great. Keep at improving the patches and we will get your
changes into the kernel source. If you want to sent your first patchset
then a tool like "quilt" or "git" might be helpful.