| Message ID | 20191122074826.1373-1-tao3.xu@intel.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Build ACPI Heterogeneous Memory Attribute Table (HMAT) | expand |
Patchew URL: https://patchew.org/QEMU/20191122074826.1373-1-tao3.xu@intel.com/
Hi,
This series failed the docker-quick@centos7 build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.
=== TEST SCRIPT BEGIN ===
#!/bin/bash
make docker-image-centos7 V=1 NETWORK=1
time make docker-test-quick@centos7 SHOW_ENV=1 J=14 NETWORK=1
=== TEST SCRIPT END ===
TEST check-qtest-x86_64: tests/numa-test
Broken pipe
/tmp/qemu-test/src/tests/libqtest.c:149: kill_qemu() detected QEMU death from signal 8 (Floating point exception) (core dumped)
ERROR - too few tests run (expected 9, got 8)
make: *** [check-qtest-x86_64] Error 1
make: *** Waiting for unfinished jobs....
TEST iotest-qcow2: 172
TEST iotest-qcow2: 174
---
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sudo', '-n', 'docker', 'run', '--label', 'com.qemu.instance.uuid=dcec428a97434211b9c0856da658ff0b', '-u', '1003', '--security-opt', 'seccomp=unconfined', '--rm', '-e', 'TARGET_LIST=', '-e', 'EXTRA_CONFIGURE_OPTS=', '-e', 'V=', '-e', 'J=14', '-e', 'DEBUG=', '-e', 'SHOW_ENV=1', '-e', 'CCACHE_DIR=/var/tmp/ccache', '-v', '/home/patchew2/.cache/qemu-docker-ccache:/var/tmp/ccache:z', '-v', '/var/tmp/patchew-tester-tmp-zr9swotk/src/docker-src.2019-11-22-03.28.32.26341:/var/tmp/qemu:z,ro', 'qemu:centos7', '/var/tmp/qemu/run', 'test-quick']' returned non-zero exit status 2.
filter=--filter=label=com.qemu.instance.uuid=dcec428a97434211b9c0856da658ff0b
make[1]: *** [docker-run] Error 1
make[1]: Leaving directory `/var/tmp/patchew-tester-tmp-zr9swotk/src'
make: *** [docker-run-test-quick@centos7] Error 2
real 12m44.362s
user 0m7.997s
The full log is available at
http://patchew.org/logs/20191122074826.1373-1-tao3.xu@intel.com/testing.docker-quick@centos7/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Patchew URL: https://patchew.org/QEMU/20191122074826.1373-1-tao3.xu@intel.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Subject: [PATCH v17 00/14] Build ACPI Heterogeneous Memory Attribute Table (HMAT)
Type: series
Message-id: 20191122074826.1373-1-tao3.xu@intel.com
=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
- [tag update] patchew/20191121000843.24844-1-beata.michalska@linaro.org -> patchew/20191121000843.24844-1-beata.michalska@linaro.org
- [tag update] patchew/20191122080039.12771-1-armbru@redhat.com -> patchew/20191122080039.12771-1-armbru@redhat.com
Switched to a new branch 'test'
9192aa6 tests/bios-tables-test: add test cases for ACPI HMAT
309fd85 tests/numa: Add case for QMP build HMAT
864da49 hmat acpi: Build Memory Side Cache Information Structure(s)
6d92931 hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s)
39ba308 hmat acpi: Build Memory Proximity Domain Attributes Structure(s)
7d0bffc numa: Extend CLI to provide memory side cache information
3fc8a54 numa: Extend CLI to provide memory latency and bandwidth information
592a96a numa: Extend CLI to provide initiator information for numa nodes
7032fc4 tests: Add test for QAPI builtin type time
2d89c93 qapi: Add builtin type time
dbe82f5 util/cutils: Add qemu_strtotime_ns()
2fef66f util/cutils: refactor do_strtosz() to support suffixes list
2cae457 util/cutils: Use qemu_strtold_finite to parse size
a691b5f util/cutils: Add Add qemu_strtold and qemu_strtold_finite
=== OUTPUT BEGIN ===
1/14 Checking commit a691b5f92191 (util/cutils: Add Add qemu_strtold and qemu_strtold_finite)
ERROR: consider using qemu_strtold in preference to strtold
#61: FILE: util/cutils.c:636:
+ *result = strtold(nptr, &ep);
total: 1 errors, 0 warnings, 69 lines checked
Patch 1/14 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
2/14 Checking commit 2cae457669b9 (util/cutils: Use qemu_strtold_finite to parse size)
3/14 Checking commit 2fef66fd2a82 (util/cutils: refactor do_strtosz() to support suffixes list)
4/14 Checking commit dbe82f50cf86 (util/cutils: Add qemu_strtotime_ns())
5/14 Checking commit 2d89c9350115 (qapi: Add builtin type time)
6/14 Checking commit 7032fc4756a0 (tests: Add test for QAPI builtin type time)
7/14 Checking commit 592a96a0ff21 (numa: Extend CLI to provide initiator information for numa nodes)
8/14 Checking commit 3fc8a542a4b7 (numa: Extend CLI to provide memory latency and bandwidth information)
WARNING: line over 80 characters
#130: FILE: hw/core/numa.c:299:
+ /* Set lb_info_provided bit 0 as 1, latency information is provided */
total: 0 errors, 1 warnings, 462 lines checked
Patch 8/14 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
9/14 Checking commit 7d0bffca6382 (numa: Extend CLI to provide memory side cache information)
10/14 Checking commit 39ba308b5612 (hmat acpi: Build Memory Proximity Domain Attributes Structure(s))
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#72:
new file mode 100644
total: 0 errors, 1 warnings, 185 lines checked
Patch 10/14 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
11/14 Checking commit 6d92931a0a40 (hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s))
12/14 Checking commit 864da49c0ce4 (hmat acpi: Build Memory Side Cache Information Structure(s))
13/14 Checking commit 309fd85d39fc (tests/numa: Add case for QMP build HMAT)
WARNING: line over 80 characters
#124: FILE: tests/numa-test.c:433:
+ g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }")));
WARNING: line over 80 characters
#159: FILE: tests/numa-test.c:468:
+ g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }")));
WARNING: line over 80 characters
#206: FILE: tests/numa-test.c:515:
+ g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }")));
total: 0 errors, 3 warnings, 206 lines checked
Patch 13/14 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
14/14 Checking commit 9192aa6b273f (tests/bios-tables-test: add test cases for ACPI HMAT)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#106:
new file mode 100644
total: 0 errors, 1 warnings, 65 lines checked
Patch 14/14 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20191122074826.1373-1-tao3.xu@intel.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
On Fri, 22 Nov 2019 01:17:12 -0800 (PST) no-reply@patchew.org wrote: > Patchew URL: https://patchew.org/QEMU/20191122074826.1373-1-tao3.xu@intel.com/ > do not ignore warnings "line over 80 characters", just fix them > > Hi, > > This series seems to have some coding style problems. See output below for > more information: > > Subject: [PATCH v17 00/14] Build ACPI Heterogeneous Memory Attribute Table (HMAT) > Type: series > Message-id: 20191122074826.1373-1-tao3.xu@intel.com > > === TEST SCRIPT BEGIN === > #!/bin/bash > git rev-parse base > /dev/null || exit 0 > git config --local diff.renamelimit 0 > git config --local diff.renames True > git config --local diff.algorithm histogram > ./scripts/checkpatch.pl --mailback base.. > === TEST SCRIPT END === > > Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 > From https://github.com/patchew-project/qemu > - [tag update] patchew/20191121000843.24844-1-beata.michalska@linaro.org -> patchew/20191121000843.24844-1-beata.michalska@linaro.org > - [tag update] patchew/20191122080039.12771-1-armbru@redhat.com -> patchew/20191122080039.12771-1-armbru@redhat.com > Switched to a new branch 'test' > 9192aa6 tests/bios-tables-test: add test cases for ACPI HMAT > 309fd85 tests/numa: Add case for QMP build HMAT > 864da49 hmat acpi: Build Memory Side Cache Information Structure(s) > 6d92931 hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s) > 39ba308 hmat acpi: Build Memory Proximity Domain Attributes Structure(s) > 7d0bffc numa: Extend CLI to provide memory side cache information > 3fc8a54 numa: Extend CLI to provide memory latency and bandwidth information > 592a96a numa: Extend CLI to provide initiator information for numa nodes > 7032fc4 tests: Add test for QAPI builtin type time > 2d89c93 qapi: Add builtin type time > dbe82f5 util/cutils: Add qemu_strtotime_ns() > 2fef66f util/cutils: refactor do_strtosz() to support suffixes list > 2cae457 util/cutils: Use qemu_strtold_finite to parse size > a691b5f util/cutils: Add Add qemu_strtold and qemu_strtold_finite > > === OUTPUT BEGIN === > 1/14 Checking commit a691b5f92191 (util/cutils: Add Add qemu_strtold and qemu_strtold_finite) > ERROR: consider using qemu_strtold in preference to strtold > #61: FILE: util/cutils.c:636: > + *result = strtold(nptr, &ep); > > total: 1 errors, 0 warnings, 69 lines checked > > Patch 1/14 has style problems, please review. If any of these errors > are false positives report them to the maintainer, see > CHECKPATCH in MAINTAINERS. > > 2/14 Checking commit 2cae457669b9 (util/cutils: Use qemu_strtold_finite to parse size) > 3/14 Checking commit 2fef66fd2a82 (util/cutils: refactor do_strtosz() to support suffixes list) > 4/14 Checking commit dbe82f50cf86 (util/cutils: Add qemu_strtotime_ns()) > 5/14 Checking commit 2d89c9350115 (qapi: Add builtin type time) > 6/14 Checking commit 7032fc4756a0 (tests: Add test for QAPI builtin type time) > 7/14 Checking commit 592a96a0ff21 (numa: Extend CLI to provide initiator information for numa nodes) > 8/14 Checking commit 3fc8a542a4b7 (numa: Extend CLI to provide memory latency and bandwidth information) > WARNING: line over 80 characters > #130: FILE: hw/core/numa.c:299: > + /* Set lb_info_provided bit 0 as 1, latency information is provided */ > > total: 0 errors, 1 warnings, 462 lines checked > > Patch 8/14 has style problems, please review. If any of these errors > are false positives report them to the maintainer, see > CHECKPATCH in MAINTAINERS. > 9/14 Checking commit 7d0bffca6382 (numa: Extend CLI to provide memory side cache information) > 10/14 Checking commit 39ba308b5612 (hmat acpi: Build Memory Proximity Domain Attributes Structure(s)) > WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? > #72: > new file mode 100644 > > total: 0 errors, 1 warnings, 185 lines checked > > Patch 10/14 has style problems, please review. If any of these errors > are false positives report them to the maintainer, see > CHECKPATCH in MAINTAINERS. > 11/14 Checking commit 6d92931a0a40 (hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s)) > 12/14 Checking commit 864da49c0ce4 (hmat acpi: Build Memory Side Cache Information Structure(s)) > 13/14 Checking commit 309fd85d39fc (tests/numa: Add case for QMP build HMAT) > WARNING: line over 80 characters > #124: FILE: tests/numa-test.c:433: > + g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }"))); > > WARNING: line over 80 characters > #159: FILE: tests/numa-test.c:468: > + g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }"))); > > WARNING: line over 80 characters > #206: FILE: tests/numa-test.c:515: > + g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }"))); > > total: 0 errors, 3 warnings, 206 lines checked > > Patch 13/14 has style problems, please review. If any of these errors > are false positives report them to the maintainer, see > CHECKPATCH in MAINTAINERS. > 14/14 Checking commit 9192aa6b273f (tests/bios-tables-test: add test cases for ACPI HMAT) > WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? > #106: > new file mode 100644 > > total: 0 errors, 1 warnings, 65 lines checked > > Patch 14/14 has style problems, please review. If any of these errors > are false positives report them to the maintainer, see > CHECKPATCH in MAINTAINERS. > === OUTPUT END === > > Test command exited with code: 1 > > > The full log is available at > http://patchew.org/logs/20191122074826.1373-1-tao3.xu@intel.com/testing.checkpatch/?type=message. > --- > Email generated automatically by Patchew [https://patchew.org/]. > Please send your feedback to patchew-devel@redhat.com
On 11/22/2019 8:38 PM, Igor Mammedov wrote: > On Fri, 22 Nov 2019 01:17:12 -0800 (PST) > no-reply@patchew.org wrote: > >> Patchew URL: https://patchew.org/QEMU/20191122074826.1373-1-tao3.xu@intel.com/ >> > do not ignore warnings "line over 80 characters", > just fix them OK I will fin them in the next version > >> >> Hi, >> >> This series seems to have some coding style problems. See output below for >> more information: >> >> Subject: [PATCH v17 00/14] Build ACPI Heterogeneous Memory Attribute Table (HMAT) >> Type: series >> Message-id: 20191122074826.1373-1-tao3.xu@intel.com >> >> === TEST SCRIPT BEGIN === >> #!/bin/bash >> git rev-parse base > /dev/null || exit 0 >> git config --local diff.renamelimit 0 >> git config --local diff.renames True >> git config --local diff.algorithm histogram >> ./scripts/checkpatch.pl --mailback base.. >> === TEST SCRIPT END === >> >> Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 >> From https://github.com/patchew-project/qemu >> - [tag update] patchew/20191121000843.24844-1-beata.michalska@linaro.org -> patchew/20191121000843.24844-1-beata.michalska@linaro.org >> - [tag update] patchew/20191122080039.12771-1-armbru@redhat.com -> patchew/20191122080039.12771-1-armbru@redhat.com >> Switched to a new branch 'test' >> 9192aa6 tests/bios-tables-test: add test cases for ACPI HMAT >> 309fd85 tests/numa: Add case for QMP build HMAT >> 864da49 hmat acpi: Build Memory Side Cache Information Structure(s) >> 6d92931 hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s) >> 39ba308 hmat acpi: Build Memory Proximity Domain Attributes Structure(s) >> 7d0bffc numa: Extend CLI to provide memory side cache information >> 3fc8a54 numa: Extend CLI to provide memory latency and bandwidth information >> 592a96a numa: Extend CLI to provide initiator information for numa nodes >> 7032fc4 tests: Add test for QAPI builtin type time >> 2d89c93 qapi: Add builtin type time >> dbe82f5 util/cutils: Add qemu_strtotime_ns() >> 2fef66f util/cutils: refactor do_strtosz() to support suffixes list >> 2cae457 util/cutils: Use qemu_strtold_finite to parse size >> a691b5f util/cutils: Add Add qemu_strtold and qemu_strtold_finite >> >> === OUTPUT BEGIN === >> 1/14 Checking commit a691b5f92191 (util/cutils: Add Add qemu_strtold and qemu_strtold_finite) >> ERROR: consider using qemu_strtold in preference to strtold >> #61: FILE: util/cutils.c:636: >> + *result = strtold(nptr, &ep); >> >> total: 1 errors, 0 warnings, 69 lines checked >> >> Patch 1/14 has style problems, please review. If any of these errors >> are false positives report them to the maintainer, see >> CHECKPATCH in MAINTAINERS. >> >> 2/14 Checking commit 2cae457669b9 (util/cutils: Use qemu_strtold_finite to parse size) >> 3/14 Checking commit 2fef66fd2a82 (util/cutils: refactor do_strtosz() to support suffixes list) >> 4/14 Checking commit dbe82f50cf86 (util/cutils: Add qemu_strtotime_ns()) >> 5/14 Checking commit 2d89c9350115 (qapi: Add builtin type time) >> 6/14 Checking commit 7032fc4756a0 (tests: Add test for QAPI builtin type time) >> 7/14 Checking commit 592a96a0ff21 (numa: Extend CLI to provide initiator information for numa nodes) >> 8/14 Checking commit 3fc8a542a4b7 (numa: Extend CLI to provide memory latency and bandwidth information) >> WARNING: line over 80 characters >> #130: FILE: hw/core/numa.c:299: >> + /* Set lb_info_provided bit 0 as 1, latency information is provided */ >> >> total: 0 errors, 1 warnings, 462 lines checked >> >> Patch 8/14 has style problems, please review. If any of these errors >> are false positives report them to the maintainer, see >> CHECKPATCH in MAINTAINERS. >> 9/14 Checking commit 7d0bffca6382 (numa: Extend CLI to provide memory side cache information) >> 10/14 Checking commit 39ba308b5612 (hmat acpi: Build Memory Proximity Domain Attributes Structure(s)) >> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? >> #72: >> new file mode 100644 >> >> total: 0 errors, 1 warnings, 185 lines checked >> >> Patch 10/14 has style problems, please review. If any of these errors >> are false positives report them to the maintainer, see >> CHECKPATCH in MAINTAINERS. >> 11/14 Checking commit 6d92931a0a40 (hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s)) >> 12/14 Checking commit 864da49c0ce4 (hmat acpi: Build Memory Side Cache Information Structure(s)) >> 13/14 Checking commit 309fd85d39fc (tests/numa: Add case for QMP build HMAT) >> WARNING: line over 80 characters >> #124: FILE: tests/numa-test.c:433: >> + g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }"))); >> >> WARNING: line over 80 characters >> #159: FILE: tests/numa-test.c:468: >> + g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }"))); >> >> WARNING: line over 80 characters >> #206: FILE: tests/numa-test.c:515: >> + g_assert(!qmp_rsp_is_err(qtest_qmp(qs, "{ 'execute': 'x-exit-preconfig' }"))); >> >> total: 0 errors, 3 warnings, 206 lines checked >> >> Patch 13/14 has style problems, please review. If any of these errors >> are false positives report them to the maintainer, see >> CHECKPATCH in MAINTAINERS. >> 14/14 Checking commit 9192aa6b273f (tests/bios-tables-test: add test cases for ACPI HMAT) >> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? >> #106: >> new file mode 100644 >> >> total: 0 errors, 1 warnings, 65 lines checked >> >> Patch 14/14 has style problems, please review. If any of these errors >> are false positives report them to the maintainer, see >> CHECKPATCH in MAINTAINERS. >> === OUTPUT END === >> >> Test command exited with code: 1 >> >> >> The full log is available at >> http://patchew.org/logs/20191122074826.1373-1-tao3.xu@intel.com/testing.checkpatch/?type=message. >> --- >> Email generated automatically by Patchew [https://patchew.org/]. >> Please send your feedback to patchew-devel@redhat.com >
This series of patches will build Heterogeneous Memory Attribute Table (HMAT) according to the command line. The ACPI HMAT describes the memory attributes, such as memory side cache attributes and bandwidth and latency details, related to the Memory Proximity Domain. The software is expected to use HMAT information as hint for optimization. In the linux kernel, the codes in drivers/acpi/hmat/hmat.c parse and report the platform's HMAT tables. The V16 patches link: https://patchwork.kernel.org/cover/11245293/ Changelog: v17: - Add check when user input latency or bandwidth 0, the lb_info_provided should also be 0. Because in ACPI 6.3 5.2.27.4, 0 means the corresponding latency or bandwidth information is not provided. - Fix the infinite loop when node->latency is 0. - Use NumaHmatCacheOptions to replace HMAT_Cache_Info (Igor) - Add check for unordered cache level input (Igor) - Add some fail test cases (Igor) v16: - Add and use qemu_strtold_finite to parse size, support full 64bit precision, modify related test cases (Eduardo and Markus) - Simplify struct HMAT_LB_Info and related code, unify latency and bandwidth (Igor) - Add cross check with hmat_lb data (Igor) - Fields in Cache Attributes are promoted to uint32_t before shifting (Igor) - Add case for QMP build HMAT (Igor) v15: - Add a new patch to refactor do_strtosz() (Eduardo) - Make tests without breaking CI (Michael) v14: - Reuse the codes of do_strtosz to build qemu_strtotime_ns (Eduardo) - Squash patch v13 01/12 and 02/12 together (Daniel and Eduardo) - Drop time unit picosecond (Eric) - Use qemu ctz64 and clz64 instead of builtin function v13: - Modify some text description - Drop "initiator_valid" field in struct NodeInfo - Reuse Garray to store the raw bandwidth and bandwidth data - Calculate common base unit using range bitmap - Add a patch to alculate hmat latency and bandwidth entry list - Drop the total_levels option and use readable cache size - Remove the unnecessary head file - Use decimal notation with appropriate suffix for cache size v12: - Fix a bug that a memory-only node without initiator setting doesn't report error. (reported by Danmei Wei) - Fix a bug that if HMAT is enabled and without hmat-lb setting, QEMU will crash. (reported by Danmei Wei) v11: - Move numa option patches forward. - Add num_initiator in Numa_state to record the number of initiators. - Simplify struct HMAT_LB_Info, use uint64_t array to store data. - Drop hmat_get_base(). - Calculate base in build_hmat_lb(). Liu Jingqi (5): numa: Extend CLI to provide memory latency and bandwidth information numa: Extend CLI to provide memory side cache information hmat acpi: Build Memory Proximity Domain Attributes Structure(s) hmat acpi: Build System Locality Latency and Bandwidth Information Structure(s) hmat acpi: Build Memory Side Cache Information Structure(s) Tao Xu (9): util/cutils: Add Add qemu_strtold and qemu_strtold_finite util/cutils: Use qemu_strtold_finite to parse size util/cutils: refactor do_strtosz() to support suffixes list util/cutils: Add qemu_strtotime_ns() qapi: Add builtin type time tests: Add test for QAPI builtin type time numa: Extend CLI to provide initiator information for numa nodes tests/numa: Add case for QMP build HMAT tests/bios-tables-test: add test cases for ACPI HMAT hw/acpi/Kconfig | 7 +- hw/acpi/Makefile.objs | 1 + hw/acpi/hmat.c | 268 +++++++++++++++++++++++++ hw/acpi/hmat.h | 42 ++++ hw/core/machine.c | 64 ++++++ hw/core/numa.c | 279 ++++++++++++++++++++++++++ hw/i386/acpi-build.c | 5 + include/qapi/visitor-impl.h | 4 + include/qapi/visitor.h | 8 + include/qemu/cutils.h | 4 + include/sysemu/numa.h | 63 ++++++ qapi/machine.json | 178 +++++++++++++++- qapi/opts-visitor.c | 22 ++ qapi/qapi-visit-core.c | 12 ++ qapi/qobject-input-visitor.c | 18 ++ qapi/trace-events | 1 + qemu-options.hx | 96 ++++++++- scripts/qapi/schema.py | 1 + tests/bios-tables-test-allowed-diff.h | 8 + tests/bios-tables-test.c | 44 ++++ tests/data/acpi/pc/APIC.acpihmat | 0 tests/data/acpi/pc/DSDT.acpihmat | 0 tests/data/acpi/pc/HMAT.acpihmat | 0 tests/data/acpi/pc/SRAT.acpihmat | 0 tests/data/acpi/q35/APIC.acpihmat | 0 tests/data/acpi/q35/DSDT.acpihmat | 0 tests/data/acpi/q35/HMAT.acpihmat | 0 tests/data/acpi/q35/SRAT.acpihmat | 0 tests/numa-test.c | 194 ++++++++++++++++++ tests/test-cutils.c | 214 ++++++++++++++++---- tests/test-keyval.c | 136 +++++++++---- tests/test-qemu-opts.c | 39 +--- tests/test-qobject-input-visitor.c | 29 +++ util/cutils.c | 147 ++++++++++---- 34 files changed, 1726 insertions(+), 158 deletions(-) create mode 100644 hw/acpi/hmat.c create mode 100644 hw/acpi/hmat.h create mode 100644 tests/data/acpi/pc/APIC.acpihmat create mode 100644 tests/data/acpi/pc/DSDT.acpihmat create mode 100644 tests/data/acpi/pc/HMAT.acpihmat create mode 100644 tests/data/acpi/pc/SRAT.acpihmat create mode 100644 tests/data/acpi/q35/APIC.acpihmat create mode 100644 tests/data/acpi/q35/DSDT.acpihmat create mode 100644 tests/data/acpi/q35/HMAT.acpihmat create mode 100644 tests/data/acpi/q35/SRAT.acpihmat