Thread (29 messages) 29 messages, 8 authors, 2007-04-10

Re: [patch 06/10] Add notify die hooks and remove some redundant debugger hooks

From: Paul Mackerras <hidden>
Date: 2007-03-24 03:17:04

Anton Blanchard writes:
Keep reading :) Later on I remove all the debugger() indirect branches.
Well, you replace one indirect branch with another.  I'm not sure how
that is any better.
Keep in mind with xmon and kprobes disabled these calls are fast. I
verified it in sim after Christophs 300 cycle rant, a pessimistic
estimate is 20 cycles on power5, including function call overhead.
With xmon disabled the debugger() call is already fast.  Generally I
see notifiers as being useful when you have some event, or some group
of events, where you can imagine lots of different software modules
(e.g. random device drivers) being interested in it.  I don't see that
that is the case with any of the kprobes notifiers.

Also I don't see any real point to forcing these disparate things to
go through the one notify_die() interface, with the consequent
demultiplexing that you then have to do in the receiving function.

Paul.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help