Skip to content

Commit 75ad6e8

Browse files
vittyvkbonzini
authored andcommitted
KVM: selftests: fix vmx_preemption_timer_test build with GCC10
GCC10 fails to build vmx_preemption_timer_test: gcc -Wall -Wstrict-prototypes -Wuninitialized -O2 -g -std=gnu99 -fno-stack-protector -fno-PIE -I../../../../tools/include -I../../../../tools/arch/x86/include -I../../../../usr/include/ -Iinclude -Ix86_64 -Iinclude/x86_64 -I.. -pthread -no-pie x86_64/evmcs_test.c ./linux/tools/testing/selftests/kselftest_harness.h ./linux/tools/testing/selftests/kselftest.h ./linux/tools/testing/selftests/kvm/libkvm.a -o ./linux/tools/testing/selftests/kvm/x86_64/evmcs_test /usr/bin/ld: ./linux/tools/testing/selftests/kvm/libkvm.a(vmx.o): ./linux/tools/testing/selftests/kvm/include/x86_64/vmx.h:603: multiple definition of `ctrl_exit_rev'; /tmp/ccMQpvNt.o: ./linux/tools/testing/selftests/kvm/include/x86_64/vmx.h:603: first defined here /usr/bin/ld: ./linux/tools/testing/selftests/kvm/libkvm.a(vmx.o): ./linux/tools/testing/selftests/kvm/include/x86_64/vmx.h:602: multiple definition of `ctrl_pin_rev'; /tmp/ccMQpvNt.o: ./linux/tools/testing/selftests/kvm/include/x86_64/vmx.h:602: first defined here ... ctrl_exit_rev/ctrl_pin_rev/basic variables are only used in vmx_preemption_timer_test.c, just move them there. Fixes: 8d7fbf0 ("KVM: selftests: VMX preemption timer migration test") Reported-by: Marcelo Bandeira Condotta <[email protected]> Signed-off-by: Vitaly Kuznetsov <[email protected]> Message-Id: <[email protected]> Signed-off-by: Paolo Bonzini <[email protected]>
1 parent 5ae1452 commit 75ad6e8

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

tools/testing/selftests/kvm/include/x86_64/vmx.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -598,10 +598,6 @@ union vmx_ctrl_msr {
598598
};
599599
};
600600

601-
union vmx_basic basic;
602-
union vmx_ctrl_msr ctrl_pin_rev;
603-
union vmx_ctrl_msr ctrl_exit_rev;
604-
605601
struct vmx_pages *vcpu_alloc_vmx(struct kvm_vm *vm, vm_vaddr_t *p_vmx_gva);
606602
bool prepare_for_vmx_operation(struct vmx_pages *vmx);
607603
void prepare_vmcs(struct vmx_pages *vmx, void *guest_rip, void *guest_rsp);

tools/testing/selftests/kvm/x86_64/vmx_preemption_timer_test.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,10 @@ bool l2_save_restore_done;
3131
static u64 l2_vmx_pt_start;
3232
volatile u64 l2_vmx_pt_finish;
3333

34+
union vmx_basic basic;
35+
union vmx_ctrl_msr ctrl_pin_rev;
36+
union vmx_ctrl_msr ctrl_exit_rev;
37+
3438
void l2_guest_code(void)
3539
{
3640
u64 vmx_pt_delta;

0 commit comments

Comments
 (0)