Skip to content

Commit f03ae8d

Browse files
KaigeFulijinxia
authored andcommitted
HV: instr_emul: Rearrange logic of instr_emul*
Originally, there is cross-references between instr_emul.h and instr_emul_wrapper.h. User must include both of them when calling instruction emulation functions. This will raise up some confusion and inconvenience. So we rearrange the logic of instruction emulation code as following: - External API -- defined in instr_emul.h * decode_instruction(struct vcpu *vcpu) * emulate_instruction(struct vcpu *vcpu) - Make all other functions as static in instr_emul.c - Remove instr_emul_wrapper.c/h No functional change. Signed-off-by: Kaige Fu <kaige.fu@intel.com> Reviewed-by: Eddie Dong <eddie.dong@intel.com> Reviewed-by: Yin Fengwei <fengwei.yin@intel.com>
1 parent ce79d3a commit f03ae8d

File tree

9 files changed

+670
-787
lines changed

9 files changed

+670
-787
lines changed

hypervisor/Makefile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,6 @@ C_SRCS += arch/x86/pm.c
139139
S_SRCS += arch/x86/wakeup.S
140140
C_SRCS += arch/x86/guest/vcpu.c
141141
C_SRCS += arch/x86/guest/vm.c
142-
C_SRCS += arch/x86/guest/instr_emul_wrapper.c
143142
C_SRCS += arch/x86/guest/vlapic.c
144143
C_SRCS += arch/x86/guest/guest.c
145144
C_SRCS += arch/x86/guest/vmcall.c

hypervisor/arch/x86/ept.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
#include <hypervisor.h>
88

9-
#include "guest/instr_emul_wrapper.h"
109
#include "guest/instr_emul.h"
1110

1211
#define ACRN_DBG_EPT 6U

0 commit comments

Comments
 (0)