On 10.12.2025 15:25, Andrew Cooper wrote: > On 08/12/2025 9:05 am, Jan Beulich wrote: >> On 28.11.2025 21:19, Andrew Cooper wrote: >>> The rest of Xen has no buisness knowing this structure, and it is currently >>> included via xen/sched.h into most code. Create a new private svm/vmcb.h. >>> >>> No functional change. >>> >>> Signed-off-by: Andrew Cooper <[email protected]> >>> --- >>> CC: Jan Beulich <[email protected]> >>> CC: Roger Pau Monné <[email protected]> >>> --- >>> xen/arch/x86/hvm/svm/asid.c | 1 + >>> xen/arch/x86/hvm/svm/emulate.c | 1 + >>> xen/arch/x86/hvm/svm/intr.c | 1 + >>> xen/arch/x86/hvm/svm/nestedsvm.c | 1 + >>> xen/arch/x86/hvm/svm/svm.c | 1 + >>> xen/arch/x86/hvm/svm/svmdebug.c | 2 + >>> xen/arch/x86/hvm/svm/vmcb.c | 2 + >>> xen/arch/x86/hvm/svm/vmcb.h | 617 ++++++++++++++++++++++++ >>> xen/arch/x86/include/asm/hvm/svm/vmcb.h | 606 ----------------------- >>> 9 files changed, 626 insertions(+), 606 deletions(-) >>> create mode 100644 xen/arch/x86/hvm/svm/vmcb.h >> I was actually hoping for the file to be moved. I notice that a few things >> are >> left in the original file, and I wonder if they couldn't be moved elsewhere >> up >> front. > > No, they can't be moved yet. There's other cleanup on the list, and > more header dis-entangling needed first. > > But I do agree the name is wrong and wants to go.
Well, then let's go this route: Acked-by: Jan Beulich <[email protected]> Jan
