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.
Thanks
Eric
>
> Thanks & regards
> Vivek
>
