[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160320193407.GB1907@black.fi.intel.com>
Date: Sun, 20 Mar 2016 22:34:07 +0300
From: "Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>
To: Linus Torvalds <torvalds@...ux-foundation.org>
Cc: Al Viro <viro@...iv.linux.org.uk>,
Christoph Lameter <cl@...ux.com>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Andrew Morton <akpm@...ux-foundation.org>,
linux-fsdevel <linux-fsdevel@...r.kernel.org>,
Matthew Wilcox <willy@...ux.intel.com>,
Vineet Gupta <vgupta@...opsys.com>,
linux-mm <linux-mm@...ck.org>
Subject: Re: [PATCH 01/71] arc: get rid of PAGE_CACHE_* and
page_cache_{get,release} macros
On Sun, Mar 20, 2016 at 12:13:47PM -0700, Linus Torvalds wrote:
> On Sun, Mar 20, 2016 at 12:00 PM, Al Viro <viro@...iv.linux.org.uk> wrote:
> >>
> >> It doesn't help legibility or testing, so let's just do it in one big go.
> >
> > Might make sense splitting it by the thing being removed, though - easier
> > to visually verify that it's doing the right thing when all replacements
> > are of the same sort...
>
> Yeah, that might indeed make each patch easier to read, and if
> something goes wrong (which looks unlikely, but hey, shit happens), it
> also makes it easier to see just what went wrong.
Hm. Okay. Re-split this way would take some time. I'll post updated
patchset tomorrow.
--
Kirill A. Shutemov
Powered by blists - more mailing lists