understanding csum tree

2015-11-12 Thread Lakshmipathi.G
/WzjJuzFB Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] btrfs-progs: print root item's last_snapshot value.

2015-10-07 Thread Lakshmipathi.G
Include last_snapshot value in print_root(). With btrfs-debug-tree, it helps to identify whether its a snapshot-ed subvolume or not. Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- print-tree.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/print-

Re: Pointers to Btrfs Testplan/Testcases

2016-02-01 Thread Lakshmipathi.G
I didn't know that xfstests covers more than xfs. Will look into their repo for btrfs specific cases. thanks for the response. Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More

Re: [PATCH][btrfs-progs] Include file verification with convert-tests

2016-02-17 Thread Lakshmipathi.G
it & expect btrfs-check to catch issue? Any thoughts on above list, suggestions/comments? thanks! Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel

[PATCH][btrfs-progs] populate fs with small dataset for convert-tests

2016-03-08 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/convert-tests.sh | 84 +++--- 1 file changed, 80 insertions(+), 4 deletions(-) diff --git a/tests/convert-tests.sh b/tests/convert-tests.sh index 0bfb41f..c1f3de0 100644 --- a

Re: [PATCH] btrfs-progs: fsck-tests: verify 'btrfs check --repair' fixes corrupted nlink field

2017-02-01 Thread Lakshmipathi.G
O_HELPER $TOP/btrfs check $TEST_DEV > > What about using check_image() here? I didn't know about check_image(), looks like above 3 lines can be replaced with check_image. Will make the change. > > Thanks, > Qu > Cheers. Lakshmipathi.G -- To unsubscribe from this list: send

[PATCH v3] btrfs-progs: fsck-tests: verify 'btrfs check --repair' fixes corrupted nlink field

2017-02-02 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/fsck-tests/026-check-inode-link/test.sh | 30 +++ 1 file changed, 30 insertions(+) create mode 100755 tests/fsck-tests/026-check-inode-link/test.sh diff --git a/tests/fsck-tests/026-check-inod

Re: Very slow balance / btrfs-transaction

2017-02-03 Thread Lakshmipathi.G
>Should quota support generally be disabled during balances? If this true and quota impacts balance throughput, at-least there should an alert message like "Running Balance with quota will affect performance" or similar before starting. Cheers, Lakshmipathi.G -- To uns

How to dump/find parity of RAID-5 file?

2017-02-03 Thread Lakshmipathi.G
4112 # Contains file data-stripe-2 dev_uuid 637b3666-9d8f-4ec4-9969-53b0b933b9b1 thanks. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://v

[PATCH] btrfs-progs: fsck-tests: script to verify init-csum-tree

2017-01-26 Thread Lakshmipathi.G
Corrupt csum entry of a file and re-create csum tree using --init-csum Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- .../fsck-tests/025-verify-init-csum-option/test.sh | 38 ++ 1 file changed, 38 insertions(+) create mode 100755 tests/fsck-tests/025-

[PATCH] xfstests: btrfs/047: check btrfs-convert with extent and non-extent source

2017-01-27 Thread Lakshmipathi.G
This is used to check the source which contains combination of Ext3 files in non-extent format and Ext4 extent-files. And validate the file md5sums before and after conversion. btrfs/012: BTRFS_CONVERT_PROG,E2FSCK_PROG definitions reused from common/config Signed-off-by: Lakshmipathi.G

Re: Fresh Raid-1 setup, dump-tree shows invalid owner id

2017-01-30 Thread Lakshmipathi.G
Raid1 is irrelevant, looks like this happen with simple case too. $./mkfs.btrfs tests/test.img $./btrfs-debug-tree tests/test.img possible issue with ./btrfs-debug-tree stdout? On Mon, Jan 30, 2017 at 7:24 AM, Lakshmipathi.G <lakshmipath...@giis.co.in> wrote: > After creat

Re: Fresh Raid-1 setup, dump-tree shows invalid owner id

2017-01-30 Thread Lakshmipathi.G
rce in > include/uapi/linux/btrfs_tree.h > > e.g.: > > #define BTRFS_DATA_RELOC_TREE_OBJECTID -9ULL > Thanks for the details. This owner number looked different from other owner ids, so wanted to check on the same, now understood. Cheers. Lakshmipathi.G -- To unsubs

[PATCH] btrfs-progs: fsck-tests: verify 'btrfs check --repair' fixes corrupted nlink field

2017-01-28 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/fsck-tests/026-check-inode-link/test.sh | 34 +++ 1 file changed, 34 insertions(+) create mode 100755 tests/fsck-tests/026-check-inode-link/test.sh diff --git a/tests/fsck-tests/026-check-inod

Fresh Raid-1 setup, dump-tree shows invalid owner id

2017-01-29 Thread Lakshmipathi.G
f743117f2 chunk uuid 6477561c-cbca-45e4-980d-56727a8dc9d9 -- or is that expected output? Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] btfs-progs: fsck-tests: corrupt nlink value test

2017-01-25 Thread Lakshmipathi.G
> Both approaches have their pros and cons so I'll accept both. The > functionality provided by the corrupt block utility can be used, any > changes to the command line UI will be also applied to the test scripts. okay, I'll continue mixing both approaches. -- To unsubscribe from this list: send

Re: [PATCH] btrfs-progs: fsck-tests: missing csum test script

2017-01-25 Thread Lakshmipathi.G
On Tue, Jan 17, 2017 at 04:06:27PM +0100, David Sterba wrote: > It's not clear from the test what's the purpose. There's one corrupted > csum but the whole csum tree rebuild option is used. This is a pretty > basic check that the --init-csum-tree works, so it should be mentioned > somewhere in the

Re: [RFC] btrfs-progs: btrfs-corrupt-block refactor

2017-01-25 Thread Lakshmipathi.G
and updates/fixes drive-2 entries. Something like that possible or i miss understood something? Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] xfstests: btrfs/047: check btrfs-convert with extent and non-extent source

2017-01-25 Thread Lakshmipathi.G
On Wed, Jan 18, 2017 at 02:39:53PM +0800, Eryu Guan wrote: > On Wed, Jan 18, 2017 at 07:17:02AM +0530, Lakshmipathi.G wrote: > > Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> > > Need detailed test description in commit log too. Okay will include them. >

[PATCH] btrfs-progs: cli-tests: Convert non-raid filesystem to raid

2017-01-29 Thread Lakshmipathi.G
Simple script to verify non-raid filesystem conversion. Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/cli-tests/005-convert-btrfs-to-raid/test.sh | 32 +++ 1 file changed, 32 insertions(+) create mode 100755 tests/cli-tests/005-convert-btrfs-t

Re: Raid1 won't mount degraded

2017-01-29 Thread Lakshmipathi.G
you sure 'device add' was successful? Whats the new device name? 'btrfs fi show' shows only device (sdb1). Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at h

Re: [PATCH] btrfs-progs: RAID5:Inject data stripe corruption and verify scrub fixes it.

2017-02-21 Thread Lakshmipathi.G
es and 64 parity stripe location seems fine. It constantly hit the parity bug with the script. If the script gets accepted, will add slightly other corruption variants likes: - corrupt all even stripe (D2,D4..) - corrupt all odd stripe (D1,D3..) - corrupt all parity stripes - corrupt all both da

Re: Large no.of extents for a single small file with a fresh RAID5 setup

2017-02-21 Thread Lakshmipathi.G
pping from logical->physical chunks on RAID5. I was wondering how these 4KB extents will fit into 64KB data-stripe? I didn't have idea about read-modify-write, just found few wiki links on it, will check. thanks Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the l

Re: Large no.of extents for a single small file with a fresh RAID5 setup

2017-02-21 Thread Lakshmipathi.G
iextent output. one more curious thing, is it fine to have extents with size 4096 on RAID5 setup? Cheers. Lakshmipathi.G #time /root/check_extents.sh f2MB 32 real1m30.284s user1m22.796s sys 0m39.324s laks/btrfs-progs# ./btrfs-debugfs -f tests/mnt/f2MB (263 0): ram 110592 disk 151015

Re: Large no.of extents for a single small file with a fresh RAID5 setup

2017-02-20 Thread Lakshmipathi.G
On Mon, Feb 20, 2017 at 03:53:15PM +0800, Qu Wenruo wrote: > > > At 02/18/2017 10:07 AM, Lakshmipathi.G wrote: > >Hi, > > > >With RAID5, using 3 loop-back devices each has 4GB. On a fresh setup, > >created a 2mb file. > >While checking no.of extents,it

[PATCH v3] btrfs-progs: RAID5: Inject data stripe corruption and verify scrub fixes it and retains correct parity.

2017-02-20 Thread Lakshmipathi.G
Test script to create file with specific data-stripe layout.Computes stripe locations. Inject corruption into data-stripe and verify scrubbing process fixes corrupted block. It also validates the corresponding parity stripe. Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co

