mbox series

[v1,0/5] x86/asm: cleanups after toolchain baseline upgrade

Message ID 20250401222105.79309-1-dmukhin@ford.com (mailing list archive)
Headers show
Series x86/asm: cleanups after toolchain baseline upgrade | expand

Message

Denis Mukhin April 1, 2025, 10:21 p.m. UTC 
Patches 1-4 remove various compiler workarounds in the hypervisor code
for Intel's VMX instructions.

Patch 5 removes compiler workaround for missing clac/stac instructions.

CI: https://gitlab.com/xen-project/people/dmukhin/xen/-/pipelines/1747394875

Denis Mukhin (5):
  x86/vmx: remove HAVE_AS_VMX
  x86/vmx: remove HAVE_AS_EPT
  x86/vmx: remove GAS_VMX_OP()
  x86/vmx: remove *_OPCODE
  x86/asm: remove HAVE_AS_CLAC_STAC

 xen/arch/x86/arch.mk                   |   5 +-
 xen/arch/x86/include/asm/asm-defns.h   |  10 ---
 xen/arch/x86/include/asm/hvm/vmx/vmx.h | 103 ++++---------------------
 3 files changed, 15 insertions(+), 103 deletions(-)

Comments

Andrew Cooper April 1, 2025, 10:48 p.m. UTC | #1 
On 01/04/2025 11:21 pm, dmkhn@proton.me wrote:
> Patches 1-4 remove various compiler workarounds in the hypervisor code
> for Intel's VMX instructions.
>
> Patch 5 removes compiler workaround for missing clac/stac instructions.
>
> CI: https://gitlab.com/xen-project/people/dmukhin/xen/-/pipelines/1747394875
>
> Denis Mukhin (5):
>   x86/vmx: remove HAVE_AS_VMX
>   x86/vmx: remove HAVE_AS_EPT
>   x86/vmx: remove GAS_VMX_OP()
>   x86/vmx: remove *_OPCODE

These are fine, but probably ought to be merged because they're all
really one thing.

When you've pulled the __vmx{on,off}() cleanup into a separate patch,
the remains of patch 4 becomes straight deletion, which also means that
Resolves: https://gitlab.com/xen-project/xen/-/work_items/202 is the
commit with almost all of the work in it.

~Andrew