Hi Eric,
> From: Auger Eric <eric.auger@xxxxxxxxxx>
> Sent: Friday, February 12, 2021 5:58 PM
>
> Hi Vivek, Yi,
>
> On 2/12/21 8:14 AM, Vivek Gautam wrote:
> > Hi Yi,
> >
> >
> > On Sat, Jan 23, 2021 at 2:29 PM Liu, Yi L <yi.l.liu@xxxxxxxxx> wrote:
> >>
> >> Hi Eric,
> >>
> >>> From: Auger Eric <eric.auger@xxxxxxxxxx>
> >>> Sent: Tuesday, January 19, 2021 6:03 PM
> >>>
> >>> Hi Yi, Vivek,
> >>>
> >> [...]
> >>>> I see. I think there needs a change in the code there. Should also expect
> >>>> a nesting_info returned instead of an int anymore. @Eric, how about your
> >>>> opinion?
> >>>>
> >>>> domain = iommu_get_domain_for_dev(&vdev->pdev->dev);
> >>>> ret = iommu_domain_get_attr(domain, DOMAIN_ATTR_NESTING,
> >>> &info);
> >>>> if (ret || !(info.features & IOMMU_NESTING_FEAT_PAGE_RESP)) {
> >>>> /*
> >>>> * No need go futher as no page request service support.
> >>>> */
> >>>> return 0;
> >>>> }
> >>> Sure I think it is "just" a matter of synchro between the 2 series. Yi,
> >>
> >> exactly.
> >>
> >>> do you have plans to respin part of
> >>> [PATCH v7 00/16] vfio: expose virtual Shared Virtual Addressing to VMs
> >>> or would you allow me to embed this patch in my series.
> >>
> >> My v7 hasn’t touch the prq change yet. So I think it's better for you to
> >> embed it to your series. ^_^>>
> >
> > Can you please let me know if you have an updated series of these
> > patches? It will help me to work with virtio-iommu/arm side changes.
>
> As per the previous discussion, I plan to take those 2 patches in my
> SMMUv3 nested stage series:
>
> [PATCH v7 01/16] iommu: Report domain nesting info
> [PATCH v7 02/16] iommu/smmu: Report empty domain nesting info
>
> we need to upgrade both since we do not want to report an empty nesting
> info anymore, for arm.
sorry for the late response. I've sent out the updated version. Also,
yeah, please feel free to take the patch in your series.
https://lore.kernel.org/linux-iommu/20210302203545.436623-2-yi.l.liu@xxxxxxxxx/
Regards,
Yi Liu
> Thanks
>
> Eric
> >
> > Thanks & regards
> > Vivek
> >
