Hi Shameer,
On 1/29/20 11:35 AM, Shameerali Kolothum Thodi wrote:
> Hi Eric,
>
>> -----Original Message-----
>> From: Qemu-devel
>> [mailto:qemu-devel-bounces+shameerali.kolothum.thodi=huawei.com@nongn
>> u.org] On Behalf Of Auger Eric
>> Sent: 28 January 2020 16:29
>> To: Shameerali Kolothum Thodi <shameerali.kolothum.th...@huawei.com>;
>> qemu-devel@nongnu.org; qemu-...@nongnu.org; imamm...@redhat.com
>> Cc: peter.mayd...@linaro.org; xiaoguangrong.e...@gmail.com;
>> m...@redhat.com; Linuxarm <linux...@huawei.com>; xuwei (O)
>> <xuw...@huawei.com>; shannon.zha...@gmail.com; ler...@redhat.com
>> Subject: Re: [PATCH v2 7/7] tests/bios-tables-test: Update arm/virt memhp
>> test
>>
>> Hi Shameer,
>>
>> On 1/17/20 6:45 PM, Shameer Kolothum wrote:
>>> Since we now have both pc-dimm and nvdimm support, update
>>> test_acpi_virt_tcg_memhp() to include those.
>>>
>>> Signed-off-by: Shameer Kolothum <shameerali.kolothum.th...@huawei.com>
>>> ---
>>> tests/data/acpi/virt/NFIT.memhp | 0
>>> tests/data/acpi/virt/SSDT.memhp | 0
>> Is it normal to have those 2 above void files? I lost track about the
>> process.
>
> I guess so :). From tests/qtest/bios-tables-test.c,
>
> /*
> * How to add or update the tests:
> * Contributor:
> * 1. add empty files for new tables, if any, under tests/data/acpi
> * 2. list any changed files in tests/bios-tables-test-allowed-diff.h
> * 3. commit the above *before* making changes that affect the tables
> ...
Thank you for reminding me of the process and doc location
>
> After reading that again, I am not sure those empty files can be in this
> Patch or not. I can move it to 6/7.
yep, maybe better then to put them in the same patch.
>
>>> tests/qtest/bios-tables-test.c | 9 +++++++--
>>> 3 files changed, 7 insertions(+), 2 deletions(-)
>>> create mode 100644 tests/data/acpi/virt/NFIT.memhp
>>> create mode 100644 tests/data/acpi/virt/SSDT.memhp
>>>
>>> diff --git a/tests/data/acpi/virt/NFIT.memhp
>> b/tests/data/acpi/virt/NFIT.memhp
>>> new file mode 100644
>>> index 0000000000..e69de29bb2
>>> diff --git a/tests/data/acpi/virt/SSDT.memhp
>> b/tests/data/acpi/virt/SSDT.memhp
>>> new file mode 100644
>>> index 0000000000..e69de29bb2
>>> diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c
>>> index f1ac2d7e96..695d2e7fac 100644
>>> --- a/tests/qtest/bios-tables-test.c
>>> +++ b/tests/qtest/bios-tables-test.c
>>> @@ -913,12 +913,17 @@ static void test_acpi_virt_tcg_memhp(void)
>>> };
>>>
>>> data.variant = ".memhp";
>>> - test_acpi_one(" -cpu cortex-a57"
>>> + test_acpi_one(" -machine nvdimm=on"
>> nit: maybe keep the same order as before ...
>>> + " -cpu cortex-a57"
>>> " -m 256M,slots=3,maxmem=1G"
>> and simply add ,nvdimm=on to above line.
>>> " -object memory-backend-ram,id=ram0,size=128M"
>>> " -object memory-backend-ram,id=ram1,size=128M"
>>> " -numa node,memdev=ram0 -numa
>> node,memdev=ram1"
>>> - " -numa dist,src=0,dst=1,val=21",
>>> + " -numa dist,src=0,dst=1,val=21"
>>> + " -object memory-backend-ram,id=ram2,size=128M"
>>> + " -object memory-backend-ram,id=nvm0,size=128M"
>>> + " -device pc-dimm,id=dimm0,memdev=ram2,node=0"
>>> + " -device nvdimm,id=dimm1,memdev=nvm0,node=1",
>>> &data);
>>>
>>> free_test_data(&data);
>>>
>
> Ok. Noted.
Thanks
Eric
>
> Thanks,
> Shameer
>
