Message ID | 20231026205553.143556-1-amir73il@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | MAINTAINERS: create an entry for exportfs | expand |
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.
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.
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 :)
On Fri, Oct 27, 2023 at 10:30 AM Christoph Hellwig <hch@infradead.org> wrote: > > 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 :) I don't have a strong opinion about this. I am submitting this patch on behalf of the current export maintainers who requested to stay as maintainers of fs/exportfs and offered that I join as a reviewer of fs/exportfs. I cannot join as a reviewer of nfsd, hence this needs an entry. Thanks, Amir.
On Thu, 26 Oct 2023 23:55:53 +0300, Amir Goldstein wrote: > Split the exportfs entry from the nfsd entry and add myself as reviewer. > > Looks useful to clarify this. I've create vfs-6.7.* tags yesterday I plan on sending this one either at the end of next week or in week 2 together with other possible fixes. --- Applied to the vfs.fixes branch of the vfs/vfs.git tree. Patches in the vfs.fixes branch should appear in linux-next soon. Please report any outstanding bugs that were missed during review in a new review to the original patch series allowing us to drop it. It's encouraged to provide Acked-bys and Reviewed-bys even though the patch has now been applied. If possible patch trailers will be updated. Note that commit hashes shown below are subject to change due to rebase, trailer updates or similar. If in doubt, please check the listed branch. tree: https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git branch: vfs.fixes [1/1] MAINTAINERS: create an entry for exportfs https://git.kernel.org/vfs/vfs/c/7723a9a55be9
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/