sunrpc: move pc_count out of struct svc_procinfo

pc_count is the only writeable memeber of struct svc_procinfo, which is
a good candidate to be const-ified as it contains function pointers.

This patch moves it into out out struct svc_procinfo, and into a
separate writable array that is pointed to by struct svc_version.

Signed-off-by: Christoph Hellwig <hch@lst.de>

10 files changed, 28 insertions, 7 deletions

diff --git a/fs/lockd/svc.c b/fs/lockd/svc.c
index 5d481e8a1b5d..cc6abe6280bc 100644
--- a/fs/lockd/svc.c
+++ b/fs/lockd/svc.c
@@ -739,23 +739,29 @@ module_exit(exit_nlm);
 /*
  * Define NLM program and procedures
  */
+static unsigned int nlmsvc_version1_count[17];
 static struct svc_version       nlmsvc_version1 = {
                 .vs_vers        = 1,
                 .vs_nproc       = 17,
                 .vs_proc        = nlmsvc_procedures,
+                .vs_count       = nlmsvc_version1_count,
                 .vs_xdrsize     = NLMSVC_XDRSIZE,
 };
+static unsigned int nlmsvc_version3_count[24];
 static struct svc_version       nlmsvc_version3 = {
                 .vs_vers        = 3,
                 .vs_nproc       = 24,
                 .vs_proc        = nlmsvc_procedures,
+                .vs_count       = nlmsvc_version3_count,
                 .vs_xdrsize     = NLMSVC_XDRSIZE,
 };
 #ifdef CONFIG_LOCKD_V4
+static unsigned int nlmsvc_version4_count[24];
 static struct svc_version       nlmsvc_version4 = {
                 .vs_vers        = 4,
                 .vs_nproc       = 24,
                 .vs_proc        = nlmsvc_procedures4,
+                .vs_count       = nlmsvc_version4_count,
                 .vs_xdrsize     = NLMSVC_XDRSIZE,
 };
 #endif
diff --git a/fs/nfs/callback_xdr.c b/fs/nfs/callback_xdr.c
index acf75dc63e14..ecd46b8c0985 100644
--- a/fs/nfs/callback_xdr.c
+++ b/fs/nfs/callback_xdr.c
@@ -1011,20 +1011,24 @@ static struct svc_procedure nfs4_callback_procedures1[] = {
         }
 };
 
+static unsigned int nfs4_callback_count1[ARRAY_SIZE(nfs4_callback_procedures1)];
 struct svc_version nfs4_callback_version1 = {
         .vs_vers = 1,
         .vs_nproc = ARRAY_SIZE(nfs4_callback_procedures1),
         .vs_proc = nfs4_callback_procedures1,
+        .vs_count = nfs4_callback_count1,
         .vs_xdrsize = NFS4_CALLBACK_XDRSIZE,
         .vs_dispatch = NULL,
         .vs_hidden = true,
         .vs_need_cong_ctrl = true,
 };
 
+static unsigned int nfs4_callback_count4[ARRAY_SIZE(nfs4_callback_procedures1)];
 struct svc_version nfs4_callback_version4 = {
         .vs_vers = 4,
         .vs_nproc = ARRAY_SIZE(nfs4_callback_procedures1),
         .vs_proc = nfs4_callback_procedures1,
+        .vs_count = nfs4_callback_count4,
         .vs_xdrsize = NFS4_CALLBACK_XDRSIZE,
         .vs_dispatch = NULL,
         .vs_hidden = true,
diff --git a/fs/nfsd/nfs2acl.c b/fs/nfsd/nfs2acl.c
index fc6b179c8fff..026edfe73fd5 100644
--- a/fs/nfsd/nfs2acl.c
+++ b/fs/nfsd/nfs2acl.c
@@ -378,10 +378,12 @@ static struct svc_procedure		nfsd_acl_procedures2[] = {
   PROC(access,  access,         access,         access,   RC_NOCACHE, ST+AT+1),
 };
 
+static unsigned int nfsd_acl_count2[ARRAY_SIZE(nfsd_acl_procedures2)];
 struct svc_version      nfsd_acl_version2 = {
                 .vs_vers        = 2,
                 .vs_nproc       = 5,
                 .vs_proc        = nfsd_acl_procedures2,
+                .vs_count       = nfsd_acl_count2,
                 .vs_dispatch    = nfsd_dispatch,
                 .vs_xdrsize     = NFS3_SVC_XDRSIZE,
 };
diff --git a/fs/nfsd/nfs3acl.c b/fs/nfsd/nfs3acl.c
index 9437b758cbfd..73c0970ccefb 100644
--- a/fs/nfsd/nfs3acl.c
+++ b/fs/nfsd/nfs3acl.c
@@ -263,10 +263,12 @@ static struct svc_procedure		nfsd_acl_procedures3[] = {
   PROC(setacl,  setacl,         setacl,         fhandle,  RC_NOCACHE, ST+pAT),
 };
 
+static unsigned int nfsd_acl_count3[ARRAY_SIZE(nfsd_acl_procedures3)];
 struct svc_version      nfsd_acl_version3 = {
                 .vs_vers        = 3,
                 .vs_nproc       = 3,
                 .vs_proc        = nfsd_acl_procedures3,
+                .vs_count       = nfsd_acl_count3,
                 .vs_dispatch    = nfsd_dispatch,
                 .vs_xdrsize     = NFS3_SVC_XDRSIZE,
 };
diff --git a/fs/nfsd/nfs3proc.c b/fs/nfsd/nfs3proc.c
index 17c90c41a3a6..b5823802e278 100644
--- a/fs/nfsd/nfs3proc.c
+++ b/fs/nfsd/nfs3proc.c
@@ -890,10 +890,12 @@ static struct svc_procedure		nfsd_procedures3[22] = {
         },
 };
 
