diff mbox series

MAINTAINERS: create an entry for exportfs

Message ID 20231026205553.143556-1-amir73il@gmail.com (mailing list archive)
State New, archived
Headers show
Series MAINTAINERS: create an entry for exportfs | expand

Commit Message

Amir Goldstein Oct. 26, 2023, 8:55 p.m. UTC 
Split the exportfs entry from the nfsd entry and add myself as reviewer.

Suggested-by: Chuck Lever <chuck.lever@oracle.com>
Acked-by: Chuck Lever <chuck.lever@oracle.com>
Acked-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Amir Goldstein <amir73il@gmail.com>
---

Christian,

Chuck has suggested that I send this patch to you.
If you prefer that Chuck merges this patch though his tree
that's fine by me.

Thanks,
Amir.

 MAINTAINERS | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

Comments

Christoph Hellwig Oct. 27, 2023, 6:08 a.m. UTC | #1 
On Thu, Oct 26, 2023 at 11:55:53PM +0300, Amir Goldstein wrote:
> Split the exportfs entry from the nfsd entry and add myself as reviewer.

I think exportfs is by now very much VFS code.
Amir Goldstein Oct. 27, 2023, 6:35 a.m. UTC | #2 
On Fri, Oct 27, 2023 at 9:08 AM Christoph Hellwig <hch@infradead.org> wrote:
>
> On Thu, Oct 26, 2023 at 11:55:53PM +0300, Amir Goldstein wrote:
> > Split the exportfs entry from the nfsd entry and add myself as reviewer.
>
> I think exportfs is by now very much VFS code.
>

Yes, that's the idea of making it a vfs sub-entry in MAINTAINERS.

Is that an ACK? or did you mean that something needs to change
in the patch?

Thanks,
Amir.
Christoph Hellwig Oct. 27, 2023, 7:30 a.m. UTC | #3 
On Fri, Oct 27, 2023 at 09:35:29AM +0300, Amir Goldstein wrote:
> On Fri, Oct 27, 2023 at 9:08 AM Christoph Hellwig <hch@infradead.org> wrote:
> >
> > On Thu, Oct 26, 2023 at 11:55:53PM +0300, Amir Goldstein wrote:
> > > Split the exportfs entry from the nfsd entry and add myself as reviewer.
> >
> > I think exportfs is by now very much VFS code.
> >
> 
> Yes, that's the idea of making it a vfs sub-entry in MAINTAINERS.
> 
> Is that an ACK? or did you mean that something needs to change
> in the patch?

I as mostly thinking of dropping the diretory and the separate entry.
That would also go along with your patch of making it a bool.

But if you feel strongly about that we can add you as an extra reviewer
for it :)
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 2894f0777537..a194e6b0bcd1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8051,6 +8051,18 @@  F:	include/linux/fs_types.h
 F:	include/uapi/linux/fs.h
 F:	include/uapi/linux/openat2.h
 
+FILESYSTEMS [EXPORTFS]
+M:	Chuck Lever <chuck.lever@oracle.com>
+M:	Jeff Layton <jlayton@kernel.org>
+R:	Amir Goldstein <amir73il@gmail.com>
+L:	linux-fsdevel@vger.kernel.org
+L:	linux-nfs@vger.kernel.org
+S:	Supported
+F:	Documentation/filesystems/nfs/exporting.rst
+F:	fs/exportfs/
+F:	fs/fhandle.c
+F:	include/linux/exportfs.h
+
 FINTEK F75375S HARDWARE MONITOR AND FAN CONTROLLER DRIVER
 M:	Riku Voipio <riku.voipio@iki.fi>
 L:	linux-hwmon@vger.kernel.org
@@ -11420,7 +11432,6 @@  S:	Supported
 W:	http://nfs.sourceforge.net/
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/cel/linux.git
 F:	Documentation/filesystems/nfs/
-F:	fs/exportfs/
 F:	fs/lockd/
 F:	fs/nfs_common/
 F:	fs/nfsd/