Re: [PATCH][5/8] Arch agnostic completely out of line locks / ppc64

From: Linus Torvalds
Date: Thu Sep 09 2004 - 19:55:07 EST




On Fri, 10 Sep 2004, Anton Blanchard wrote:
>
> With preempt off profile_pc works as expected, timer ticks in spinlocks
> are apportioned to the calling function. Turn preempt on and you end up
> with one big bucket called __preempt_spin_lock where most of the
> spinlock ticks end up.

But that's because "__preempt_spin_lock" on ppc is in the wrong section,
no?

Just change it from "__sched" to "__lockfunc", and move it to
kernel/spinlock.c while you're at it, and everything works right. Do the
same for __preempt_write_lock() too.

Oh, and you need to do the "is_sched_function()" change too that I
outlined in the previous email.

All in all, about four lines of code changes (+ some movement to make it
all saner)

Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/