diff mbox series

[v2] powerpc/xmon: symbol lookup length fixed

Message ID 20241024191232.1570894-2-mchauras@linux.ibm.com (mailing list archive)
State Accepted
Commit b196db2f536645eda7684655f3fae913e33fda4b
Headers show
Series [v2] powerpc/xmon: symbol lookup length fixed | expand

Commit Message

Mukesh Kumar Chaurasiya Oct. 24, 2024, 7:12 p.m. UTC
Currently this cannot lookup symbol beyond 64 characters in some cases
like "ls", "lp" and "t"

Fix this by using KSYM_NAME_LEN instead of fixed 64 characters

Signed-off-by: Mukesh Kumar Chaurasiya <mchauras@linux.ibm.com>
---
 arch/powerpc/xmon/xmon.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

ChangeLog v1 -> v2:
  - Commit message updated with commands

Comments

Michael Ellerman Nov. 17, 2024, 12:09 p.m. UTC | #1
On Fri, 25 Oct 2024 00:42:33 +0530, Mukesh Kumar Chaurasiya wrote:
> Currently this cannot lookup symbol beyond 64 characters in some cases
> like "ls", "lp" and "t"
> 
> Fix this by using KSYM_NAME_LEN instead of fixed 64 characters
> 
> 

Applied to powerpc/next.

[1/1] powerpc/xmon: symbol lookup length fixed
      https://git.kernel.org/powerpc/c/b196db2f536645eda7684655f3fae913e33fda4b

cheers
diff mbox series

Patch

diff --git a/arch/powerpc/xmon/xmon.c b/arch/powerpc/xmon/xmon.c
index e6cddbb2305f..22b8b5cc4df0 100644
--- a/arch/powerpc/xmon/xmon.c
+++ b/arch/powerpc/xmon/xmon.c
@@ -3662,7 +3662,7 @@  symbol_lookup(void)
 	int type = inchar();
 	unsigned long addr, cpu;
 	void __percpu *ptr = NULL;
-	static char tmp[64];
+	static char tmp[KSYM_NAME_LEN];
 
 	switch (type) {
 	case 'a':
@@ -3671,7 +3671,7 @@  symbol_lookup(void)
 		termch = 0;
 		break;
 	case 's':
-		getstring(tmp, 64);
+		getstring(tmp, KSYM_NAME_LEN);
 		if (setjmp(bus_error_jmp) == 0) {
 			catch_memory_errors = 1;
 			sync();
@@ -3686,7 +3686,7 @@  symbol_lookup(void)
 		termch = 0;
 		break;
 	case 'p':
-		getstring(tmp, 64);
+		getstring(tmp, KSYM_NAME_LEN);
 		if (setjmp(bus_error_jmp) == 0) {
 			catch_memory_errors = 1;
 			sync();