lkml.org 
[lkml]   [2004]   [Sep]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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/

\
 
 \ /
  Last update: 2005-03-22 14:06    [W:0.403 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site