[PATCH v2 0/9] x86/hvm: {svm,vmx}.{c,h} cleanup

Xenia Ragiadakou posted 9 patches 1 year, 2 months ago
Test gitlab-ci failed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20230222120023.3004624-1-burzalodowa@gmail.com
xen/arch/x86/hvm/svm/nestedsvm.c       |   1 +
xen/arch/x86/hvm/svm/svm.c             |   2 +
xen/arch/x86/hvm/svm/svm.h             |  40 +++
xen/arch/x86/hvm/svm/vmcb.c            |   4 +-
xen/arch/x86/hvm/vmx/intr.c            |   2 +
xen/arch/x86/hvm/vmx/realmode.c        |   2 +
xen/arch/x86/hvm/vmx/vmcs.c            |  14 +
xen/arch/x86/hvm/vmx/vmx.c             |  73 ++--
xen/arch/x86/hvm/vmx/vmx.h             | 458 ++++++++++++++++++++++++
xen/arch/x86/hvm/vmx/vvmx.c            |   2 +
xen/arch/x86/include/asm/hvm/svm/svm.h |  33 --
xen/arch/x86/include/asm/hvm/vmx/vmx.h | 468 +------------------------
12 files changed, 564 insertions(+), 535 deletions(-)
create mode 100644 xen/arch/x86/hvm/svm/svm.h
create mode 100644 xen/arch/x86/hvm/vmx/vmx.h
[PATCH v2 0/9] x86/hvm: {svm,vmx}.{c,h} cleanup
Posted by Xenia Ragiadakou 1 year, 2 months ago
This patch series attempts a cleanup of files {svm,vmx}.{c,h} by removing
redundant headers and sorting the rest, reducing the scope of declarations
and definitions, moving functions used only by internal {svm,vmx} code to
private headers.

This series aims to address the comments made on the first version.

Main changes from the v1 series:
  - split the changes into smaller reviewable patches
  - drop the huge code rearrangment done in vmx.c
  - instead of spreading arround the declarations in c files to hide them,
    use private headers

There are more detailed per-patch changesets.

Xenia Ragiadakou (9):
  x86/svm: remove unused forward declaration of struct vcpu from svm.h
  x86/svm: opencode SVM_PAUSE{FILTER,THRESH}_INIT
  x86/svm: move declarations used only by svm code from svm.h to private
    header
  x86/vmx: remove unused included headers from vmx.h
  x86/vmx: reduce scope of GAS_VMX_OP definition
  x86/vmx: move declations used only by vmx code from vmx.h to private
    header
  x86/vmx: remove unused included headers from vmx.c
  x86/vmx: declare nvmx_enqueue_n2_exceptions() static
  x86/vmx: move vmx_update_debug_state() in vmcs.c and declare it static

 xen/arch/x86/hvm/svm/nestedsvm.c       |   1 +
 xen/arch/x86/hvm/svm/svm.c             |   2 +
 xen/arch/x86/hvm/svm/svm.h             |  40 +++
 xen/arch/x86/hvm/svm/vmcb.c            |   4 +-
 xen/arch/x86/hvm/vmx/intr.c            |   2 +
 xen/arch/x86/hvm/vmx/realmode.c        |   2 +
 xen/arch/x86/hvm/vmx/vmcs.c            |  14 +
 xen/arch/x86/hvm/vmx/vmx.c             |  73 ++--
 xen/arch/x86/hvm/vmx/vmx.h             | 458 ++++++++++++++++++++++++
 xen/arch/x86/hvm/vmx/vvmx.c            |   2 +
 xen/arch/x86/include/asm/hvm/svm/svm.h |  33 --
 xen/arch/x86/include/asm/hvm/vmx/vmx.h | 468 +------------------------
 12 files changed, 564 insertions(+), 535 deletions(-)
 create mode 100644 xen/arch/x86/hvm/svm/svm.h
 create mode 100644 xen/arch/x86/hvm/vmx/vmx.h

-- 
2.37.2