On 01/03/2022 11:06, Jan Beulich wrote:
> With bed9ae54df44 ("x86/time: switch platform timer hooks to altcall")
> in place we can further arrange for ENDBR removal from the functions no
> longer subject to indirect calls. Note that plt_tsc is left untouched,
> for not holding any pointer eligible for ENDBR removal.I'd be tempted to include it, for consistency sake alone. It is less likely to go wrong in the future if another hook is introduced. > Signed-off-by: Jan Beulich <[email protected]> With the commit message, I'm not not certain if this is linked to the previous patch. Overall it looks fine, but I'd like to get clarity on this point. > --- > I did consider converting most of the plt_* to const (plt_hpet and > plt_pmtimer cannot be converted), but this would entail quite a few > further changes. It's all initdata. const is not terribly interesting, especially if it is invasive and incomplete to do. ~Andrew
