Re: [xfstests PATCH v6 0/8] overlay: add fsck.overlay basic tests

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



On 2018/1/24 17:41, Amir Goldstein Wrote:
> On Wed, Jan 24, 2018 at 9:33 AM, zhangyi (F) <yi.zhang@xxxxxxxxxx> wrote:
>> Hi All:
>>
>> Here is the sixth version of test cases for the fsck.overlay[1].
> 
> You know, I wanted to say this earlier.
> This series is a mix of 2 independent series and I don't know if
> it is best for you to post them together.
> 
> The 3 tests added to overlay/fsck group only require the minimal
> helper _overlay_fsck_dirs.
> 
> The rest of the patches really do something very different than testing
> overlay.fsck itself.
> 
> If you hadn't combined the 2 series together, the overlay/fsck series
> would have completed review 2-3 revisions ago.
> 
> I'll leave it to you to decide if to post the next revision as one series
> or two, but I think it would be better if you at least re-order the
> patches so that the already reviewed overlay/fsck patches
> with the simple _overlay_fsck_dirs helper are before the rest of
> the patches.
> 
Sure, it's better to split into two independent series, will do.

Thanks,
Yi.

--
To unsubscribe from this list: send the line "unsubscribe fstests" 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 Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux