Re: [Xen-devel] [PATCHv2 for-4.6] p2m/ept: Work around hardware errata setting A bit

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 28.09.15 at 16:48, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 28/09/15 15:03, Jan Beulich wrote:
>>>>> On 28.09.15 at 14:43, <tim@xxxxxxx> wrote:
>>> At 13:39 +0100 on 28 Sep (1443447574), Ross Lagerwall wrote:
>>>> @@ -1150,6 +1152,9 @@ int ept_p2m_init(struct p2m_domain *p2m)
>>>>      p2m->memory_type_changed = ept_memory_type_changed;
>>>>      p2m->audit_p2m = NULL;
>>>>  
>>>> +    /* Work around Errata AVR41 on Avaton processors. */
>>>> +    cpu_has_ept_ad = boot_cpu_data.x86_model != 0x4d;
>>>> +
>>> Shouldn't this check the family (a.k.a. boot_cpu_data.x86) too?
>> Definitely. Considering that the spec update says that the firmware
>> may contain a workaround (microcode update) perhaps even the
>> stepping and microcode level would need looking at.
> 
> There are no details available as to what firmware fix might be
> available, nor how to determine whether the issue is fixed.

Which is why we've been awaiting input from Intel forever since
the submission of the first version of the patch. But of course I
agree that we can always relax the check later on.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel