[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/3] x86/smpboot: Re-position the call to tboot_wake_ap()


  • To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Tue, 19 Jan 2021 15:38:14 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=FG46/5jfbqtW7jdeIaBcf2b6zG+35QeJUJ8B67129j0=; b=gQbOzT+NnxDJ4NsJu2KmJ2jHS0ndQAjrDP0JWBAzZ3kEC5HzrfELfVMos3b/F77evIr38NKFV7x5arQclT3q26P5Z3lwghOXsdqbrNZyqP/ttzuG/xsWgp916rl51ufNovcMmvoNvuE+Mcmu1lz0e6FXTf9Y/W8hrqsgKK0Cu3Lk3Sj6442/302o82NsTE1/li/vbyMa9ndROa4iwiRUOjU6rH21HeqLcchw4/3xdVDlc7YB5ZKLSfxnEZQbf+F/ovE9byKZOev3NL6ZhfdcLLaqUcBk9KtzLmmALy/vrYaz53UwvXgpYVeYY5649I8YRVBYT2AFrlibAowhwOkXAA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=eSRB+QBup0iujWehQxAfhE8zO8/9AMd4aWwwLVDPaJ2kD533p0mcBjkh3LOJEBuV4wsDCTvzCGWTdCc7iIVm/tpHKveyvG4mntFZrEpDwR+d/E/gyrrWDNRIPYAw955wR4Oa0NaNlA3SGi9VxmF1EVJB6j77+yBy556ms5rFHPwMTiFWNjAYPjJRHvPMNUyNaWRAEuIZF1W1TDqwvHmJhq2X84D44WWXDWMUNnghmTxh8sAqz/LdBcme9gHaLwZY0yzpOyaHhNdSA78sIGV4cRow5EuK5A9PM8kHqBKuH/A6Y1Ok1CBhvPdYaMOTs9KM8QfmKIvYx/Eb3kM6mVhHGA==
  • Authentication-results: esa3.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>, Wei Liu <wl@xxxxxxx>, Marek Kasiewicz <marek.kasiewicz@xxxxxxxxx>, Norbert Kamiński <norbert.kaminski@xxxxxxxxx>, Michal Zygowski <michal.zygowski@xxxxxxxxx>, Piotr Krol <piotr.krol@xxxxxxxx>, Krystian Hebel <krystian.hebel@xxxxxxxxx>, "Daniel P . Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>, Rich Persaud <persaur@xxxxxxxxx>, Christopher Clark <christopher.w.clark@xxxxxxxxx>
  • Delivery-date: Tue, 19 Jan 2021 14:38:43 +0000
  • Ironport-sdr: NSQ9gwJn7Xz3scB1NEjCyYLhIriraIYxVg6SaV4x0FyW0FmEILJFuww3h9p6jIy3mcP8ewoL34 HTisXtCo3wsxC6Ics9+xLe/QHpQPdQUBQXpHBFeq4zOiGMZ2MZujgC1n2tl9p/xJbeyqxuiNTl bi3tVy0QEKllmw21gzAUNpoweI2pQoEzrSRg9B/tyGS5e/mgO22mg1cKYDZOJkWM78mqfAeeDj a7EBYn6rrXXH6YIMVfBP8hbhRqq28+IIxfpMKuHZtfHYqbVttNsn3+kAqzDdne//Nlxv6ouZgh vIs=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Fri, Jan 15, 2021 at 11:10:44PM +0000, Andrew Cooper wrote:
> So all the moving parts are in one function.
> 
> No functional change.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> ---
> CC: Jan Beulich <JBeulich@xxxxxxxx>
> CC: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> CC: Wei Liu <wl@xxxxxxx>
> CC: Marek Kasiewicz <marek.kasiewicz@xxxxxxxxx>
> CC: Norbert Kamiński <norbert.kaminski@xxxxxxxxx>
> CC: Michal Zygowski <michal.zygowski@xxxxxxxxx>
> CC: Piotr Krol <piotr.krol@xxxxxxxx>
> CC: Krystian Hebel <krystian.hebel@xxxxxxxxx>
> CC: Daniel P. Smith <dpsmith@xxxxxxxxxxxxxxxxxxxx>
> CC: Rich Persaud <persaur@xxxxxxxxx>
> CC: Christopher Clark <christopher.w.clark@xxxxxxxxx>
> ---
>  xen/arch/x86/smpboot.c | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/xen/arch/x86/smpboot.c b/xen/arch/x86/smpboot.c
> index 67e727cebd..9eca452ce1 100644
> --- a/xen/arch/x86/smpboot.c
> +++ b/xen/arch/x86/smpboot.c
> @@ -426,6 +426,13 @@ static int wakeup_secondary_cpu(int phys_apicid, 
> unsigned long start_eip)
>      int maxlvt, timeout, i;
>  
>      /*
> +     * Some versions of tboot might be able to handle the entire wake 
> sequence
> +     * on our behalf.
> +     */
> +    if ( tboot_in_measured_env() && tboot_wake_ap(phys_apicid, start_eip) )

I think you are missing a ! in front of tboot_wake_ap?

Thanks, Roger.



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.