Messages in this thread | | | Date | Thu, 16 Sep 2004 08:53:42 +0200 | From | Andi Kleen <> | Subject | Re: [PATCH] remove LOCK_SECTION from x86_64 spin_lock asm |
| |
On Thu, Sep 16, 2004 at 08:51:01AM +0200, Ingo Molnar wrote: > > * Andi Kleen <ak@suse.de> wrote: > > > I think the idea was that the spinlock functions should be small > > enough that they don't have any stack local variables. [...] > > this might work for x64, but even there, are you sure it works even with > CONFIG_PREEMPT enabled (there the spinlocks are more complex)? It sure
I would expect so. x86-64 should have enough callee clobbered registers by default to handle it. Unless someone makes them complex enough that it needs more than 4 variables or so or adds another function call. But I hope this won't happen.
> wont work on x86 so i think we need a generic 'soft PC' solution. > > the alternative would be to unwind the stack - quite some task on some > platforms ...
Sometimes call graph profiling would be very useful, but I wouldn't want the profiler to do it by default, especially not for this silly simple case. dwarf2 unwinding is complex enough that just requiring frame pointers for the CG case would look attractive.
-Andi
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |