Subject: CVS commit: [vmlocking] src/sys/arch
To: None <source-changes@NetBSD.org>
From: Andrew Doran <ad@netbsd.org>
List: source-changes
Date: 08/29/2007 22:36:08
Module Name:	src
Committed By:	ad
Date:		Wed Aug 29 22:36:08 UTC 2007

Modified Files:
	src/sys/arch/amd64/amd64 [vmlocking]: vector.S
	src/sys/arch/i386/i386 [vmlocking]: vector.S

Log Message:
In the TLB shootdown handlers, don't ack the request until the invalidation
is complete. We could end up loading bogus translations due to a speculative
load if the calling CPU recycles a PDE/PTE, and these may not be shot down
before the handler returns. Pointed out by yamt. I think this is very
unlikely to happen but it's better to be on the safe side. To be revisted
later, as the invalidations are time consuming and it would be nice to
release the caller sooner rather than later.


To generate a diff of this commit:
cvs rdiff -r1.10.6.3 -r1.10.6.4 src/sys/arch/amd64/amd64/vector.S
cvs rdiff -r1.24.6.4 -r1.24.6.5 src/sys/arch/i386/i386/vector.S

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.