Re: [PATCH] SN2 user-MMIO CPU migration

From: Luck, Tony <tony.luck_at_intel.com>
Date: 2006-01-25 04:33:16
On Tue, Jan 24, 2006 at 10:02:35AM +0100, Ingo Molnar wrote:
> what's the problem with doing that? It's a single comparison [of two 
> values we already have accessed in that function!] and a rarely taken 
> branch in a single subarch of a single arch's switch_to() function. If 
> this use (of migration related arch functionality) becomes more 
> widespread then we can generalize it, but right now i just dont see the 
> point, given that the two solutions are almost totally equivalent in 
> terms of "overhead".

As coded the comparison wasn't inline, it was inside the machvec function,
so we'd have to make an indirect function call in order to make the
comparison and then do nothing most of the time.  I'd be happier with:

    if (unlikely(foo != bar)) /* migration */
	platform_switch_from(next);

But I'd also like to see how invasive a "task has migrated" bit in
thread_info.flags (as suggested elsewhere in this thread) gets to
be.

-Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Wed Jan 25 04:33:59 2006

This archive was generated by hypermail 2.1.8 : 2006-01-25 04:34:07 EST