[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH v6 07/11] xen, common: move the memory_mapping DOMCTL hypercall to common code
This commit moves to common code the implementation of the memory_mapping DOMCTL, currently available only for the x86 architecture. This commit also adds to ARM-related code an unmap_mmio_regions() function to match the corresponding one available for x86. NOTE: this commit also adds an empty definition for an ARM-specific memory_type_changed() function. Such a function's implementation is not added in this patchset. Signed-off-by: Arianna Avanzini <avanzini.arianna@xxxxxxxxx> Cc: Dario Faggioli <dario.faggioli@xxxxxxxxxx> Cc: Paolo Valente <paolo.valente@xxxxxxxxxx> Cc: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx> Cc: Julien Grall <julien.grall@xxxxxxxxxx> Cc: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx> Cc: Jan Beulich <JBeulich@xxxxxxxx> Cc: Keir Fraser <keir@xxxxxxx> Cc: Tim Deegan <tim@xxxxxxx> Cc: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> Cc: Eric Trudeau <etrudeau@xxxxxxxxxxxx> Cc: Viktor Kleinik <viktor.kleinik@xxxxxxxxxxxxxxx> --- v6: - Pass p2m_invalid as last parameter to unmap_mmio_regions() for ARM as it is not (and currently must not be) used. - Add an empty definition of the memory_type_changed() function for ARM. v5: - Rename new header to p2m-common.h. v4: - Use a define for paddr_bits instead of a new variable. - Define prototypes for common functions map_mmio_regions() and unmap_mmio_regions() only once in a common header. v3: - Add a paddr_bits variable for ARM. v2: - Move code to xen/arm/domctl.c. --- xen/arch/arm/p2m.c | 16 ++++++++++ xen/arch/x86/domctl.c | 70 -------------------------------------------- xen/common/domctl.c | 70 ++++++++++++++++++++++++++++++++++++++++++++ xen/include/asm-arm/p2m.h | 12 ++++---- xen/include/asm-x86/p2m.h | 13 +------- xen/include/xen/p2m-common.h | 16 ++++++++++ 6 files changed, 108 insertions(+), 89 deletions(-) create mode 100644 xen/include/xen/p2m-common.h diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c index 54a0afa..a215959 100644 --- a/xen/arch/arm/p2m.c +++ b/xen/arch/arm/p2m.c @@ -14,6 +14,10 @@ #define P2M_FIRST_ORDER 1 #define P2M_FIRST_ENTRIES (LPAE_ENTRIES<<P2M_FIRST_ORDER) +void memory_type_changed(struct domain *d) +{ +} + void dump_p2m_lookup(struct domain *d, paddr_t addr) { struct p2m_domain *p2m = &d->arch.p2m; @@ -522,6 +526,18 @@ int map_mmio_regions(struct domain *d, MATTR_DEV, p2m_mmio_direct); } +int unmap_mmio_regions(struct domain *d, + unsigned long start_gfn, + unsigned long end_gfn, + unsigned long mfn) +{ + return apply_p2m_changes(d, REMOVE, + pfn_to_paddr(start_gfn), + pfn_to_paddr(end_gfn + 1), + pfn_to_paddr(mfn), + MATTR_DEV, p2m_invalid); +} + int guest_physmap_add_entry(struct domain *d, unsigned long gpfn, unsigned long mfn, diff --git a/xen/arch/x86/domctl.c b/xen/arch/x86/domctl.c index 04c6007..1ffcaa8 100644 --- a/xen/arch/x86/domctl.c +++ b/xen/arch/x86/domctl.c @@ -641,76 +641,6 @@ long arch_do_domctl( } break; - case XEN_DOMCTL_memory_mapping: - { - unsigned long gfn = domctl->u.memory_mapping.first_gfn; - unsigned long mfn = domctl->u.memory_mapping.first_mfn; - unsigned long nr_mfns = domctl->u.memory_mapping.nr_mfns; - unsigned long mfn_end = mfn + nr_mfns - 1; - unsigned long gfn_end = gfn + nr_mfns - 1; - int add = domctl->u.memory_mapping.add_mapping; - - ret = -EINVAL; - if ( mfn_end < mfn || /* wrap? */ - ((mfn | mfn_end) >> (paddr_bits - PAGE_SHIFT)) || - gfn_end < gfn ) /* wrap? */ - return ret; - - ret = -EPERM; - if ( !iomem_access_permitted(current->domain, mfn, mfn_end) ) - break; - - ret = xsm_iomem_mapping(XSM_HOOK, d, mfn, mfn_end, add); - if ( ret ) - break; - - if ( add ) - { - printk(XENLOG_G_INFO - "memory_map: add: dom%d gfn=%lx mfn=%lx nr=%lx\n", - d->domain_id, gfn, mfn, nr_mfns); - - ret = iomem_permit_access(d, mfn, mfn_end); - if ( !ret ) - { - ret = map_mmio_regions(d, gfn, gfn_end, mfn); - if ( ret ) - { - printk(XENLOG_G_WARNING - "memory_map: fail: dom%d gfn=%lx mfn=%lx ret:%ld\n", - d->domain_id, gfn, mfn, ret); - if ( iomem_deny_access(d, mfn, mfn_end) && - is_hardware_domain(current->domain) ) - printk(XENLOG_ERR - "memory_map: failed to deny dom%d access " - "to [%lx,%lx]\n", - d->domain_id, mfn, mfn_end); - } - } - } - else - { - int tmp_rc = 0; - - printk(XENLOG_G_INFO - "memory_map: remove: dom%d gfn=%lx mfn=%lx nr=%lx\n", - d->domain_id, gfn, mfn, nr_mfns); - - tmp_rc = unmap_mmio_regions(d, gfn, gfn_end, mfn); - ret = iomem_deny_access(d, mfn, mfn_end); - if ( !ret ) - ret = tmp_rc; - if ( ret && is_hardware_domain(current->domain) ) - printk(XENLOG_ERR - "memory_map: error %ld %s dom%d access to [%lx,%lx]\n", - ret, tmp_rc ? "removing" : "denying", d->domain_id, - mfn, mfn_end); - } - /* Do this unconditionally to cover errors on above failure paths. */ - memory_type_changed(d); - } - break; - case XEN_DOMCTL_ioport_mapping: { #define MAX_IOPORTS 0x10000 diff --git a/xen/common/domctl.c b/xen/common/domctl.c index af3614b..c550c9f 100644 --- a/xen/common/domctl.c +++ b/xen/common/domctl.c @@ -822,6 +822,76 @@ long do_domctl(XEN_GUEST_HANDLE_PARAM(xen_domctl_t) u_domctl) } break; + case XEN_DOMCTL_memory_mapping: + { + unsigned long gfn = op->u.memory_mapping.first_gfn; + unsigned long mfn = op->u.memory_mapping.first_mfn; + unsigned long nr_mfns = op->u.memory_mapping.nr_mfns; + unsigned long mfn_end = mfn + nr_mfns - 1; + unsigned long gfn_end = gfn + nr_mfns - 1; + int add = op->u.memory_mapping.add_mapping; + + ret = -EINVAL; + if ( mfn_end < mfn || /* wrap? */ + ((mfn | mfn_end) >> (paddr_bits - PAGE_SHIFT)) || + gfn_end < gfn ) /* wrap? */ + return ret; + + ret = -EPERM; + if ( !iomem_access_permitted(current->domain, mfn, mfn_end) ) + break; + + ret = xsm_iomem_mapping(XSM_HOOK, d, mfn, mfn_end, add); + if ( ret ) + break; + + if ( add ) + { + printk(XENLOG_G_INFO + "memory_map: add: dom%d gfn=%lx mfn=%lx nr=%lx\n", + d->domain_id, gfn, mfn, nr_mfns); + + ret = iomem_permit_access(d, mfn, mfn_end); + if ( !ret ) + { + ret = map_mmio_regions(d, gfn, gfn_end, mfn); + if ( ret ) + { + printk(XENLOG_G_WARNING + "memory_map: fail: dom%d gfn=%lx mfn=%lx ret:%ld\n", + d->domain_id, gfn, mfn, ret); + if ( iomem_deny_access(d, mfn, mfn_end) && + is_hardware_domain(current->domain) ) + printk(XENLOG_ERR + "memory_map: failed to deny dom%d access " + "to [%lx,%lx]\n", + d->domain_id, mfn, mfn_end); + } + } + } + else + { + int tmp_rc = 0; + + printk(XENLOG_G_INFO + "memory_map: remove: dom%d gfn=%lx mfn=%lx nr=%lx\n", + d->domain_id, gfn, mfn, nr_mfns); + + tmp_rc = unmap_mmio_regions(d, gfn, gfn_end, mfn); + ret = iomem_deny_access(d, mfn, mfn_end); + if ( !ret ) + ret = tmp_rc; + if ( ret && is_hardware_domain(current->domain) ) + printk(XENLOG_ERR + "memory_map: error %ld %s dom%d access to [%lx,%lx]\n", + ret, tmp_rc ? "removing" : "denying", d->domain_id, + mfn, mfn_end); + } + /* Do this unconditionally to cover errors on above failure paths. */ + memory_type_changed(d); + } + break; + case XEN_DOMCTL_settimeoffset: { domain_set_time_offset(d, op->u.settimeoffset.time_offset_seconds); diff --git a/xen/include/asm-arm/p2m.h b/xen/include/asm-arm/p2m.h index f75dd44..3bc72be 100644 --- a/xen/include/asm-arm/p2m.h +++ b/xen/include/asm-arm/p2m.h @@ -2,6 +2,9 @@ #define _XEN_P2M_H #include <xen/mm.h> +#include <xen/p2m-common.h> + +#define paddr_bits PADDR_BITS struct domain; @@ -51,6 +54,8 @@ typedef enum { #define p2m_is_foreign(_t) ((_t) == p2m_map_foreign) #define p2m_is_ram(_t) ((_t) == p2m_ram_rw || (_t) == p2m_ram_ro) +void memory_type_changed(struct domain *d); + /* Initialise vmid allocator */ void p2m_vmid_allocator_init(void); @@ -84,13 +89,6 @@ int p2m_cache_flush(struct domain *d, xen_pfn_t start_mfn, xen_pfn_t end_mfn); /* Setup p2m RAM mapping for domain d from start-end. */ int p2m_populate_ram(struct domain *d, paddr_t start, paddr_t end); -/* Map MMIO regions in the p2m: start_gfn and end_gfn is the range, inclusive - * of the end_gfn frame number, in the guest physical address space to map, - * starting from the machine frame number mfn. */ -int map_mmio_regions(struct domain *d, - unsigned long start_gfn, - unsigned long end_gfn, - unsigned long mfn); int guest_physmap_add_entry(struct domain *d, unsigned long gfn, diff --git a/xen/include/asm-x86/p2m.h b/xen/include/asm-x86/p2m.h index 1098600..0fee7f6 100644 --- a/xen/include/asm-x86/p2m.h +++ b/xen/include/asm-x86/p2m.h @@ -29,21 +29,10 @@ #include <xen/config.h> #include <xen/paging.h> +#include <xen/p2m-common.h> #include <asm/mem_sharing.h> #include <asm/page.h> /* for pagetable_t */ -/* Map MMIO regions in the p2m: start_gfn and end_gfn is the range, inclusive - * of the end_gfn frame number, in the guest physical address space to map, - * starting from the machine frame number mfn. */ -int map_mmio_regions(struct domain *d, - unsigned long start_gfn, - unsigned long end_gfn, - unsigned long mfn); -int unmap_mmio_regions(struct domain *d, - unsigned long start_gfn, - unsigned long end_gfn, - unsigned long mfn); - extern bool_t opt_hap_1gb, opt_hap_2mb; /* diff --git a/xen/include/xen/p2m-common.h b/xen/include/xen/p2m-common.h new file mode 100644 index 0000000..c1210fc --- /dev/null +++ b/xen/include/xen/p2m-common.h @@ -0,0 +1,16 @@ +#ifndef _XEN_P2M_COMMON_H +#define _XEN_P2M_COMMON_H + +/* Map MMIO regions in the p2m: start_gfn and end_gfn is the range, inclusive + * of the end_gfn frame number, in the guest physical address space to map, + * starting from the machine frame number mfn. */ +int map_mmio_regions(struct domain *d, + unsigned long start_gfn, + unsigned long end_gfn, + unsigned long mfn); +int unmap_mmio_regions(struct domain *d, + unsigned long start_gfn, + unsigned long end_gfn, + unsigned long mfn); + +#endif /* _XEN_P2M_COMMON_H */ -- 1.9.1 _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |