[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <af23974b-20e8-47fc-8063-e7c1440e46aa@default>
Date: Wed, 21 Sep 2011 12:13:17 -0700 (PDT)
From: Dan Magenheimer <dan.magenheimer@...cle.com>
To: Konrad Wilk <konrad.wilk@...cle.com>
Cc: linux-kernel@...r.kernel.org, linux-mm@...ck.org, jeremy@...p.org,
hughd@...gle.com, ngupta@...are.org, JBeulich@...ell.com,
Kurt Hackel <kurt.hackel@...cle.com>, npiggin@...nel.dk,
akpm@...ux-foundation.org, riel@...hat.com, hannes@...xchg.org,
matthew@....cx, Chris Mason <chris.mason@...cle.com>,
sjenning@...ux.vnet.ibm.com, kamezawa.hiroyu@...fujitsu.com,
jackdachef@...il.com, cyclonusj@...il.com, levinsasha928@...il.com
Subject: RE: [PATCH V10 5/6] mm: cleancache: update to match akpm frontswap
feedback
> From: Konrad Wilk
>
> On Thu, Sep 15, 2011 at 02:34:46PM -0700, Dan Magenheimer wrote:
> > From: Dan Magenheimer <dan.magenheimer@...cle.com>
> > Subject: [PATCH V10 5/6] mm: cleancache: update to match akpm frontswap feedback
>
> That is a pretty bad title. Think about it - in a year, you are going to
> try to track down something using 'git annotate' and this git commit is going
> to come up. And you look at it and this is the title. It does not carry
> the technical details of what is in the patch.
>
> Also in case you are thinking "But there are so more git commits" - don't
> fret about them. It is OK to have many of them. So don't by shy with them.
OK, thanks for the feedback. Since there are no code changes,
I won't flood the list with another version with the patches divided
differently (and thanks for your help reorganizing them and your offlist
review so I know it meets your approval!); instead I will just make a
new v11 git branch. Since kernel.org is still down, if anyone
wants to look at the latest, it can be found at:
git://oss.oracle.com/git/djm/tmem.git#frontswap-v11
I'll also be asking Stephen Rothwell to pull v11 into linux-next
because the frontswap version that is there is getting a bit stale.
Hopefully everything is finally now ready for merging for the
3.2 window!
Thanks,
Dan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists