Message ID | 20191028075220.25673-1-tao3.xu@intel.com |
---|---|
Headers | show |
Series | Build ACPI Heterogeneous Memory Attribute Table (HMAT) | expand |
Patchew URL: https://patchew.org/QEMU/20191028075220.25673-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 === Looking for expected file 'tests/data/acpi/pc/SRAT.acpihmat' Looking for expected file 'tests/data/acpi/pc/SRAT' ** ERROR:/tmp/qemu-test/src/tests/bios-tables-test.c:354:load_expected_aml: assertion failed: (exp_sdt.aml_file) ERROR - Bail out! ERROR:/tmp/qemu-test/src/tests/bios-tables-test.c:354:load_expected_aml: assertion failed: (exp_sdt.aml_file) make: *** [check-qtest-x86_64] Error 1 make: *** Waiting for unfinished jobs.... TEST iotest-qcow2: 035 TEST iotest-qcow2: 036 --- raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command '['sudo', '-n', 'docker', 'run', '--label', 'com.qemu.instance.uuid=1a4bf9c587854f81918532b75d1e65b8', '-u', '1001', '--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/patchew/.cache/qemu-docker-ccache:/var/tmp/ccache:z', '-v', '/var/tmp/patchew-tester-tmp-3ruj_rni/src/docker-src.2019-10-28-04.25.27.15934:/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=1a4bf9c587854f81918532b75d1e65b8 make[1]: *** [docker-run] Error 1 make[1]: Leaving directory `/var/tmp/patchew-tester-tmp-3ruj_rni/src' make: *** [docker-run-test-quick@centos7] Error 2 real 11m10.684s user 0m8.369s The full log is available at http://patchew.org/logs/20191028075220.25673-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
Ping for comments:) On 10/28/2019 3:52 PM, Xu, Tao3 wrote: > 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 V13 patches link: > https://patchwork.kernel.org/cover/11200909/ > > Changelog: > 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(). > v10: > - Add qemu_strtotime_ps() to convert strings with time suffixes > to numbers, and add some tests for it. > - Add qapi buildin type time, and add some tests for it. > - Add machine oprion properties "-machine hmat=on|off" for > enabling or disabling HMAT in QEMU. > > 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 (6): > 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 > numa: Calculate hmat latency and bandwidth entry list > tests/bios-tables-test: add test cases for ACPI HMAT > > hw/acpi/Kconfig | 7 +- > hw/acpi/Makefile.objs | 1 + > hw/acpi/hmat.c | 263 ++++++++++++++++++++++++++ > hw/acpi/hmat.h | 42 +++++ > hw/core/machine.c | 64 +++++++ > hw/core/numa.c | 284 ++++++++++++++++++++++++++++- > hw/i386/acpi-build.c | 5 + > include/qapi/visitor-impl.h | 4 + > include/qapi/visitor.h | 8 + > include/qemu/cutils.h | 1 + > include/sysemu/numa.h | 104 +++++++++++ > 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.c | 44 +++++ > tests/test-cutils.c | 204 +++++++++++++++++++++ > tests/test-keyval.c | 122 +++++++++++++ > tests/test-qobject-input-visitor.c | 29 +++ > util/cutils.c | 89 +++++---- > 23 files changed, 1555 insertions(+), 44 deletions(-) > create mode 100644 hw/acpi/hmat.c > create mode 100644 hw/acpi/hmat.h >