From: Fengguang Wu <fengguang.wu@intel.com>
Date: Wed, 10 Jul 2013 01:17:14 +0000 (+0800)
Subject: rpc_pipe: rpc_dir_inode_operations can be static
X-Git-Tag: v4.0~3678^2~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4f8568cb5290295c384d5c1328c52790e33a8a0d;p=platform%2Fkernel%2Flinux-amlogic.git

rpc_pipe: rpc_dir_inode_operations can be static

Hi Jeff,

FYI, there are new sparse warnings show up in

tree:   git://git.linux-nfs.org/projects/trondmy/linux-nfs.git nfs-for-next
head:   296afe1f58d55fd56ed85daaafafcfee39f59ece
commit: 76fa66657900071016f2bae61de28f059f3f2abf [2/5] rpc_pipe: set dentry operations at d_alloc time

>> net/sunrpc/rpc_pipe.c:496:31: sparse: symbol 'rpc_dir_inode_operations' was not declared. Should it be static?

Please consider folding the attached diff :-)

Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
---

diff --git a/net/sunrpc/rpc_pipe.c b/net/sunrpc/rpc_pipe.c
index c5f6812..61239a2 100644
--- a/net/sunrpc/rpc_pipe.c
+++ b/net/sunrpc/rpc_pipe.c
@@ -493,7 +493,7 @@ rpc_lookup(struct inode *dir, struct dentry *dentry, unsigned int flags)
 	return NULL;
 }
 
-const struct inode_operations rpc_dir_inode_operations = {
+static const struct inode_operations rpc_dir_inode_operations = {
 	.lookup		= rpc_lookup,
 };