mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-08-30 11:03:07 -04:00
sunrpc: remove unused svc_version vs_count field
Now that svc_seq_show() and the nfsd netlink stats handler both use the per-netns svc_stat vs_count arrays, the global per-version vs_count percpu counters are no longer read by anything. Remove the vs_count field from struct svc_version and all the associated DEFINE_PER_CPU_ALIGNED arrays and initializers across nfsd, lockd, and the NFS client callback service. Assisted-by: LLM Signed-off-by: Jeff Layton <jlayton@kernel.org> Link: https://patch.msgid.link/20260717-exportd-netlink-v7-4-b7ce17b83b60@kernel.org Signed-off-by: Chuck Lever <cel@kernel.org>
This commit is contained in:
@@ -1423,14 +1423,10 @@ union nlm4svc_xdrstore {
|
||||
struct nlm4_shareres_wrapper shareres;
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nlm4svc_call_counters[ARRAY_SIZE(nlm4svc_procedures)]);
|
||||
|
||||
const struct svc_version nlmsvc_version4 = {
|
||||
.vs_vers = 4,
|
||||
.vs_nproc = ARRAY_SIZE(nlm4svc_procedures),
|
||||
.vs_proc = nlm4svc_procedures,
|
||||
.vs_count = nlm4svc_call_counters,
|
||||
.vs_dispatch = nlmsvc_dispatch,
|
||||
.vs_xdrsize = sizeof(union nlm4svc_xdrstore),
|
||||
};
|
||||
|
||||
@@ -1436,25 +1436,18 @@ union nlmsvc_xdrstore {
|
||||
* NLMv1 defines only procedures 1 - 15. Linux lockd also implements
|
||||
* procedures 0 (NULL) and 16 (SM_NOTIFY).
|
||||
*/
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long, nlm1svc_call_counters[17]);
|
||||
|
||||
const struct svc_version nlmsvc_version1 = {
|
||||
.vs_vers = 1,
|
||||
.vs_nproc = 17,
|
||||
.vs_proc = nlmsvc_procedures,
|
||||
.vs_count = nlm1svc_call_counters,
|
||||
.vs_dispatch = nlmsvc_dispatch,
|
||||
.vs_xdrsize = sizeof(union nlmsvc_xdrstore),
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nlm3svc_call_counters[ARRAY_SIZE(nlmsvc_procedures)]);
|
||||
|
||||
const struct svc_version nlmsvc_version3 = {
|
||||
.vs_vers = 3,
|
||||
.vs_nproc = ARRAY_SIZE(nlmsvc_procedures),
|
||||
.vs_proc = nlmsvc_procedures,
|
||||
.vs_count = nlm3svc_call_counters,
|
||||
.vs_dispatch = nlmsvc_dispatch,
|
||||
.vs_xdrsize = sizeof(union nlmsvc_xdrstore),
|
||||
};
|
||||
|
||||
@@ -1090,26 +1090,20 @@ static const struct svc_procedure nfs4_callback_procedures1[] = {
|
||||
}
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nfs4_callback_count1[ARRAY_SIZE(nfs4_callback_procedures1)]);
|
||||
const 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 = nfs_callback_dispatch,
|
||||
.vs_hidden = true,
|
||||
.vs_need_cong_ctrl = true,
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nfs4_callback_count4[ARRAY_SIZE(nfs4_callback_procedures1)]);
|
||||
const 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 = nfs_callback_dispatch,
|
||||
.vs_hidden = true,
|
||||
|
||||
@@ -210,14 +210,11 @@ static const struct svc_procedure localio_procedures1[] = {
|
||||
};
|
||||
|
||||
#define LOCALIO_NR_PROCEDURES ARRAY_SIZE(localio_procedures1)
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
localio_count[LOCALIO_NR_PROCEDURES]);
|
||||
const struct svc_version localio_version1 = {
|
||||
.vs_vers = 1,
|
||||
.vs_nproc = LOCALIO_NR_PROCEDURES,
|
||||
.vs_proc = localio_procedures1,
|
||||
.vs_dispatch = nfsd_dispatch,
|
||||
.vs_count = localio_count,
|
||||
.vs_xdrsize = XDR_QUADLEN(UUID_SIZE),
|
||||
.vs_hidden = true,
|
||||
};
|
||||
|
||||
@@ -388,13 +388,10 @@ static const struct svc_procedure nfsd_acl_procedures2[5] = {
|
||||
},
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nfsd_acl_count2[ARRAY_SIZE(nfsd_acl_procedures2)]);
|
||||
const struct svc_version nfsd_acl_version2 = {
|
||||
.vs_vers = 2,
|
||||
.vs_nproc = ARRAY_SIZE(nfsd_acl_procedures2),
|
||||
.vs_proc = nfsd_acl_procedures2,
|
||||
.vs_count = nfsd_acl_count2,
|
||||
.vs_dispatch = nfsd_dispatch,
|
||||
.vs_xdrsize = NFS3_SVC_XDRSIZE,
|
||||
};
|
||||
|
||||
@@ -278,13 +278,10 @@ static const struct svc_procedure nfsd_acl_procedures3[3] = {
|
||||
},
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nfsd_acl_count3[ARRAY_SIZE(nfsd_acl_procedures3)]);
|
||||
const struct svc_version nfsd_acl_version3 = {
|
||||
.vs_vers = 3,
|
||||
.vs_nproc = ARRAY_SIZE(nfsd_acl_procedures3),
|
||||
.vs_proc = nfsd_acl_procedures3,
|
||||
.vs_count = nfsd_acl_count3,
|
||||
.vs_dispatch = nfsd_dispatch,
|
||||
.vs_xdrsize = NFS3_SVC_XDRSIZE,
|
||||
};
|
||||
|
||||
@@ -1108,13 +1108,10 @@ static const struct svc_procedure nfsd_procedures3[22] = {
|
||||
},
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nfsd_count3[ARRAY_SIZE(nfsd_procedures3)]);
|
||||
const struct svc_version nfsd_version3 = {
|
||||
.vs_vers = 3,
|
||||
.vs_nproc = ARRAY_SIZE(nfsd_procedures3),
|
||||
.vs_proc = nfsd_procedures3,
|
||||
.vs_dispatch = nfsd_dispatch,
|
||||
.vs_count = nfsd_count3,
|
||||
.vs_xdrsize = NFS3_SVC_XDRSIZE,
|
||||
};
|
||||
|
||||
@@ -4262,13 +4262,10 @@ static const struct svc_procedure nfsd_procedures4[2] = {
|
||||
},
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nfsd_count4[ARRAY_SIZE(nfsd_procedures4)]);
|
||||
const struct svc_version nfsd_version4 = {
|
||||
.vs_vers = 4,
|
||||
.vs_nproc = ARRAY_SIZE(nfsd_procedures4),
|
||||
.vs_proc = nfsd_procedures4,
|
||||
.vs_count = nfsd_count4,
|
||||
.vs_dispatch = nfsd_dispatch,
|
||||
.vs_xdrsize = NFS4_SVC_XDRSIZE,
|
||||
.vs_rpcb_optnl = true,
|
||||
|
||||
@@ -845,13 +845,10 @@ static const struct svc_procedure nfsd_procedures2[18] = {
|
||||
},
|
||||
};
|
||||
|
||||
static DEFINE_PER_CPU_ALIGNED(unsigned long,
|
||||
nfsd_count2[ARRAY_SIZE(nfsd_procedures2)]);
|
||||
const struct svc_version nfsd_version2 = {
|
||||
.vs_vers = 2,
|
||||
.vs_nproc = ARRAY_SIZE(nfsd_procedures2),
|
||||
.vs_proc = nfsd_procedures2,
|
||||
.vs_count = nfsd_count2,
|
||||
.vs_dispatch = nfsd_dispatch,
|
||||
.vs_xdrsize = NFS2_SVC_XDRSIZE,
|
||||
};
|
||||
|
||||
@@ -407,7 +407,6 @@ struct svc_version {
|
||||
u32 vs_vers; /* version number */
|
||||
u32 vs_nproc; /* number of procedures */
|
||||
const struct svc_procedure *vs_proc; /* per-procedure info */
|
||||
unsigned long __percpu *vs_count; /* call counts */
|
||||
u32 vs_xdrsize; /* xdrsize needed for this version */
|
||||
|
||||
/* Don't register with rpcbind */
|
||||
|
||||
@@ -1344,9 +1344,6 @@ svc_generic_init_request(struct svc_rqst *rqstp,
|
||||
memset(rqstp->rq_argp, 0, procp->pc_argzero);
|
||||
memset(rqstp->rq_resp, 0, procp->pc_ressize);
|
||||
|
||||
/* Bump per-procedure stats counter */
|
||||
this_cpu_inc(versp->vs_count[rqstp->rq_proc]);
|
||||
|
||||
/* Bump per-net per-procedure stats counter */
|
||||
if (rqstp->rq_server->sv_stats &&
|
||||
rqstp->rq_server->sv_stats->program == progp &&
|
||||
|
||||
Reference in New Issue
Block a user