@@ -690,13 +690,9 @@ int arch_domain_create(struct domain *d,
if ( (rc = p2m_init(d)) != 0 )
goto fail;
- rc = -ENOMEM;
- if ( (d->shared_info = alloc_xenheap_pages(0, 0)) == NULL )
+ if ( (rc = alloc_shared_info(d, 0)) != 0 )
goto fail;
- clear_page(d->shared_info);
- share_xen_page_with_guest(virt_to_page(d->shared_info), d, SHARE_rw);
-
switch ( config->arch.gic_version )
{
case XEN_DOMCTL_CONFIG_GIC_V2:
@@ -767,7 +763,7 @@ void arch_domain_destroy(struct domain *d)
p2m_teardown(d);
domain_vgic_free(d);
domain_vuart_free(d);
- free_xenheap_page(d->shared_info);
+ free_shared_info(d);
#ifdef CONFIG_ACPI
free_xenheap_pages(d->arch.efi_acpi_table,
get_order_from_bytes(d->arch.efi_acpi_len));
@@ -1424,7 +1424,7 @@ int xenmem_add_to_physmap_one(
if ( idx != 0 )
return -EINVAL;
- mfn = virt_to_mfn(d->shared_info);
+ mfn = d->shared_info.mfn;
t = p2m_ram_rw;
break;
@@ -612,12 +612,9 @@ int arch_domain_create(struct domain *d,
* The shared_info machine address must fit in a 32-bit field within a
* 32-bit guest's start_info structure. Hence we specify MEMF_bits(32).
*/
- if ( (d->shared_info = alloc_xenheap_pages(0, MEMF_bits(32))) == NULL )
+ if ( (rc = alloc_shared_info(d, MEMF_bits(32))) != 0 )
goto fail;
- clear_page(d->shared_info);
- share_xen_page_with_guest(virt_to_page(d->shared_info), d, SHARE_rw);
-
if ( (rc = init_domain_irq_mapping(d)) != 0 )
goto fail;
@@ -665,7 +662,7 @@ int arch_domain_create(struct domain *d,
psr_domain_free(d);
iommu_domain_destroy(d);
cleanup_domain_irq_mapping(d);
- free_xenheap_page(d->shared_info);
+ free_shared_info(d);
xfree(d->arch.cpuid);
xfree(d->arch.msr);
if ( paging_initialised )
@@ -694,7 +691,7 @@ void arch_domain_destroy(struct domain *d)
pv_domain_destroy(d);
free_perdomain_mappings(d);
- free_xenheap_page(d->shared_info);
+ free_shared_info(d);
cleanup_domain_irq_mapping(d);
psr_domain_free(d);
@@ -720,7 +717,7 @@ void arch_domain_unpause(struct domain *d)
int arch_domain_soft_reset(struct domain *d)
{
- struct page_info *page = virt_to_page(d->shared_info), *new_page;
+ struct page_info *page = mfn_to_page(d->shared_info.mfn), *new_page;
int ret = 0;
struct domain *owner;
mfn_t mfn;
@@ -4538,7 +4538,7 @@ int xenmem_add_to_physmap_one(
{
case XENMAPSPACE_shared_info:
if ( idx == 0 )
- mfn = virt_to_mfn(d->shared_info);
+ mfn = d->shared_info.mfn;
break;
case XENMAPSPACE_grant_table:
rc = gnttab_map_frame(d, idx, gpfn, &mfn);
@@ -743,7 +743,7 @@ int __init dom0_construct_pv(struct domain *d,
clear_page(si);
si->nr_pages = nr_pages;
- si->shared_info = virt_to_maddr(d->shared_info);
+ si->shared_info = mfn_to_maddr(d->shared_info.mfn);
if ( !pv_shim )
si->flags = SIF_PRIVILEGED | SIF_INITDOMAIN;
@@ -282,7 +282,7 @@ static void write_start_info(struct domain *d)
snprintf(si->magic, sizeof(si->magic), "xen-3.0-x86_%s",
is_pv_32bit_domain(d) ? "32p" : "64");
si->nr_pages = domain_tot_pages(d);
- si->shared_info = virt_to_maddr(d->shared_info);
+ si->shared_info = mfn_to_maddr(d->shared_info.mfn);
si->flags = 0;
BUG_ON(xen_hypercall_hvm_get_param(HVM_PARAM_STORE_PFN, &si->store_mfn));
BUG_ON(xen_hypercall_hvm_get_param(HVM_PARAM_STORE_EVTCHN, ¶m));
@@ -47,6 +47,10 @@
#include <asm/guest.h>
#endif
+/* Override macros from asm/page.h to make them work with mfn_t */
+#undef virt_to_mfn
+#define virt_to_mfn(v) _mfn(__virt_to_mfn(v))
+
/* Linux config option: propageted to domain0 */
/* xen_processor_pmbits: xen control Cx, Px, ... */
unsigned int xen_processor_pmbits = XEN_PROCESSOR_PM_PX;
@@ -1644,6 +1648,28 @@ int continue_hypercall_on_cpu(
return 0;
}
+int alloc_shared_info(struct domain *d, unsigned int memflags)
+{
+ if ( (d->shared_info.virt = alloc_xenheap_pages(0, memflags)) == NULL )
+ return -ENOMEM;
+
+ d->shared_info.mfn = virt_to_mfn(d->shared_info.virt);
+
+ clear_page(d->shared_info.virt);
+ share_xen_page_with_guest(mfn_to_page(d->shared_info.mfn), d, SHARE_rw);
+
+ return 0;
+}
+
+void free_shared_info(struct domain *d)
+{
+ if ( !d->shared_info.virt )
+ return;
+
+ free_xenheap_page(d->shared_info.virt);
+ d->shared_info.virt = NULL;
+}
+
/*
* Local variables:
* mode: C
@@ -196,7 +196,7 @@ void getdomaininfo(struct domain *d, struct xen_domctl_getdomaininfo *info)
info->outstanding_pages = d->outstanding_pages;
info->shr_pages = atomic_read(&d->shr_pages);
info->paged_pages = atomic_read(&d->paged_pages);
- info->shared_info_frame = mfn_to_gmfn(d, virt_to_mfn(d->shared_info));
+ info->shared_info_frame = mfn_to_gmfn(d, mfn_x(d->shared_info.mfn));
BUG_ON(SHARED_M2P(info->shared_info_frame));
info->cpupool = cpupool_get_id(d);
@@ -110,9 +110,9 @@ void update_domain_wallclock_time(struct domain *d)
shared_info(d, wc_nsec) = wc_nsec;
#ifdef CONFIG_X86
if ( likely(!has_32bit_shinfo(d)) )
- d->shared_info->native.wc_sec_hi = sec >> 32;
+ d->shared_info.virt->native.wc_sec_hi = sec >> 32;
else
- d->shared_info->compat.arch.wc_sec_hi = sec >> 32;
+ d->shared_info.virt->compat.arch.wc_sec_hi = sec >> 32;
#else
shared_info(d, wc_sec_hi) = sec >> 32;
#endif
@@ -1,24 +1,25 @@
#ifndef __XEN_X86_SHARED_H__
#define __XEN_X86_SHARED_H__
-#define nmi_reason(d) (!has_32bit_shinfo(d) ? \
- (u32 *)&(d)->shared_info->native.arch.nmi_reason : \
- (u32 *)&(d)->shared_info->compat.arch.nmi_reason)
+#define nmi_reason(d) \
+ (!has_32bit_shinfo(d) ? \
+ (u32 *)&(d)->shared_info.virt->native.arch.nmi_reason : \
+ (u32 *)&(d)->shared_info.virt->compat.arch.nmi_reason)
#define GET_SET_SHARED(type, field) \
static inline type arch_get_##field(const struct domain *d) \
{ \
return !has_32bit_shinfo(d) ? \
- d->shared_info->native.arch.field : \
- d->shared_info->compat.arch.field; \
+ d->shared_info.virt->native.arch.field : \
+ d->shared_info.virt->compat.arch.field; \
} \
static inline void arch_set_##field(struct domain *d, \
type val) \
{ \
if ( !has_32bit_shinfo(d) ) \
- d->shared_info->native.arch.field = val; \
+ d->shared_info.virt->native.arch.field = val; \
else \
- d->shared_info->compat.arch.field = val; \
+ d->shared_info.virt->compat.arch.field = val; \
}
#define GET_SET_VCPU(type, field) \
@@ -130,4 +130,7 @@ struct vnuma_info {
void vnuma_destroy(struct vnuma_info *vnuma);
+int alloc_shared_info(struct domain *d, unsigned int memflags);
+void free_shared_info(struct domain *d);
+
#endif /* __XEN_DOMAIN_H__ */
@@ -346,7 +346,10 @@ struct domain
unsigned int max_vcpus;
struct vcpu **vcpu;
- shared_info_t *shared_info; /* shared data area */
+ struct {
+ mfn_t mfn;
+ shared_info_t *virt;
+ } shared_info; /* shared data area */
spinlock_t domain_lock;
@@ -43,7 +43,7 @@ typedef struct vcpu_info vcpu_info_t;
extern vcpu_info_t dummy_vcpu_info;
-#define shared_info(d, field) __shared_info(d, (d)->shared_info, field)
+#define shared_info(d, field) __shared_info(d, (d)->shared_info.virt, field)
#define vcpu_info(v, field) __vcpu_info(v, (v)->vcpu_info, field)
#endif /* __XEN_SHARED_H__ */
... and save the MFN. This patch modifies the 'shared_info' field of struct domain to be a structure comprising an MFN and a virtual address. Allocations are still done from xenheap, so the virtual address still equates to virt_to_mfn() called on the MFN but subsequent patch will change this. Hence the need to save the MFN. NOTE: Whist defining the new helpers, virt_to_mfn() in common/domain.c is made type safe. The definition of nmi_reason() in asm-x86/shared.h is also re- flowed to avoid overly long lines. Signed-off-by: Paul Durrant <pdurrant@amazon.com> --- Cc: Stefano Stabellini <sstabellini@kernel.org> Cc: Julien Grall <julien@xen.org> Cc: Volodymyr Babchuk <Volodymyr_Babchuk@epam.com> Cc: Andrew Cooper <andrew.cooper3@citrix.com> Cc: George Dunlap <george.dunlap@citrix.com> Cc: Ian Jackson <ian.jackson@eu.citrix.com> Cc: Jan Beulich <jbeulich@suse.com> Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Cc: Wei Liu <wl@xen.org> Cc: "Roger Pau Monné" <roger.pau@citrix.com> --- xen/arch/arm/domain.c | 8 ++------ xen/arch/arm/mm.c | 2 +- xen/arch/x86/domain.c | 11 ++++------- xen/arch/x86/mm.c | 2 +- xen/arch/x86/pv/dom0_build.c | 2 +- xen/arch/x86/pv/shim.c | 2 +- xen/common/domain.c | 26 ++++++++++++++++++++++++++ xen/common/domctl.c | 2 +- xen/common/time.c | 4 ++-- xen/include/asm-x86/shared.h | 15 ++++++++------- xen/include/xen/domain.h | 3 +++ xen/include/xen/sched.h | 5 ++++- xen/include/xen/shared.h | 2 +- 13 files changed, 55 insertions(+), 29 deletions(-)