[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: 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

[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: 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

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: 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 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: [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

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

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

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: 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

[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

[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

[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

[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-

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. >

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: [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

[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

[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] 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: 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

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

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-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

[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

[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: 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

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

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

[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: 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

[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: 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] 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] 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

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

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-

<    1   2