Re: [PATCH v2] btrfs-progs: misc-tests: Superblock corruption and recovery using backup.

2017-02-16 Thread Lakshmipathi.G
On Thu, Feb 16, 2017 at 09:05:02AM +0800, Qu Wenruo wrote: > > > At 02/16/2017 04:50 AM, Lakshmipathi.G wrote: > >Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> > > Looks good to me. > > Reviewed by: Qu Wenruo <quwen...@cn.fujitsu.com> &g

[PATCH v3] btrfs-progs: misc-tests: Superblock corruption and recovery using backup.

2017-02-16 Thread Lakshmipathi.G
Test script to recover damaged primary superblock using backup superblock. Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- .../019-fix-superblock-corruption/test.sh | 36 ++ 1 file changed, 36 insertions(+) create mode 100755 tests/misc-tests/0

[PATCH v2] btrfs-progs: RAID5:Inject data stripe corruption and verify scrub fixes it.

2017-02-16 Thread Lakshmipathi.G
Test script to create file with specific data-stripe layout and pre-computed parity values. Inject corruption into data-stripe and verify scrubbing process fixes corrupted block. It also attempts to validate the corresponding parity block value. Signed-off-by: Lakshmipathi.G <lakshmip

Large no.of extents for a single small file with a fresh RAID5 setup

2017-02-17 Thread Lakshmipathi.G
disk size 2113536 logical size 2097152 ratio 0.99 Is this expected behaviour? Is there any mount option to minimize no.of extents for a smaller file? Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message

List of corruption cases for scrub

2017-02-23 Thread Lakshmipathi.G
entries or suggest new corrupt ideas! Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: How to dump/find parity of RAID-5 file?

2017-02-14 Thread Lakshmipathi.G
3-drives with 128KB file-size. Will try to understand Raid-6 little later. thanks. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: How to dump/find parity of RAID-5 file?

2017-02-14 Thread Lakshmipathi.G
sometime in future we will have your finished patches :) 'physical-find' and 'physical-find' commands will be really useful for debugging/testing and learning purposes. thanks. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH] btrfs-progs: misc-tests: Primary Superblock corruption and recovery using backup Superblock.

2017-02-14 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- .../019-fix-corrupted-superblock/test.sh | 37 ++ 1 file changed, 37 insertions(+) create mode 100755 tests/misc-tests/019-fix-corrupted-superblock/test.sh diff --git a/tests/misc-tests/019-fix-cor

Re: [PATCH] btrfs-progs: misc-tests: Primary Superblock corruption and recovery using backup Superblock.

2017-02-15 Thread Lakshmipathi.G
btrfs check' and 'btrfs check --repair' but it seems like --repair don't fix the corruption. So just moved away from using it. After you mentioned, now I see 'btrfs check' will be helpful. Will make these changes. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsu

[PATCH] btrfs-progs: RAID5:Inject data stripe corruption and verify scrub fixes it.

2017-02-15 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- .../020-raid5-datastripe-corruption/test.sh| 224 + 1 file changed, 224 insertions(+) create mode 100755 tests/misc-tests/020-raid5-datastripe-corruption/test.sh diff --git a/tests/misc-tests/020

Re: [PATCH] btrfs-progs: RAID5:Inject data stripe corruption and verify scrub fixes it.

2017-02-16 Thread Lakshmipathi.G
On Thu, Feb 16, 2017 at 09:12:31AM +0800, Qu Wenruo wrote: > > > At 02/16/2017 04:56 AM, Lakshmipathi.G wrote: > >On Wed, Feb 15, 2017 at 05:29:33PM +0800, Qu Wenruo wrote: > >> > >> > >>At 02/15/2017 05:03 PM, Lakshmipathi.G wrote: > >>>Si

Re: How to dump/find parity of RAID-5 file?

2017-02-16 Thread Lakshmipathi.G
it should be possible to access parity/data-stripes more sensible manner instead using expensive "cat /device/ | hexdump | grep" combination. thanks. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH v2] btrfs-progs: misc-tests: Superblock corruption and recovery using backup.

2017-02-15 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- .../019-fix-superblock-corruption/test.sh | 38 ++ 1 file changed, 38 insertions(+) create mode 100755 tests/misc-tests/019-fix-superblock-corruption/test.sh diff --git a/tests/misc-tests/0

