[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v7 9/9] qemu-options.hx: Document the virtio-iommu-pci aw-bit
From: |
Eric Auger |
Subject: |
Re: [PATCH v7 9/9] qemu-options.hx: Document the virtio-iommu-pci aw-bits option |
Date: |
Thu, 7 Mar 2024 08:40:53 +0100 |
User-agent: |
Mozilla Thunderbird |
On 3/6/24 21:48, Philippe Mathieu-Daudé wrote:
> On 6/3/24 21:32, Eric Auger wrote:
>> Document the new aw-bits option.
>>
>> Signed-off-by: Eric Auger <eric.auger@redhat.com>
>> Reviewed-by: Cédric Le Goater <clg@redhat.com>
>>
>> ---
>>
>> v4 -> v5
>> - tweek the aw-bits option description according to Cédric's
>> suggestion
>> ---
>> qemu-options.hx | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/qemu-options.hx b/qemu-options.hx
>> index 757df3eac0..87959ede08 100644
>> --- a/qemu-options.hx
>> +++ b/qemu-options.hx
>> @@ -1180,6 +1180,9 @@ SRST
>> This decides the default granule to be be exposed by the
>> virtio-iommu. If host, the granule matches the host page size.
>> + ``aw-bits=val`` (val between 32 and 64, default depends on
>> machine)
>> + This decides the address width of IOVA address space. It
>> defaults
>
> "It defaults"?
Yes needs to be removed.
>
> IMO this should be squashed in patch 6/9 "virtio-iommu: Add an option
> to define the input range width".
in [6/9] the default does not yet depend on machine, hence my choice to
keep it separate and after 8/9
Eric
>
>> ERST
>> DEF("name", HAS_ARG, QEMU_OPTION_name,
>
- Re: [PATCH v7 4/9] qemu-options.hx: Document the virtio-iommu-pci granule option, (continued)
[PATCH v7 6/9] virtio-iommu: Add an option to define the input range width, Eric Auger, 2024/03/06
[PATCH v7 3/9] virtio-iommu: Change the default granule to the host page size, Eric Auger, 2024/03/06
[PATCH v7 9/9] qemu-options.hx: Document the virtio-iommu-pci aw-bits option, Eric Auger, 2024/03/06
[PATCH v7 5/9] virtio-iommu: Trace domain range limits as unsigned int, Eric Auger, 2024/03/06
[PATCH v7 7/9] hw/i386/q35: Set virtio-iommu aw-bits default value to 39, Eric Auger, 2024/03/06
[PATCH v7 8/9] hw/arm/virt: Set virtio-iommu aw-bits default value to 48, Eric Auger, 2024/03/06