From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.suse.de ([195.135.220.15]:49883 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751615AbdLAKwR (ORCPT ); Fri, 1 Dec 2017 05:52:17 -0500 Date: Fri, 1 Dec 2017 11:52:15 +0100 From: Jan Kara To: Chris Mason Cc: Tejun Heo , Jan Kara , axboe@kernel.dk, jbacik@fb.com, kernel-team@fb.com, linux-kernel@vger.kernel.org, linux-btrfs@vger.kernel.org Subject: Re: [PATCHSET v2] cgroup, writeback, btrfs: make sure btrfs issues metadata IOs from the root cgroup Message-ID: <20171201105215.GB8365@quack2.suse.cz> References: <20171010155441.753966-1-tj@kernel.org> <20171129165608.GA26740@quack2.suse.cz> <20171129170330.GV983427@devbig577.frc2.facebook.com> <20171129170501.GW983427@devbig577.frc2.facebook.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-btrfs-owner@vger.kernel.org List-ID: On Wed 29-11-17 13:38:26, Chris Mason wrote: > On 11/29/2017 12:05 PM, Tejun Heo wrote: > >On Wed, Nov 29, 2017 at 09:03:30AM -0800, Tejun Heo wrote: > >>Hello, > >> > >>On Wed, Nov 29, 2017 at 05:56:08PM +0100, Jan Kara wrote: > >>>What has happened with this patch set? > >> > >>No idea. cc'ing Chris directly. Chris, if the patchset looks good, > >>can you please route them through the btrfs tree? > > > >lol looking at the patchset again, I'm not sure that's obviously the > >right tree. It can either be cgroup, block or btrfs. If no one > >objects, I'll just route them through cgroup. > > We'll have to coordinate a bit during the next merge window but I don't have > a problem with these going in through cgroup. Dave does this sound good to > you? Also I was wondering about another thing: How does this play with Josef's series for metadata writeback (Metadata specific accouting and dirty writeout)? Would the per-inode selection of cgroup writeback still be needed when Josef's series is applied since metadata writeback then won't be associated with any particular mapping anymore? Honza -- Jan Kara SUSE Labs, CR