Re: [PATCH 1/5] xfstest: add ubifs support

Dongsheng Yang
2015-08-03T04:53:35Z

Sorry, I click "send" to early. :(

On 08/03/2015 12:44 PM, Dongsheng Yang wrote:
> On 08/03/2015 12:12 PM, Dave Chinner wrote:
>> On Thu, Jul 30, 2015 at 02:06:53PM +0800, Dongsheng Yang wrote:
>>> Signed-off-by: Dongsheng Yang <yangds.fnst@cn.fujitsu.com>
>>> ---
>>> check | 2 ++
>>> common/config | 9 +++++----
[...]
>>> # no way to check consistency for tmpfs
>>> ;;
>>> + ubifs)
>>> + ;;
>>
>> There is no fsck.ubifs? How do you validate the filesystem is
>> consistent?

Yes, currently, we have no fsck.ubifs, but I am planing to add it.
>>
>> Keep in minds that the people you are asking to review/commit this
>> code have no idea what UBIFS is or how it works, so you need to
>> explain why xfstests requires non-standard configuration and
>> bypassing of the config sanity testing...

There is a document for ubifs, the main reason for this patchset is
ubifs is running on ubi volume which is not a block device.

http://www.linux-mtd.infradead.org/doc/ubifs.html

Thanx
Yang
>>
>> Cheers,
>>
>> Dave.
>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> .
>