> -----Original Message-----
> From: Nathan Chen <[email protected]>
> Sent: 17 March 2026 18:38
> To: [email protected]; [email protected]
> Cc: Eric Auger <[email protected]>; Peter Maydell
> <[email protected]>; Shannon Zhao <[email protected]>;
> Michael S . Tsirkin <[email protected]>; Igor Mammedov
> <[email protected]>; Ani Sinha <[email protected]>; Paolo Bonzini
> <[email protected]>; Daniel P . Berrangé <[email protected]>; Eric
> Blake <[email protected]>; Markus Armbruster <[email protected]>;
> Shameer Kolothum Thodi <[email protected]>; Matt Ochs
> <[email protected]>; Nicolin Chen <[email protected]>; Nathan Chen
> <[email protected]>
> Subject: [PATCH v3 1/8] hw/arm/smmuv3-accel: Check ATS compatibility
> between host and guest
>
> From: Nathan Chen <[email protected]>
>
> Compare the host SMMUv3 ATS support bit with the guest SMMUv3 ATS
> support bit in IDR0 and fail the compatibility check if ATS support is opted
> as
> enabled on the guest SMMUv3 when it is not supported on host SMMUv3.
>
> Fixes: f7f5013a55a3 ("hw/arm/smmuv3-accel: Add support for ATS")
> Reviewed-by: Eric Auger <[email protected]>
> Signed-off-by: Nathan Chen <[email protected]>
> ---
> hw/arm/smmuv3-accel.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/hw/arm/smmuv3-accel.c b/hw/arm/smmuv3-accel.c index
> 17306cd04b..2bb142c47f 100644
> --- a/hw/arm/smmuv3-accel.c
> +++ b/hw/arm/smmuv3-accel.c
> @@ -101,6 +101,12 @@
> smmuv3_accel_check_hw_compatible(SMMUv3State *s,
> smmuv3_oas_bits(FIELD_EX32(s->idr[5], IDR5, OAS)));
> return false;
> }
> + /* Check ATS value opted is compatible with Host SMMUv3 */
> + if (FIELD_EX32(info->idr[0], IDR0, ATS) <
> + FIELD_EX32(s->idr[0], IDR0, ATS)) {
> + error_setg(errp, "Host SMMUv3 doesn't support Address Translation
> Services");
> + return false;
> + }
Reviewed-by: Shameer Kolothum <[email protected]>
Thanks,
Shameer
