public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Sean Christopherson <seanjc@google.com>
To: Varad Gautam <varad.gautam@suse.com>
Cc: kvm@vger.kernel.org, pbonzini@redhat.com, drjones@redhat.com,
	marcorr@google.com, zxwang42@gmail.com, erdemaktas@google.com,
	rientjes@google.com, brijesh.singh@amd.com,
	Thomas.Lendacky@amd.com, jroedel@suse.de, bp@suse.de
Subject: Re: [kvm-unit-tests PATCH 0/9] SMP Support for x86 UEFI Tests
Date: Fri, 8 Apr 2022 14:35:04 +0000	[thread overview]
Message-ID: <YlBIGDreEtR9u+lh@google.com> (raw)
In-Reply-To: <20220408103127.19219-1-varad.gautam@suse.com>

On Fri, Apr 08, 2022, Varad Gautam wrote:
> This series brings multi-vcpu support to UEFI tests on x86.
> 
> Most of the necessary AP bringup code already exists within kvm-unit-tests'
> cstart64.S, and has now been either rewritten in C or moved to a common location
> to be shared between EFI and non-EFI test builds.
> 
> A call gate is used to transition from 16-bit to 32-bit mode, since EFI may
> not load the 32-bit entrypoint low enough to be reachable from the SIPI vector.
> 
> Git branch: https://github.com/varadgautam/kvm-unit-tests/commits/ap-boot-v1
> 
> Varad Gautam (9):
>   x86: Move ap_init() to smp.c
>   x86: Move load_idt() to desc.c
>   x86: desc: Split IDT entry setup into a generic helper
>   x86: efi, smp: Transition APs from 16-bit to 32-bit mode
>   x86: Move 32-bit bringup routines to start32.S
>   x86: efi, smp: Transition APs from 32-bit to 64-bit mode
>   x86: Move load_gdt_tss() to desc.c
>   x86: Provide a common 64-bit AP entrypoint for EFI and non-EFI
>   x86: setup: Serialize ap_start64 with a spinlock

This series doesn't apply cleanly on upstream master.  I feel bad for asking, but
in addition to rebasing to master, can you also rebase on top of my series[*] that
fixes SMP bugs that were introduced by the initial UEFI support?  I don't think
there will be semantic conflicts, but the whitespace cleanups (spaces => tabs) do
conflict, and I'd really like to start purging the spaces mess from KUT.

Paolo / Andrew, ping on my series, it still applies cleanly.

[*] https://lore.kernel.org/all/20220121231852.1439917-1-seanjc@google.com

  parent reply	other threads:[~2022-04-08 14:35 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-08 10:31 [kvm-unit-tests PATCH 0/9] SMP Support for x86 UEFI Tests Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 1/9] x86: Move ap_init() to smp.c Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 2/9] x86: Move load_idt() to desc.c Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 3/9] x86: desc: Split IDT entry setup into a generic helper Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 4/9] x86: efi, smp: Transition APs from 16-bit to 32-bit mode Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 5/9] x86: Move 32-bit bringup routines to start32.S Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 6/9] x86: efi, smp: Transition APs from 32-bit to 64-bit mode Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 7/9] x86: Move load_gdt_tss() to desc.c Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 8/9] x86: Provide a common 64-bit AP entrypoint for EFI and non-EFI Varad Gautam
2022-04-08 10:31 ` [kvm-unit-tests PATCH 9/9] x86: setup: Serialize ap_start64 with a spinlock Varad Gautam
2022-04-08 14:35 ` Sean Christopherson [this message]
2022-04-12 17:40   ` [kvm-unit-tests PATCH 0/9] SMP Support for x86 UEFI Tests Varad Gautam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YlBIGDreEtR9u+lh@google.com \
    --to=seanjc@google.com \
    --cc=Thomas.Lendacky@amd.com \
    --cc=bp@suse.de \
    --cc=brijesh.singh@amd.com \
    --cc=drjones@redhat.com \
    --cc=erdemaktas@google.com \
    --cc=jroedel@suse.de \
    --cc=kvm@vger.kernel.org \
    --cc=marcorr@google.com \
    --cc=pbonzini@redhat.com \
    --cc=rientjes@google.com \
    --cc=varad.gautam@suse.com \
    --cc=zxwang42@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox