Re: [git pull] kgdb-light -v10

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Andi Kleen <andi@...>
Cc: <linux-kernel@...>, Frank Ch. Eigler <fche@...>, Roland McGrath <roland@...>, Thomas Gleixner <tglx@...>, H. Peter Anvin <hpa@...>, Linus Torvalds <torvalds@...>, Andrew Morton <akpm@...>
Date: Tuesday, February 12, 2008 - 11:28 am

* Andi Kleen <andi@firstfloor.org> wrote:


well, this is a small detail, but still you are wrong, and on a 
correctly working system this will not occur. (if yes, tell me how)

KGDB does a very straightforward "all CPUs enter controlled state" 
transition when the session begins, and at the end an "all CPUs 
continue" transition.

I'm not sure what you mean exactly under "stopping all CPUs for 
indefinite amount of time" (your statement is sufficiently vague to be 
hard to counter via specifics) - that does not happen, unless the system 
is so buggy that a CPU is not able to process an NMI anymore [which is 
rather rare] - in that case the whole system is likely locked up anyway. 
In that case the simplest and safest behavior is what kgdb-light does 
currently: it will only proceed if all CPUs have responded. Note that 
you are wrong to suggest that "KGDB locks up", the system _has already 
locked up_.

yes, we could "time out" and force a KGDB session even if some CPUs do 
not respond. But it's obviously not a completely safe system state, 
because other CPUs might be changing things under the feet of the 
debugger. So the safest first-level approach is to not enter the 
debugger in this case.

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

Messages in current thread:
kgdb in git-x86#mm review, Andi Kleen, (Sun Feb 10, 9:53 pm)
Re: kgdb in git-x86#mm review, Mark Lord, (Mon Feb 11, 12:03 pm)
Re: kgdb in git-x86#mm review, Frank Ch. Eigler, (Mon Feb 11, 11:32 am)
Re: [git pull] kgdb-light -v8,, Jan Kiszka, (Mon Feb 11, 12:41 pm)
Re: [git pull] kgdb-light -v8,, Ingo Molnar, (Mon Feb 11, 12:54 pm)
[git pull] kgdb-light -v9, Ingo Molnar, (Mon Feb 11, 7:03 pm)
Re: [git pull] kgdb-light -v9, Andi Kleen, (Tue Feb 12, 6:03 am)
[git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 7:27 am)
Re: [git pull] kgdb-light -v10, Domenico Andreoli, (Tue Feb 12, 9:18 am)
Re: [git pull] kgdb-light -v10, Jason Wessel, (Tue Feb 12, 9:59 am)
Re: [git pull] kgdb-light -v10, Domenico Andreoli, (Tue Feb 12, 11:45 am)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 8:19 am)
Re: [git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 8:38 am)
Re: [git pull] kgdb-light -v10, Jason Wessel, (Tue Feb 12, 9:30 am)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 10:39 am)
Re: [git pull] kgdb-light -v10, Jason Wessel, (Tue Feb 12, 10:35 am)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 11:36 am)
Re: [git pull] kgdb-light -v10, Jason Wessel, (Tue Feb 12, 12:21 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 1:10 pm)
Re: [git pull] kgdb-light -v10, Jason Wessel, (Tue Feb 12, 12:48 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 9:50 am)
Re: [git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 11:28 am)
Re: [git pull] kgdb-light -v10, Linus Torvalds, (Tue Feb 12, 12:46 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 2:20 pm)
Re: [git pull] kgdb-light -v10, Andrew Morton, (Tue Feb 12, 2:20 pm)
Re: [git pull] kgdb-light -v10, Frank Ch. Eigler, (Tue Feb 12, 3:34 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 4:16 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 3:16 pm)
Re: [git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 5:01 pm)
Re: [git pull] kgdb-light -v10, Linus Torvalds, (Tue Feb 12, 2:11 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 3:22 pm)
Re: [git pull] kgdb-light -v10, Linus Torvalds, (Tue Feb 12, 3:01 pm)
Re: [git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 1:01 pm)
Re: [git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 1:10 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 12:11 pm)
Re: [git pull] kgdb-light -v10, Linus Torvalds, (Tue Feb 12, 12:25 pm)
Re: [git pull] kgdb-light -v10, Jason Wessel, (Fri Feb 15, 4:36 pm)
Re: [git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 12:42 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 1:07 pm)
Re: [RFC][PATCH] modular kgdb-light, Jason Wessel, (Fri Feb 15, 4:24 pm)
Re: [git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 12:24 pm)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 1:01 pm)
Re: [git pull] kgdb-light -v10, Ingo Molnar, (Tue Feb 12, 11:16 am)
Re: [git pull] kgdb-light -v10, Andi Kleen, (Tue Feb 12, 11:28 am)
Re: [git pull] kgdb-light -v9, Roland McGrath, (Tue Feb 12, 6:26 am)
Re: [git pull] kgdb-light -v9, Ingo Molnar, (Tue Feb 12, 6:34 am)
Re: [git pull] kgdb-light -v9, Sam Ravnborg, (Tue Feb 12, 5:35 am)
Re: kgdb in git-x86#mm review, Andi Kleen, (Mon Feb 11, 12:11 pm)