Return-Path: linux-nfs-owner@vger.kernel.org Received: from mx12.netapp.com ([216.240.18.77]:42432 "EHLO mx12.netapp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751533AbaEATw4 (ORCPT ); Thu, 1 May 2014 15:52:56 -0400 Message-ID: <5362A60C.7080805@netapp.com> Date: Thu, 1 May 2014 15:52:44 -0400 From: Anna Schumaker MIME-Version: 1.0 To: Weston Andros Adamson , CC: Subject: Re: [PATCH 0/3] nfs: get rid of nfs_pgio_data, related cleanup References: <1398971693-82399-1-git-send-email-dros@primarydata.com> In-Reply-To: <1398971693-82399-1-git-send-email-dros@primarydata.com> Content-Type: text/plain; charset="ISO-8859-1" Sender: linux-nfs-owner@vger.kernel.org List-ID: I like the idea, and I don't have a problem with these at a glance. I'll let you know if I hit anything whenever I get something set up for testing. On 05/01/2014 03:14 PM, Weston Andros Adamson wrote: > I'd like to know what people think of this proposed cleanup - it is to be > applied on top of the "pgio" branch at: > > git://git.linux-nfs.org/projects/dros/linux-nfs.git > > The basic idea is that we no longer need a separate nfs_pgio_data structure, > because the nfs_pgio_header structure always has one - and only one. > > I know this changes a lot of files, but I think now is the time to do this, > as we have to test everything anyways due to my subpage work and Anna's > pgio cleanup. > > I should note that there is little chance we'll ever go back to having more > than one nfs_pgio_data per nfs_pgio_header - since the nfs_page subpage > splitting work allows us to describe arbitrary ranges at that level. > > I think I can go a bit further by getting rid of the pgio_header refcnt. > I'll look closer at this soon... > > -dros > > Weston Andros Adamson (3): > nfs: move nfs_pgio_data and remove nfs_rw_header > nfs: rename members of nfs_pgio_data > nfs: merge nfs_pgio_data into _header > > fs/nfs/blocklayout/blocklayout.c | 99 +++++++++++---------- > fs/nfs/direct.c | 8 +- > fs/nfs/internal.h | 10 +-- > fs/nfs/nfs3proc.c | 21 +++-- > fs/nfs/nfs4_fs.h | 6 +- > fs/nfs/nfs4filelayout.c | 169 +++++++++++++++++------------------ > fs/nfs/nfs4proc.c | 105 +++++++++++----------- > fs/nfs/nfs4trace.h | 28 +++--- > fs/nfs/objlayout/objio_osd.c | 24 +++-- > fs/nfs/objlayout/objlayout.c | 81 ++++++++--------- > fs/nfs/objlayout/objlayout.h | 8 +- > fs/nfs/pageio.c | 186 ++++++++++++++++----------------------- > fs/nfs/pnfs.c | 96 ++++++++------------ > fs/nfs/pnfs.h | 10 +-- > fs/nfs/proc.c | 27 +++--- > fs/nfs/read.c | 48 +++++----- > fs/nfs/write.c | 65 +++++++------- > include/linux/nfs_fs.h | 2 +- > include/linux/nfs_page.h | 13 +-- > include/linux/nfs_xdr.h | 31 +++---- > 20 files changed, 489 insertions(+), 548 deletions(-) >