[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH] x86/HAP: drop pointless assertions
Just like for shadow code, hap_map_domain_page() is just a thin wrapper around map_domain_page(), which cannot fail. Asserting its success is redundant. hap_unmap_domain_page() should also not be used for something not mapped through hap_map_domain_page(). Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx> --- a/xen/arch/x86/mm/hap/hap.c +++ b/xen/arch/x86/mm/hap/hap.c @@ -233,9 +233,8 @@ static struct page_info *hap_alloc(struc d->arch.paging.hap.free_pages--; p = __map_domain_page(pg); - ASSERT(p != NULL); clear_page(p); - hap_unmap_domain_page(p); + unmap_domain_page(p); return pg; } @@ -375,7 +374,6 @@ static void hap_install_xen_entries_in_l l4_pgentry_t *l4e; l4e = hap_map_domain_page(l4mfn); - ASSERT(l4e != NULL); /* Copy the common Xen mappings from the idle domain */ memcpy(&l4e[ROOT_PAGETABLE_FIRST_XEN_SLOT], Attachment:
x86-HAP-pointless-asserts.patch _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |