Re: [PATCH v5 04/15] x86/mtrr: support setting MTRR state for software defined MTRRs

From: Juergen Gross
Date: Tue Apr 11 2023 - 11:57:19 EST

On 11.04.23 16:26, Borislav Petkov wrote:
On Tue, Apr 11, 2023 at 04:04:00PM +0200, Juergen Gross wrote:
Yeah, the condition needs to be:

if (!(hv_is_isolation_supported() ||
cc_platform_has(CC_ATTR_GUEST_SEV_SNP)) &&
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Why is that needed at all?


Short answer: You asked me to add it in V2 of the series.

Longer answer: SEV_SNP guests and TDX guests would cause #VE when writing
MTRR MSRs.


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature