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

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

dmkhn@proton.me posted 5 patches 7 months ago

Diff against v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/20250401222105.79309-1-dmukhin@ford.com

There is a newer version of this series

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(-)

Expand all Fold all

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

Posted by dmkhn@proton.me 7 months ago

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(-)

-- 
2.34.1

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

Posted by Andrew Cooper 7 months ago

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