On 3/7/24 11:21, Philippe Mathieu-Daudé wrote:
> On 7/3/24 08:32, Eric Auger wrote:
>> Hi Philippe,
>>
>> On 3/6/24 21:44, Philippe Mathieu-Daudé wrote:
>>> On 6/3/24 21:32, Eric Auger wrote:
>>>> We are missing an entry for the virtio-iommu-pci device. Add the
>>>> information on which machine it is currently supported and document
>>>> the new granule option.
>>>>
>>>> Signed-off-by: Eric Auger <eric.au...@redhat.com>
>>>> ---
>>>> qemu-options.hx | 8 ++++++++
>>>> 1 file changed, 8 insertions(+)
>>>>
>>>> diff --git a/qemu-options.hx b/qemu-options.hx
>>>> index 9a47385c15..757df3eac0 100644
>>>> --- a/qemu-options.hx
>>>> +++ b/qemu-options.hx
>>>> @@ -1172,6 +1172,14 @@ SRST
>>>> Please also refer to the wiki page for general scenarios of
>>>> VT-d
>>>> emulation in QEMU: https://wiki.qemu.org/Features/VT-d.
>>>> +``-device virtio-iommu-pci[,option=...]``
>>>> + This is only supported by ``-machine q35`` and ``-machine virt``.
>>>
>>> Don't we need to precise x86 and arm here?
>> Well I got inspired of intel_iommu doc. I think this is sufficiently
>> precise, no?
>
> Apparently not:
>
> $ git grep -w '"virt"' hw
> hw/arm/virt.c:96: mc->alias = "virt"; \
> hw/m68k/virt.c:314: .name = MACHINE_TYPE_NAME("virt"),
> hw/openrisc/virt.c:37:#define TYPE_VIRT_MACHINE MACHINE_TYPE_NAME("virt")
> hw/riscv/virt.c:1754: .name = MACHINE_TYPE_NAME("virt"),
> hw/xtensa/virt.c:127:DEFINE_MACHINE("virt", xtensa_virt_machine_init)
ah ok. I will precise then.
Thanks!
Eric
