Re: [PATCH v2 1/5] perf/x86/rapl: move RAPL support to common x86 code

[Stephane Eranian]

On Thu, Jun 4, 2020 at 6:11 AM Johannes Hirte
<johannes.hirte@xxxxxxxxxxxxx> wrote:
>
> On 2020 Jun 01, Stephane Eranian wrote:
> > On Mon, Jun 1, 2020 at 5:39 AM Johannes Hirte
> > <johannes.hirte@xxxxxxxxxxxxx> wrote:
> > >
> > > On 2020 Mai 27, Stephane Eranian wrote:
> > >
> > > ...
> > > > diff --git a/arch/x86/events/Makefile b/arch/x86/events/Makefile
> > > > index 6f1d1fde8b2de..12c42eba77ec3 100644
> > > > --- a/arch/x86/events/Makefile
> > > > +++ b/arch/x86/events/Makefile
> > > > @@ -1,5 +1,6 @@
> > > >  # SPDX-License-Identifier: GPL-2.0-only
> > > >  obj-y                                        += core.o probe.o
> > > > +obj-$(PERF_EVENTS_INTEL_RAPL)                += rapl.o
> > > >  obj-y                                        += amd/
> > > >  obj-$(CONFIG_X86_LOCAL_APIC)            += msr.o
> > > >  obj-$(CONFIG_CPU_SUP_INTEL)          += intel/
> > >
> > > With this change, rapl won't be build. Must be:
> > >
> > > obj-$(CONFIG_PERF_EVENTS_INTEL_RAPL)                += rapl.o
> > >
> > Correct. I posted a patch last week to fix that.
> > Thanks.
>
> Yes, it just wasn't in tip when I've tested. Sorry for the noise.
>
It is now. All is good.
Thanks.

>
> --
> Regards,
>   Johannes Hirte
>