Hi Peter,
On 7/8/25 5:40 PM, Shameer Kolothum wrote:
> Commit d6afe18b7242 ("hw/arm/virt-acpi-build: Fix ACPI IORT and MADT tables
> when its=off") moved ITS group node generation under the its=on condition.
> However, it still creates rc_its_idmaps unconditionally, which results in
> duplicate ID mappings in the IORT table.
>
> Fixes:d6afe18b7242 ("hw/arm/virt-acpi-build: Fix ACPI IORT and MADT tables
> when its=off")
At least please could you take this fix?
Thanks
Eric
> Reviewed-by: Jonathan Cameron <jonathan.came...@huawei.com>
> Reviewed-by: Eric Auger <eric.au...@redhat.com>
> Reviewed-by: Donald Dutile <ddut...@redhat.com>
> Tested-by: Eric Auger <eric.au...@redhat.com>
> Signed-off-by: Shameer Kolothum <shameerali.kolothum.th...@huawei.com>
> ---
> hw/arm/virt-acpi-build.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/hw/arm/virt-acpi-build.c b/hw/arm/virt-acpi-build.c
> index cd90c47976..724fad5ffa 100644
> --- a/hw/arm/virt-acpi-build.c
> +++ b/hw/arm/virt-acpi-build.c
> @@ -329,12 +329,6 @@ build_iort(GArray *table_data, BIOSLinker *linker,
> VirtMachineState *vms)
> /* Sort the smmu idmap by input_base */
> g_array_sort(rc_smmu_idmaps, iort_idmap_compare);
>
> - /*
> - * Knowing the ID ranges from the RC to the SMMU, it's possible to
> - * determine the ID ranges from RC that are directed to the ITS.
> - */
> - create_rc_its_idmaps(rc_its_idmaps, rc_smmu_idmaps);
> -
> nb_nodes = 2; /* RC and SMMUv3 */
> rc_mapping_count = rc_smmu_idmaps->len;
>
