Messages in this thread | | | Subject | Re: [PATCH v4 15/16] driver/edac: enable Hygon support to AMD64 EDAC driver | From | Pu Wen <> | Date | Wed, 22 Aug 2018 21:18:40 +0800 |
| |
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
| |