>>> On 21.02.18 at 15:02, <julien.gr...@arm.com> wrote:
> --- a/xen/arch/x86/x86_64/mm.c
> +++ b/xen/arch/x86/x86_64/mm.c
> @@ -40,6 +40,10 @@ asm(".file \"" __FILE__ "\"");
> #include <asm/mem_sharing.h>
> #include <public/memory.h>
>
> +/* Override macros from asm/page.h to make them work with mfn_t */
> +#undef page_to_mfn
> +#define page_to_mfn(pg) _mfn(__page_to_mfn(pg))
I can't spot where this is needed in this file.
> @@ -234,7 +238,7 @@ void vunmap(const void *va)
> #ifndef _PAGE_NONE
> destroy_xen_mappings(addr, addr + PAGE_SIZE * pages);
> #else /* Avoid tearing down intermediate page tables. */
> - map_pages_to_xen(addr, 0, pages, _PAGE_NONE);
> + map_pages_to_xen(addr, _mfn(0), pages, _PAGE_NONE);
INVALID_MFN?
Also please again see about using PFN_DOWN() for some of the
sizes passed to the function as you touch those places anyway.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
