Date: Fri, 19 Feb 2016 11:15:40 -0500
From: Brian Foster <bfoster@redhat.com>
To: linux-fsdevel@vger.kernel.org
Cc: Al Viro <viro@ZenIV.linux.org.uk>
Subject: Re: [PATCH v7 0/2] improve sync efficiency with sb inode wb list
Message-ID: <20160219161540.GB27263@bfoster.bfoster>
References: <1453496510-32485-1-git-send-email-bfoster@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1453496510-32485-1-git-send-email-bfoster@redhat.com>
Sender: linux-fsdevel-owner@vger.kernel.org

On Fri, Jan 22, 2016 at 04:01:48PM -0500, Brian Foster wrote:
> Hi all,
> 
> No code changes in this version... I've just updated the commit log
> description for patch 1 as requested by Jan and added the associated
> reviewed-by tags. Thanks.
> 
> Brian
> 

Ping. Al, any thoughts on this?

Brian

> v7:
> - Updated patch 1/2 commit log description to reference performance
>   impact.
> v6: http://marc.info/?l=linux-fsdevel&m=145322635828644&w=2
> - Use rcu locking instead of s_inode_list_lock spinlock in
>   wait_sb_inodes().
> - Refactor wait_sb_inodes() to keep inode on wb list.
> - Drop remaining, unnecessary lazy list removal bits and relocate inode
>   list check to clear_inode().
> - Fix up some comments, etc.
> v5: http://marc.info/?l=linux-fsdevel&m=145262374402798&w=2
> - Converted from per-bdi list to per-sb list. Also marked as RFC and
>   dropped testing/review tags.
> - Updated to use new irq-safe lock for wb list.
> - Dropped lazy list removal. Inodes are removed when the mapping is
>   cleared of the writeback tag.
> - Tweaked wait_sb_inodes() to remove deferred iput(), other cleanups.
> - Added wb list tracepoint patch.
> v4: http://marc.info/?l=linux-fsdevel&m=143511628828000&w=2
> 
> Brian Foster (1):
>   wb: inode writeback list tracking tracepoints
> 
> Dave Chinner (1):
>   sb: add a new writeback list for sync
> 
>  fs/fs-writeback.c                | 111 ++++++++++++++++++++++++++++++---------
>  fs/inode.c                       |   2 +
>  fs/super.c                       |   2 +
>  include/linux/fs.h               |   4 ++
>  include/linux/writeback.h        |   3 ++
>  include/trace/events/writeback.h |  22 ++++++--
>  mm/page-writeback.c              |  18 +++++++
>  7 files changed, 133 insertions(+), 29 deletions(-)
> 
> -- 
> 2.4.3
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html