Re: [PATCH] btrfs-progs: RAID5:Inject data stripe corruption and verify scrub fixes it.

2017-02-15 Thread Lakshmipathi.G
On Wed, Feb 15, 2017 at 05:29:33PM +0800, Qu Wenruo wrote: > > > At 02/15/2017 05:03 PM, Lakshmipathi.G wrote: > >Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> > >--- > > .../020-raid5-datastripe-corruption/test.sh| 224 > > ++

[PATCH] xfstests: btrfs/047: check btrfs-convert with extent and non-extent source

2017-01-17 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/btrfs/047 | 108 tests/btrfs/047.out | 1 + tests/btrfs/group | 1 + 3 files changed, 110 insertions(+) create mode 100755 tests/btrfs/047 create mode 100644

Re: List of corruption cases for scrub

2017-02-27 Thread Lakshmipathi.G
On Mon, Feb 27, 2017 at 11:40:31AM +0800, Qu Wenruo wrote: > > > At 02/24/2017 10:32 AM, Lakshmipathi.G wrote: > >Hi. > > > >I tried to a create list of corruption test scenarios for scrubbing process > >with RAID5. > >Here's the list: >

Re: List of corruption cases for scrub

2017-02-26 Thread Lakshmipathi.G
On Fri, Feb 24, 2017 at 06:39:20PM +0100, Goffredo Baroncelli wrote: > Hi, > On 2017-02-24 03:32, Lakshmipathi.G wrote: > > Hi. > > > > I tried to a create list of corruption test scenarios for scrubbing process > > with RAID5. > > Here's the list: > &g

[PATCH]btrfs-progs: Post btrfs-convert verify permissions and acls

2016-09-05 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/common.convert | 95 +++- 1 file changed, 94 insertions(+), 1 deletion(-) diff --git a/tests/common.convert b/tests/common.convert index 4e3d49c..67c99b1 100644 --- a

[PATCH]btrfs-progs: btrfs-debugfs: cleanup unused variables reported by pylint

2016-10-10 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- btrfs-debugfs | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/btrfs-debugfs b/btrfs-debugfs index 0a654a6..dfb8853 100755 --- a/btrfs-debugfs +++ b/btrfs-debugfs @@ -4,7 +4,7 @@ # LGPLv2 license # Cop

[PATCH]btrfs-progs: Add fast,slow symlinks,fifo types to convert test

2016-10-09 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/common.convert | 20 +--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/tests/common.convert b/tests/common.convert index 1e00d38..5083e65 100644 --- a/tests/common.convert +++ b

[PATCH]btrfs-progs: Add fast,slow symlinks and fifo types to convert test

2016-09-15 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/common.convert | 18 +++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/tests/common.convert b/tests/common.convert index 67c99b1..2790be5 100644 --- a/tests/common.convert +++ b

[PATCH]btrfs-progs: btrfs-convert.c : check source file system state

2016-09-15 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- btrfs-convert.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/btrfs-convert.c b/btrfs-convert.c index c10dc17..27da9ce 100644 --- a/btrfs-convert.c +++ b/btrfs-convert.c @@ -2171,6 +2171,17 @@ stati

btrfs-convert: migrating files with flags set using chattr

2016-10-09 Thread Lakshmipathi.G
-convert test.img [root@ mnt]# lsattr a b a b Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org

Re: btrfs-convert: migrating files with flags set using chattr

2016-10-10 Thread Lakshmipathi.G
Hi Qu, Thanks for resolving the issue and update. I'll re-use your testcase with common.convert/generate_dataset() when it becomes available on master branch. Cheers, Lakshmipathi.G On Mon, Oct 10, 2016 at 6:54 AM, Qu Wenruo <quwen...@cn.fujitsu.com> wrote: > Thanks for the repo

[PATCH] btrfs-progs: Corruption-framework: Include inode fields

2017-01-05 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- btrfs-corrupt-block.c | 48 1 file changed, 48 insertions(+) diff --git a/btrfs-corrupt-block.c b/btrfs-corrupt-block.c index a2f35ab..0e1eb52 100644 --- a/btrfs-corrupt-block.c

[PATCH] btrfs-progs: fsck-tests: missing csum test script

2017-01-05 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/fsck-tests/027-missing-data-csum/test.sh | 39 ++ 1 file changed, 39 insertions(+) create mode 100755 tests/fsck-tests/027-missing-data-csum/test.sh diff --git a/tests/fsck-tests/027-missing-dat

