From: Dongsheng Yang <yangds.fnst@cn.fujitsu.com>
To: Dave Chinner <david@fromorbit.com>
Cc: eguan@redhat.com, fstests@vger.kernel.org, dedekind1@gmail.com,
richard.weinberger@gmail.com, linux-mtd@lists.infradead.org
Subject: Re: [PATCH 1/5] xfstest: add ubifs support
Date: Mon, 3 Aug 2015 12:47:36 +0800 [thread overview]
Message-ID: <55BEF268.8020404@cn.fujitsu.com> (raw)
In-Reply-To: <55BEF1A3.9060103@cn.fujitsu.com>
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
> .
>
WARNING: multiple messages have this Message-ID (diff)
From: Dongsheng Yang <yangds.fnst@cn.fujitsu.com>
To: Dave Chinner <david@fromorbit.com>
Cc: <eguan@redhat.com>, <fstests@vger.kernel.org>,
<dedekind1@gmail.com>, <richard.weinberger@gmail.com>,
<linux-mtd@lists.infradead.org>
Subject: Re: [PATCH 1/5] xfstest: add ubifs support
Date: Mon, 3 Aug 2015 12:47:36 +0800 [thread overview]
Message-ID: <55BEF268.8020404@cn.fujitsu.com> (raw)
In-Reply-To: <55BEF1A3.9060103@cn.fujitsu.com>
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
> .
>
next prev parent reply other threads:[~2015-08-03 4:53 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-30 6:06 [PATCH 0/5] xfstests: introduce ubifs in xfstests Dongsheng Yang
2015-07-30 6:06 ` Dongsheng Yang
2015-07-30 6:06 ` [PATCH 1/5] xfstest: add ubifs support Dongsheng Yang
2015-07-30 6:06 ` Dongsheng Yang
2015-08-03 4:12 ` Dave Chinner
2015-08-03 4:44 ` Dongsheng Yang
2015-08-03 4:44 ` Dongsheng Yang
2015-08-03 4:47 ` Dongsheng Yang [this message]
2015-08-03 4:47 ` Dongsheng Yang
2015-08-03 5:51 ` Dave Chinner
2015-07-30 6:06 ` [PATCH 2/5] xfstest: generic/219 add _require_odirect Dongsheng Yang
2015-07-30 6:06 ` Dongsheng Yang
2015-08-03 3:11 ` Eryu Guan
2015-08-03 3:59 ` Dave Chinner
2015-08-03 5:01 ` Dongsheng Yang
2015-08-03 5:01 ` Dongsheng Yang
2015-08-03 5:55 ` Dave Chinner
2015-07-30 6:06 ` [PATCH 3/5] xfstest: generic/125 " Dongsheng Yang
2015-07-30 6:06 ` Dongsheng Yang
2015-08-03 3:12 ` Eryu Guan
2015-07-30 6:06 ` [PATCH 4/5] xfstest: introduce subtestset for ubifs Dongsheng Yang
2015-07-30 6:06 ` Dongsheng Yang
2015-08-03 3:28 ` Eryu Guan
2015-08-03 5:12 ` Dongsheng Yang
2015-08-03 5:12 ` Dongsheng Yang
2015-07-30 6:06 ` [PATCH 5/5] xfstest: add quota tests to ubifs Dongsheng Yang
2015-07-30 6:06 ` Dongsheng Yang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55BEF268.8020404@cn.fujitsu.com \
--to=yangds.fnst@cn.fujitsu.com \
--cc=david@fromorbit.com \
--cc=dedekind1@gmail.com \
--cc=eguan@redhat.com \
--cc=fstests@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=richard.weinberger@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.