On 1/8/25 08:05, Sairaj Kodilkar wrote:
The ats_enabled field is set using HTTUNEN, which is wrong.
Fix this by removing the field as it is never used.

Fixes: d29a09ca68428 ("hw/i386: Introduce AMD IOMMU")
Signed-off-by: Sairaj Kodilkar <sarun...@amd.com>
Reviewed-by: Vasant Hegde <vasant.he...@amd.com>
---
  hw/i386/amd_iommu.c | 5 ++---
  hw/i386/amd_iommu.h | 1 -
  2 files changed, 2 insertions(+), 4 deletions(-)


@@ -1626,7 +1624,8 @@ static const VMStateDescription 
vmstate_amdvi_sysbus_migratable = {
        /* Updated in  amdvi_handle_control_write() */
        VMSTATE_BOOL(enabled, AMDVIState),
        VMSTATE_BOOL(ga_enabled, AMDVIState),
-      VMSTATE_BOOL(ats_enabled, AMDVIState),
+      /* bool ats_enabled is obsolete */
+      VMSTATE_UNUSED(1),

         VMSTATE_UNUSED(1), /* was ats_enabled */

Otherwise,

Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>


Reply via email to