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
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.