[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 2/5] virtio-iommu: Implement RESV_MEM probe request
From: |
Auger Eric |
Subject: |
Re: [PATCH v6 2/5] virtio-iommu: Implement RESV_MEM probe request |
Date: |
Sat, 27 Jun 2020 09:10:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
Hi Markus,
On 6/27/20 9:08 AM, Markus Armbruster wrote:
> Eric Auger <eric.auger@redhat.com> writes:
>
>> This patch implements the PROBE request. At the moment,
>> only THE RESV_MEM property is handled. The first goal is
>> to report iommu wide reserved regions such as the MSI regions
>> set by the machine code. On x86 this will be the IOAPIC MSI
>> region, [0xFEE00000 - 0xFEEFFFFF], on ARM this may be the ITS
>> doorbell.
>>
>> In the future we may introduce per device reserved regions.
>> This will be useful when protecting host assigned devices
>> which may expose their own reserved regions
>>
>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>> Reviewed-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
>>
>> ---
>>
>> v5 -> v6:
>> - removed validation of s->reserved_regions[i].type in the
>> probe request as it should rather happen in the realize()
>
> Consider keeping the assertion. I'd makes it locally obvious that the
> assignment to subtype is not supposed to truncate. The realize method
> is in another file. This is not a demand.
OK I will add the assert again
Thanks
Eric
>
> [...]
>
- [PATCH v6 0/5] VIRTIO-IOMMU probe request support and MSI bypass on ARM, Eric Auger, 2020/06/26
- [PATCH v6 2/5] virtio-iommu: Implement RESV_MEM probe request, Eric Auger, 2020/06/26
- [PATCH v6 1/5] qdev: Introduce DEFINE_PROP_RESERVED_REGION, Eric Auger, 2020/06/26
- [PATCH v6 3/5] virtio-iommu: Handle reserved regions in the translation process, Eric Auger, 2020/06/26
- [PATCH v6 4/5] virtio-iommu-pci: Add array of Interval properties, Eric Auger, 2020/06/26
- [PATCH v6 5/5] hw/arm/virt: Let the virtio-iommu bypass MSIs, Eric Auger, 2020/06/26