2022-10-31 14:17:28

by Chuck Lever III

Subject: [PATCH 2] MAINTAINERS: NFSD should be responsible for fs/exportfs

We recently received a patch for fs/exportfs/expfs.c, but there
isn't a subsystem maintainer listed for fs/exportfs:

Christian Brauner <[email protected]> (commit_signer:2/2=100%,authored:1/2=50%,added_lines:3/6=50%,removed_lines:2/6=33%)
Al Viro <[email protected]> (commit_signer:1/2=50%,authored:1/2=50%,added_lines:3/6=50%,removed_lines:4/6=67%)
Miklos Szeredi <[email protected]> (commit_signer:1/2=50%)
Amir Goldstein <[email protected]> (commit_signer:1/2=50%)
[email protected] (open list)

Neil says:
> Looking at recent commits, patches come in through multiple
> different trees.
> nfsd certainly has an interest in expfs.c. The only other user is
> name_to_handle/open_by_handle API.
> I see it as primarily nfsd functionality which is useful enough to
> be exported directly to user-space.
> (It was created by me when I was nfsd maintainer - does that
> count?)

Suggested-by: Neil Brown <[email protected]>
Signed-off-by: Chuck Lever <[email protected]>
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)

The patch description in v1 was truncated. Here's a refresh.

diff --git a/MAINTAINERS b/MAINTAINERS
index 379945f82a64..61fb45cfc825 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11129,6 +11129,7 @@ L: [email protected]
S: Supported
W: http://nfs.sourceforge.net/
T: git git://git.kernel.org/pub/scm/linux/kernel/git/cel/linux.git
+F: fs/exportfs/
F: fs/lockd/
F: fs/nfs_common/
F: fs/nfsd/

2022-10-31 17:12:43

by Jeff Layton

Subject: Re: [PATCH 2] MAINTAINERS: NFSD should be responsible for fs/exportfs

On Mon, 2022-10-31 at 10:01 -0400, Chuck Lever wrote:
> We recently received a patch for fs/exportfs/expfs.c, but there
> isn't a subsystem maintainer listed for fs/exportfs:
>
> Christian Brauner <[email protected]> (commit_signer:2/2=100%,authored:1/2=50%,added_lines:3/6=50%,removed_lines:2/6=33%)
> Al Viro <[email protected]> (commit_signer:1/2=50%,authored:1/2=50%,added_lines:3/6=50%,removed_lines:4/6=67%)
> Miklos Szeredi <[email protected]> (commit_signer:1/2=50%)
> Amir Goldstein <[email protected]> (commit_signer:1/2=50%)
> [email protected] (open list)
>
> Neil says:
> > Looking at recent commits, patches come in through multiple
> > different trees.
> > nfsd certainly has an interest in expfs.c. The only other user is
> > name_to_handle/open_by_handle API.
> > I see it as primarily nfsd functionality which is useful enough to
> > be exported directly to user-space.
> > (It was created by me when I was nfsd maintainer - does that
> > count?)
>
> Suggested-by: Neil Brown <[email protected]>
> Signed-off-by: Chuck Lever <[email protected]>
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> The patch description in v1 was truncated. Here's a refresh.
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 379945f82a64..61fb45cfc825 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -11129,6 +11129,7 @@ L: [email protected]
> S: Supported
> W: http://nfs.sourceforge.net/
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/cel/linux.git
> +F: fs/exportfs/
> F: fs/lockd/
> F: fs/nfs_common/
> F: fs/nfsd/
>
>

Acked-by: Jeff Layton <[email protected]>