]> xenbits.xensource.com Git - xen.git/commit
x86: don't use VA for cache flush when also flushing TLB
authorJan Beulich <jbeulich@suse.com>
Wed, 28 May 2014 07:51:07 +0000 (09:51 +0200)
committerJan Beulich <jbeulich@suse.com>
Wed, 28 May 2014 07:51:07 +0000 (09:51 +0200)
commit50df6f7429f73364bbddb0970a3a34faa01a7790
tree453f534764393674b459f67ac71f83042008ece6
parent6b4d71d028f445cba7426a144751fddc8bfdd67b
x86: don't use VA for cache flush when also flushing TLB

Doing both flushes at once is a strong indication for the address
mapping to either having got dropped (in which case the cache flush,
when done via INVLPG, would fault) or its physical address having
changed (in which case the cache flush would end up being done on the
wrong address range). There is no adverse effect (other than the
obvious performance one) using WBINVD in this case regardless of the
range's size; only map_pages_to_xen() uses combined flushes at present.

This problem was observed with the 2nd try backport of d6cb14b3 ("VT-d:
suppress UR signaling for desktop chipsets") to 4.2 (where ioremap()
needs to be replaced with set_fixmap_nocache(); the now commented out
__set_fixmap(, 0, 0) there to undo the mapping resulted in the first of
the above two scenarios).

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
xen/arch/x86/flushtlb.c