Re: [Xen-devel] [PATCH v4 00/15] Add support for Hygon Dhyana Family 18h processor

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 03.04.19 at 17:21, <puwen@xxxxxxxx> wrote:
> On 2019/4/3 0:15, Jan Beulich wrote:
>> On 02.04.19 at 18:00, <puwen@xxxxxxxx> wrote:
>>> On 2019/4/2 23:14, Andrew Cooper wrote:
>>>> On 30/03/2019 10:40, Pu Wen wrote:
>>>>> This patch series have been applied and tested successfully on Hygon
>>>>> Dhyana processor, also been tested on AMD EPYC (family 17h) processor.
>>>>> It works fine and makes no harm to the existing code.
>>>> One thing I don't see in this series is anything about microcode
>>>> loading.  Presumably you'll follow the AMD patchloading mechanism, with
>>>> a blob you provide yourself?
>>> When the Hygon patchloading mechanism is practicable and it's necessary
>>> to load the Hygon microcode outside of BIOS, we'll send the patchloading
>>> patches to Xen and Linux mailing lists.
>> 
>> I suppose both Intel and AMD had this same intention of ucode loading
>> being a firmware job only, and we see where we are right now. As long
>> as updated firmware does not become available in a timely manner (or
>> perhaps not at all, because of a vendor considering a certain system
>> EOL), there's going to be a need to be able to load it from the OS.
> 
> It's reasonable. But I'll not put the microcode loading patch in this
> series, and will sent it individually later after sufficiently tested.

Fine with me, but Andrew will speak for himself.

> Or just add Hygon vendor checking to follow the AMD patchloading
> mechanism right now and do the adjustment for Hygon if needed in the
> future?

I'd prefer not to see this "blindly" enabled. As you say, it should
be tested. If no other changes are needed, the patch will be
easy to created and process.

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel