| To: | chandan <chandan@xxxxxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] xfstests: generic/255: Execute only if blocksize <= 4096 |
| From: | Eric Sandeen <sandeen@xxxxxxxxxx> |
| Date: | Mon, 26 Aug 2013 09:29:31 -0500 |
| Cc: | Eric Sandeen <sandeen@xxxxxxxxxxx>, linux-btrfs@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, sekharan@xxxxxxxxxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <44437668.FEX1DiRokR@xxxxxxxxxxxxxxxxxxxxx> |
| References: | <2909046.09LYLQvZ6q@xxxxxxxxxxxxxxxxxxxxx> <521A36C6.40006@xxxxxxxxxxx> <44437668.FEX1DiRokR@xxxxxxxxxxxxxxxxxxxxx> |
| User-agent: | Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/20130801 Thunderbird/17.0.8 |
On 8/26/13 12:38 AM, chandan wrote: > On Sunday, August 25, 2013 11:54:30 AM Eric Sandeen wrote: >> Can you explain why this is necessary? >> >> What failures do you see, on what filesystems? > > generic/255 currently fails on Btrfs on a ppc64 machine with 64k page size and > hence 64k block size. > > generic/255 has been written to test the corner cases for 4k block size. I did > try to make it work with variable sized block sizes, But I got stuck > working with md5sum (since we would need two sets of md5sums, due to > _test_generic_punch() being invoked with and without '-k' option per block > size). > > Since 4k block size support for Btrfs on ppc64 is already being worked on, I > think its better to prevent execution of generic/255 for block sizes greater > than 4k. > > Apologies for not including the above description in the patch. It happens. ;) To be honest I haven't really looked at how _test_generic_punch & generic/255 works lately. Just as a sanity check, does it also fail on xfs for 64k block sizes on ppc64? Thanks, -Eric |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] Re: XFS: Assertion failed: first <= last && last < BBTOB(bp->b_length), file: fs/xfs/xfs_trans_buf.c, line: 568, Brian Foster |
|---|---|
| Next by Date: | Re: [patch] xfs: check for underflow in xfs_iformat_fork(), Dan Carpenter |
| Previous by Thread: | Re: [PATCH] xfstests: generic/255: Execute only if blocksize <= 4096, chandan |
| Next by Thread: | Re: [PATCH] xfstests: generic/255: Execute only if blocksize <= 4096, chandan |
| Indexes: | [Date] [Thread] [Top] [All Lists] |