[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241031192350.GA26688@lichtman.org>
Date: Thu, 31 Oct 2024 19:23:50 +0000
From: Nir Lichtman <nir@...htman.org>
To: jason.wessel@...driver.com, daniel.thompson@...aro.org,
dianders@...omium.org, linux-kernel@...r.kernel.org
Subject: [PATCH] kdb: Fix incorrect naming of history arrow keys in code
Problem: The kdb CLI code that handles the history up and down
navigation incorrectly names the up and down arrows as ctrl p and n.
Details: This could be some kind of left over legacy.
(maybe inspired by ddb which only reacts to ctrl p and n for history nav).
kdb doesn't react to ctrl p and n, and following the code flow with GDB
reveals that these values map to the up and down arrows.
Solution: Rename the macros accordingly and rename the function name
to reflect that it relates to arrows and not ctrl commands.
Signed-off-by: Nir Lichtman <nir@...htman.org>
---
kernel/debug/kdb/kdb_main.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/kernel/debug/kdb/kdb_main.c b/kernel/debug/kdb/kdb_main.c
index f5f7d7fb5936..d4b407afb888 100644
--- a/kernel/debug/kdb/kdb_main.c
+++ b/kernel/debug/kdb/kdb_main.c
@@ -1123,22 +1123,22 @@ int kdb_parse(const char *cmdstr)
}
-static int handle_ctrl_cmd(char *cmd)
+static int handle_arrow_cmd(char *cmd)
{
-#define CTRL_P 16
-#define CTRL_N 14
+#define ARROW_UP 16
+#define ARROW_DOWN 14
/* initial situation */
if (cmd_head == cmd_tail)
return 0;
switch (*cmd) {
- case CTRL_P:
+ case ARROW_UP:
if (cmdptr != cmd_tail)
cmdptr = (cmdptr + KDB_CMD_HISTORY_COUNT - 1) %
KDB_CMD_HISTORY_COUNT;
strscpy(cmd_cur, cmd_hist[cmdptr], CMD_BUFLEN);
return 1;
- case CTRL_N:
+ case ARROW_DOWN:
if (cmdptr != cmd_head)
cmdptr = (cmdptr+1) % KDB_CMD_HISTORY_COUNT;
strscpy(cmd_cur, cmd_hist[cmdptr], CMD_BUFLEN);
@@ -1351,7 +1351,7 @@ static int kdb_local(kdb_reason_t reason, int error, struct pt_regs *regs,
*(cmd_hist[cmd_head] +
strlen(cmd_hist[cmd_head])-1) = '\0';
}
- if (!handle_ctrl_cmd(cmdbuf))
+ if (!handle_arrow_cmd(cmdbuf))
*(cmd_cur+strlen(cmd_cur)-1) = '\0';
cmdbuf = cmd_cur;
goto do_full_getstr;
--
2.39.2
Powered by blists - more mailing lists