| To: | Theodore Tso <tytso@xxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH 6/8] xfstests: add fiemap operation to fsstress |
| From: | Dmitry Monakhov <dmonakhov@xxxxxxxxxx> |
| Date: | Thu, 03 Nov 2011 15:04:17 +0400 |
| Cc: | Theodore Tso <tytso@xxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, aelder@xxxxxxx, hch@xxxxxx, xfs@xxxxxxxxxxx |
| Dkim-signature: | v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=sender:from:to:cc:subject:in-reply-to:references:user-agent:date :message-id:mime-version:content-type:content-transfer-encoding; bh=bNHOksAqZbqwzWp6bM196NcZfTdqBLvR/eKNxDh4OSM=; b=x+dE80T9BhHeDEDkE1JIWd7hBhQosILLjLM86RjtHtmNPG14IHYJ9USENzBkhiYe6w pny1sHS9iImffcV3ri/I2l0JK2RJmEGhJUB/cy0JFxCz+4JyNxy/A/3CZTfkeNUYGkiV 6pgF9XeTz7ZVz+74yT7ScWgB0CFBEmbgpuRoY= |
| In-reply-to: | <FA2EDC7F-92EC-449E-9FBC-B7955283DFAC@xxxxxxx> |
| References: | <1319849297-3506-1-git-send-email-dmonakhov@xxxxxxxxxx> <1319849297-3506-7-git-send-email-dmonakhov@xxxxxxxxxx> <20111102195534.GB22500@xxxxxxxxxxxxx> <FA2EDC7F-92EC-449E-9FBC-B7955283DFAC@xxxxxxx> |
| Sender: | Dmitry Monakhov <rjevskiy@xxxxxxxxx> |
| User-agent: | Notmuch/0.5-69-g3e4a9d6 (http://notmuchmail.org) Emacs/23.1.1 (i486-pc-linux-gnu) |
On Thu, 3 Nov 2011 06:54:16 -0400, Theodore Tso <tytso@xxxxxxx> wrote: > > On Nov 2, 2011, at 3:55 PM, Christoph Hellwig wrote: > > > Alex, Eric, Dave - should we add new tests with the new operations > > Dmitry added, or is adding new ops to the existing tests fine? > > One argument for adding new ops to existing tests is that it makes the run > time of the entire test suite take longer. A QA pass is already taking > quite a while, and it would be nice if we could keep xfstests as efficient as > possible in terms of the maximum testing coverage per time spent running the > test suite…. Yes, but regression test with explicit seed option should be preserved. Number of such test is not too big, so it is reasonable to hardcode set of operations in such tests and let all others use new features. > > -- Ted > |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: xfs_repair fails with corrupt dinode 17491441757, extent total = 1, nblocks = 0. This is a bug., Christoph Hellwig |
|---|---|
| Next by Date: | Re: [Patch] xfs: serialise unaligned direct IOs, Amit Sahrawat |
| Previous by Thread: | Re: [PATCH 6/8] xfstests: add fiemap operation to fsstress, Theodore Tso |
| Next by Thread: | Re: [PATCH 6/8] xfstests: add fiemap operation to fsstress, Ted Ts'o |
| Indexes: | [Date] [Thread] [Top] [All Lists] |