lkml.org 
[lkml]   [2023]   [Feb]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v3 05/12] x86/xen: set MTRR state when running as Xen PV initial domain
From
On 27.02.23 14:52, Boris Ostrovsky wrote:
>
>
> On 2/27/23 2:12 AM, Juergen Gross wrote:
>> 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.
>
>
> TBH I am not sure what the right way is to handle errors here. What if the
> hypercall fails on second iteration?

The main reason would be that only one variable MTRR is available.

Its not as if there are very complicated scenarios leading to failures here.

Either the interface is usable and then it will work, or it isn't usable
and we can fall back to today's handling by ignoring MTRRs.


Juergen
[unhandled content-type:application/pgp-keys][unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2023-03-27 00:37    [W:0.042 / U:5.664 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site