From: Andrew Cooper <andrew.cooper3@citrix.com>
Using an interface based on addresses directly, not modules.
No functional change.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Signed-off-by: Daniel P. Smith <dpsmith@apertussolutions.com>
Reviewed-by: Jason Andryuk <jason.andryuk@amd.com>
---
Changes since v5:
- moved bootstrap_map_addr() to being static to setup.c
---
xen/arch/x86/setup.c | 19 +++++++++++++------
1 file changed, 13 insertions(+), 6 deletions(-)
diff --git a/xen/arch/x86/setup.c b/xen/arch/x86/setup.c
index 8261a87d9fb4..852454c161ee 100644
--- a/xen/arch/x86/setup.c
+++ b/xen/arch/x86/setup.c
@@ -442,24 +442,22 @@ static void __init normalise_cpu_order(void)
* Ensure a given physical memory range is present in the bootstrap mappings.
* Use superpage mappings to ensure that pagetable memory needn't be allocated.
*/
-void *__init bootstrap_map(const module_t *mod)
+static void *__init bootstrap_map_addr(paddr_t start, paddr_t end)
{
static unsigned long __initdata map_cur = BOOTSTRAP_MAP_BASE;
- uint64_t start, end, mask = (1L << L2_PAGETABLE_SHIFT) - 1;
+ uint64_t mask = (1L << L2_PAGETABLE_SHIFT) - 1;
void *ret;
if ( system_state != SYS_STATE_early_boot )
- return mod ? mfn_to_virt(mod->mod_start) : NULL;
+ return end ? maddr_to_virt(start) : NULL;
- if ( !mod )
+ if ( !end )
{
destroy_xen_mappings(BOOTSTRAP_MAP_BASE, BOOTSTRAP_MAP_LIMIT);
map_cur = BOOTSTRAP_MAP_BASE;
return NULL;
}
- start = (uint64_t)mod->mod_start << PAGE_SHIFT;
- end = start + mod->mod_end;
if ( start >= end )
return NULL;
@@ -475,6 +473,15 @@ void *__init bootstrap_map(const module_t *mod)
return ret;
}
+void *__init bootstrap_map(const module_t *mod)
+{
+ if ( !mod )
+ return bootstrap_map_addr(0, 0);
+
+ return bootstrap_map_addr(pfn_to_paddr(mod->mod_start),
+ pfn_to_paddr(mod->mod_start) + mod->mod_end);
+}
+
static void __init move_memory(
uint64_t dst, uint64_t src, unsigned int size)
{
--
2.30.2