[PATCH 0/2] Remove unnecessary asm/hvm/support.h includes

Xenia Ragiadakou posted 2 patches 1 year, 2 months ago
Test gitlab-ci passed
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20230206124251.911744-1-burzalodowa@gmail.com
xen/arch/x86/cpu/amd.c                  | 2 +-
xen/arch/x86/cpu/hygon.c                | 1 -
xen/arch/x86/cpu/intel.c                | 2 +-
xen/arch/x86/cpu/vpmu.c                 | 1 -
xen/arch/x86/cpu/vpmu_intel.c           | 1 -
xen/arch/x86/crash.c                    | 1 -
xen/arch/x86/domain.c                   | 1 -
xen/arch/x86/domctl.c                   | 1 -
xen/arch/x86/emul-i8254.c               | 2 +-
xen/arch/x86/hvm/hpet.c                 | 2 +-
xen/arch/x86/hvm/io.c                   | 1 -
xen/arch/x86/hvm/irq.c                  | 2 +-
xen/arch/x86/hvm/pmtimer.c              | 3 ++-
xen/arch/x86/hvm/rtc.c                  | 3 ++-
xen/arch/x86/hvm/save.c                 | 2 --
xen/arch/x86/hvm/stdvga.c               | 1 -
xen/arch/x86/hvm/svm/emulate.c          | 1 -
xen/arch/x86/hvm/svm/intr.c             | 1 -
xen/arch/x86/hvm/svm/vmcb.c             | 1 -
xen/arch/x86/hvm/viridian/synic.c       | 1 -
xen/arch/x86/hvm/viridian/time.c        | 1 -
xen/arch/x86/hvm/viridian/viridian.c    | 1 -
xen/arch/x86/hvm/vmx/intr.c             | 1 -
xen/arch/x86/hvm/vmx/vmcs.c             | 1 -
xen/arch/x86/hvm/vmx/vvmx.c             | 1 +
xen/arch/x86/hvm/vpic.c                 | 2 +-
xen/arch/x86/hvm/vpt.c                  | 2 +-
xen/arch/x86/i387.c                     | 1 -
xen/arch/x86/include/asm/hvm/vmx/vmx.h  | 2 +-
xen/arch/x86/include/asm/spec_ctrl.h    | 2 +-
xen/arch/x86/mm/altp2m.c                | 1 -
xen/arch/x86/mm/hap/nested_ept.c        | 1 -
xen/arch/x86/mm/hap/nested_hap.c        | 1 -
xen/arch/x86/oprofile/op_model_athlon.c | 2 +-
xen/arch/x86/oprofile/xenoprof.c        | 1 -
xen/arch/x86/smp.c                      | 1 -
xen/arch/x86/sysctl.c                   | 1 -
xen/arch/x86/x86_64/traps.c             | 1 -
xen/arch/x86/xstate.c                   | 1 -
xen/drivers/passthrough/x86/hvm.c       | 1 -
40 files changed, 15 insertions(+), 40 deletions(-)
[PATCH 0/2] Remove unnecessary asm/hvm/support.h includes
Posted by Xenia Ragiadakou 1 year, 2 months ago
While working on the separation between AMD and Intel code, I noticed that
the header asm/hvm/support.h is included in many files even if none of its
declarations and macro definitions is referred in those files.

This patch series aims to remove any redundant includes of asm/hvm/support.h.
The changes are sent as 2 patches to aid review (especially the change in
asm/spec_ctrl.h).

Xenia Ragiadakou (2):
  x86: do not include asm/hvm/support.h when not used
  x86/hygon: do not include asm/hvm/support.h when not used

 xen/arch/x86/cpu/amd.c                  | 2 +-
 xen/arch/x86/cpu/hygon.c                | 1 -
 xen/arch/x86/cpu/intel.c                | 2 +-
 xen/arch/x86/cpu/vpmu.c                 | 1 -
 xen/arch/x86/cpu/vpmu_intel.c           | 1 -
 xen/arch/x86/crash.c                    | 1 -
 xen/arch/x86/domain.c                   | 1 -
 xen/arch/x86/domctl.c                   | 1 -
 xen/arch/x86/emul-i8254.c               | 2 +-
 xen/arch/x86/hvm/hpet.c                 | 2 +-
 xen/arch/x86/hvm/io.c                   | 1 -
 xen/arch/x86/hvm/irq.c                  | 2 +-
 xen/arch/x86/hvm/pmtimer.c              | 3 ++-
 xen/arch/x86/hvm/rtc.c                  | 3 ++-
 xen/arch/x86/hvm/save.c                 | 2 --
 xen/arch/x86/hvm/stdvga.c               | 1 -
 xen/arch/x86/hvm/svm/emulate.c          | 1 -
 xen/arch/x86/hvm/svm/intr.c             | 1 -
 xen/arch/x86/hvm/svm/vmcb.c             | 1 -
 xen/arch/x86/hvm/viridian/synic.c       | 1 -
 xen/arch/x86/hvm/viridian/time.c        | 1 -
 xen/arch/x86/hvm/viridian/viridian.c    | 1 -
 xen/arch/x86/hvm/vmx/intr.c             | 1 -
 xen/arch/x86/hvm/vmx/vmcs.c             | 1 -
 xen/arch/x86/hvm/vmx/vvmx.c             | 1 +
 xen/arch/x86/hvm/vpic.c                 | 2 +-
 xen/arch/x86/hvm/vpt.c                  | 2 +-
 xen/arch/x86/i387.c                     | 1 -
 xen/arch/x86/include/asm/hvm/vmx/vmx.h  | 2 +-
 xen/arch/x86/include/asm/spec_ctrl.h    | 2 +-
 xen/arch/x86/mm/altp2m.c                | 1 -
 xen/arch/x86/mm/hap/nested_ept.c        | 1 -
 xen/arch/x86/mm/hap/nested_hap.c        | 1 -
 xen/arch/x86/oprofile/op_model_athlon.c | 2 +-
 xen/arch/x86/oprofile/xenoprof.c        | 1 -
 xen/arch/x86/smp.c                      | 1 -
 xen/arch/x86/sysctl.c                   | 1 -
 xen/arch/x86/x86_64/traps.c             | 1 -
 xen/arch/x86/xstate.c                   | 1 -
 xen/drivers/passthrough/x86/hvm.c       | 1 -
 40 files changed, 15 insertions(+), 40 deletions(-)

-- 
2.37.2