Hi,
On 3/10/25 5:23 PM, Alireza Sanaee via wrote:
> Test new PPTT topolopy with cache representation.
>
> Signed-off-by: Alireza Sanaee <alireza.san...@huawei.com>
> Reviewed-by: Jonathan Cameron <jonathan.came...@huawei.com>
> ---
> tests/qtest/bios-tables-test.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
> index 0a333ec43536..6bdc25f4df1e 100644
> --- a/tests/qtest/bios-tables-test.c
> +++ b/tests/qtest/bios-tables-test.c
> @@ -2142,6 +2142,10 @@ static void test_acpi_aarch64_virt_tcg_topology(void)
> };
>
> test_acpi_one("-cpu cortex-a57 "
> + "-M
> virt,smp-cache.0.cache=l1i,smp-cache.0.topology=cluster,"
> + "smp-cache.1.cache=l1d,smp-cache.1.topology=cluster,"
> + "smp-cache.2.cache=l2,smp-cache.2.topology=cluster,"
> + "smp-cache.3.cache=l3,smp-cache.3.topology=cluster "
In the coverletter you used different topologies: core, cluster, socket.
Don't you want to test the same config here?
Is it better to reuse the existing ".topology" variant or to create
another one? (I have no strong opinion though)
Thanks
Eric
> "-smp sockets=1,clusters=2,cores=2,threads=2", &data);
> free_test_data(&data);
> }
