Re: [PATCH] cpuidle: governor: export cpuidle governor functions

[Maulik Shah <mkshah@codeaurora.org>]

Hi Daniel/Rafael,

 >> would it make sense to convert the governors into modules

i am not aware if this was not pursued further due to any issue.

Do you see any concerns to allow existing governors compiled as loadable 
module?
if not i can work on same and post. please do let me know your thoughts 
on this.

i have CCed Saravana and Todd for awareness.

Thanks,
Maulik

On 10/10/2020 1:56 AM, Daniel Lezcano wrote:
> Hi Rafael,
>
> On 22/09/2020 19:27, Rafael J. Wysocki wrote:
>> Hi Lina,
>>
>> On Tue, Sep 22, 2020 at 6:12 PM Lina Iyer <ilina@codeaurora.org> wrote:
>>> Hi Rafael,
>>>
>>> On Tue, Sep 22 2020 at 10:00 -0600, Rafael J. Wysocki wrote:
>>>> Sorry for the delay.
>>>>
>>>> On Wed, Sep 9, 2020 at 12:15 AM Lina Iyer <ilina@codeaurora.org> wrote:
>>>>> Commit 83788c0caed3 ("cpuidle: remove unused exports") removed
>>>>> capability of registering cpuidle governors, which was unused at that
>>>>> time. By exporting the symbol, let's allow platform specific modules to
>>>>> register cpuidle governors and use cpuidle_governor_latency_req() to get
>>>>> the QoS for the CPU.
>>>> Which platform-specific modules may want to do that and why?
>>>>
>>> We are planning a custom cpuidle governor for QCOM SoCs. With Android,
>>> the idea is to make them loadable modules so they can be in a separate
>>> partition.
>> Well, the $subject patch is not applicable without a mainline user
>> requiring this, so it needs to be posted along with that user.
> Putting apart the custom cpuidle governor mentioned above, would it make
> sense to convert the governors into modules ? It is pointless to have
> all of them compiled in, especially with distros doing make
> allmodconfig, no?
>
>
-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation