Re: [patch V2 28/28] x86/perf/intel/rapl: Make it modular

From: Thomas Gleixner
Date: Mon Feb 22 2016 - 14:12:49 EST


Andi,

On Mon, 22 Feb 2016, Andi Kleen wrote:

> Thomas Gleixner <tglx@xxxxxxxxxxxxx> writes:
>
> > Add the necessary exit functions so it can be built as a module.
>
> If you make it a module you also need to add MODULE_DEVICE_TABLE
> for the PCI IDs and also add x86_cpu_id tables/annotations
> for the model numbers.
>
> Otherwise the module would always need to be loaded manually
> which would break existing setups.

I know. That's why I made the default Yes. We can drop that patch or add some
explanation to the help text that 'M' is for manual module loading until
someone adds the necessary magic for auto probing.

Thanks,

tglx