[PATCH] btrfs-progs: btrfs-debugfs: Display usage hint with no arguments

2017-01-03 Thread Lakshmipathi.G
--- btrfs-debugfs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/btrfs-debugfs b/btrfs-debugfs index dfb8853..70419fa 100755 --- a/btrfs-debugfs +++ b/btrfs-debugfs @@ -392,7 +392,9 @@ parser.add_argument('-f', '--file', action='store_const', const=1, help='get fil

[PATCH] btrfs-progs: btrfs-debugfs: Display usage hint with no arguments

2017-01-03 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- btrfs-debugfs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/btrfs-debugfs b/btrfs-debugfs index dfb8853..70419fa 100755 --- a/btrfs-debugfs +++ b/btrfs-debugfs @@ -392,7 +392,9 @@ parser.add_argume

Re: [PATCH] btrfs-progs: btrfs-debugfs: Display usage hint with no arguments

2017-01-03 Thread Lakshmipathi.G
, Lakshmipathi.G FOSS Programmer. www.giis.co.in On Tue, Jan 3, 2017 at 8:16 PM, David Sterba <dste...@suse.cz> wrote: > This is what I see when no arguments pare passed: > > $ ./btrfs-debugfs > usage: btrfs-debugfs [-h] [-b] [-f] path [path ...] > btrfs-debugfs: err

Re:[PATCH] btrfs-progs: btrfs-debugfs: Display usage hint with no arguments

2017-01-03 Thread Lakshmipathi.G
Yes. /mnt/file.txt is a mandatory argument. And -h/-b/-f are optional arugments. But the issue is, one of these optional argument is must. If we run: btrfs-debugfs /mnt/file.txt doesn't produce any output at all. From time to time, I run 'btrfs-debugfs /path/to/file' and wonder why no output

Re: [PATCH] btrfs-progs: btrfs-debugfs: Display usage hint with no arguments

2017-01-04 Thread Lakshmipathi.G
I think we can achieve above help string with the help of custom usage function instead of using argparse default usage. I'll check and send a new patch. Cheers, Lakshmipathi.G FOSS Programmer. www.giis.co.in On Wed, Jan 4, 2017 at 6:14 AM, Qu Wenruo <quwen...@cn.fujitsu.com>

[PATCH] btrfs-progs: Corruption-framework: Include inode nlink field

2017-01-03 Thread Lakshmipathi.G
Will include other fields, if this gets accepted. Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- btrfs-corrupt-block.c | 8 1 file changed, 8 insertions(+) diff --git a/btrfs-corrupt-block.c b/btrfs-corrupt-block.c index 16680df..64376ca 100644 --- a/btrfs-c

[PATCH] btrfs-progs: Corruption-framework: Include inode nlink field

2017-01-03 Thread Lakshmipathi.G
Will include other fields, if this gets accepted. Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- btrfs-corrupt-block.c | 8 1 file changed, 8 insertions(+) diff --git a/btrfs-corrupt-block.c b/btrfs-corrupt-block.c index 16680df..64376ca 100644 --- a/btrfs-c

[PATCH] btrfs-progs: Corruption-framework: Include inode nlink field

2017-01-05 Thread Lakshmipathi.G
Patch with fix for David Sterba review comment. Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- btrfs-corrupt-block.c | 8 1 file changed, 8 insertions(+) diff --git a/btrfs-corrupt-block.c b/btrfs-corrupt-block.c index 16680df..a2f35ab 100644 --- a/btrfs-corrupt-b

[PATCH] btfs-progs: fsck-tests: corrupt nlink value test

2017-01-07 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/fsck-tests/025-wrong-inode-nlink/test.sh | 37 ++ 1 file changed, 37 insertions(+) create mode 100755 tests/fsck-tests/025-wrong-inode-nlink/test.sh diff --git a/tests/fsck-tests/025-wrong-inode

[PATCH v4] btrfs-progs: misc-tests: Superblock corruption and recovery using backup

2017-04-15 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- .../020-fix-superblock-corruption/test.sh | 35 ++ 1 file changed, 35 insertions(+) create mode 100755 tests/misc-tests/020-fix-superblock-corruption/test.sh diff --git a/tests/misc-tests/0

Re: [PATCH] btrfs-progs: cli-tests: Convert non-raid filesystem to raid

2017-04-15 Thread Lakshmipathi.G
On Mon, Mar 13, 2017 at 06:47:50PM +0100, David Sterba wrote: > On Sun, Jan 29, 2017 at 08:14:55PM +0530, Lakshmipathi.G wrote: > > Simple script to verify non-raid filesystem conversion. > > The cli (command line interface) tests are supposed to cover the common > useca

[PATCH v4] btrfs-progs: fsck-tests: verify 'btrfs check --repair' fixes corrupted nlink field

2017-04-15 Thread Lakshmipathi.G
commit 5cdcc57b8d599f03fe692e0285ee9e17f32267fa Author: Lakshmipathi.G <lakshmipath...@giis.co.in> Date: Sat Apr 15 14:26:20 2017 +0530 btrfs-progs: fsck-tests: verify 'btrfs check --repair' fixes corrupted nlink field Signed-off-by: Lakshmipathi.G <lakshmipath...@g

Re: [PATCH v3] btrfs-progs: fsck-tests: verify 'btrfs check --repair' fixes corrupted nlink field

2017-04-15 Thread Lakshmipathi.G
On Thu, Mar 30, 2017 at 05:52:11PM +0200, David Sterba wrote: > On Thu, Feb 02, 2017 at 01:35:02PM +0530, Lakshmipathi.G wrote: > > Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> > > --- > > tests/fsck-tests/026-check-inode-link/test.sh | 30 > > ++

Re: [PATCH v3] btrfs-progs: misc-tests: Superblock corruption and recovery using backup.

2017-04-15 Thread Lakshmipathi.G
On Mon, Mar 13, 2017 at 06:15:04PM +0100, David Sterba wrote: > On Fri, Feb 17, 2017 at 04:01:59AM +0530, Lakshmipathi.G wrote: > > Test script to recover damaged primary superblock using backup superblock. > > > > Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in&g

Re: [PATCH 9/9] btrfs-progs: modify: Introduce option to specify the pattern to fill mirror

2017-04-18 Thread Lakshmipathi.G
Nice. With this new option, it should be possible to achieve something like RAID56 corruption test-script say, https://patchwork.kernel.org/patch/9583455/ right? Cheers, Lakshmipathi.G FOSS Programmer. http://www.giis.co.in http://www.webminal.org On Mon, Apr 17, 2017 at 8:56 AM, Qu Wenruo

[PATCH] btrfs-progs: tests: add variable quotation to convert-tests

2017-04-23 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/convert-tests.sh | 2 +- tests/convert-tests/001-ext2-basic/test.sh | 4 ++-- tests/convert-tests/002-ext3-basic/test.sh | 4 ++-- tests/convert-tests/003-ext4-basic/t

Re: [PATCH 9/9] btrfs-progs: modify: Introduce option to specify the pattern to fill mirror

2017-04-23 Thread Lakshmipathi.G
pected to find this and fix it) where as optype xor => should flip the bits at given offset. -- Cheers, Lakshmipathi.G FOSS Programmer. http://www.giis.co.in http://www.webminal.org On Thu, Apr 20, 2017 at 6:30 AM, Qu Wenruo <quwen...@cn.fujitsu.com> wrote: > > >

