From 114e0af84ecc08cdb4bd7b871d34be104ca4c59b Mon Sep 17 00:00:00 2001 From: Michael S. Tsirkin Date: Tue, 8 Aug 2017 19:01:56 +0300 Subject: acpi-test: update expected DSDT files Signed-off-by: Michael S. Tsirkin --- tests/acpi-test-data/q35/DSDT | Bin 7782 -> 7782 bytes tests/acpi-test-data/q35/DSDT.bridge | Bin 7799 -> 7799 bytes tests/acpi-test-data/q35/DSDT.cphp | Bin 8245 -> 8245 bytes tests/acpi-test-data/q35/DSDT.ipmibt | Bin 7857 -> 7857 bytes tests/acpi-test-data/q35/DSDT.memhp | Bin 9147 -> 9147 bytes 5 files changed, 0 insertions(+), 0 deletions(-) (limited to 'tests') diff --git a/tests/acpi-test-data/q35/DSDT b/tests/acpi-test-data/q35/DSDT index a6138c8291..a080e2ace2 100644 Binary files a/tests/acpi-test-data/q35/DSDT and b/tests/acpi-test-data/q35/DSDT differ diff --git a/tests/acpi-test-data/q35/DSDT.bridge b/tests/acpi-test-data/q35/DSDT.bridge index 6b90f606f8..31a76732e5 100644 Binary files a/tests/acpi-test-data/q35/DSDT.bridge and b/tests/acpi-test-data/q35/DSDT.bridge differ diff --git a/tests/acpi-test-data/q35/DSDT.cphp b/tests/acpi-test-data/q35/DSDT.cphp index 976755ef2d..3e23cb638f 100644 Binary files a/tests/acpi-test-data/q35/DSDT.cphp and b/tests/acpi-test-data/q35/DSDT.cphp differ diff --git a/tests/acpi-test-data/q35/DSDT.ipmibt b/tests/acpi-test-data/q35/DSDT.ipmibt index b3fa4359ff..9c5e03fd2f 100644 Binary files a/tests/acpi-test-data/q35/DSDT.ipmibt and b/tests/acpi-test-data/q35/DSDT.ipmibt differ diff --git a/tests/acpi-test-data/q35/DSDT.memhp b/tests/acpi-test-data/q35/DSDT.memhp index 7341c405bf..1b7c484a5e 100644 Binary files a/tests/acpi-test-data/q35/DSDT.memhp and b/tests/acpi-test-data/q35/DSDT.memhp differ -- cgit v1.2.3-55-g7522 From 24dd1e17696fbd11b3df50718dddcbdfe2c45f24 Mon Sep 17 00:00:00 2001 From: Jens Freimann Date: Tue, 8 Aug 2017 22:38:59 +0200 Subject: libqtest: always set up signal handler for SIGABRT Currently abort handlers only work for the first test function in a testcase, because the list of abort handlers is not properly cleared when qtest_quit() is called. qtest_quit() only deletes the kill_qemu_hook but doesn't completely clear the abrt_hooks list. The effect is that abrt_hooks.is_setup is never set to false and in a following test the abrt_hooks list is not initialized and setup_sigabrt_handler() is not called. One way to solve this is to clear the list in qtest_quit(), but that means only asserts between qtest_start and qtest_quit will be catched by the abort handler. We can make abort handlers work in all cases if we always setup the signal handler for SIGABRT in qtest_init. Signed-off-by: Jens Freimann Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/libqtest.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests') diff --git a/tests/libqtest.c b/tests/libqtest.c index 4a5492a603..b9a1f180e1 100644 --- a/tests/libqtest.c +++ b/tests/libqtest.c @@ -140,8 +140,8 @@ void qtest_add_abrt_handler(GHookFunc fn, const void *data) /* Only install SIGABRT handler once */ if (!abrt_hooks.is_setup) { g_hook_list_init(&abrt_hooks, sizeof(GHook)); - setup_sigabrt_handler(); } + setup_sigabrt_handler(); hook = g_hook_alloc(&abrt_hooks); hook->func = fn; -- cgit v1.2.3-55-g7522