Thread (12 messages) 12 messages, 3 authors, 2021-10-08

Re: [PATCH v2 4/4] common/log: Fix *_dump_log routines for ext4

From: Allison Henderson <hidden>
Date: 2021-10-08 17:49:33
Also in: fstests


On 10/6/21 5:26 PM, Catherine Hoang wrote:
dumpe2fs -h displays the superblock contents, not the journal contents.
Use the logdump utility to dump the contents of the journal.

Signed-off-by: Catherine Hoang <redacted>
Looks good to me.  Thanks for the fix
Reviewed-by: Allison Henderson <redacted>
quoted hunk ↗ jump to hunk
---
  common/log | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/common/log b/common/log
index 0a9aaa7f..154f3959 100644
--- a/common/log
+++ b/common/log
@@ -229,7 +229,7 @@ _scratch_dump_log()
  		$DUMP_F2FS_PROG $SCRATCH_DEV
  		;;
  	ext4)
-		$DUMPE2FS_PROG -h $SCRATCH_DEV
+		$DEBUGFS_PROG -R "logdump -a" $SCRATCH_DEV
  		;;
  	*)
  		;;
@@ -246,7 +246,7 @@ _test_dump_log()
  		$DUMP_F2FS_PROG $TEST_DEV
  		;;
  	ext4)
-		$DUMPE2FS_PROG -h $TEST_DEV
+		$DEBUGFS_PROG -R "logdump -a" $TEST_DEV
  		;;
  	*)
  		;;
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help