Re: [Xen-devel] [PATCH v8 3/6] iommu: make iommu_inclusive_mapping a suboption of dom0-iommu

[Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>]

On 9/7/18 4:07 PM, Roger Pau Monne wrote:
> Introduce a new dom0-iommu=map-inclusive generic option that
> supersedes iommu_inclusive_mapping. The previous behavior is preserved
> and the option should only be enabled by default on Intel hardware.
>
> Signed-off-by: Roger Pau Monné<roger.pau@xxxxxxxxxx>
> Reviewed-by: Paul Durrant<paul.durrant@xxxxxxxxxx>
> Reviewed-by: Jan Beulich<jbeulich@xxxxxxxx>
> Reviewed-by: Kevin Tian<kevin.tian@xxxxxxxxx>
> ---
> Changes since v7:
>   - Use -1, 0, 1 in iommu_hwdom_inclusive.
>   - Print a warning message if map-inclusive is set on ARM.
>
> Changes since v4:
>   - Use an if to set the default option value.
>   - Set the option to false unconditionally on ARM.
>
> Changes since v2:
>   - Fix typo in commit message.
>   - Change style and text of the documentation in xen command line.
>   - Set the defaults in {intel/amd}_iommu_hwdom_init for inclusive.
>   - Re-add the iommu_dom0_passthrough || !is_pv_domain(d) check.
>
> Changes since v1:
>   - Use dom0-iommu instead of the iommu option.
>   - Only enable by default on Intel hardware.
> ---
> Cc: Andrew Cooper<andrew.cooper3@xxxxxxxxxx>
> Cc: George Dunlap<George.Dunlap@xxxxxxxxxxxxx>
> Cc: Ian Jackson<ian.jackson@xxxxxxxxxxxxx>
> Cc: Jan Beulich<jbeulich@xxxxxxxx>
> Cc: Julien Grall<julien.grall@xxxxxxx>
> Cc: Konrad Rzeszutek Wilk<konrad.wilk@xxxxxxxxxx>
> Cc: Stefano Stabellini<sstabellini@xxxxxxxxxx>
> Cc: Tim Deegan<tim@xxxxxxx>
> Cc: Wei Liu<wei.liu2@xxxxxxxxxx>
> Cc: Suravee Suthikulpanit<suravee.suthikulpanit@xxxxxxx>
> Cc: Brian Woods<brian.woods@xxxxxxx>
> Cc: Kevin Tian<kevin.tian@xxxxxxxxx>
For AMD-related changes.

Acked-by: Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>

Thank you,
Suravee

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel