tests: cleanup ptimer-test
1) ptimer-test is not a qtest---it runs the ptimer.c code directly in the ptimer-test process 2) ptimer-test has its own stubs file, so there is no need to add more stubs to stubs/vmstate.c Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Dmitry Osipenko <digetx@gmail.com> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
		
							parent
							
								
									7a2334f720
								
							
						
					
					
						commit
						24b9462544
					
				| 
						 | 
				
			
			@ -3,11 +3,6 @@
 | 
			
		|||
#include "migration/vmstate.h"
 | 
			
		||||
 | 
			
		||||
const VMStateDescription vmstate_dummy = {};
 | 
			
		||||
const VMStateInfo vmstate_info_uint8;
 | 
			
		||||
const VMStateInfo vmstate_info_uint32;
 | 
			
		||||
const VMStateInfo vmstate_info_uint64;
 | 
			
		||||
const VMStateInfo vmstate_info_int64;
 | 
			
		||||
const VMStateInfo vmstate_info_timer;
 | 
			
		||||
 | 
			
		||||
int vmstate_register_with_alias_id(DeviceState *dev,
 | 
			
		||||
                                   int instance_id,
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -116,6 +116,8 @@ check-unit-$(CONFIG_REPLICATION) += tests/test-replication$(EXESUF)
 | 
			
		|||
check-unit-y += tests/test-bufferiszero$(EXESUF)
 | 
			
		||||
gcov-files-check-bufferiszero-y = util/bufferiszero.c
 | 
			
		||||
check-unit-y += tests/test-uuid$(EXESUF)
 | 
			
		||||
check-unit-y += tests/ptimer-test$(EXESUF)
 | 
			
		||||
gcov-files-ptimer-test-y = hw/core/ptimer.c
 | 
			
		||||
 | 
			
		||||
check-block-$(CONFIG_POSIX) += tests/qemu-iotests-quick.sh
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -307,7 +309,6 @@ check-qtest-xtensaeb-y = $(check-qtest-xtensa-y)
 | 
			
		|||
check-qtest-s390x-y = tests/boot-serial-test$(EXESUF)
 | 
			
		||||
 | 
			
		||||
check-qtest-generic-y += tests/qom-test$(EXESUF)
 | 
			
		||||
check-qtest-generic-y += tests/ptimer-test$(EXESUF)
 | 
			
		||||
 | 
			
		||||
qapi-schema += alternate-any.json
 | 
			
		||||
qapi-schema += alternate-array.json
 | 
			
		||||
| 
						 | 
				
			
			@ -515,6 +516,7 @@ tests/test-timed-average$(EXESUF): tests/test-timed-average.o qemu-timer.o \
 | 
			
		|||
	$(test-util-obj-y)
 | 
			
		||||
tests/test-base64$(EXESUF): tests/test-base64.o \
 | 
			
		||||
	libqemuutil.a libqemustub.a
 | 
			
		||||
tests/ptimer-test$(EXESUF): tests/ptimer-test.o tests/ptimer-test-stubs.o hw/core/ptimer.o libqemustub.a
 | 
			
		||||
 | 
			
		||||
tests/test-logging$(EXESUF): tests/test-logging.o $(test-util-obj-y)
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -677,7 +679,6 @@ tests/test-filter-redirector$(EXESUF): tests/test-filter-redirector.o $(qtest-ob
 | 
			
		|||
tests/test-x86-cpuid-compat$(EXESUF): tests/test-x86-cpuid-compat.o $(qtest-obj-y)
 | 
			
		||||
tests/ivshmem-test$(EXESUF): tests/ivshmem-test.o contrib/ivshmem-server/ivshmem-server.o $(libqos-pc-obj-y)
 | 
			
		||||
tests/vhost-user-bridge$(EXESUF): tests/vhost-user-bridge.o
 | 
			
		||||
tests/ptimer-test$(EXESUF): tests/ptimer-test.o tests/ptimer-test-stubs.o hw/core/ptimer.o
 | 
			
		||||
tests/test-uuid$(EXESUF): tests/test-uuid.o $(test-util-obj-y)
 | 
			
		||||
 | 
			
		||||
tests/migration/stress$(EXESUF): tests/migration/stress.o
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -11,9 +11,16 @@
 | 
			
		|||
#include "qemu/osdep.h"
 | 
			
		||||
#include "qemu/main-loop.h"
 | 
			
		||||
#include "sysemu/replay.h"
 | 
			
		||||
#include "migration/vmstate.h"
 | 
			
		||||
 | 
			
		||||
#include "ptimer-test.h"
 | 
			
		||||
 | 
			
		||||
const VMStateInfo vmstate_info_uint8;
 | 
			
		||||
const VMStateInfo vmstate_info_uint32;
 | 
			
		||||
const VMStateInfo vmstate_info_uint64;
 | 
			
		||||
const VMStateInfo vmstate_info_int64;
 | 
			
		||||
const VMStateInfo vmstate_info_timer;
 | 
			
		||||
 | 
			
		||||
struct QEMUBH {
 | 
			
		||||
    QEMUBHFunc *cb;
 | 
			
		||||
    void *opaque;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -505,47 +505,47 @@ static void add_ptimer_tests(uint8_t policy)
 | 
			
		|||
        g_sprintf(policy_name, "default");
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/set_count policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_set_count);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/set_limit policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_set_limit);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/oneshot policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_oneshot);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/periodic policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_periodic);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/on_the_fly_mode_change policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_on_the_fly_mode_change);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/on_the_fly_period_change policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_on_the_fly_period_change);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/on_the_fly_freq_change policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_on_the_fly_freq_change);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/run_with_period_0 policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_run_with_period_0);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/run_with_delta_0 policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_run_with_delta_0);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/periodic_with_load_0 policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_periodic_with_load_0);
 | 
			
		||||
 | 
			
		||||
    qtest_add_data_func(
 | 
			
		||||
    g_test_add_data_func(
 | 
			
		||||
        g_strdup_printf("/ptimer/oneshot_with_load_0 policy=%s", policy_name),
 | 
			
		||||
        ppolicy, check_oneshot_with_load_0);
 | 
			
		||||
}
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue