diff options
author | Ryusei Yamaguchi <mandel59@gmail.com> | 2009-08-16 00:54:41 +0900 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2009-08-25 12:39:37 -0400 |
commit | ed2d8aed52212610d4cb79be3cbf535b04be38dc (patch) | |
tree | 6baac65cfa094ac79db8a473ad7bd4b1dbbc5c2a /fs/nfsd/nfsctl.c | |
parent | 55bb55dca0cecac2fb7b8c743db41361c011c8a8 (diff) | |
download | op-kernel-dev-ed2d8aed52212610d4cb79be3cbf535b04be38dc.zip op-kernel-dev-ed2d8aed52212610d4cb79be3cbf535b04be38dc.tar.gz |
knfsd: Replace lock_kernel with a mutex in nfsd pool stats.
lock_kernel() in knfsd was replaced with a mutex. The later
commit 03cf6c9f49a8fea953d38648d016e3f46e814991 ("knfsd:
add file to export stats about nfsd pools") did not follow
that change. This patch fixes the issue.
Also move the get and put of nfsd_serv to the open and close methods
(instead of start and stop methods) to allow atomic check and increment
of reference count in the open method (where we can still return an
error).
Signed-off-by: Ryusei Yamaguchi <mandel59@gmail.com>
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
Cc: Greg Banks <gnb@fmeh.org>
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'fs/nfsd/nfsctl.c')
-rw-r--r-- | fs/nfsd/nfsctl.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c index b764d7d..00388d2 100644 --- a/fs/nfsd/nfsctl.c +++ b/fs/nfsd/nfsctl.c @@ -174,12 +174,13 @@ static const struct file_operations exports_operations = { }; extern int nfsd_pool_stats_open(struct inode *inode, struct file *file); +extern int nfsd_pool_stats_release(struct inode *inode, struct file *file); static struct file_operations pool_stats_operations = { .open = nfsd_pool_stats_open, .read = seq_read, .llseek = seq_lseek, - .release = seq_release, + .release = nfsd_pool_stats_release, .owner = THIS_MODULE, }; |