Re: [PATCH 0/5] memcg: trivial cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat 14-01-12 16:07:42, Hugh Dickins wrote:
> On Mon, 9 Jan 2012, Johannes Weiner wrote:
> > On Sat, Dec 31, 2011 at 11:26:42PM -0800, Hugh Dickins wrote:
> > > Obviously I've missed the boat for per-memcg per-zone LRU locking in 3.3,
> > > but I've split out a shameless bunch of trivial cleanups from that work,
> > > and hoping these might still sneak in unless they're controversial.
> > > 
> > > Following on from my earlier mmotm/next patches, here's five
> > > to memcontrol.c and .h, followed by six to the rest of mm.
> > > 
> > > [PATCH 1/5] memcg: replace MEM_CONT by MEM_RES_CTLR
> > > [PATCH 2/5] memcg: replace mem and mem_cont stragglers
> > > [PATCH 3/5] memcg: lru_size instead of MEM_CGROUP_ZSTAT
> > > [PATCH 4/5] memcg: enum lru_list lru
> > > [PATCH 5/5] memcg: remove redundant returns
> > 
> > No objections from my side wrt putting them into 3.3.
> > 
> > Thanks!
> 
> I was hoping that these five memcg trivia (and my two SHM_UNLOCK fixes)
> were on their way into 3.3, but they've not yet shown up in mm-commits.
> 
> I'll resend them all again now: I've not rediffed, since they apply
> (if at different offsets) to Linus's current git tree; but I have added
> in the (somewhat disproportionate for trivia!) Acked-bys and Reviewed-bys.
> 
> Michal was not happy with 3/5: I've summarized below the --- on that one,
> do with it as you wish - I think neither Michal nor I shall slam the door
> and burst into tears if you decide against one of us.

Yes, please go on with the patch. I will not lose any sleep over
MEM_CGROUP_ZSTAT ;)
As both Kame and Johannes acked that, there is no need to discuss that
more.

-- 
Michal Hocko
SUSE Labs
SUSE LINUX s.r.o.
Lihovarska 1060/12
190 00 Praha 9    
Czech Republic

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]