Re: Building a BTRFS test machine

2017-08-14 Thread Lakshmipathi.G
ugs based on mailing-list responses will be little tricky than creating new bugs. something like that can be useful as well. I can help a bit on such testbed setup :-) Thanks! Cheers, Lakshmipathi.G http://www.giis.co.in http://www.webminal.org On Mon, Aug 14, 2017 at 6:31 AM, Ce

Re: [PATCH] Btrfs-progs: convert: do not clear nodatasum flag in inode item

2017-06-28 Thread Lakshmipathi.G
Could you please add also a test-script for this interesting issue under tests/convert-tests (If its not already there)?. thanks. Cheers, Lakshmipathi.G http://www.giis.co.in http://www.webminal.org On Thu, Jun 29, 2017 at 2:59 AM, Liu Bo <bo.li@oracle.com> wrote: > With th

btrfs-convert --no-datasum & btrfs check: check_owner_ref: BUG_ON `rec->is_root` triggered, value 1

2017-07-02 Thread Lakshmipathi.G
iggered, value 1 Is this a bug or something wrong with my environment? More info: https://bugzilla.kernel.org/show_bug.cgi?id=196217 Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org Mo

Re: btrfs-convert --no-datasum & btrfs check: check_owner_ref: BUG_ON `rec->is_root` triggered, value 1

2017-07-02 Thread Lakshmipathi.G
Looked like I missed few patch. After checking with latest branch. Its working. Cheers, Lakshmipathi.G On Sun, Jul 2, 2017 at 7:30 PM, Lakshmipathi.G <lakshmipath...@gmail.com> wrote: > Hi. > While using btrfs-convert with '--no-datasum' option & running btrfs > check

[PATCH] btrfs-progs: btrfs-convert: Add larger device support

2017-04-27 Thread Lakshmipathi.G
Bug: Fail to convert 22TB EXT4 to BTRFS https://bugzilla.kernel.org/show_bug.cgi?id=194795 Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- convert/source-ext2.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/convert/source-ext2.c b/convert/

[PATCH] btrfs-progs: tests: add variable quotation to fsck-tests

2017-04-24 Thread Lakshmipathi.G
Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- tests/fsck-tests/006-bad-root-items/test.sh| 6 +++--- tests/fsck-tests/012-leaf-corruption/test.sh | 24 +++--- tests/fsck-tests/013-extent-tree-rebuild/test.sh | 22 ++-- test

[PATCH] btrfs-progs: Introduce 'btrfs inspect-internal dump-csum' option

2017-04-24 Thread Lakshmipathi.G
Example usage: btrfs inspect-internal dump-csum /btrfs/50gbfile /dev/sda4 csum for /btrfs/50gbfile dumped to /btrfs/50gbfile.csumdump Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- Makefile | 3 +- cmds-inspect-dump-csum.c

Re: [PATCH 1/2 v2] btrfs-progs: fix btrfs send & receive with -e flag

2017-04-28 Thread Lakshmipathi.G
Hi. Adding the bug reporter, Nazar for the discussion (as I'm not familiar with send/receive feature/code). Cheers, Lakshmipathi.G http://www.giis.co.in http://www.webminal.org On Fri, Apr 28, 2017 at 3:25 PM, Christian Brauner <christian.brau...@canonical.com> wrote: > > Hi

Re: [PATCH] btrfs-progs: btrfs-convert: Add larger device support

2017-04-27 Thread Lakshmipathi.G
Hi Qu, Seems like its a known issue with older bitmaps. http://linux-ext4.vger.kernel.narkive.com/wkNbJe0b/patch-debugfs-open-with-ext2-flag-64bits Cheers. Lakshmipathi.G On Fri, Apr 28, 2017 at 09:48:10AM +0800, Qu Wenruo wrote: > > > At 04/28/2017 09:27 AM, Qu Wenruo wrote: > &g

Re: [PATCH 4/5] btrfs-progs: tests: fix typo in convert-tests/008-readonly-image

2017-07-31 Thread Lakshmipathi.G
> if [ $? -ne 1 ]; then > echo "after convert ext2_save/image is not read-only" Though this mayn't affect test-script result, please change the typo in this location also. thanks. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe lin

Re: [PATCH] btrfs-progs: btrfs-convert: Add larger device support

2017-05-03 Thread Lakshmipathi.G
rage. Sure, will send the above test script in upcoming days, then look into mkfs.ext4 options. thanks. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2] btrfs-progs: btrfs-convert: Add larger device support

2017-05-15 Thread Lakshmipathi.G
k_count from u32 to u64? do we also need to change other fields like inodes_count and free_inode_count? Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] btrfs-progs: btrfs-convert: Add larger device support

2017-05-09 Thread Lakshmipathi.G
-- Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH v2] btrfs-progs: btrfs-convert: Add larger device support

2017-05-13 Thread Lakshmipathi.G
, ext2fs_inode_data_blocks2,ext2fs_read_ext_attr2 bug: https://bugzilla.kernel.org/show_bug.cgi?id=194795 Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- convert/source-ext2.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/convert/source-ext2.c b/c

Re: [PATCH v3 00/19] Btrfs-progs offline scrub

2017-05-13 Thread Lakshmipathi.G
> > Ping? > > Any comments? > > Thanks, > Qu Can I inject corruption with existing script [1] and expect offline scrub to fix it? If so, I'll give it try and let you know the results. [1] https://patchwork.kernel.org/patch/9583455/ Cheers, Lakshmipathi.G -- To unsub

Re: [PATCH] btrfs-progs: tests: remove variable quotation from convert-tests

2017-05-15 Thread Lakshmipathi.G
t; > > Tsutomu Itoh t-i...@jp.fujitsu.com > -- > To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Cheers, Lakshmipathi.G http://www.giis.co.in http://www.webminal.org -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] btrfs-progs: btrfs-convert: Add larger device support

