Re: unionmount-testsuite: backward compat support for REDIRECT_DIR=n

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

 



On Tue, May 30, 2017 at 11:35 PM, Vivek Goyal <vgoyal@xxxxxxxxxx> wrote:
> On Sun, May 28, 2017 at 05:43:45PM +0300, Amir Goldstein wrote:
...
>>
>> Vivek,
>>
>> I added support for running my maser branch with old kernels
>> or with REDIRECT_DIR=n.
>>
>> User needs to request backward compat mode explicitly.
>> When using new command line arg --xdev,
>> dir rename tests that would return EXDEV are skipped.
>>
...
>
> So --xdev will skip tests which are not supposed to pass without
> REDIRECT_DIR=n?

Short answer - yes.

Long answer:
Technically, those tests (rename-empty-dir, rename-pop-dir, rename-move-dir)
pass on David's master branch by expecting and getting EXDEV.
My master branch, OTOH, does not verify getting EXDEV with the --xdev
flag, so it will not uncover a hypothetical regression with REDIRECT_DIR=n
where those tests no longer return EXDEV.

The reason I did not revert eae160a ("Relax failure to rename lower directory")
is because I have added more directory rename tests since then, so the revert
itself is not enough and I chose the easier path.

>
> Would be great if we could automate this. That is test which are not valid
> with current kernel/overlay configuration are skipped automatically. It
> totally up to you. :-)
>

Yes, that would be great.
Need to test "`cat /sys/module/overlay/parameters/redirect_dir`" = Y
But need to take care to modprobe overlay before the test.

It's up to me or anyone that wants to send a patch ;-)

Thanks,
Amir.
--
To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



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

  Powered by Linux