Re: [GIT PULL] x86/cpu for v5.17
From: Linus Torvalds
Date: Mon Jan 10 2022 - 13:18:27 EST
On Mon, Jan 10, 2022 at 3:16 AM Borislav Petkov <bp@xxxxxxx> wrote:
>
> - Avoid writing MSR_CSTAR on Intel due to TDX guests raising a #VE trap
This is all fine, but my reaction to this is that I would have
expected it to be either a wrmsrl_safe(), or using an actual CPU
feature check.
Checking for a particular vendor seems a bit hacky. We generally try
to avoid things like that, don't we?
Not a big deal, I just thought I'd mention it since I reacted to it.
And we don't seem to have those vendor checks in any of the other code
that uses MSR_CSTAR (just grepping for that and seeing it mentioned in
kvm code etc)
Linus