* [PATCH v2] xfs/021: filter out detailed array element info from xfs_db
@ 2016-11-10 4:29 Eryu Guan
2016-11-10 4:33 ` Zorro Lang
2016-11-13 12:29 ` Christoph Hellwig
0 siblings, 2 replies; 3+ messages in thread
From: Eryu Guan @ 2016-11-10 4:29 UTC (permalink / raw)
To: fstests; +Cc: linux-xfs, Eryu Guan
Starting from xfsprogs commit 384283555871 ("xfs_db: print one array
element per line"), xfs_db prints one array element per line. This
breaks the filter in xfs/021, which now fails as:
hdr.freemap[0-2] = [base,size] [FREEMAP..]
+0:[104,1892]
+1:[0,0]
+2:[0,0]
entries[0-2] = [hashval,nameidx,incomplete,root,local] [ENTRIES..]
So we have extra lines that need to be filtered out,
Signed-off-by: Eryu Guan <eguan@redhat.com>
---
- be more precise on matching the array output, address Zorro's comment
tests/xfs/021 | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/xfs/021 b/tests/xfs/021
index 67a4346..0471b95 100755
--- a/tests/xfs/021
+++ b/tests/xfs/021
@@ -151,7 +151,7 @@ s/^(hdr.info.magic =) 0x3bee/\1 0xfbee/;
s/^(hdr.firstused =) (\d+)/\1 FIRSTUSED/;
s/^(hdr.freemap\[0-2] = \[base,size]).*/\1 [FREEMAP..]/;
s/^(entries\[0-2] = \[hashval,nameidx,incomplete,root,local]).*/\1 [ENTRIES..]/;
- print;'
+ print unless /^\d+:\[.*/;'
echo "*** done"
exit
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH v2] xfs/021: filter out detailed array element info from xfs_db
2016-11-10 4:29 [PATCH v2] xfs/021: filter out detailed array element info from xfs_db Eryu Guan
@ 2016-11-10 4:33 ` Zorro Lang
2016-11-13 12:29 ` Christoph Hellwig
1 sibling, 0 replies; 3+ messages in thread
From: Zorro Lang @ 2016-11-10 4:33 UTC (permalink / raw)
To: Eryu Guan; +Cc: fstests, linux-xfs
On Thu, Nov 10, 2016 at 12:29:43PM +0800, Eryu Guan wrote:
> Starting from xfsprogs commit 384283555871 ("xfs_db: print one array
> element per line"), xfs_db prints one array element per line. This
> breaks the filter in xfs/021, which now fails as:
>
> hdr.freemap[0-2] = [base,size] [FREEMAP..]
> +0:[104,1892]
> +1:[0,0]
> +2:[0,0]
> entries[0-2] = [hashval,nameidx,incomplete,root,local] [ENTRIES..]
>
> So we have extra lines that need to be filtered out,
>
> Signed-off-by: Eryu Guan <eguan@redhat.com>
> ---
V2 looks better to me,
Reviewed-by: Zorro Lang <zlang@redhat.com>
> - be more precise on matching the array output, address Zorro's comment
>
> tests/xfs/021 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/xfs/021 b/tests/xfs/021
> index 67a4346..0471b95 100755
> --- a/tests/xfs/021
> +++ b/tests/xfs/021
> @@ -151,7 +151,7 @@ s/^(hdr.info.magic =) 0x3bee/\1 0xfbee/;
> s/^(hdr.firstused =) (\d+)/\1 FIRSTUSED/;
> s/^(hdr.freemap\[0-2] = \[base,size]).*/\1 [FREEMAP..]/;
> s/^(entries\[0-2] = \[hashval,nameidx,incomplete,root,local]).*/\1 [ENTRIES..]/;
> - print;'
> + print unless /^\d+:\[.*/;'
>
> echo "*** done"
> exit
> --
> 2.7.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH v2] xfs/021: filter out detailed array element info from xfs_db
2016-11-10 4:29 [PATCH v2] xfs/021: filter out detailed array element info from xfs_db Eryu Guan
2016-11-10 4:33 ` Zorro Lang
@ 2016-11-13 12:29 ` Christoph Hellwig
1 sibling, 0 replies; 3+ messages in thread
From: Christoph Hellwig @ 2016-11-13 12:29 UTC (permalink / raw)
To: Eryu Guan; +Cc: fstests, linux-xfs
Looks good,
Reviewed-by: Christoph Hellwig <hch@lst.de>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-11-13 12:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-10 4:29 [PATCH v2] xfs/021: filter out detailed array element info from xfs_db Eryu Guan
2016-11-10 4:33 ` Zorro Lang
2016-11-13 12:29 ` Christoph Hellwig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).