Re: [PATCH] memcg: add per cgroup dirty page accounting

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

 



On Mon, Mar 09, 2015 at 11:29:05AM -0400, Greg Thelen wrote:
> On Mon, Mar 9, 2015 at 9:52 AM, Tejun Heo <tj@xxxxxxxxxx> wrote:
> > Hello, Greg.
> >
> > On Sun, Mar 08, 2015 at 09:50:32PM -0700, Greg Thelen wrote:
> >> When modifying PG_Dirty on cached file pages, update the new
> >> MEM_CGROUP_STAT_DIRTY counter.  This is done in the same places where
> >> global NR_FILE_DIRTY is managed.  The new memcg stat is visible in the
> >> per memcg memory.stat cgroupfs file.  The most recent past attempt at
> >> this was http://thread.gmane.org/gmane.linux.kernel.cgroups/8632
> >
> > Awesome.  I had a similar but inferior (haven't noticed the irqsave
> > problem) patch in my series.  Replaced that with this one.  I'm
> > getting ready to post the v2 of the cgroup writeback patchset.  Do you
> > mind routing this patch together in the patchset?
> 
> I don't object to routing this patch with the larger writeback series.
> But I do have small concern that merging the writeback series might
> take a while and this patch has independent value.  For now, I'd say:
> go for it.  If the series gets stalled we might want to split it off.

Yeah, sure, either is fine for me.  Hmm... I'm gonna move this patch
to the head of the series so that it can go either way.

Thanks.

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux