Re: [PATCH 1/2][v2] cpuidle-haltpoll: Replace default_idle with arch_cpu_idle

From: Rafael J. Wysocki
Date: Thu Dec 08 2022 - 08:02:12 EST


On Thu, Dec 8, 2022 at 1:43 PM Li,Rongqing <lirongqing@xxxxxxxxx> wrote:
>
> > >
> > > When cpuidle-haltpoll is built as module,
> >
> > But it isn't now.
>
> Centos is compiling it as module, it will fail;
> Other user wants to compile it as module, they will fail,
> Syzbot random configuration building will fail
>
> Unless prohibit to build it as module as below:
>
> config HALTPOLL_CPUIDLE
> - tristate "Halt poll cpuidle driver"
> + bool "Halt poll cpuidle driver"
> depends on X86 && KVM_GUEST
> default y
> help

Ah, OK. Sorry about the confusion.

Yes, the driver (not the governor, though), can be modular, so yes you
need the export, but please change it to EXPORT_SYMBOL_GPL().