From: Boaz Harrosh Subject: Re: [pnfs] [PATCH 5/6] nfsd: Source files #include cleanups Date: Mon, 23 Nov 2009 19:26:10 +0200 Message-ID: <4B0AC5B2.3030703@panasas.com> References: <4B0AB039.6020608@panasas.com> <1258991991-25446-1-git-send-email-bharrosh@panasas.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: linux-fsdevel , Randy Dunlap , Al Viro To: "J. Bruce Fields" , Benny Halevy , NFS list , pNFS Mailing List , Trond Myklebust Received: from daytona.panasas.com ([67.152.220.89]:9127 "EHLO daytona.int.panasas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751865AbZKWR0I (ORCPT ); Mon, 23 Nov 2009 12:26:08 -0500 In-Reply-To: <1258991991-25446-1-git-send-email-bharrosh@panasas.com> Sender: linux-nfs-owner@vger.kernel.org List-ID: On 11/23/2009 05:59 PM, Boaz Harrosh wrote: > Now that the headers are fixed and carry their own wait, all fs/nfsd/ > source files can include a minimal set of headers. and still compile just > fine. > > This patch should improve the compilation speed of the nfsd module. > > Signed-off-by: Boaz Harrosh > --- > fs/nfsd/auth.c | 4 ---- > fs/nfsd/export.c | 15 --------------- > fs/nfsd/lockd.c | 5 ----- > fs/nfsd/nfs2acl.c | 4 ---- > fs/nfsd/nfs3acl.c | 3 --- > fs/nfsd/nfs3proc.c | 13 ------------- > fs/nfsd/nfs3xdr.c | 11 ----------- > fs/nfsd/nfs4acl.c | 8 -------- > fs/nfsd/nfs4callback.c | 13 ------------- > fs/nfsd/nfs4idmap.c | 15 +-------------- > fs/nfsd/nfs4proc.c | 10 ---------- > fs/nfsd/nfs4recover.c | 8 -------- > fs/nfsd/nfs4state.c | 15 --------------- > fs/nfsd/nfs4xdr.c | 12 +----------- > fs/nfsd/nfscache.c | 8 -------- > fs/nfsd/nfsctl.c | 25 ------------------------- > fs/nfsd/nfsfh.c | 9 --------- > fs/nfsd/nfsproc.c | 13 ------------- > fs/nfsd/nfssvc.c | 15 --------------- > fs/nfsd/nfsxdr.c | 8 -------- > fs/nfsd/stats.c | 5 ----- > fs/nfsd/vfs.c | 21 --------------------- > 22 files changed, 2 insertions(+), 238 deletions(-) > > diff --git a/fs/nfsd/auth.c b/fs/nfsd/auth.c > index 36fcabb..ad354d2 100644 > --- a/fs/nfsd/auth.c > +++ b/fs/nfsd/auth.c > @@ -4,12 +4,8 @@ > * Copyright (C) 1995, 1996 Olaf Kirch > */ > > -#include > #include > -#include > -#include > #include > -#include > #include "auth.h" > > int nfsexp_flags(struct svc_rqst *rqstp, struct svc_export *exp) > diff --git a/fs/nfsd/export.c b/fs/nfsd/export.c > index b73baba..68e63f4 100644 > --- a/fs/nfsd/export.c > +++ b/fs/nfsd/export.c > @@ -14,27 +14,12 @@ > * Copyright (C) 1995, 1996 Olaf Kirch, > */ > > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > -#include > #include > #include > > -#include > #include > -#include > #include > -#include > -#include > -#include > #include > > #define NFSDDBG_FACILITY NFSDDBG_EXPORT > diff --git a/fs/nfsd/lockd.c b/fs/nfsd/lockd.c > index b2786a5..19a16e1 100644 > --- a/fs/nfsd/lockd.c > +++ b/fs/nfsd/lockd.c > @@ -8,12 +8,7 @@ > * Copyright (C) 1996, Olaf Kirch > */ > > -#include > -#include > #include > -#include > -#include > -#include > #include > #include > > diff --git a/fs/nfsd/nfs2acl.c b/fs/nfsd/nfs2acl.c > index e2a17f0..64c368b 100644 > --- a/fs/nfsd/nfs2acl.c > +++ b/fs/nfsd/nfs2acl.c > @@ -6,13 +6,9 @@ > * Copyright (C) 2002-2003 Andreas Gruenbacher > */ > > -#include > -#include > #include > #include > -#include > #include > -#include > #include > > #define NFSDDBG_FACILITY NFSDDBG_PROC > diff --git a/fs/nfsd/nfs3acl.c b/fs/nfsd/nfs3acl.c > index ff73596..8e44949 100644 > --- a/fs/nfsd/nfs3acl.c > +++ b/fs/nfsd/nfs3acl.c > @@ -6,12 +6,9 @@ > * Copyright (C) 2002-2003 Andreas Gruenbacher > */ > > -#include > -#include > #include > #include > #include > -#include > #include > > #define RETURN_STATUS(st) { resp->status = (st); return (st); } > diff --git a/fs/nfsd/nfs3proc.c b/fs/nfsd/nfs3proc.c > index a713c41..ed6112e 100644 > --- a/fs/nfsd/nfs3proc.c > +++ b/fs/nfsd/nfs3proc.c > @@ -6,25 +6,12 @@ > * Copyright (C) 1996, 1997, 1998 Olaf Kirch > */ > > -#include > -#include > -#include > #include > #include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > #include > > -#include > -#include > #include > #include > -#include > > #define NFSDDBG_FACILITY NFSDDBG_PROC > > diff --git a/fs/nfsd/nfs3xdr.c b/fs/nfsd/nfs3xdr.c > index edf926e..97b3619 100644 > --- a/fs/nfsd/nfs3xdr.c > +++ b/fs/nfsd/nfs3xdr.c > @@ -8,18 +8,7 @@ > * 2003-08-09 Jamie Lokier: Use htonl() for nanoseconds, not htons()! > */ > > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > -#include > -#include > -#include > -#include > #include > #include "auth.h" > > diff --git a/fs/nfsd/nfs4acl.c b/fs/nfsd/nfs4acl.c > index 6d9c6aa..d6f0cea 100644 > --- a/fs/nfsd/nfs4acl.c > +++ b/fs/nfsd/nfs4acl.c > @@ -36,15 +36,7 @@ > * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > */ > > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > -#include > #include > > > diff --git a/fs/nfsd/nfs4callback.c b/fs/nfsd/nfs4callback.c > index 24e8d78..4fe3960 100644 > --- a/fs/nfsd/nfs4callback.c > +++ b/fs/nfsd/nfs4callback.c > @@ -33,22 +33,9 @@ > * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > */ > > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > #include > #include > -#include > -#include > -#include > > #define NFSDDBG_FACILITY NFSDDBG_PROC > > diff --git a/fs/nfsd/nfs4idmap.c b/fs/nfsd/nfs4idmap.c > index ba2c199..8e518cd 100644 > --- a/fs/nfsd/nfs4idmap.c > +++ b/fs/nfsd/nfs4idmap.c > @@ -35,22 +35,9 @@ > */ > > #include > -#include > - > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > -#include > #include > -#include > +#include > > /* > * Cache entry > diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c > index bebc0c2..e3c1e6d 100644 > --- a/fs/nfsd/nfs4proc.c > +++ b/fs/nfsd/nfs4proc.c > @@ -34,20 +34,10 @@ > * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS > * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > */ > - > -#include > -#include > -#include > #include > > -#include > -#include > #include > -#include > -#include > #include > -#include > -#include > > #define NFSDDBG_FACILITY NFSDDBG_PROC > > diff --git a/fs/nfsd/nfs4recover.c b/fs/nfsd/nfs4recover.c > index b534840..be9e712 100644 > --- a/fs/nfsd/nfs4recover.c > +++ b/fs/nfsd/nfs4recover.c > @@ -33,20 +33,12 @@ > * > */ > > -#include > -#include > #include > -#include > #include > -#include > -#include > #include > #include > -#include > -#include > #include > #include > -#include > > #define NFSDDBG_FACILITY NFSDDBG_PROC > > diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c > index c8b621a..93f801c 100644 > --- a/fs/nfsd/nfs4state.c > +++ b/fs/nfsd/nfs4state.c > @@ -34,26 +34,11 @@ > * > */ > > -#include > -#include > -#include > - > -#include > -#include > -#include > #include > -#include > -#include > #include > -#include > -#include > -#include > #include > #include > #include > -#include > -#include > -#include > #include > #include > > diff --git a/fs/nfsd/nfs4xdr.c b/fs/nfsd/nfs4xdr.c > index 0fbd50c..444eeca 100644 > --- a/fs/nfsd/nfs4xdr.c > +++ b/fs/nfsd/nfs4xdr.c > @@ -40,22 +40,12 @@ > * at the end of nfs4svc_decode_compoundargs. > */ > > -#include > -#include > -#include > #include > -#include > +#include > #include > -#include > -#include > -#include > -#include > -#include > #include > #include > -#include > #include > -#include > #include > > #define NFSDDBG_FACILITY NFSDDBG_XDR > diff --git a/fs/nfsd/nfscache.c b/fs/nfsd/nfscache.c > index 4638635..96694b8 100644 > --- a/fs/nfsd/nfscache.c > +++ b/fs/nfsd/nfscache.c > @@ -10,14 +10,6 @@ > * Copyright (C) 1995, 1996 Olaf Kirch > */ > > -#include > -#include > -#include > -#include > -#include > -#include > - > -#include > #include > #include > > diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c > index 5c01fc1..e4f49fd 100644 > --- a/fs/nfsd/nfsctl.c > +++ b/fs/nfsd/nfsctl.c > @@ -6,42 +6,17 @@ > * Copyright (C) 1995, 1996 Olaf Kirch > */ > > -#include > - > -#include > -#include > -#include > -#include > #include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > #include > > -#include > #include > -#include > -#include > #include > #include > #include > -#include > #include > #include > #include > > -#include > -#include > - > /* > * We have a single directory with 9 nodes in it. > */ > diff --git a/fs/nfsd/nfsfh.c b/fs/nfsd/nfsfh.c > index 01965b2..01f142f 100644 > --- a/fs/nfsd/nfsfh.c > +++ b/fs/nfsd/nfsfh.c > @@ -9,17 +9,8 @@ > * ... and again Southern-Winter 2001 to support export_operations > */ > > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > > -#include > -#include > #include > #include > #include "auth.h" > diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c > index c5393d1..2307693 100644 > --- a/fs/nfsd/nfsproc.c > +++ b/fs/nfsd/nfsproc.c > @@ -7,21 +7,8 @@ > * Copyright (C) 1995-1997 Olaf Kirch > */ > > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > -#include > > -#include > -#include > -#include > #include > #include > > diff --git a/fs/nfsd/nfssvc.c b/fs/nfsd/nfssvc.c > index 67ea83e..ab4b321 100644 > --- a/fs/nfsd/nfssvc.c > +++ b/fs/nfsd/nfssvc.c > @@ -8,30 +8,15 @@ > * Copyright (C) 1995, 1996, 1997 Olaf Kirch > */ > > -#include > #include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > -#include > #include > #include > -#include > #include > > -#include > #include > -#include > #include > -#include > #include > -#include > #include > -#include > #include > #include > #include > diff --git a/fs/nfsd/nfsxdr.c b/fs/nfsd/nfsxdr.c > index afd08e2..5e0603d 100644 > --- a/fs/nfsd/nfsxdr.c > +++ b/fs/nfsd/nfsxdr.c > @@ -6,15 +6,7 @@ > * Copyright (C) 1995, 1996 Olaf Kirch > */ > > -#include > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > #include "auth.h" > > #define NFSDDBG_FACILITY NFSDDBG_XDR > diff --git a/fs/nfsd/stats.c b/fs/nfsd/stats.c > index 71944cd..e3e411e 100644 > --- a/fs/nfsd/stats.c > +++ b/fs/nfsd/stats.c > @@ -23,14 +23,9 @@ > * Copyright (C) 1995, 1996, 1997 Olaf Kirch > */ > > -#include > -#include > -#include > #include > -#include > #include > > -#include > #include > #include > #include > diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c > index 6385739..a899429 100644 > --- a/fs/nfsd/vfs.c > +++ b/fs/nfsd/vfs.c > @@ -16,43 +16,22 @@ > * Zerocpy NFS support (C) 2002 Hirokazu Takahashi > */ > > -#include > -#include > -#include > #include > #include > -#include > -#include > #include > -#include > -#include > #include > -#include > -#include > -#include > -#include > -#include > -#include > #include > -#include > -#include This one is needed on some arches. This patch will have to sit in linux-next for a while and collect some breakage. I'll monitor it's progress. I'm sending a ver 2 patch. Boaz > -#include > #include > #ifdef CONFIG_NFSD_V3 > -#include > #include > #endif /* CONFIG_NFSD_V3 */ > -#include > #include > #include > -#include > #include > #include > #ifdef CONFIG_NFSD_V4 > -#include > #include > #include > -#include > #endif /* CONFIG_NFSD_V4 */ > #include > #include