x86/ept: fix shattering of special pages

The current logic in epte_get_entry_emt() will split any page marked
as special with order greater than zero, without checking whether the
super page is all special.

Fix this by only splitting the page only if it's not all marked as
special, in order to prevent unneeded super page shuttering.

The unconditional special super page shattering has caused a
performance regression on some XenServer GPU pass through workloads.

Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Kevin Tian <kevin.tian@intel.com>
This commit is contained in:
Roger Pau Monne authored and Andrew Cooper committed 2022-06-30 18:07:13 +01:00
1 parent fef244b179
commit 60d1adfa18
1 file changed
+11 -9
+11 -9
View File
@@ -491,7 +491,7 @@ int epte_get_entry_emt(struct domain *d, gfn_t gfn, mfn_t mfn,
{
int gmtrr_mtype, hmtrr_mtype;
struct vcpu *v = current;
unsigned long i;
unsigned long i, special_pgs;
*ipat = false;
@@ -525,15 +525,17 @@ int epte_get_entry_emt(struct domain *d, gfn_t gfn, mfn_t mfn,
return MTRR_TYPE_WRBACK;
}
for ( i = 0; i < (1ul << order); i++ )
{
for ( special_pgs = i = 0; i < (1ul << order); i++ )
if ( is_special_page(mfn_to_page(mfn_add(mfn, i))) )
{
if ( order )
return -1;
*ipat = true;
return MTRR_TYPE_WRBACK;
}
special_pgs++;
if ( special_pgs )
{
if ( special_pgs != (1ul << order) )
return -1;
*ipat = true;
return MTRR_TYPE_WRBACK;
}
switch ( type )
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.