On 24/08/2021 15:27, Jan Beulich wrote: > --- a/xen/drivers/passthrough/vtd/utils.c > +++ b/xen/drivers/passthrough/vtd/utils.c > @@ -106,11 +106,6 @@ void print_vtd_entries(struct vtd_iommu > } > > root_entry = (struct root_entry *)map_vtd_domain_page(iommu->root_maddr);
Seeing as you're actually cleaning up mapping calls, drop this cast too? Either way, Acked-by: Andrew Cooper <[email protected]> > - if ( root_entry == NULL ) > - { > - printk(" root_entry == NULL\n"); > - return; > - } > > printk(" root_entry[%02x] = %"PRIx64"\n", bus, root_entry[bus].val); > if ( !root_present(root_entry[bus]) ) >
