Re: [ANNOUNCE] Merkey's Kernel Debugger

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Andi Kleen
Date: Tuesday, August 5, 2008 - 8:08 pm

Nick Piggin <nickpiggin@yahoo.com.au> writes:

I don't think kgdb and a simple assembler debugger 
are directly comparable. kgdb always requires a remote machine,
which has many advantages, but is also often very inconvenient
or impossible to arrange. An low overhead assembler debugger
can be always compiled in just in case.

Also at least for the x86 port the debugger interfaces should
be general enough now (see die hooks as a "debug vfs") that it would
be quite possible to have a multitude of debuggers just using 
them. In fact that's already the cases, kprobes and kgdb and 
kdump are all kinds of debuggers using such hooks.

As long as it doesn't impact the core code and the mdb 
code itself is considered merge worthy and has clean interfaces 
that would seem fine to me.It essentially would just live somewhere in 
its own directory using the existing interfaces. My standard
test for seeing if a debugger has clean interfaces is to see
if it can be loaded as a module.

There are enough different debugging styles around that offering
developers different tools of which they can pick whatever suits
them is not a bad idea. Also as everyone knows debugging
is often a major time eater and if more tools are available that 
can only help the kernel.

That said I haven't read the mdb code, not judging on its general
merge-worthiness or am really completely sure what are all the details
of a "netware style debugger", just a general high level comment on
debuggers. At least judging based on the patch sizes it at least
doesn't seem particularly bloated.  But of course it would need full
proper review first.

-Andi
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Sun Aug 3, 10:22 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Sun Aug 3, 12:36 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Rene Herman, (Sun Aug 3, 1:00 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Josh Boyer, (Sun Aug 3, 5:14 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Sun Aug 3, 7:19 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Stefan Richter, (Mon Aug 4, 6:41 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Mon Aug 4, 7:33 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Geert Uytterhoeven, (Tue Aug 5, 2:41 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Tue Aug 5, 8:02 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Tue Aug 5, 8:19 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Tue Aug 5, 8:32 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Nick Piggin, (Tue Aug 5, 8:33 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Nick Piggin, (Tue Aug 5, 8:45 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Chris Friesen, (Tue Aug 5, 9:04 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Nick Piggin, (Tue Aug 5, 9:38 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Tue Aug 5, 9:39 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Tue Aug 5, 9:45 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Tue Aug 5, 10:10 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Paul Mundt, (Tue Aug 5, 10:21 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Tue Aug 5, 8:08 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Nick Piggin, (Tue Aug 5, 10:50 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Bill Davidsen, (Wed Aug 6, 6:11 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Stefan Richter, (Wed Aug 6, 6:37 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Wed Aug 6, 6:45 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Olivier Galibert, (Wed Aug 6, 6:54 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Nick Piggin, (Wed Aug 6, 7:16 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Jason Wessel, (Wed Aug 6, 10:21 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Wed Aug 6, 11:57 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Rene Herman, (Wed Aug 6, 12:47 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Thu Aug 7, 5:45 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Peter Zijlstra, (Thu Aug 7, 8:17 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Thu Aug 7, 8:52 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Thu Aug 7, 9:07 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Stefan Richter, (Thu Aug 7, 10:04 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Christoph Lameter, (Thu Aug 7, 10:45 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Thu Aug 7, 10:53 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Stefan Richter, (Thu Aug 7, 11:08 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Christoph Lameter, (Thu Aug 7, 12:10 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Thu Aug 7, 12:34 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Jay Lan, (Thu Aug 7, 12:47 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Thu Aug 7, 1:06 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Bernhard Walle, (Thu Aug 7, 1:07 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Thu Aug 7, 1:09 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Bernhard Walle, (Thu Aug 7, 1:11 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Daniel Barkalow, (Thu Aug 7, 1:43 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Thu Aug 7, 2:02 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Thu Aug 7, 2:04 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger , Keith Owens, (Thu Aug 7, 3:28 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Stefan Richter, (Thu Aug 7, 5:28 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Vivek Goyal, (Thu Aug 7, 6:15 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Vivek Goyal, (Thu Aug 7, 6:26 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Thu Aug 7, 7:29 pm)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Nick Piggin, (Fri Aug 8, 1:40 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Cliff Wickman, (Fri Aug 8, 5:08 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Fri Aug 8, 5:20 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Fri Aug 8, 6:19 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Vivek Goyal, (Fri Aug 8, 6:29 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Cliff Wickman, (Fri Aug 8, 7:50 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Cliff Wickman, (Fri Aug 8, 8:06 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Jay Lan, (Fri Aug 8, 9:57 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Fri Aug 8, 11:03 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jidong xiao, (Mon Aug 11, 3:36 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Vivek Goyal, (Mon Aug 11, 5:56 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Vivek Goyal, (Mon Aug 11, 6:02 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Mon Aug 11, 6:11 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Andi Kleen, (Mon Aug 11, 6:50 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, jmerkey, (Mon Aug 11, 9:16 am)
Re: [ANNOUNCE] Merkey's Kernel Debugger, Takenori Nagano, (Mon Aug 18, 2:12 am)