On Tue, Jul 2, 2024 at 2:31 PM Li Zhijian <lizhijian(a)fujitsu.com> wrote:
Previously, if LIST_HEAD_FORMAT was not set, `list -r` will traverse
the
list in order, that doesn't obey the -r(reverse) semantics.
Per the code, -r(LIST_HEAD_REVERSE) only work with LIST_HEAD_FORMAT and
'-r' usage also says that "For a list linked with list_head
structures",
so it's reasonable to enable LIST_HEAD_FORMAT for it.
Cc: Lianbo Jiang <lijiang(a)redhat.com>
Cc: Tao Liu <ltao(a)redhat.com>
Signed-off-by: Li Zhijian <lizhijian(a)fujitsu.com>
---
V2: enable LIST_HEAD_FORMAT for -r option instead of fatal when -h/-H is
not specified for -r. # Lianbo
---
tools.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools.c b/tools.c
index 0f2db108838a..1022d579071a 100644
--- a/tools.c
+++ b/tools.c
@@ -3370,6 +3370,7 @@ cmd_list(void)
break;
case 'r':
+ ld->flags |= LIST_HEAD_FORMAT;
ld->flags |= LIST_HEAD_REVERSE;
break;
Thank you for the update, and this looks good to me. So for the v2: Ack.
Lianbo
--
2.29.2