2017-05-09 Thread Lakshmipathi.G
run_check fails to detect mke2fs command. Something wrong with my setup or issue with run_check? -- Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.

btrfs-convert: whats minimum free space requirement?

2017-06-08 Thread Lakshmipathi.G
Hi. Just wanted to check whether do we have any numbers on whats the minimum free space requirement on source file system for btrfs-convert to work? ex: Like 5% of ext3/4 free space is needed for btrfs-convert to succeed? Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line

Re: btrfs-convert: whats minimum free space requirement?

2017-06-09 Thread Lakshmipathi.G
rt. > > Thanks, > Qu Thanks for the details, Qu. I was thinking about adding a free-space check (based on total blocks/free block stat from source superblock) before btrfs-convert begins, looks like its more complex than i thought. Cheers. Lakshmipathi.G -- To unsubscribe from this

[PATCH] Fix 'btrfs device stats --check' cli option

2017-06-22 Thread Lakshmipathi.G
Reported by Tomas Thiemel Bug 194961 - btrfs device stats --check does not work --- cmds-device.c | 1 + 1 file changed, 1 insertion(+) diff --git a/cmds-device.c b/cmds-device.c index 5e016a7..4337eb2 100644 --- a/cmds-device.c +++ b/cmds-device.c @@ -400,6 +400,7 @@

[PATCH v2] btrfs-progs: Fix 'btrfs device stats --check' cli option

2017-06-22 Thread Lakshmipathi.G
Bug 194961 - btrfs device stats --check does not work https://bugzilla.kernel.org/show_bug.cgi?id=194961 Reported-by: Tomas Thiemel<thie...@centrum.cz> Signed-off-by: Lakshmipathi.G <lakshmipath...@giis.co.in> --- cmds-device.c | 1 + 1 file changed, 1 insertion(+) diff --git a/cmd

Re: Stability status of btrfs-convert...

2017-06-22 Thread Lakshmipathi.G
te number of people converted their EXT4 to BTRFS . IMO, btrfs-convert is fairly stable, especially if you have only GB's of data Cheers, Lakshmipathi.G http://www.giis.co.in http://www.webminal.org -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of

Re: [PATCH v4] btrfs-progs: btrfs-convert: Add larger device support

2017-06-26 Thread Lakshmipathi.G
is simple. Okay sure, thanks. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v3 00/19] Btrfs-progs offline scrub

2017-05-23 Thread Lakshmipathi.G
/btrfs_offline_scrub -- Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v3 00/19] Btrfs-progs offline scrub

2017-05-23 Thread Lakshmipathi.G
og file (/var/log/nginx_access_20160101.log) I don't need to panic, as I can live without that file. Or possibly delete that file, so that scrub doesn't report errors. -- Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message

Re: [PATCH v3 00/19] Btrfs-progs offline scrub

2017-05-22 Thread Lakshmipathi.G
tch/9583455/ > > No fixing yet. > > This part is focusing on error detection and report. > > Feel free to inject corruption and to see if it behave as expected. > > Thanks, > Qu > Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe l

Re: [PATCH v3 00/19] Btrfs-progs offline scrub

2017-05-22 Thread Lakshmipathi.G
Tree extents scrubbed: 8 Data bytes scrubbed: 192512 Data extents scrubbed: 3 Data bytes without csum: 131072 Read error: 0 Verify error: 0 Csum error: 1 Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message

Re: [PATCH v3 00/19] Btrfs-progs offline scrub

2017-05-22 Thread Lakshmipathi.G
s, > Qu > Ah,my bad. I thought the entire binary is changed, didn't realize its an option. Will check again with 'offline' option. -- Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vge

Re: [PATCH v3 00/19] Btrfs-progs offline scrub

2017-05-22 Thread Lakshmipathi.G
although I have done a lot during > the test) to see if it's good enough to be used as a better scrub > equivalent. > > And of course, any idea on the interface design (--offline under scrub > or other place) is welcomed. > > Thanks, > Qu > > At 05/22/2017 04:30

Re: [PATCH 2/2] btrfs-progs: test for restoring multiple devices fs into a single device

2017-05-19 Thread Lakshmipathi.G
uot; can be done after comparing file integrity? This way, if integrity fails, the setup will be there to debug further. thanks. -- Cheers, Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org

Re: [PATCH v2] btrfs-progs: btrfs-convert: Add larger device support

2017-06-01 Thread Lakshmipathi.G
patch. Cheers. Lakshmipathi.G -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

  1   2   >