|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH 2/2] x86/vmx: Drop ept_get_*() helpers
The ept_get_*() helpers are not used consistently, and are more verbose than
the code they wrap. Drop the wrappers and use the internal union names
consistently.
While making these adjustments, drop the redundant ept_* prefix from mt, wl
and ad, and rename the asr field to mfn for consistency with Xen's existing
terminology.
No functional change.
Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
CC: Jan Beulich <JBeulich@xxxxxxxx>
CC: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
CC: Jun Nakajima <jun.nakajima@xxxxxxxxx>
CC: Kevin Tian <kevin.tian@xxxxxxxxx>
---
xen/arch/x86/hvm/vmx/vmcs.c | 6 +++---
xen/arch/x86/hvm/vmx/vmx.c | 6 +++---
xen/arch/x86/hvm/vmx/vvmx.c | 9 +++-----
xen/arch/x86/mm/p2m-ept.c | 44 +++++++++++++++++++-------------------
xen/include/asm-x86/hvm/vmx/vmcs.h | 16 ++++++--------
5 files changed, 37 insertions(+), 44 deletions(-)
diff --git a/xen/arch/x86/hvm/vmx/vmcs.c b/xen/arch/x86/hvm/vmx/vmcs.c
index 59ef199..3fe1783 100644
--- a/xen/arch/x86/hvm/vmx/vmcs.c
+++ b/xen/arch/x86/hvm/vmx/vmcs.c
@@ -1247,8 +1247,8 @@ static int construct_vmcs(struct vcpu *v)
struct p2m_domain *p2m = p2m_get_hostp2m(d);
struct ept_data *ept = &p2m->ept;
- ept->asr = pagetable_get_pfn(p2m_get_pagetable(p2m));
- __vmwrite(EPT_POINTER, ept_get_eptp(ept));
+ ept->mfn = pagetable_get_pfn(p2m_get_pagetable(p2m));
+ __vmwrite(EPT_POINTER, ept->eptp);
}
if ( paging_mode_hap(d) )
@@ -1593,7 +1593,7 @@ void vmx_domain_update_eptp(struct domain *d)
ASSERT(atomic_read(&d->pause_count));
for_each_vcpu ( d, v )
- vmx_vcpu_update_eptp(v, ept_get_eptp(&p2m->ept));
+ vmx_vcpu_update_eptp(v, p2m->ept.eptp);
ept_sync_domain(p2m);
}
diff --git a/xen/arch/x86/hvm/vmx/vmx.c b/xen/arch/x86/hvm/vmx/vmx.c
index eca4f7d..9078cf9 100644
--- a/xen/arch/x86/hvm/vmx/vmx.c
+++ b/xen/arch/x86/hvm/vmx/vmx.c
@@ -1996,11 +1996,11 @@ static void vmx_vcpu_update_eptp(struct vcpu *v)
p2m = p2m_get_hostp2m(d);
ept = &p2m->ept;
- ept->asr = pagetable_get_pfn(p2m_get_pagetable(p2m));
+ ept->mfn = pagetable_get_pfn(p2m_get_pagetable(p2m));
vmx_vmcs_enter(v);
- __vmwrite(EPT_POINTER, ept_get_eptp(ept));
+ __vmwrite(EPT_POINTER, ept->eptp);
if ( v->arch.hvm_vmx.secondary_exec_control &
SECONDARY_EXEC_ENABLE_VIRT_EXCEPTIONS )
@@ -3953,7 +3953,7 @@ void vmx_vmenter_helper(const struct cpu_user_regs *regs)
if ( cpumask_test_cpu(cpu, ept->invalidate) )
{
cpumask_clear_cpu(cpu, ept->invalidate);
- __invept(INVEPT_SINGLE_CONTEXT, ept_get_eptp(ept), 0);
+ __invept(INVEPT_SINGLE_CONTEXT, ept->eptp, 0);
}
}
diff --git a/xen/arch/x86/hvm/vmx/vvmx.c b/xen/arch/x86/hvm/vmx/vvmx.c
index 9caebe5..5acb88a 100644
--- a/xen/arch/x86/hvm/vmx/vvmx.c
+++ b/xen/arch/x86/hvm/vmx/vvmx.c
@@ -1043,16 +1043,13 @@ uint64_t get_shadow_eptp(struct vcpu *v)
struct p2m_domain *p2m = p2m_get_nestedp2m(v, np2m_base);
struct ept_data *ept = &p2m->ept;
- ept->asr = pagetable_get_pfn(p2m_get_pagetable(p2m));
- return ept_get_eptp(ept);
+ ept->mfn = pagetable_get_pfn(p2m_get_pagetable(p2m));
+ return ept->eptp;
}
static uint64_t get_host_eptp(struct vcpu *v)
{
- struct domain *d = v->domain;
- struct ept_data *ept_data = &p2m_get_hostp2m(d)->ept;
-
- return ept_get_eptp(ept_data);
+ return p2m_get_hostp2m(v->domain)->ept.eptp;
}
static bool_t nvmx_vpid_enabled(const struct vcpu *v)
diff --git a/xen/arch/x86/mm/p2m-ept.c b/xen/arch/x86/mm/p2m-ept.c
index 04878f5..6c1b835 100644
--- a/xen/arch/x86/mm/p2m-ept.c
+++ b/xen/arch/x86/mm/p2m-ept.c
@@ -430,7 +430,7 @@ static int ept_invalidate_emt_range(struct p2m_domain *p2m,
int wrc, rc = 0, ret = GUEST_TABLE_MAP_FAILED;
table = map_domain_page(_mfn(pagetable_get_pfn(p2m_get_pagetable(p2m))));
- for ( i = ept_get_wl(&p2m->ept); i > target; --i )
+ for ( i = p2m->ept.wl; i > target; --i )
{
ret = ept_next_level(p2m, 1, &table, &gfn_remainder, i);
if ( ret == GUEST_TABLE_MAP_FAILED )
@@ -500,8 +500,8 @@ static int ept_invalidate_emt_range(struct p2m_domain *p2m,
static int resolve_misconfig(struct p2m_domain *p2m, unsigned long gfn)
{
struct ept_data *ept = &p2m->ept;
- unsigned int level = ept_get_wl(ept);
- unsigned long mfn = ept_get_asr(ept);
+ unsigned int level = ept->wl;
+ unsigned long mfn = ept->mfn;
ept_entry_t *epte;
int wrc, rc = 0;
@@ -687,7 +687,7 @@ ept_set_entry(struct p2m_domain *p2m, unsigned long gfn,
mfn_t mfn,
* 3. passing a valid order.
*/
if ( ((gfn | mfn_x(mfn)) & ((1UL << order) - 1)) ||
- ((u64)gfn >> ((ept_get_wl(ept) + 1) * EPT_TABLE_ORDER)) ||
+ ((u64)gfn >> ((ept->wl + 1) * EPT_TABLE_ORDER)) ||
(order % EPT_TABLE_ORDER) )
return -EINVAL;
@@ -704,7 +704,7 @@ ept_set_entry(struct p2m_domain *p2m, unsigned long gfn,
mfn_t mfn,
table = map_domain_page(_mfn(pagetable_get_pfn(p2m_get_pagetable(p2m))));
ret = GUEST_TABLE_MAP_FAILED;
- for ( i = ept_get_wl(ept); i > target; i-- )
+ for ( i = ept->wl; i > target; i-- )
{
ret = ept_next_level(p2m, 0, &table, &gfn_remainder, i);
if ( !ret )
@@ -898,7 +898,7 @@ static mfn_t ept_get_entry(struct p2m_domain *p2m,
/* This pfn is higher than the highest the p2m map currently holds */
if ( gfn > p2m->max_mapped_pfn )
{
- for ( i = ept_get_wl(ept); i > 0; --i )
+ for ( i = ept->wl; i > 0; --i )
if ( (gfn & ~((1UL << (i * EPT_TABLE_ORDER)) - 1)) >
p2m->max_mapped_pfn )
break;
@@ -907,7 +907,7 @@ static mfn_t ept_get_entry(struct p2m_domain *p2m,
/* Should check if gfn obeys GAW here. */
- for ( i = ept_get_wl(ept); i > 0; i-- )
+ for ( i = ept->wl; i > 0; i-- )
{
retry:
if ( table[gfn_remainder >> (i * EPT_TABLE_ORDER)].recalc )
@@ -1009,7 +1009,7 @@ void ept_walk_table(struct domain *d, unsigned long gfn)
goto out;
}
- for ( i = ept_get_wl(ept); i >= 0; i-- )
+ for ( i = ept->wl; i >= 0; i-- )
{
ept_entry_t *ept_entry, *next;
u32 index;
@@ -1043,12 +1043,12 @@ void ept_walk_table(struct domain *d, unsigned long gfn)
static void ept_change_entry_type_global(struct p2m_domain *p2m,
p2m_type_t ot, p2m_type_t nt)
{
- unsigned long mfn = ept_get_asr(&p2m->ept);
+ unsigned long mfn = p2m->ept.mfn;
if ( !mfn )
return;
- if ( ept_invalidate_emt(_mfn(mfn), 1, ept_get_wl(&p2m->ept)) )
+ if ( ept_invalidate_emt(_mfn(mfn), 1, p2m->ept.wl) )
ept_sync_domain(p2m);
}
@@ -1057,11 +1057,11 @@ static int ept_change_entry_type_range(struct
p2m_domain *p2m,
unsigned long first_gfn,
unsigned long last_gfn)
{
- unsigned int i, wl = ept_get_wl(&p2m->ept);
+ unsigned int i, wl = p2m->ept.wl;
unsigned long mask = (1 << EPT_TABLE_ORDER) - 1;
int rc = 0, sync = 0;
- if ( !ept_get_asr(&p2m->ept) )
+ if ( p2m->ept.mfn )
return -EINVAL;
for ( i = 0; i <= wl; )
@@ -1101,12 +1101,12 @@ static int ept_change_entry_type_range(struct
p2m_domain *p2m,
static void ept_memory_type_changed(struct p2m_domain *p2m)
{
- unsigned long mfn = ept_get_asr(&p2m->ept);
+ unsigned long mfn = p2m->ept.mfn;
if ( !mfn )
return;
- if ( ept_invalidate_emt(_mfn(mfn), 0, ept_get_wl(&p2m->ept)) )
+ if ( ept_invalidate_emt(_mfn(mfn), 0, p2m->ept.wl) )
ept_sync_domain(p2m);
}
@@ -1179,7 +1179,7 @@ static void ept_enable_pml(struct p2m_domain *p2m)
return;
/* Enable EPT A/D bit for PML */
- p2m->ept.ept_ad = 1;
+ p2m->ept.ad = 1;
vmx_domain_update_eptp(p2m->domain);
}
@@ -1191,7 +1191,7 @@ static void ept_disable_pml(struct p2m_domain *p2m)
vmx_domain_disable_pml(p2m->domain);
/* Disable EPT A/D bit */
- p2m->ept.ept_ad = 0;
+ p2m->ept.ad = 0;
vmx_domain_update_eptp(p2m->domain);
}
@@ -1216,10 +1216,10 @@ int ept_p2m_init(struct p2m_domain *p2m)
p2m->tlb_flush = ept_tlb_flush;
/* Set the memory type used when accessing EPT paging structures. */
- ept->ept_mt = EPT_DEFAULT_MT;
+ ept->mt = EPT_DEFAULT_MT;
/* set EPT page-walk length, now it's actual walk length - 1, i.e. 3 */
- ept->ept_wl = 3;
+ ept->wl = 3;
if ( cpu_has_vmx_pml )
{
@@ -1289,7 +1289,7 @@ static void ept_dump_p2m_table(unsigned char key)
gfn_remainder = gfn;
table =
map_domain_page(_mfn(pagetable_get_pfn(p2m_get_pagetable(p2m))));
- for ( i = ept_get_wl(ept); i > 0; i-- )
+ for ( i = ept->wl; i > 0; i-- )
{
ept_entry = table + (gfn_remainder >> (i * EPT_TABLE_ORDER));
if ( ept_entry->emt == MTRR_NUM_TYPES )
@@ -1337,8 +1337,8 @@ void p2m_init_altp2m_ept(struct domain *d, unsigned int i)
p2m->min_remapped_gfn = gfn_x(INVALID_GFN);
p2m->max_remapped_gfn = 0;
ept = &p2m->ept;
- ept->asr = pagetable_get_pfn(p2m_get_pagetable(p2m));
- d->arch.altp2m_eptp[i] = ept_get_eptp(ept);
+ ept->mfn = pagetable_get_pfn(p2m_get_pagetable(p2m));
+ d->arch.altp2m_eptp[i] = ept->eptp;
}
unsigned int p2m_find_altp2m_by_eptp(struct domain *d, uint64_t eptp)
@@ -1357,7 +1357,7 @@ unsigned int p2m_find_altp2m_by_eptp(struct domain *d,
uint64_t eptp)
p2m = d->arch.altp2m_p2m[i];
ept = &p2m->ept;
- if ( eptp == ept_get_eptp(ept) )
+ if ( eptp == ept->eptp )
goto out;
}
diff --git a/xen/include/asm-x86/hvm/vmx/vmcs.h
b/xen/include/asm-x86/hvm/vmx/vmcs.h
index d71de04..33fe7d6 100644
--- a/xen/include/asm-x86/hvm/vmx/vmcs.h
+++ b/xen/include/asm-x86/hvm/vmx/vmcs.h
@@ -58,12 +58,12 @@ struct vmx_msr_state {
struct ept_data {
union {
- struct {
- u64 ept_mt :3,
- ept_wl :3,
- ept_ad :1, /* bit 6 - enable EPT A/D bits */
- rsvd :5,
- asr :52;
+ struct {
+ uint64_t mt:3, /* Memory Type. */
+ wl:3, /* Walk length -1. */
+ ad:1, /* Enable EPT A/D bits. */
+ :5, /* rsvd. */
+ mfn:52;
};
u64 eptp;
};
@@ -95,10 +95,6 @@ struct pi_desc {
u32 rsvd[6];
} __attribute__ ((aligned (64)));
-#define ept_get_wl(ept) ((ept)->ept_wl)
-#define ept_get_asr(ept) ((ept)->asr)
-#define ept_get_eptp(ept) ((ept)->eptp)
-
#define NR_PML_ENTRIES 512
struct pi_blocking_vcpu {
--
2.1.4
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |