From: Helge Deller <deller@gmx.de>
To: Boaz Harrosh <bharrosh@panasas.com>
Cc: "J. Bruce Fields" <bfields@citi.umich.edu>,
linux-parisc <linux-parisc@vger.kernel.org>,
Grant Grundler <grundler@parisc-linux.org>,
Kyle McMartin <kyle@mcmartin.ca>,
"James E.J. Bottomley" <jejb@parisc-linux.org>,
Linux Kernel <linux-kernel@vger.kernel.org>,
sparclinux <sparclinux@vger.kernel.org>,
linux-s390 <linux-s390@vger.kernel.org>,
NFS list <linux-nfs@vger.kernel.org>,
pNFS Mailing List <pnfs@linux-nfs.org>,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 1/5] parsic: remove unused nfsd #includes
Date: Thu, 26 Nov 2009 22:47:53 +0100 [thread overview]
Message-ID: <4B0EF789.8060805@gmx.de> (raw)
In-Reply-To: <4B0E94D4.5090800@panasas.com>
On 11/26/2009 03:46 PM, Boaz Harrosh wrote:
> On 11/24/2009 11:27 PM, Grant Grundler wrote:
>> On Tue, Nov 24, 2009 at 07:58:15PM +0200, Boaz Harrosh wrote:
>>> Some arch files had unneeded #includes directives from linux/nfsd/*.
>>> Some vfs files had unneeded #includes directives from linux/nfsd/*.
>> ...
>>> patches:
>>> [PATCH 1/5] parsic: remove unused nfsd #includes
>>> Some kind soul, please compile with this patch? compile
>>> is all that's needed.
>>
>> Boaz,
>> linux-2.6 compiles for parisc with this patch. Please add:
>> Tested-by: Grant Grundler<grundler@parisc-linux.org>
>>
>> thanks,
>> grant
>>
>
> CCing registered parsic maintainers. Could we please get an ACK on
> this patch, and cary this through the nfsd tree?
Sure.
Acked-by: Helge Deller <deller@gmx.de>
Helge
>
> Thanks
> Boaz
>
> On 11/24/2009 07:59 PM, Boaz Harrosh wrote:
>>
>> Some unused includes removed.
>>
>> This patch is in an effort to cleanup nfsd headers and move
>> private definitions to source directory.
>>
>> Signed-off-by: Boaz Harrosh<bharrosh@panasas.com>
>> ---
>> arch/parisc/kernel/sys_parisc32.c | 6 ------
>> 1 files changed, 0 insertions(+), 6 deletions(-)
>>
>> diff --git a/arch/parisc/kernel/sys_parisc32.c b/arch/parisc/kernel/sys_parisc32.c
>> index 561388b..4a8ca63 100644
>> --- a/arch/parisc/kernel/sys_parisc32.c
>> +++ b/arch/parisc/kernel/sys_parisc32.c
>> @@ -26,13 +26,7 @@
>> #include<linux/shm.h>
>> #include<linux/slab.h>
>> #include<linux/uio.h>
>> -#include<linux/nfs_fs.h>
>> #include<linux/ncp_fs.h>
>> -#include<linux/sunrpc/svc.h>
>> -#include<linux/nfsd/nfsd.h>
>> -#include<linux/nfsd/cache.h>
>> -#include<linux/nfsd/xdr.h>
>> -#include<linux/nfsd/syscall.h>
>> #include<linux/poll.h>
>> #include<linux/personality.h>
>> #include<linux/stat.h>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
WARNING: multiple messages have this Message-ID (diff)
From: Helge Deller <deller@gmx.de>
To: Boaz Harrosh <bharrosh@panasas.com>
Cc: "J. Bruce Fields" <bfields@citi.umich.edu>,
linux-parisc <linux-parisc@vger.kernel.org>,
Grant Grundler <grundler@parisc-linux.org>,
Kyle McMartin <kyle@mcmartin.ca>,
"James E.J. Bottomley" <jejb@parisc-linux.org>,
Linux Kernel <linux-kernel@vger.kernel.org>,
sparclinux <sparclinux@vger.kernel.org>,
linux-s390 <linux-s390@vger.kernel.org>,
NFS list <linux-nfs@vger.kernel.org>,
pNFS Mailing List <pnfs@linux-nfs.org>,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 1/5] parsic: remove unused nfsd #includes
Date: Thu, 26 Nov 2009 21:47:53 +0000 [thread overview]
Message-ID: <4B0EF789.8060805@gmx.de> (raw)
In-Reply-To: <4B0E94D4.5090800@panasas.com>
On 11/26/2009 03:46 PM, Boaz Harrosh wrote:
> On 11/24/2009 11:27 PM, Grant Grundler wrote:
>> On Tue, Nov 24, 2009 at 07:58:15PM +0200, Boaz Harrosh wrote:
>>> Some arch files had unneeded #includes directives from linux/nfsd/*.
>>> Some vfs files had unneeded #includes directives from linux/nfsd/*.
>> ...
>>> patches:
>>> [PATCH 1/5] parsic: remove unused nfsd #includes
>>> Some kind soul, please compile with this patch? compile
>>> is all that's needed.
>>
>> Boaz,
>> linux-2.6 compiles for parisc with this patch. Please add:
>> Tested-by: Grant Grundler<grundler@parisc-linux.org>
>>
>> thanks,
>> grant
>>
>
> CCing registered parsic maintainers. Could we please get an ACK on
> this patch, and cary this through the nfsd tree?
Sure.
Acked-by: Helge Deller <deller@gmx.de>
Helge
>
> Thanks
> Boaz
>
> On 11/24/2009 07:59 PM, Boaz Harrosh wrote:
>>
>> Some unused includes removed.
>>
>> This patch is in an effort to cleanup nfsd headers and move
>> private definitions to source directory.
>>
>> Signed-off-by: Boaz Harrosh<bharrosh@panasas.com>
>> ---
>> arch/parisc/kernel/sys_parisc32.c | 6 ------
>> 1 files changed, 0 insertions(+), 6 deletions(-)
>>
>> diff --git a/arch/parisc/kernel/sys_parisc32.c b/arch/parisc/kernel/sys_parisc32.c
>> index 561388b..4a8ca63 100644
>> --- a/arch/parisc/kernel/sys_parisc32.c
>> +++ b/arch/parisc/kernel/sys_parisc32.c
>> @@ -26,13 +26,7 @@
>> #include<linux/shm.h>
>> #include<linux/slab.h>
>> #include<linux/uio.h>
>> -#include<linux/nfs_fs.h>
>> #include<linux/ncp_fs.h>
>> -#include<linux/sunrpc/svc.h>
>> -#include<linux/nfsd/nfsd.h>
>> -#include<linux/nfsd/cache.h>
>> -#include<linux/nfsd/xdr.h>
>> -#include<linux/nfsd/syscall.h>
>> #include<linux/poll.h>
>> #include<linux/personality.h>
>> #include<linux/stat.h>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
next prev parent reply other threads:[~2009-11-26 21:47 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-24 17:58 [PATCHSET 0-5] remove unneeded nfsd #includes Boaz Harrosh
2009-11-24 17:58 ` Boaz Harrosh
2009-11-24 17:58 ` Boaz Harrosh
2009-11-24 17:58 ` Boaz Harrosh
2009-11-24 17:59 ` [PATCH 1/5] parsic: remove unused " Boaz Harrosh
2009-11-24 17:59 ` Boaz Harrosh
2009-11-24 17:59 ` Boaz Harrosh
2009-11-24 17:59 ` Boaz Harrosh
2009-11-26 14:46 ` Boaz Harrosh
2009-11-26 14:46 ` Boaz Harrosh
2009-11-26 14:46 ` Boaz Harrosh
2009-11-26 14:46 ` Boaz Harrosh
2009-11-26 21:47 ` Helge Deller [this message]
2009-11-26 21:47 ` Helge Deller
2009-11-24 18:01 ` [PATCH 2/5] sparc: " Boaz Harrosh
2009-11-24 18:01 ` Boaz Harrosh
2009-11-24 18:01 ` Boaz Harrosh
2009-11-24 18:01 ` Boaz Harrosh
2009-11-25 23:19 ` David Miller
2009-11-25 23:19 ` David Miller
2009-11-24 18:02 ` [PATCH 3/5] s390: " Boaz Harrosh
2009-11-24 18:02 ` Boaz Harrosh
2009-11-24 18:02 ` Boaz Harrosh
2009-11-24 18:02 ` Boaz Harrosh
2009-11-25 8:38 ` Martin Schwidefsky
2009-11-25 8:38 ` Martin Schwidefsky
2009-11-24 18:03 ` [PATCH 4/5] lockd: Remove unused nfsd headers #includes Boaz Harrosh
2009-11-24 18:03 ` Boaz Harrosh
2009-11-24 18:03 ` Boaz Harrosh
2009-11-24 18:03 ` Boaz Harrosh
2009-11-25 20:10 ` J. Bruce Fields
2009-11-25 20:10 ` J. Bruce Fields
2009-11-25 20:10 ` J. Bruce Fields
2009-11-25 20:23 ` J. Bruce Fields
2009-11-25 20:23 ` J. Bruce Fields
2009-11-25 20:23 ` J. Bruce Fields
2009-11-26 12:39 ` Boaz Harrosh
2009-11-26 12:39 ` Boaz Harrosh
2009-12-03 17:27 ` J. Bruce Fields
2009-12-03 17:27 ` J. Bruce Fields
2009-12-03 17:27 ` J. Bruce Fields
2009-11-24 18:04 ` [PATCH 5/5] vfs: nfsctl.c unused nfsd #includes Boaz Harrosh
2009-11-24 18:04 ` Boaz Harrosh
2009-11-24 18:04 ` Boaz Harrosh
2009-11-25 20:13 ` J. Bruce Fields
2009-11-25 20:13 ` J. Bruce Fields
2009-11-25 20:13 ` J. Bruce Fields
2009-11-24 18:04 ` Boaz Harrosh
2009-11-24 18:05 ` [RFC] nfsd: Move private headers to source directory Boaz Harrosh
2009-11-24 18:05 ` Boaz Harrosh
2009-11-24 18:05 ` Boaz Harrosh
2009-11-24 18:05 ` Boaz Harrosh
2009-11-25 20:14 ` J. Bruce Fields
2009-11-25 20:14 ` J. Bruce Fields
2009-11-25 20:14 ` J. Bruce Fields
2009-11-25 20:14 ` J. Bruce Fields
2009-11-24 21:27 ` [PATCHSET 0-5] remove unneeded nfsd #includes Grant Grundler
2009-11-24 21:27 ` Grant Grundler
2009-11-24 21:27 ` Grant Grundler
2009-11-24 21:27 ` Grant Grundler
2009-11-24 23:08 ` J. Bruce Fields
2009-11-24 23:08 ` J. Bruce Fields
2009-11-25 8:42 ` Boaz Harrosh
2009-11-25 8:42 ` Boaz Harrosh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B0EF789.8060805@gmx.de \
--to=deller@gmx.de \
--cc=akpm@linux-foundation.org \
--cc=bfields@citi.umich.edu \
--cc=bharrosh@panasas.com \
--cc=grundler@parisc-linux.org \
--cc=jejb@parisc-linux.org \
--cc=kyle@mcmartin.ca \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-nfs@vger.kernel.org \
--cc=linux-parisc@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=pnfs@linux-nfs.org \
--cc=sparclinux@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.