OpenVZ Forum: Devel » [PATCH 0/5] nfs: few cleanup patches inspired by sparse

Home » Mailing lists » Devel » [PATCH 0/5] nfs: few cleanup patches inspired by sparse

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

[PATCH 0/5] nfs: few cleanup patches inspired by sparse [message #48185]

Tue, 02 October 2012 10:18

Stanislav Kinsbursky is currently offline

Stanislav Kinsbursky
Messages: 683
Registered: October 2011

Senior Member

First two patched fixes problems reported by Fengguang Wu
<fengguang.wu@intel.com>.

The following series implements...

---

Stanislav Kinsbursky (5):
nfs: include NFSv4 header in netns.h
nfs: declare nfs_callback_tcp_port in header
nfs: declare nfs_xdev_mount as static
nfs: include nfs4_fh.h in nfs4sysctl.c
nfs: include internah.h in getroot.h

fs/nfs/callback.h | 1 +
fs/nfs/getroot.c | 2 ++
fs/nfs/netns.h | 1 +
fs/nfs/nfs4sysctl.c | 1 +
fs/nfs/super.c | 2 +-
5 files changed, 6 insertions(+), 1 deletions(-)

Report message to a moderator

[PATCH 1/5] nfs: include NFSv4 header in netns.h [message #48186 is a reply to message #48185]

Tue, 02 October 2012 10:18

Stanislav Kinsbursky
Messages: 683
Registered: October 2011

Senior Member

Build error:
fs/nfs/netns.h:27:15: error: 'NFS4_MAX_MINOR_VERSION' undeclared here (not in
a function)

Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
---
fs/nfs/netns.h | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/fs/nfs/netns.h b/fs/nfs/netns.h
index b9c7f9b..8ee1fab 100644
--- a/fs/nfs/netns.h
+++ b/fs/nfs/netns.h
@@ -5,6 +5,7 @@
#ifndef __NFS_NETNS_H__
#define __NFS_NETNS_H__

+#include <linux/nfs4.h>
#include <net/net_namespace.h>
#include <net/netns/generic.h>

Report message to a moderator

[PATCH 2/5] nfs: declare nfs_callback_tcp_port in header [message #48187 is a reply to message #48185]

Tue, 02 October 2012 10:18

Stanislav Kinsbursky
Messages: 683
Registered: October 2011

Senior Member

Sparse warning:
fs/nfs/super.c:2638:16: sparse: symbol 'nfs_callback_tcpport' was not
declared. Should it be static?

Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
---
fs/nfs/callback.h | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/fs/nfs/callback.h b/fs/nfs/callback.h
index c07a8d4..4251c2a 100644
--- a/fs/nfs/callback.h
+++ b/fs/nfs/callback.h
@@ -208,5 +208,6 @@ extern int nfs4_set_callback_sessionid(struct nfs_client *clp);
#define NFS41_BC_MAX_CALLBACKS 1

extern unsigned int nfs_callback_set_tcpport;
+extern unsigned short nfs_callback_tcpport;

#endif /* __LINUX_FS_NFS_CALLBACK_H */

Report message to a moderator

[PATCH 3/5] nfs: declare nfs_xdev_mount as static [message #48188 is a reply to message #48185]

Tue, 02 October 2012 10:19

Stanislav Kinsbursky
Messages: 683
Registered: October 2011

Senior Member

Sparse warning:
fs/nfs/super.c:2517:15: warning: symbol 'nfs_xdev_mount' was not declared.
Should it be static?

Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
---
fs/nfs/super.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/fs/nfs/super.c b/fs/nfs/super.c
index a719bc0..b4079bb 100644
--- a/fs/nfs/super.c
+++ b/fs/nfs/super.c
@@ -2514,7 +2514,7 @@ EXPORT_SYMBOL_GPL(nfs_kill_super);
/*
* Clone an NFS2/3/4 server record on xdev traversal (FSID-change)
*/
-struct dentry *
+static struct dentry *
nfs_xdev_mount(struct file_system_type *fs_type, int flags,
const char *dev_name, void *raw_data)
{

Report message to a moderator

[PATCH 4/5] nfs: include nfs4_fh.h in nfs4sysctl.c [message #48189 is a reply to message #48185]

Tue, 02 October 2012 10:19

Stanislav Kinsbursky
Messages: 683
Registered: October 2011

Senior Member

Sparse warnings:
fs/nfs/nfs4sysctl.c:56:5: warning: symbol 'nfs4_register_sysctl' was not
declared. Should it be static?
fs/nfs/nfs4sysctl.c:64:6: warning: symbol 'nfs4_unregister_sysctl' was not
declared. Should it be static?

Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
---
fs/nfs/nfs4sysctl.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/fs/nfs/nfs4sysctl.c b/fs/nfs/nfs4sysctl.c
index 5729bc8..2628d92 100644
--- a/fs/nfs/nfs4sysctl.c
+++ b/fs/nfs/nfs4sysctl.c
@@ -9,6 +9,7 @@
#include <linux/nfs_idmap.h>
#include <linux/nfs_fs.h>

+#include "nfs4_fs.h"
#include "callback.h"

static const int nfs_set_port_min = 0;

Report message to a moderator

[PATCH 5/5] nfs: include internah.h in getroot.h [message #48190 is a reply to message #48185]

Tue, 02 October 2012 10:19

Stanislav Kinsbursky
Messages: 683
Registered: October 2011

Senior Member

Sparse warning:
fs/nfs/nfs4getroot.c:11:5: warning: symbol 'nfs4_get_rootfh' was not declared.
Should it be static?

Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
---
fs/nfs/getroot.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/fs/nfs/getroot.c b/fs/nfs/getroot.c
index 4654ced..033803c 100644
--- a/fs/nfs/getroot.c
+++ b/fs/nfs/getroot.c
@@ -32,6 +32,8 @@

#include <asm/uaccess.h>

+#include "internal.h"
+
#define NFSDBG_FACILITY NFSDBG_CLIENT

/*

Report message to a moderator

Previous Topic:	[RFC 0/4] bypass charges if memcg is not used
Next Topic:	[PATCH v3 00/16] slab accounting for memcg

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Mon Feb 16 04:58:08 GMT 2026

Total time taken to generate the page: 0.31332 seconds