On 19.01.2021 15:55, Andrew Cooper wrote: > On 19/01/2021 14:38, Roger Pau Monné wrote: >> 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 <[email protected]> >>> --- >>> CC: Jan Beulich <[email protected]> >>> CC: Roger Pau Monné <[email protected]> >>> CC: Wei Liu <[email protected]> >>> CC: Marek Kasiewicz <[email protected]> >>> CC: Norbert Kamiński <[email protected]> >>> CC: Michal Zygowski <[email protected]> >>> CC: Piotr Krol <[email protected]> >>> CC: Krystian Hebel <[email protected]> >>> CC: Daniel P. Smith <[email protected]> >>> CC: Rich Persaud <[email protected]> >>> CC: Christopher Clark <[email protected]> >>> --- >>> 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? > > Oh - so I am. That function is totally backwards. > > Fixed.
And then Acked-by: Jan Beulich <[email protected]> Jan
