login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2008
»
February
»
20
Re: [PATCH 1/2] nfsd: fix sparse warnings
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [thread] [
date
] [
author
]
[view in full thread]
From: J. Bruce Fields
Subject:
Re: [PATCH 1/2] nfsd: fix sparse warnings
Date: Wednesday, February 20, 2008 - 4:37 pm
On Wed, Feb 20, 2008 at 12:49:00PM -0800, Harvey Harrison wrote:
quoted text
> Add extern to nfsd/nfsd.h > fs/nfsd/nfssvc.c:146:5: warning: symbol 'nfsd_nrthreads' was not declared. Should it be static? > fs/nfsd/nfssvc.c:261:5: warning: symbol 'nfsd_nrpools' was not declared. Should it be static? > fs/nfsd/nfssvc.c:269:5: warning: symbol 'nfsd_get_nrthreads' was not declared. Should it be static? > fs/nfsd/nfssvc.c:281:5: warning: symbol 'nfsd_set_nrthreads' was not declared. Should it be static? > fs/nfsd/export.c:1534:23: warning: symbol 'nfs_exports_op' was not declared. Should it be static? > > Add include of auth.h > fs/nfsd/auth.c:27:5: warning: symbol 'nfsd_setuser' was not declared. Should it be static? > > Make static, move forward declaration closer to where it's needed. > fs/nfsd/nfs4state.c:1877:1: warning: symbol 'laundromat_main' was not declared. Should it be static? > > Make static, forward declaration was already marked static. > fs/nfsd/nfs4idmap.c:206:1: warning: symbol 'idtoname_parse' was not declared. Should it be static? > fs/nfsd/vfs.c:1156:1: warning: symbol 'nfsd_create_setattr' was not declared. Should it be static?
Applied, thanks. --b.
quoted text
> > Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com> > --- > fs/nfsd/auth.c | 1 + > fs/nfsd/nfs4idmap.c | 2 +- > fs/nfsd/nfs4state.c | 10 +++++----- > fs/nfsd/nfsctl.c | 7 ------- > fs/nfsd/vfs.c | 2 +- > include/linux/nfsd/nfsd.h | 8 ++++++++ > 6 files changed, 16 insertions(+), 14 deletions(-) > > diff --git a/fs/nfsd/auth.c b/fs/nfsd/auth.c > index d13403e..294992e 100644 > --- a/fs/nfsd/auth.c > +++ b/fs/nfsd/auth.c > @@ -10,6 +10,7 @@ > #include <linux/sunrpc/svcauth.h> > #include <linux/nfsd/nfsd.h> > #include <linux/nfsd/export.h> > +#include "auth.h" > > int nfsexp_flags(struct svc_rqst *rqstp, struct svc_export *exp) > { > diff --git a/fs/nfsd/nfs4idmap.c b/fs/nfsd/nfs4idmap.c > index 996bd88..5b39842 100644 > --- a/fs/nfsd/nfs4idmap.c > +++ b/fs/nfsd/nfs4idmap.c > @@ -202,7 +202,7 @@ static struct cache_detail idtoname_cache = { > .alloc = ent_alloc, > }; > > -int > +static int > idtoname_parse(struct cache_detail *cd, char *buf, int buflen) > { > struct ent ent, *res; > diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c > index bcb97d8..c7c92ae 100644 > --- a/fs/nfsd/nfs4state.c > +++ b/fs/nfsd/nfs4state.c > @@ -1762,10 +1762,6 @@ out: > return status; > } > > -static struct workqueue_struct *laundry_wq; > -static void laundromat_main(struct work_struct *); > -static DECLARE_DELAYED_WORK(laundromat_work, laundromat_main); > - > __be32 > nfsd4_renew(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, > clientid_t *clid) > @@ -1873,7 +1869,11 @@ nfs4_laundromat(void) > return clientid_val; > } > > -void > +static struct workqueue_struct *laundry_wq; > +static void laundromat_main(struct work_struct *); > +static DECLARE_DELAYED_WORK(laundromat_work, laundromat_main); > + > +static void > laundromat_main(struct work_struct *not_used) > { > time_t t; > diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c > index 8516137..73d3f28 100644 > --- a/fs/nfsd/nfsctl.c > +++ b/fs/nfsd/nfsctl.c > @@ -149,7 +149,6 @@ static const struct file_operations transaction_ops = { > .release = simple_transaction_release, > }; > > -extern struct seq_operations nfs_exports_op; > static int exports_open(struct inode *inode, struct file *file) > { > return seq_open(file, &nfs_exports_op); > @@ -347,8 +346,6 @@ static ssize_t write_filehandle(struct file *file, char *buf, size_t size) > return mesg - buf; > } > > -extern int nfsd_nrthreads(void); > - > static ssize_t write_threads(struct file *file, char *buf, size_t size) > { > /* if size > 0, look for a number of threads and call nfsd_svc > @@ -371,10 +368,6 @@ static ssize_t write_threads(struct file *file, char *buf, size_t size) > return strlen(buf); > } > > -extern int nfsd_nrpools(void); > -extern int nfsd_get_nrthreads(int n, int *); > -extern int nfsd_set_nrthreads(int n, int *); > - > static ssize_t write_pool_threads(struct file *file, char *buf, size_t size) > { > /* if size > 0, look for an array of number of threads per node > diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c > index 46f59d5..0265310 100644 > --- a/fs/nfsd/vfs.c > +++ b/fs/nfsd/vfs.c > @@ -1152,7 +1152,7 @@ nfsd_commit(struct svc_rqst *rqstp, struct svc_fh *fhp, > } > #endif /* CONFIG_NFSD_V3 */ > > -__be32 > +static __be32 > nfsd_create_setattr(struct svc_rqst *rqstp, struct svc_fh *resfhp, > struct iattr *iap) > { > diff --git a/include/linux/nfsd/nfsd.h b/include/linux/nfsd/nfsd.h > index 8caf4c4..f4de14d 100644 > --- a/include/linux/nfsd/nfsd.h > +++ b/include/linux/nfsd/nfsd.h > @@ -56,12 +56,20 @@ extern struct svc_program nfsd_program; > extern struct svc_version nfsd_version2, nfsd_version3, > nfsd_version4; > extern struct svc_serv *nfsd_serv; > + > +extern struct seq_operations nfs_exports_op; > + > /* > * Function prototypes. > */ > int nfsd_svc(unsigned short port, int nrservs); > int nfsd_dispatch(struct svc_rqst *rqstp, __be32 *statp); > > +int nfsd_nrthreads(void); > +int nfsd_nrpools(void); > +int nfsd_get_nrthreads(int n, int *); > +int nfsd_set_nrthreads(int n, int *); > + > /* nfsd/vfs.c */ > int fh_lock_parent(struct svc_fh *, struct dentry *); > int nfsd_racache_init(int); > -- > 1.5.4.2.200.g99e75 > >
--
unsubscribe notice
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/
Previous message: [
thread
] [
date
] [
author
]
Next message: [thread] [
date
] [
author
]
Messages in current thread:
[PATCH 1/2] nfsd: fix sparse warnings
, Harvey Harrison
, (Wed Feb 20, 1:49 pm)
Re: [PATCH 1/2] nfsd: fix sparse warnings
, J. Bruce Fields
, (Wed Feb 20, 4:37 pm)
Navigation
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Greg KH
Og dreams of kernels
jimmy bahuleyan
Re: how about mutual compatibility between Linux's GPLv2 and GPLv3?
Andrew Morton
Re: 2.6.26-rc2-mm1
James Bottomley
Re: INITIO scsi driver fails to work properly
Kamalesh Babulal
Re: [BUG] 2.6.25-rc5-mm1 kernel panic with "Exception: 501 " on powerpc
git-commits-head
:
Linux Kernel Mailing List
amba: struct device - replace bus_id with dev_name(), dev_set_name()
Linux Kernel Mailing List
cpumask: make irq_set_affinity() take a const struct cpumask
Linux Kernel Mailing List
[ARM] move include/asm-arm to arch/arm/include/asm
Linux Kernel Mailing List
FS-Cache: Implement the cookie management part of the netfs API
Linux Kernel Mailing List
x86: remove subarchitecture support
git
:
Junio C Hamano
Re: mingw, windows, crlf/lf, and git
Santi Béjar
Re: How to use git-fmt-merge-msg?
Johannes Sixt
Re: Trying to use git-filter-branch to compress history by removing large, obsol...
Nicolas Pitre
Re: [ANNOUNCE] Example Cogito Addon - cogito-bundle
Junio C Hamano
Re: [PATCH 1/2] Teach git-describe to display distances from tags.
freebsd-current
:
Michael Moll
Re: Unable to boot on ECS K7S5A
FreeBSD Tinderbox
[head tinderbox] failure on ia64/ia64
Jack Vogel
RFC: Capability addition for IEEE 1588
Julian Elischer
oh-oh .. 128 processors..
Stefan Lambrev
rtfree: 0xc741ee88 has 1 refs
linux-netdev
:
Arnaud Ebalard
Re: [REGRESSION,BISECTED] MIPv6 support broken by f4f914b58019f0
jamal
RE: [PATCH 1/4] [NET_SCHED] explict hold dev tx lock
Patrick Ohly
[RFC PATCH 04/13] net: implement generic SOF_TIMESTAMPING_TX_* support
Ben Hutchings
[PATCH 11/11] sfc: Replace stats_enabled flag with a disable count
Jesse Barnes
Re: [PATCH] pci: fix kexec with power state D3
Colocation donated by:
Syndicate