... as the only user of sl1mfn would prefer it that way. No functional change.
Signed-off-by: Andrew Cooper <[email protected]> --- CC: Jan Beulich <[email protected]> CC: Tim Deegan <[email protected]> CC: Wei Liu <[email protected]> CC: Roger Pau Monné <[email protected]> --- xen/arch/x86/mm/shadow/common.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/xen/arch/x86/mm/shadow/common.c b/xen/arch/x86/mm/shadow/common.c index c9640b9..28d1dd4 100644 --- a/xen/arch/x86/mm/shadow/common.c +++ b/xen/arch/x86/mm/shadow/common.c @@ -3834,7 +3834,7 @@ int shadow_track_dirty_vram(struct domain *d, memcpy(dirty_bitmap, dirty_vram->dirty_bitmap, dirty_size); else { - unsigned long map_mfn = mfn_x(INVALID_MFN); + mfn_t map_mfn = INVALID_MFN; void *map_sl1p = NULL; /* Iterate over VRAM to track dirty bits. */ @@ -3872,13 +3872,13 @@ int shadow_track_dirty_vram(struct domain *d, /* Hopefully the most common case: only one mapping, * whose dirty bit we can use. */ l1_pgentry_t *sl1e; - unsigned long sl1mfn = paddr_to_pfn(sl1ma); + mfn_t sl1mfn = maddr_to_mfn(sl1ma); - if ( sl1mfn != map_mfn ) + if ( !mfn_eq(sl1mfn, map_mfn) ) { if ( map_sl1p ) unmap_domain_page(map_sl1p); - map_sl1p = map_domain_page(_mfn(sl1mfn)); + map_sl1p = map_domain_page(sl1mfn); map_mfn = sl1mfn; } sl1e = map_sl1p + (sl1ma & ~PAGE_MASK); -- 2.1.4 _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
