On Fri, Oct 30, 2020 at 06:26:31AM -0400, Michael S. Tsirkin wrote: > On Thu, Oct 08, 2020 at 03:22:54PM -0600, Alex Williamson wrote: > > On Thu, 8 Oct 2020 19:15:56 +0200 > > Jean-Philippe Brucker <[email protected]> wrote: > > > > > From: Bharat Bhushan <[email protected]> > > > > > > Set IOMMU supported page size mask same as host Linux supported page > > > size mask. > > > > > > Signed-off-by: Bharat Bhushan <[email protected]> > > > Signed-off-by: Jean-Philippe Brucker <[email protected]> > > > --- > > > hw/vfio/common.c | 8 ++++++++ > > > 1 file changed, 8 insertions(+) > > > > > > diff --git a/hw/vfio/common.c b/hw/vfio/common.c > > > index 13471ae2943..e66054b02a7 100644 > > > --- a/hw/vfio/common.c > > > +++ b/hw/vfio/common.c > > > @@ -636,6 +636,14 @@ static void vfio_listener_region_add(MemoryListener > > > *listener, > > > int128_get64(llend), > > > iommu_idx); > > > > > > + ret = memory_region_iommu_set_page_size_mask(giommu->iommu, > > > + container->pgsizes, > > > + &err); > > > + if (ret) { > > > + g_free(giommu); > > > + goto fail; > > > + } > > > + > > > ret = memory_region_register_iommu_notifier(section->mr, > > > &giommu->n, > > > &err); > > > if (ret) { > > > > Acked-by: Alex Williamson <[email protected]> > > This one too, seems independent of the rest of the > patchset and can be merged separately, right?
This depends on patch 7 which introduces the function Thanks, Jean > If so > > Acked-by: Michael S. Tsirkin <[email protected]> >
