Re: [PATCH 07/14] sunrpc: add a generic netlink family for cache upcalls

From: Chuck Lever

Date: Thu Mar 19 2026 - 15:15:32 EST


On 3/16/26 11:14 AM, Jeff Layton wrote:

> diff --git a/fs/nfsd/netlink.c b/fs/nfsd/netlink.c
> index 4e08c1a6b3943cda5b44c2b64bcf3a00173a08db..81c943345d13db849483bf0d6773458115ff0134 100644
> --- a/fs/nfsd/netlink.c
> +++ b/fs/nfsd/netlink.c
> @@ -59,7 +59,7 @@ static const struct genl_split_ops nfsd_nl_ops[] = {
> .cmd = NFSD_CMD_THREADS_SET,
> .doit = nfsd_nl_threads_set_doit,
> .policy = nfsd_threads_set_nl_policy,
> - .maxattr = NFSD_A_SERVER_MAX,
> + .maxattr = NFSD_A_SERVER_FH_KEY,
> .flags = GENL_ADMIN_PERM | GENL_CMD_CAP_DO,
> },
> {

This hunk is clearly not related to adding "a generic netlink family for
cache upcalls". Should I apply it instead to the appropriate FH-signing
patch, which is still in my nfsd-testing branch?


--
Chuck Lever