Re: [PATCH v9 0/4 REBASED] fstests: add idmapped mounts tests

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



On Mon, Mar 22, 2021 at 10:37:04AM +0800, Eryu Guan wrote:
> On Sun, Mar 21, 2021 at 04:32:00PM +0100, Christian Brauner wrote:
> > On Sun, Mar 21, 2021 at 10:26:35PM +0800, Eryu Guan wrote:
> > > On Tue, Mar 16, 2021 at 11:36:23AM +0100, Christian Brauner wrote:
> > > > Hey everyone,
> > > > 
> > > > /* v9 */
> > > > Rebased onto current master.
> > > > Hopefully we can merge this soon so we can have regular testing asap and
> > > > the rebasing is getting a bit tedious too. ;)
> > > 
> > > Sorry for the late review.. You don't have to rebase on every release if
> > > updating the test seq number is the only thing changed. Seq number
> > > conflicts all the time, usually I can fix it on commit.
> > > 
> > > Patch 2 is missing from the list, maybe it's too big to post, could you
> > 
> > _Sigh_, this seems like another vger issue. The individual Ccs/Tos
> > should've all gotten the patch but it's missing from the list. This is a
> > pattern I've seen quite a few times recently with vger. There's nothing
> > to do apart from complain about it to the vger maintainer. :)
> > 
> > > please provide a git repo so I can pull from there?
> > 
> > For sure. It's available in three locations:
> > https://git.kernel.org/pub/scm/linux/kernel/git/brauner/xfstests-dev.git/log/?h=idmapped_mounts_v2
> > https://gitlab.com/brauner/xfstests/-/tree/idmapped_mounts_v2
> > https://github.com/brauner/xfstests/tree/idmapped_mounts_v2
> 
> Ok, thanks!
> 
> > 
> > (Are you fixing the things you pointed out in the individual patches
> > yourself or should I resend a new version?)
> 
> New version please, I can fix trival things like typo, renumber the

I'll give you a new version hopefully today (CET).

> test, code style, etc.

Thanks!

Christian



[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