Re: [PATCH v1 0/7] x86: Disentangle <asm/processor.h> dependency on CPUID APIs
"Ahmed S. Darwish" <[email protected]>
| Newsgroups | dev.linux.lists.x86-cpuid,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <aG6ri0-PlE-9lxyE@lx-t490> |
Hi!
On Wed, 09 Jul 2025, Sean Christopherson wrote:
>
> On Wed, Jul 09, 2025, Ahmed S. Darwish wrote:
> >
> > Beside the call sites converted at CPUID model v3 above, this series also
> > switches below files:
> >
> > arch/x86/kernel/cpu/microcode/core.c
> > arch/x86/kernel/cpu/microcode/intel.c
> > arch/x86/kernel/cpu/mshyperv.c
> > arch/x86/kvm/lapic.c
> > arch/x86/kvm/svm/sev.c
>
> I'm officially confused. Neither lapic.c nor svm/sev.c are modified in
> this series. Does "this series" actually mean something other than
> _this_ seris?
>
Oh, correct. That list should've only been:
arch/x86/kernel/cpu/microcode/core.c
arch/x86/kernel/cpu/microcode/intel.c
arch/x86/kernel/cpu/mshyperv.c
drivers/cpufreq/longrun.c
drivers/cpufreq/powernow-k7.c
drivers/cpufreq/powernow-k8.c
I had a script that scanned the CPUID call sites which included
<asm/cpuid/api.h> implicitly through <asm/processor.h>. I filtered the
false positives but forgot to remove them from the cover letter… :-(
For the arch/x86/kvm/ files, they got updated in one go by modifying
their internal "cpuid.h" header (arch/x86/kvm/cpuid.h) to explicitly
include the CPUID API.
Thanks,
--
Ahmed S. Darwish
Linutronix GmbH