* [PATCH] cpuidle: governor: export cpuidle governor functions @ 2020-09-08 22:15 Lina Iyer 2020-09-22 16:00 ` Rafael J. Wysocki 0 siblings, 1 reply; 7+ messages in thread From: Lina Iyer @ 2020-09-08 22:15 UTC (permalink / raw) To: rjw, daniel.lezcano; +Cc: linux-pm, linux-arm-msm, Lina Iyer 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. Signed-off-by: Lina Iyer <ilina@codeaurora.org> --- drivers/cpuidle/governor.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/cpuidle/governor.c b/drivers/cpuidle/governor.c index 29acaf48e575..0e51ed25665e 100644 --- a/drivers/cpuidle/governor.c +++ b/drivers/cpuidle/governor.c @@ -102,6 +102,7 @@ int cpuidle_register_governor(struct cpuidle_governor *gov) return ret; } +EXPORT_SYMBOL_GPL(cpuidle_register_governor); /** * cpuidle_governor_latency_req - Compute a latency constraint for CPU @@ -118,3 +119,4 @@ s64 cpuidle_governor_latency_req(unsigned int cpu) return (s64)device_req * NSEC_PER_USEC; } +EXPORT_SYMBOL_GPL(cpuidle_governor_latency_req); -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] cpuidle: governor: export cpuidle governor functions 2020-09-08 22:15 [PATCH] cpuidle: governor: export cpuidle governor functions Lina Iyer @ 2020-09-22 16:00 ` Rafael J. Wysocki 2020-09-22 16:12 ` Lina Iyer 0 siblings, 1 reply; 7+ messages in thread From: Rafael J. Wysocki @ 2020-09-22 16:00 UTC (permalink / raw) To: Lina Iyer; +Cc: Rafael J. Wysocki, Daniel Lezcano, Linux PM, linux-arm-msm 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? > Signed-off-by: Lina Iyer <ilina@codeaurora.org> > --- > drivers/cpuidle/governor.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/cpuidle/governor.c b/drivers/cpuidle/governor.c > index 29acaf48e575..0e51ed25665e 100644 > --- a/drivers/cpuidle/governor.c > +++ b/drivers/cpuidle/governor.c > @@ -102,6 +102,7 @@ int cpuidle_register_governor(struct cpuidle_governor *gov) > > return ret; > } > +EXPORT_SYMBOL_GPL(cpuidle_register_governor); > > /** > * cpuidle_governor_latency_req - Compute a latency constraint for CPU > @@ -118,3 +119,4 @@ s64 cpuidle_governor_latency_req(unsigned int cpu) > > return (s64)device_req * NSEC_PER_USEC; > } > +EXPORT_SYMBOL_GPL(cpuidle_governor_latency_req); > -- > The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] cpuidle: governor: export cpuidle governor functions 2020-09-22 16:00 ` Rafael J. Wysocki @ 2020-09-22 16:12 ` Lina Iyer 2020-09-22 17:27 ` Rafael J. Wysocki 0 siblings, 1 reply; 7+ messages in thread From: Lina Iyer @ 2020-09-22 16:12 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Rafael J. Wysocki, Daniel Lezcano, Linux PM, linux-arm-msm 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. Thanks, Lina >> Signed-off-by: Lina Iyer <ilina@codeaurora.org> >> --- >> drivers/cpuidle/governor.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/cpuidle/governor.c b/drivers/cpuidle/governor.c >> index 29acaf48e575..0e51ed25665e 100644 >> --- a/drivers/cpuidle/governor.c >> +++ b/drivers/cpuidle/governor.c >> @@ -102,6 +102,7 @@ int cpuidle_register_governor(struct cpuidle_governor *gov) >> >> return ret; >> } >> +EXPORT_SYMBOL_GPL(cpuidle_register_governor); >> >> /** >> * cpuidle_governor_latency_req - Compute a latency constraint for CPU >> @@ -118,3 +119,4 @@ s64 cpuidle_governor_latency_req(unsigned int cpu) >> >> return (s64)device_req * NSEC_PER_USEC; >> } >> +EXPORT_SYMBOL_GPL(cpuidle_governor_latency_req); >> -- >> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, >> a Linux Foundation Collaborative Project >> ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] cpuidle: governor: export cpuidle governor functions 2020-09-22 16:12 ` Lina Iyer @ 2020-09-22 17:27 ` Rafael J. Wysocki 2020-10-09 20:26 ` Daniel Lezcano 0 siblings, 1 reply; 7+ messages in thread From: Rafael J. Wysocki @ 2020-09-22 17:27 UTC (permalink / raw) To: Lina Iyer Cc: Rafael J. Wysocki, Rafael J. Wysocki, Daniel Lezcano, Linux PM, linux-arm-msm 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. Cheers! ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] cpuidle: governor: export cpuidle governor functions 2020-09-22 17:27 ` Rafael J. Wysocki @ 2020-10-09 20:26 ` Daniel Lezcano 2021-08-17 9:31 ` Maulik Shah 0 siblings, 1 reply; 7+ messages in thread From: Daniel Lezcano @ 2020-10-09 20:26 UTC (permalink / raw) To: Rafael J. Wysocki, Lina Iyer; +Cc: Rafael J. Wysocki, Linux PM, linux-arm-msm 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? -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] cpuidle: governor: export cpuidle governor functions 2020-10-09 20:26 ` Daniel Lezcano @ 2021-08-17 9:31 ` Maulik Shah 2021-08-19 15:02 ` Ulf Hansson 0 siblings, 1 reply; 7+ messages in thread From: Maulik Shah @ 2021-08-17 9:31 UTC (permalink / raw) To: Daniel Lezcano, Rafael J. Wysocki Cc: Rafael J. Wysocki, Linux PM, linux-arm-msm, Saravana Kannan, Todd Kjos, Srinivas Rao L 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 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] cpuidle: governor: export cpuidle governor functions 2021-08-17 9:31 ` Maulik Shah @ 2021-08-19 15:02 ` Ulf Hansson 0 siblings, 0 replies; 7+ messages in thread From: Ulf Hansson @ 2021-08-19 15:02 UTC (permalink / raw) To: Maulik Shah Cc: Daniel Lezcano, Rafael J. Wysocki, Rafael J. Wysocki, Linux PM, linux-arm-msm, Saravana Kannan, Todd Kjos, Srinivas Rao L On Tue, 17 Aug 2021 at 11:32, Maulik Shah <mkshah@codeaurora.org> wrote: > > 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. I suggest you re-submit a new patch and put some arguments of why this is useful for us in the commit message. Then we can discuss that instead, rather than this old patch. Kind regards Uffe > > 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 > ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-08-19 15:03 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-09-08 22:15 [PATCH] cpuidle: governor: export cpuidle governor functions Lina Iyer 2020-09-22 16:00 ` Rafael J. Wysocki 2020-09-22 16:12 ` Lina Iyer 2020-09-22 17:27 ` Rafael J. Wysocki 2020-10-09 20:26 ` Daniel Lezcano 2021-08-17 9:31 ` Maulik Shah 2021-08-19 15:02 ` Ulf Hansson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).