Re: [PATCH v4 15/16] driver/edac: enable Hygon support to AMD64 EDAC driver

[Pu Wen]

On 2018/8/22 2:07, Pavel Machek wrote>> But for right now I think we 
should strive to keep the changes as small
> > as possible and only do real splitting when they start adding new
> > functionality. Which would mean having a hygon_edac.c too, for example.
> >
> > All, IMHO, of course. Sharing code between vendors is always yucky.
>
> Dunno, people do not know that 0x18 is reserved on AMD, so resulting
> code is quite confusing.
>
> Explicit vendor check is way to go, long term.
> 									Pavel

Hi Pavel,

For amd64_edac_init() entry, there will be a x86_match_cpu(amd64_cpuids)
which checking whether amd64_edac module fit current CPU. Only Hygon has
registered X86_VENDOR_HYGON with family 0x18h in amd64_cpuids[] to use
AMD edac codes.

Also with the assumption that Hygon will negotiate with AMD that only
Hygon will use Family 18h. So we assume in later codes vendor checking
can be omitted.

You are right, for short term, we are trying to keep the tight change and
enable Dhyana with minimal effort. For long term, agreed with that.

So based on the assumption that only Hygon will use family 0x18, we will
rework the patch, make the assumption clear and try to keep it small.
If that's ok, we will rework this patch.

Thanks for all the suggestions.

Regards,
Pu Wen