Re: [PATCH v3 05/12] x86/xen: set MTRR state when running as Xen PV initial domain

[Juergen Gross]

On 24.02.23 22:00, Boris Ostrovsky wrote:
> On 2/23/23 4:32 AM, Juergen Gross wrote:
> > +
> > +	for (reg = 0; reg < MTRR_MAX_VAR_RANGES; reg++) {
> > +		op.u.read_memtype.reg = reg;
> > +		if (HYPERVISOR_platform_op(&op))
> > +			break;
>
>
> If we fail on the first iteration, do we still want to mark MTRRs are 
> enabled/set in mtrr_overwrite_state()?

Hmm, good idea.

I think we should just drop the call of mtrr_overwrite_state() in this
case.


Juergen

Attachment: OpenPGP_0xB0DE9DD628BF132F.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature