RE: [PATCH] SN2 user-MMIO CPU migration

From: Brent Casavant <bcasavan_at_sgi.com>
Date: 2006-01-25 09:12:09
On Tue, 24 Jan 2006, Chen, Kenneth W wrote:

> Yeah, I'm not thrilled either.  Currently, on context switch kernel
> doesn't read thread_info.cpu, nor any other neighboring field.  With
> the patch, it needs to do a cacheline read and possibly an update.
> If unlucky, it might be two cache lines with last_cpu 92 bytes away
> from thread_info.cpu. Perhaps, last_cpu should be next to cpu field.

Done.

> On the other hand, it may not be too bad because on kernel exit, flags
> field in thread_info is accessed for TIF_* and if kernel exit path is
> taken back-to-back with context switch, the net effect maybe small.

Actually, IA64_HAS_EXTRA_STATE() reads the TIF_* flags, and is called
in __switch_to(), which is in very close proximity to checking last_cpu.
I don't think there'll be a problem here, particularly as I've now moved
last_cpu to neighbor cpu.

Updated patch following in a few minutes.

Brent

-- 
Brent Casavant                          All music is folk music.  I ain't
bcasavan@sgi.com                        never heard a horse sing a song.
Silicon Graphics, Inc.                    -- Louis Armstrong
-
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 09:13:27 2006

This archive was generated by hypermail 2.1.8 : 2006-01-25 09:13:34 EST