Changes in kgdb.c

78b5f52a - microblaze: fix typos in comments

Sat, 30 Apr 2022 19:11:15 +0000

microblaze: fix typos in commentsVarious spelling mistakes in comments.Detected with the help of Coccinelle.Signed-off-by: Julia Lawall Link: https://lore.kernel.org/r/20220430191122.8667-1-Julia.Lawall@inria.frSigned-off-by: Michal Simek List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

cc028297 - kgdb/treewide: constify struct kgdb_arch arch_kgdb_ops

Thu, 06 Dec 2018 20:07:40 +0000

kgdb/treewide: constify struct kgdb_arch arch_kgdb_opscheckpatch.pl reports the following: WARNING: struct kgdb_arch should normally be const #28: FILE: arch/mips/kernel/kgdb.c:397: +struct kgdb_arch arch_kgdb_ops = {This report makes sense, as all other ops struct, thisone should also be const. This patch does the change.Cc: Vineet Gupta Cc: Russell King Cc: Catalin Marinas Cc: Will Deacon Cc: Yoshinori Sato Cc: Richard Kuo Cc: Michal Simek Cc: Ralf Baechle Cc: Paul Burton Cc: James Hogan Cc: Ley Foon Tan Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc: Rich Felker Cc: "David S. Miller" Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Borislav Petkov Cc: x86@kernel.orgAcked-by: Daniel Thompson Acked-by: Paul Burton Signed-off-by: Christophe Leroy Acked-by: Borislav Petkov Acked-by: Michael Ellerman (powerpc)Signed-off-by: Daniel Thompson List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

55ae2f3b - microblaze: Label local function static

Fri, 05 Jun 2015 08:35:31 +0000

microblaze: Label local function staticWarnings found by sparse:arch/microblaze/kernel/dma.c:157:5: warning: symbol'dma_direct_mmap_coherent' was not declared. Should it be static?arch/microblaze/kernel/kgdb.c:35:14: warning: symbol 'pvr' was notdeclared. Should it be static?Signed-off-by: Michal Simek List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

e76fdb32 - microblaze: Use unsigned type for "for" loop because of comparison-kgdb.c

Thu, 18 Dec 2014 14:56:17 +0000

microblaze: Use unsigned type for "for" loop because of comparison-kgdb.cThis patch removes warnings reported by W=1:arch/microblaze/kernel/kgdb.c: In function 'pt_regs_to_gdb_regs':arch/microblaze/kernel/kgdb.c:43:16: warning: comparison between signedand unsigned integer expressions [-Wsign-compare]arch/microblaze/kernel/kgdb.c:51:16: warning: comparison between signedand unsigned integer expressions [-Wsign-compare]arch/microblaze/kernel/kgdb.c: In function 'gdb_regs_to_pt_regs':arch/microblaze/kernel/kgdb.c:77:16: warning: comparison between signedand unsigned integer expressions [-Wsign-compare]arch/microblaze/kernel/kgdb.c: In function'sleeping_thread_to_gdb_regs':arch/microblaze/kernel/kgdb.c:99:16: warning: comparison between signedand unsigned integer expressions [-Wsign-compare]arch/microblaze/kernel/kgdb.c:103:16: warning: comparison between signedand unsigned integer expressions [-Wsign-compare]Signed-off-by: Michal Simek List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

f396a4d2 - microblaze: Declare microblaze_kgdb_break in header

Thu, 18 Dec 2014 14:32:57 +0000

microblaze: Declare microblaze_kgdb_break in headerThis patch removes the warning:arch/microblaze/kernel/kgdb.c:81:6: warning: no previous prototype for'microblaze_kgdb_break' [-Wmissing-prototypes]Signed-off-by: Michal Simek List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

218a12f1 - microblaze: Don't mark arch_kgdb_ops as const.

Sat, 08 Jun 2013 16:52:08 +0000

microblaze: Don't mark arch_kgdb_ops as const.Other architectures don't do it, and it conflicts with the extern'd definitionin include/linux/kgdb.h.The patch fails checkpatch but it reflects currentfunctions declaration and solved compilation error.Signed-off-by: Graeme Smecher CC: Michal Simek CC: linux-kernel@vger.kernel.orgSigned-off-by: Michal Simek List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

6bd55f0b - microblaze: Fix coding style issues

Thu, 27 Dec 2012 09:40:38 +0000

microblaze: Fix coding style issuesFix coding style issues reported by checkpatch.pl.Signed-off-by: Michal Simek List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

e4f29092 - microblaze: KGDB little endian support

Tue, 28 Sep 2010 05:49:17 +0000

microblaze: KGDB little endian supportJust need to use little-endian opcode for brki r16, 0x18Signed-off-by: Michal Simek CC: Jason Wessel List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

f859f0a2 - microblaze: kgdb: Remove unused variable and fix return value

Tue, 10 Aug 2010 06:19:03 +0000

microblaze: kgdb: Remove unused variable and fix return valueI forget to remove unused variable from kgdb_arch_handle_exception.Fix return value in microblaze_kgdb_break function.Error log:arch/microblaze/kernel/kgdb.c: In function 'microblaze_kgdb_break':arch/microblaze/kernel/kgdb.c:83: warning: 'return' with a value, in function returning voidarch/microblaze/kernel/kgdb.c: In function 'kgdb_arch_handle_exception':arch/microblaze/kernel/kgdb.c:119: warning: unused variable 'cpu'Signed-off-by: Michal Simek List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c

2d5973cb - microblaze: Add KGDB support

Tue, 03 Aug 2010 09:45:08 +0000

microblaze: Add KGDB supportKgdb uses brki r16, 0x18 instruction to calllow level _debug_exception function which savecurrent state to pt_regs and call microblaze_kgdb_breakfunction. _debug_exception should be called only fromthe kernel space. User space calling is not supportedbecause user application debugging uses different handling.pt_regs_to_gdb_regs loads additional special registerswhich can't be changed * Enable KGDB in Kconfig * Remove ancient not-tested KGDB support * Remove ancient _debug_exception code from entry.SOnly MMU KGDB support is supported.Signed-off-by: Michal Simek CC: Jason Wessel CC: John Williams CC: Edgar E. Iglesias CC: linux-kernel@vger.kernel.orgAcked-by: Jason Wessel List of files: /linux-6.15/arch/microblaze/kernel/kgdb.c