On 4/5/23 03:18, Philippe Mathieu-Daudé wrote:
We want all accelerators to share the same opaque pointer in CPUState. Start with the HAX context, renaming its forward declarated structure 'hax_vcpu_state' as 'AccelvCPUState'. Signed-off-by: Philippe Mathieu-Daudé<philmd@xxxxxxxxxx> --- include/hw/core/cpu.h | 7 +++---- target/i386/hax/hax-i386.h | 3 ++- target/i386/nvmm/nvmm-all.c | 2 +- target/i386/whpx/whpx-all.c | 2 +- 4 files changed, 7 insertions(+), 7 deletions(-)
Can this be squashed with previous? It seems odd to change the name twice in a row. Is the "v" in AccelvCPUState helpful?
+ struct AccelvCPUState *accel; /* shared by kvm, hax and hvf */ bool vcpu_dirty;
Move below the comment? Or is that later? r~