From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id pA3G6A0D027296 for ; Thu, 3 Nov 2011 11:06:12 -0500 Date: Thu, 3 Nov 2011 12:05:59 -0400 From: "Ted Ts'o" Subject: Re: [PATCH 6/8] xfstests: add fiemap operation to fsstress Message-ID: <20111103160559.GB15045@thunk.org> References: <1319849297-3506-1-git-send-email-dmonakhov@openvz.org> <1319849297-3506-7-git-send-email-dmonakhov@openvz.org> <20111102195534.GB22500@infradead.org> <87wrbhxya6.fsf@dmbot.sw.ru> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <87wrbhxya6.fsf@dmbot.sw.ru> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: xfs-bounces@oss.sgi.com Errors-To: xfs-bounces@oss.sgi.com To: Dmitry Monakhov Cc: Christoph Hellwig , linux-fsdevel@vger.kernel.org, xfs@oss.sgi.com, hch@lst.de, aelder@sgi.com T24gVGh1LCBOb3YgMDMsIDIwMTEgYXQgMDM6MDQ6MTdQTSArMDQwMCwgRG1pdHJ5IE1vbmFraG92 IHdyb3RlOgo+IE9uIFRodSwgMyBOb3YgMjAxMSAwNjo1NDoxNiAtMDQwMCwgVGhlb2RvcmUgVHNv IDx0eXRzb0BtaXQuZWR1PiB3cm90ZToKPiA+IAo+ID4gT24gTm92IDIsIDIwMTEsIGF0IDM6NTUg UE0sIENocmlzdG9waCBIZWxsd2lnIHdyb3RlOgo+ID4gCj4gPiA+IEFsZXgsIEVyaWMsIERhdmUg LSBzaG91bGQgd2UgYWRkIG5ldyB0ZXN0cyB3aXRoIHRoZSBuZXcgb3BlcmF0aW9ucwo+ID4gPiBE bWl0cnkgYWRkZWQsIG9yIGlzIGFkZGluZyBuZXcgb3BzIHRvIHRoZSBleGlzdGluZyB0ZXN0cyBm aW5lPwo+ID4gCj4gPiBPbmUgYXJndW1lbnQgZm9yIGFkZGluZyBuZXcgb3BzIHRvIGV4aXN0aW5n IHRlc3RzIGlzIHRoYXQgaXQgbWFrZXMKPiA+IHRoZSBydW4gdGltZSBvZiB0aGUgZW50aXJlIHRl c3Qgc3VpdGUgdGFrZSBsb25nZXIuICBBIFFBIHBhc3MgaXMKPiA+IGFscmVhZHkgdGFraW5nIHF1 aXRlIGEgd2hpbGUsIGFuZCBpdCB3b3VsZCBiZSBuaWNlIGlmIHdlIGNvdWxkCj4gPiBrZWVwIHhm c3Rlc3RzIGFzIGVmZmljaWVudCBhcyBwb3NzaWJsZSBpbiB0ZXJtcyBvZiB0aGUgbWF4aW11bQo+ ID4gdGVzdGluZyBjb3ZlcmFnZSBwZXIgdGltZSBzcGVudCBydW5uaW5nIHRoZSB0ZXN0IHN1aXRl 4oCmLgo+Cj4gWWVzLCBidXQgcmVncmVzc2lvbiB0ZXN0IHdpdGggZXhwbGljaXQgc2VlZCBvcHRp b24gc2hvdWxkIGJlCj4gcHJlc2VydmVkLiBOdW1iZXIgb2Ygc3VjaCB0ZXN0IGlzIG5vdCB0b28g YmlnLCBzbyBpdCBpcyByZWFzb25hYmxlIHRvCj4gaGFyZGNvZGUgc2V0IG9mIG9wZXJhdGlvbnMg aW4gc3VjaCB0ZXN0cyBhbmQgbGV0IGFsbCBvdGhlcnMgdXNlIG5ldyBmZWF0dXJlcy4KClRoYXQn cyBub3Qgd2hhdCBJIHdhcyB0YWxraW5nIGFib3V0LiAgT2YgY291cnNlIHRoZXJlIHNob3VsZCBi ZSBhIHdheQp0byBydW4gYSByZWdyZXNzaW9uIHRlc3Qgd2l0aCBhbiBleHBsaWNpdCBzZWVkIG9w dGlvbiAoYWx0aG91Z2ggaW4KZ2VuZXJhbCBJIHRoaW5rIGEgc3BlY2lmaWMgdGVzdCBpbiB4ZnN0 ZXN0cyBzaG91bGQgYnkgZGVmYXVsdCB1c2UgYQpyYW5kb20gc2VlZCwgYW5kIGhhdmUgYSB3YXkg dG8gZWFzaWx5IHNwZWNpZnkgYW4gZXhwbGljaXQgc2VlZCB3aXRob3V0CmhhdmluZyB0byByZXZl cnNlIGVuZ2luZWVyIHRoZSB0ZXN0IGFuZCBydW5uaW5nIGZzc3RyZXNzIG1hbnVhbGx5KS4KCldo YXQgSSB3YXMgdGFsa2luZyBhYm91dCB3YXMgdGhlIGZhY3Qgd2UgYWxyZWFkeSBoYXZlIHNldmVy YWwgKGhhbGYgYQpkb3plbiBvciBzbywgaWYgbWVtb3J5IHNlcnZlcyBjb3JyZWN0bHkpIHhmc3Rl c3RzIHRoYXQgdXNlIGZzc3RyZXNzCndpdGggYSBkaWZmZXJlbnQgc2V0IG9mIGZzc3RyZXNzIG9w dGlvbnMuICBJbiBzb21lIGNhc2VzIGl0IG1ha2VzIHRvCmFkZCBhIG5ldyBudW1iZXJlZCB4ZnN0 ZXN0IHN1YnRlc3QsIGJ1dCBJJ2QgcmF0aGVyIG5vdCBmaW5kIHRoYXQgd2UndmUKZG91YmxlZCB0 aGUgbnVtYmVyIG9mIHRlc3RzIHVzaW5nIGZzc3RyZXNzIGluIHRoZSBmdXR1cmUsIGFuZCB3aXRo IGl0LApkb3VibGVkIHRoZSBydW4tdGltZSBvZiB0aGUgYXV0byBvciBxdWljayB4ZnN0ZXN0cyBn cm91cC4uLi4KCgkgICAgCSAgICAgCSAgICAJICAgIC0gVGVkCgpfX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fXwp4ZnMgbWFpbGluZyBsaXN0Cnhmc0Bvc3Muc2dp LmNvbQpodHRwOi8vb3NzLnNnaS5jb20vbWFpbG1hbi9saXN0aW5mby94ZnMK From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ted Ts'o Subject: Re: [PATCH 6/8] xfstests: add fiemap operation to fsstress Date: Thu, 3 Nov 2011 12:05:59 -0400 Message-ID: <20111103160559.GB15045@thunk.org> References: <1319849297-3506-1-git-send-email-dmonakhov@openvz.org> <1319849297-3506-7-git-send-email-dmonakhov@openvz.org> <20111102195534.GB22500@infradead.org> <87wrbhxya6.fsf@dmbot.sw.ru> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Christoph Hellwig , linux-fsdevel@vger.kernel.org, aelder@sgi.com, hch@lst.de, xfs@oss.sgi.com To: Dmitry Monakhov Return-path: Received: from li9-11.members.linode.com ([67.18.176.11]:57669 "EHLO test.thunk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932244Ab1KCQGI (ORCPT ); Thu, 3 Nov 2011 12:06:08 -0400 Content-Disposition: inline In-Reply-To: <87wrbhxya6.fsf@dmbot.sw.ru> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Thu, Nov 03, 2011 at 03:04:17PM +0400, Dmitry Monakhov wrote: > On Thu, 3 Nov 2011 06:54:16 -0400, Theodore Tso wrote= : > >=20 > > On Nov 2, 2011, at 3:55 PM, Christoph Hellwig wrote: > >=20 > > > Alex, Eric, Dave - should we add new tests with the new operation= s > > > Dmitry added, or is adding new ops to the existing tests fine? > >=20 > > 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=E2=80=A6. > > 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 f= eatures. That's not what I was talking about. Of course there should be a way to run a regression test with an explicit seed option (although in general I think a specific test in xfstests should by default use a random seed, and have a way to easily specify an explicit seed without having to reverse engineer the test and running fsstress manually). What I was talking about was the fact we already have several (half a dozen or so, if memory serves correctly) xfstests that use fsstress with a different set of fsstress options. In some cases it makes to add a new numbered xfstest subtest, but I'd rather not find that we've doubled the number of tests using fsstress in the future, and with it, doubled the run-time of the auto or quick xfstests group.... - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel= " in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html