Re: [PATCH v2 block/for-linus] writeback: flush inode cgroup wb switches instead of pinning super_block

From	Jan Kara <jack@suse.cz>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 block/for-linus] writeback: flush inode cgroup wb switches instead of pinning super_block
Date	2016-03-02 11:30 +0100
Message-ID	<r8ccy-6iu-3@gated-at.bofh.it> (permalink)
References	(5 earlier) <r3jol-4YQ-3@gated-at.bofh.it> <r7CVr-7Vn-1@gated-at.bofh.it> <r7Fqi-18H-29@gated-at.bofh.it> <r7WAO-3OJ-7@gated-at.bofh.it> <r7WKu-3T3-29@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tue 01-03-16 12:50:19, Tejun Heo wrote:
> Hello, Jens.
> 
> On Tue, Mar 1, 2016 at 12:46 PM, Jens Axboe <axboe@kernel.dk> wrote:
> > I queued this up for 4.5, but I'm feeling a bit uneasy about it. But it's
> > either that, or revert 5ff8eaac1636 and fix it for real in 4.6. Jan/Tejun,
> > what do you think?
> 
> Given that this only matters for cgroup writeback cases, this should
> still be fairly low impact, so I don't think it'd matter too much
> whether we fix this in this cycle or for 4.6. However, that also means
> that we're not risking much by doing it in this cycle, so I'd vote for
> doing it now.

Yeah, without CGROUP_WRITEBACK enabled this patch is NOP so I don't care
much when this gets merged.

								Honza
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR

Thread

[PATCH block/for-linus] writeback: flush inode cgroup wb switches  instead of pinning super_block Tejun Heo <tj@kernel.org> - 2016-02-29 21:50 +0100
  Re: [PATCH block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Al Viro <viro@ZenIV.linux.org.uk> - 2016-02-29 22:00 +0100
    Re: [PATCH block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Tejun Heo <tj@kernel.org> - 2016-02-29 22:00 +0100
      Re: [PATCH block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Tejun Heo <tj@kernel.org> - 2016-02-29 22:10 +0100
        Re: [PATCH block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Jan Kara <jack@suse.cz> - 2016-02-29 22:30 +0100
      Re: [PATCH block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Al Viro <viro@ZenIV.linux.org.uk> - 2016-02-29 22:10 +0100
  [PATCH v2 block/for-linus] writeback: flush inode cgroup wb switches  instead of pinning super_block Tejun Heo <tj@kernel.org> - 2016-03-01 00:30 +0100
    Re: [PATCH v2 block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Jan Kara <jack@suse.cz> - 2016-03-01 10:30 +0100
    Re: [PATCH v2 block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Jens Axboe <axboe@kernel.dk> - 2016-03-01 18:50 +0100
      Re: [PATCH v2 block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Tejun Heo <tj@kernel.org> - 2016-03-01 19:00 +0100
        Re: [PATCH v2 block/for-linus] writeback: flush inode cgroup wb  switches instead of pinning super_block Jan Kara <jack@suse.cz> - 2016-03-02 11:30 +0100
  Re: [PATCH block/for-linus] writeback: flush inode cgroup wb switches  instead of pinning super_block Tahsin Erdogan <tahsin@google.com> - 2016-03-01 14:40 +0100

csiph-web