On Thu, Dec 09, 2004 at 01:46:22AM +0000, John Levon wrote: > On Thu, Dec 09, 2004 at 11:39:06AM +1100, Greg Banks wrote: > > > But for now I don't see any drama with leaving in the ->setup() and > > ->shutdown() methods when rewriting the ops structure. Ditto for > > the ->create_files() methods. > > Wouldn't this mean that we try to set up the NMI stuff regardless of > forcing the timer ? I can imagine a flaky system where somebody needs to > avoid going near that stuff. > > timer_init() making sure to set all fields seems reasonable to me. Or > oprofile_init() could grab ->backtrace, memset the structure, then > replace ->backtrace... Ok, how about this patch? Greg. -- Greg Banks, R&D Software Engineer, SGI Australian Software Group. I don't speak for SGI.