On 01/09/2025 10:23 am, Jan Beulich wrote: > On 28.08.2025 17:03, Andrew Cooper wrote: >> FRED and IDT differ by a Supervisor Token on the base of the shstk. This >> means that the value they load into MSR_PL0_SSP differs by 8. >> >> s3_resume() in particular has logic which is otherwise invariant of FRED >> mode, >> and must not clobber a FRED MSR_PL0_SSP with an IDT one. >> >> This also simplifies the AP path too. Updating reinit_bsp_stack() is >> deferred >> until later. > This last sentence looks to be ... > >> No functional change. >> >> Signed-off-by: Andrew Cooper <[email protected]> >> --- >> CC: Jan Beulich <[email protected]> >> CC: Roger Pau Monné <[email protected]> >> >> v2: >> * Extend comment about clearing the busy bit. >> * Move reinit_bsp_stack() hunk into this patch. > ... stale, according to this. Other than that: > Reviewed-by: Jan Beulich <[email protected]>
Oops yes. I'll drop. ~Andrew