+static unsigned int nfsd_count3[ARRAY_SIZE(nfsd_procedures3)];
 struct svc_version      nfsd_version3 = {
                 .vs_vers        = 3,
                 .vs_nproc       = 22,
                 .vs_proc        = nfsd_procedures3,
+                .vs_count       = nfsd_count3,
                 .vs_dispatch    = nfsd_dispatch,
                 .vs_xdrsize     = NFS3_SVC_XDRSIZE,
 };
diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
index a947dcef5e4e..bad5fec0ebc7 100644
--- a/fs/nfsd/nfs4proc.c
+++ b/fs/nfsd/nfs4proc.c
@@ -2554,10 +2554,12 @@ static struct svc_procedure		nfsd_procedures4[2] = {
         },
 };
 
+static unsigned int nfsd_count3[ARRAY_SIZE(nfsd_procedures4)];
 struct svc_version      nfsd_version4 = {
         .vs_vers                = 4,
         .vs_nproc               = 2,
         .vs_proc                = nfsd_procedures4,
+        .vs_count               = nfsd_count3,
         .vs_dispatch            = nfsd_dispatch,
         .vs_xdrsize             = NFS4_SVC_XDRSIZE,
         .vs_rpcb_optnl          = true,
diff --git a/fs/nfsd/nfsproc.c b/fs/nfsd/nfsproc.c
index 0ef88d0e67d9..44b157553733 100644
--- a/fs/nfsd/nfsproc.c
+++ b/fs/nfsd/nfsproc.c
@@ -743,10 +743,12 @@ static struct svc_procedure		nfsd_procedures2[18] = {
 };
 
 
+static unsigned int nfsd_count2[ARRAY_SIZE(nfsd_procedures2)];
 struct svc_version      nfsd_version2 = {
                 .vs_vers        = 2,
                 .vs_nproc       = 18,
                 .vs_proc        = nfsd_procedures2,
+                .vs_count       = nfsd_count2,
                 .vs_dispatch    = nfsd_dispatch,
                 .vs_xdrsize     = NFS2_SVC_XDRSIZE,
 };
diff --git a/include/linux/sunrpc/svc.h b/include/linux/sunrpc/svc.h
index 6cfe41db7f31..9f00384153f4 100644
--- a/include/linux/sunrpc/svc.h
+++ b/include/linux/sunrpc/svc.h
@@ -397,6 +397,7 @@ struct svc_version {
         u32                     vs_vers;        /* version number */
         u32                     vs_nproc;       /* number of procedures */
         struct svc_procedure *  vs_proc;        /* per-procedure info */
+        unsigned int            *vs_count;      /* call counts */
         u32                     vs_xdrsize;     /* xdrsize needed for this version */
 
         /* Don't register with rpcbind */
@@ -429,7 +430,6 @@ struct svc_procedure {
         void                    (*pc_release)(struct svc_rqst *);
         unsigned int            pc_argsize;     /* argument struct size */
         unsigned int            pc_ressize;     /* result struct size */