Re: [PATCH hyperv-next 0/2] arch/x86, x86/hyperv: Few fixes for the AP startup

On 5/8/2025 11:39 AM, Wei Liu wrote:

On Wed, May 07, 2025 at 11:22:24AM -0700, Roman Kisel wrote:

quoted

This patchset combines two patches that depend on each other and were not applying
cleanly:
   1. Fix APIC ID and VP index confusion in hv_snp_boot_ap():
     https://lore.kernel.org/linux-hyperv/20250430204720.108962-1-romank@linux.microsoft.com/ (local)
   2. Provide the CPU number in the wakeup AP callback:
     https://lore.kernel.org/linux-hyperv/20250430204720.108962-1-romank@linux.microsoft.com/ (local)

I rebased the patches on top of the latest hyperv-next tree and updated the second patch
that broke the linux-next build. That fix that, I made one non-functional change:
updated the signature of numachip_wakeup_secondary() to match the parameter list of
wakeup_secondary_cpu().

Roman Kisel (2):
   x86/hyperv: Fix APIC ID and VP index confusion in hv_snp_boot_ap()
   arch/x86: Provide the CPU number in the wakeup AP callback

I queue these up.

Just so you know I'm experimenting a new setup. These have been applied
to hyperv-next-staging. It will take some time for them to propagate to
hyperv-next.

Thank you very much! That's exciting news about the new staging setup!!

Thanks,
Wei.

quoted

  arch/x86/coco/sev/core.c             | 13 ++-----
  arch/x86/hyperv/hv_init.c            | 33 +++++++++++++++++
  arch/x86/hyperv/hv_vtl.c             | 54 ++++------------------------
  arch/x86/hyperv/ivm.c                | 11 ++++--
  arch/x86/include/asm/apic.h          |  8 ++---
  arch/x86/include/asm/mshyperv.h      |  7 ++--
  arch/x86/kernel/acpi/madt_wakeup.c   |  2 +-
  arch/x86/kernel/apic/apic_noop.c     |  8 ++++-
  arch/x86/kernel/apic/apic_numachip.c |  2 +-
  arch/x86/kernel/apic/x2apic_uv_x.c   |  2 +-
  arch/x86/kernel/smpboot.c            | 10 +++---
  include/hyperv/hvgdk_mini.h          |  2 +-
  12 files changed, 76 insertions(+), 76 deletions(-)


base-commit: 9b0844d87b1407681b78130429f798beb366f43f
-- 
2.43.0

-- 
Thank you,
Roman