Re: [PATCH 4/5] fstests/MAINTAINERS: add some specific reviewers

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Wed, Apr 05, 2023 at 09:47:55AM +0200, Christian Brauner wrote:
> On Wed, Apr 05, 2023 at 01:14:10AM +0800, Zorro Lang wrote:
> > Some people contribute to someone specific fs testing mostly, record
> > some of them as Reviewer.
> > 
> > Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx>
> > ---
> > 
> > If someone doesn't want to be in cc list of related fstests patch, please
> > reply this email, I'll remove that reviewer line.
> > 
> > Or if someone else (who contribute to fstests very much) would like to a
> > specific reviewer, nominate yourself to get a review.
> > 
> > Thanks,
> > Zorro
> > 
> >  MAINTAINERS | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 620368cb..0ad12a38 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -108,6 +108,7 @@ Maintainers List
> >  	  or reviewer or co-maintainer can be in cc list.
> >  
> >  BTRFS
> > +R:	Filipe Manana <fdmanana@xxxxxxxx>
> >  L:	linux-btrfs@xxxxxxxxxxxxxxx
> >  S:	Supported
> >  F:	tests/btrfs/
> > @@ -137,16 +138,19 @@ F:	tests/f2fs/
> >  F:	common/f2fs
> >  
> >  FSVERITY
> > +R:	Eric Biggers <ebiggers@xxxxxxxxxx>
> >  L:	fsverity@xxxxxxxxxxxxxxx
> >  S:	Supported
> >  F:	common/verity
> >  
> >  FSCRYPT
> > +R:	Eric Biggers <ebiggers@xxxxxxxxxx>
> >  L:      linux-fscrypt@xxxxxxxxxxxxxxx
> >  S:	Supported
> >  F:	common/encrypt
> >  
> >  FS-IDMAPPED
> 
> I'd just make this VFS since src/vfs/ covers generic vfs functionality.
> 
> But up to you,

Sure, it can be "VFS". I didn't use "VFS" directly due to vfs is larger, current
src/vfs only tests a small part of it, more tests are under tests/generic directory.
And we don't have many vfs tests, e.g. we don't test new mount API in fstests.

We need a way to sort out "VFS" only files/tests (e.g. group tag), and if we use
"VFS" at here, would you like to be a reviewer of all vfs tests?

Thanks,
Zorro

> 
> Acked-by: Christian Brauner <brauner@xxxxxxxxxx>
> 
> > +R:	Christian Brauner <brauner@xxxxxxxxxx>
> >  L:	linux-fsdevel@xxxxxxxxxxxxxxx
> >  S:	Supported
> >  F:	src/vfs/
> > @@ -163,6 +167,7 @@ S:	Supported
> >  F:	tests/ocfs2/
> >  
> >  OVERLAYFS
> > +R:	Amir Goldstein <amir73il@xxxxxxxxx>
> >  L:	linux-unionfs@xxxxxxxxxxxxxxx
> >  S:	Supported
> >  F:	tests/overlay
> > @@ -174,6 +179,7 @@ S:	Supported
> >  F:	tests/udf/
> >  
> >  XFS
> > +R:	Darrick J. Wong <djwong@xxxxxxxxxx>
> >  L:	linux-xfs@xxxxxxxxxxxxxxx
> >  S:	Supported
> >  F:	common/dump
> > -- 
> > 2.39.2
> > 
> 




[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux