Re: [PATCH 0/2] Another overlay test for online inconsistency detection

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

 



On Wed, May 9, 2018 at 6:39 PM, Eryu Guan <guaneryu@xxxxxxxxx> wrote:
> On Fri, May 04, 2018 at 12:41:29PM +0300, Amir Goldstein wrote:
>> Eryu,
>>
>> Originally, I wrote overlay/049 to test online inconcsistency detection
>> feature, which I wrote as a sub feature of NFS export, so the initial
>> version of the test required and eanbled NFS export.
>>
>> Then, at some point, the test started to pass without NFS export enabled,
>> which was a by-product of another fix for inotify I did around the same
>> time. So the test got merged without requiring NFS export, but with said
>> fix commit on next branch.
>>
>> On a recent discussion with Vivek, I realized that there was another
>> "by-product" inconsistency detection introduced by the same fix commit
>> and not covered by overlay/049.
>>
>> The variant overlay/059 checks for this case of malformed multiply
>> referenced origin files. I wrote the test to support upcoming 'metacopy'
>> feature by Vivek, so it requires and enables redirect_dir, which is
>> harmless otherwise.
>>
>> I tested overlay/059 on several old kernels. The results are:
>> - notrun on kernel < v4.10, before "redirect_dir" feature
>> - pass on kernels < v4.12, before "origin" was introduced
>> - fail on kernels < v4.16, before the fix commit
>> - pass on newer kernels
>
> Thanks a lot for the detailed background information and test status!
>
> BTW, I noticed that you still send patches to address eguan@xxxxxxxxxx,
> I'm not using thaa address anymore, please send to guaneryu@xxxxxxxxx
> instead :)
>

Right. Forgot to update my scripts.
Should be good now.

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