* Confused by function names cpuidle_install_idle_handler/cpuidle_uninstall_idle_handler
@ 2014-07-18 16:07 Mohammad Merajul Islam Molla
2014-07-22 10:54 ` Mulyadi Santosa
0 siblings, 1 reply; 6+ messages in thread
From: Mohammad Merajul Islam Molla @ 2014-07-18 16:07 UTC (permalink / raw)
To: kernelnewbies
Hello,
In drivers/cpuidle/cpuidle.c, there are two functions
cpuidle_install_idle_handler & cpuidle_uninstall_idle_handler. The
names seem confusing to me as they don't install any handler, rather
set 'initialized' variable to 1/0.
In v3.0 kernel, these functions used to look as below where they
installed and uninstalled some handler function -
void cpuidle_install_idle_handler(void)
123 {
124 if (enabled_devices && (pm_idle != cpuidle_idle_call)) {
125 /* Make sure all changes finished before we switch
to new idle */
126 smp_wmb();
127 pm_idle = cpuidle_idle_call;
128 }
129 }
void cpuidle_uninstall_idle_handler(void)
135 {
136 if (enabled_devices && pm_idle_old && (pm_idle != pm_idle_old)) {
137 pm_idle = pm_idle_old;
138 cpuidle_kick_cpus();
139 }
140 }
Do these names (and corresponding comments) still hold?
--
Thanks,
-Meraj
^ permalink raw reply [flat|nested] 6+ messages in thread
* Confused by function names cpuidle_install_idle_handler/cpuidle_uninstall_idle_handler
2014-07-18 16:07 Confused by function names cpuidle_install_idle_handler/cpuidle_uninstall_idle_handler Mohammad Merajul Islam Molla
@ 2014-07-22 10:54 ` Mulyadi Santosa
2014-07-22 13:56 ` Mohammad Merajul Islam Molla
0 siblings, 1 reply; 6+ messages in thread
From: Mulyadi Santosa @ 2014-07-22 10:54 UTC (permalink / raw)
To: kernelnewbies
On Fri, Jul 18, 2014 at 11:07 PM, Mohammad Merajul Islam Molla
<meraj.enigma@gmail.com> wrote:
> Hello,
>
> In drivers/cpuidle/cpuidle.c, there are two functions
> cpuidle_install_idle_handler & cpuidle_uninstall_idle_handler. The
> names seem confusing to me as they don't install any handler, rather
> set 'initialized' variable to 1/0.
>
> In v3.0 kernel, these functions used to look as below where they
> installed and uninstalled some handler function -
>
> void cpuidle_install_idle_handler(void)
> 123 {
> 124 if (enabled_devices && (pm_idle != cpuidle_idle_call)) {
> 125 /* Make sure all changes finished before we switch
> to new idle */
> 126 smp_wmb();
> 127 pm_idle = cpuidle_idle_call;
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
pm_idle is the callback handler, and it is assigned to
cpuidle_idle_call. I think this what it means by "install"
--
regards,
Mulyadi Santosa
Freelance Linux trainer and consultant
blog: the-hydra.blogspot.com
training: mulyaditraining.blogspot.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Confused by function names cpuidle_install_idle_handler/cpuidle_uninstall_idle_handler
2014-07-22 10:54 ` Mulyadi Santosa
@ 2014-07-22 13:56 ` Mohammad Merajul Islam Molla
2014-07-22 14:43 ` Anand Moon
0 siblings, 1 reply; 6+ messages in thread
From: Mohammad Merajul Islam Molla @ 2014-07-22 13:56 UTC (permalink / raw)
To: kernelnewbies
Hello,
As mentioned, the code excerpt was from kernel version 3.0.
In recent kernel (version 3.16.0-rc4), the code for the two mentioned
functions looks as below -
/**
* cpuidle_install_idle_handler - installs the cpuidle idle loop handler
*/
void cpuidle_install_idle_handler(void)
{
if (enabled_devices) {
/* Make sure all changes finished before we switch to
new idle */
smp_wmb();
initialized = 1;
}
}
/**
* cpuidle_uninstall_idle_handler - uninstalls the cpuidle idle loop
handler
*/
void cpuidle_uninstall_idle_handler(void)
{
if (enabled_devices) {
initialized = 0;
kick_all_cpus_sync();
}
}
That is the source of confusion. There is no pm_idle handler installed
(or uninstalled) anymore.
--
Thanks,
-Meraj
On Tue, Jul 22, 2014 at 4:54 PM, Mulyadi Santosa
<mulyadi.santosa@gmail.com> wrote:
> On Fri, Jul 18, 2014 at 11:07 PM, Mohammad Merajul Islam Molla
> <meraj.enigma@gmail.com> wrote:
>> Hello,
>>
>> In drivers/cpuidle/cpuidle.c, there are two functions
>> cpuidle_install_idle_handler & cpuidle_uninstall_idle_handler. The
>> names seem confusing to me as they don't install any handler, rather
>> set 'initialized' variable to 1/0.
>>
>> In v3.0 kernel, these functions used to look as below where they
>> installed and uninstalled some handler function -
>>
>> void cpuidle_install_idle_handler(void)
>> 123 {
>> 124 if (enabled_devices && (pm_idle != cpuidle_idle_call)) {
>> 125 /* Make sure all changes finished before we switch
>> to new idle */
>> 126 smp_wmb();
>> 127 pm_idle = cpuidle_idle_call;
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> pm_idle is the callback handler, and it is assigned to
> cpuidle_idle_call. I think this what it means by "install"
>
>
> --
> regards,
>
> Mulyadi Santosa
> Freelance Linux trainer and consultant
>
> blog: the-hydra.blogspot.com
> training: mulyaditraining.blogspot.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Confused by function names cpuidle_install_idle_handler/cpuidle_uninstall_idle_handler
2014-07-22 13:56 ` Mohammad Merajul Islam Molla
@ 2014-07-22 14:43 ` Anand Moon
2014-07-22 15:07 ` Mohammad Merajul Islam Molla
0 siblings, 1 reply; 6+ messages in thread
From: Anand Moon @ 2014-07-22 14:43 UTC (permalink / raw)
To: kernelnewbies
Hi Meraj,
Please look at the following commit this will help clear you confusion
git commit id : a0bfa1373859e9d11dc92561a8667588803e42d8
patch :0001-cpuidle-stop-depending-on-pm_idle.patch
To generate following patch:
#git format-patch -1 a0bfa1373859e9d11dc92561a8667588803e42d8
0001-cpuidle-stop-depending-on-pm_idle.patch
-Anand Moon
On Tuesday, July 22, 2014 7:28 PM, Mohammad Merajul Islam Molla <meraj.enigma@gmail.com> wrote:
Hello,
As mentioned, the code excerpt was from kernel version 3.0.
In recent kernel (version 3.16.0-rc4), the code for the two mentioned
functions looks as below -
/**
* cpuidle_install_idle_handler - installs the cpuidle idle loop handler
*/
void cpuidle_install_idle_handler(void)
{
? ? ? ? if (enabled_devices) {
? ? ? ? ? ? ? ? /* Make sure all changes finished before we switch to
new idle */
? ? ? ? ? ? ? ? smp_wmb();
? ? ? ? ? ? ? ? initialized = 1;
? ? ? ? }
}
/**
* cpuidle_uninstall_idle_handler - uninstalls the cpuidle idle loop
handler
*/
void cpuidle_uninstall_idle_handler(void)
{
? ? ? ? if (enabled_devices) {
? ? ? ? ? ? ? ? initialized = 0;
? ? ? ? ? ? ? ? kick_all_cpus_sync();
? ? ? ? }
}
That is the source of confusion. There is no pm_idle handler installed
(or uninstalled) anymore.
--
Thanks,
-Meraj
On Tue, Jul 22, 2014 at 4:54 PM, Mulyadi Santosa
<mulyadi.santosa@gmail.com> wrote:
> On Fri, Jul 18, 2014 at 11:07 PM, Mohammad Merajul Islam Molla
> <meraj.enigma@gmail.com> wrote:
>> Hello,
>>
>> In drivers/cpuidle/cpuidle.c, there are two functions
>> cpuidle_install_idle_handler & cpuidle_uninstall_idle_handler. The
>> names seem confusing to me as they don't install any handler, rather
>> set 'initialized'? variable to 1/0.
>>
>> In v3.0 kernel, these functions used to look as below where they
>> installed and uninstalled some handler function? -
>>
>> void cpuidle_install_idle_handler(void)
>> 123 {
>> 124? ? ? ? if (enabled_devices && (pm_idle != cpuidle_idle_call)) {
>> 125? ? ? ? ? ? ? ? /* Make sure all changes finished before we switch
>> to new idle */
>> 126? ? ? ? ? ? ? ? smp_wmb();
>> 127? ? ? ? ? ? ? ? pm_idle = cpuidle_idle_call;
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> pm_idle is the callback handler, and it is assigned to
> cpuidle_idle_call. I think this what it means by "install"
>
>
> --
> regards,
>
> Mulyadi Santosa
> Freelance Linux trainer and consultant
>
> blog: the-hydra.blogspot.com
> training: mulyaditraining.blogspot.com
_______________________________________________
Kernelnewbies mailing list
Kernelnewbies at kernelnewbies.org
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
^ permalink raw reply [flat|nested] 6+ messages in thread
* Confused by function names cpuidle_install_idle_handler/cpuidle_uninstall_idle_handler
2014-07-22 14:43 ` Anand Moon
@ 2014-07-22 15:07 ` Mohammad Merajul Islam Molla
2014-07-22 15:43 ` Anand Moon
0 siblings, 1 reply; 6+ messages in thread
From: Mohammad Merajul Islam Molla @ 2014-07-22 15:07 UTC (permalink / raw)
To: kernelnewbies
Hi Anand,
Thanks for your reply. I looked up the patch.
Question:
Should these functions be renamed now that they don't
do what they used to do earlier?
--
Thanks,
-Meraj
On Tue, Jul 22, 2014 at 8:43 PM, Anand Moon <moon.linux@yahoo.com> wrote:
> Hi Meraj,
>
> Please look at the following commit this will help clear you confusion
>
> git commit id : a0bfa1373859e9d11dc92561a8667588803e42d8
>
>
> patch :0001-cpuidle-stop-depending-on-pm_idle.patch
>
> To generate following patch:
>
> #git format-patch -1 a0bfa1373859e9d11dc92561a8667588803e42d8
>
> 0001-cpuidle-stop-depending-on-pm_idle.patch
>
>
> -Anand Moon
>
>
> On Tuesday, July 22, 2014 7:28 PM, Mohammad Merajul Islam Molla <meraj.enigma@gmail.com> wrote:
> Hello,
>
> As mentioned, the code excerpt was from kernel version 3.0.
>
> In recent kernel (version 3.16.0-rc4), the code for the two mentioned
> functions looks as below -
>
> /**
> * cpuidle_install_idle_handler - installs the cpuidle idle loop handler
> */
> void cpuidle_install_idle_handler(void)
> {
> if (enabled_devices) {
> /* Make sure all changes finished before we switch to
> new idle */
> smp_wmb();
> initialized = 1;
> }
> }
>
> /**
> * cpuidle_uninstall_idle_handler - uninstalls the cpuidle idle loop
> handler
> */
> void cpuidle_uninstall_idle_handler(void)
> {
> if (enabled_devices) {
> initialized = 0;
> kick_all_cpus_sync();
> }
> }
>
>
> That is the source of confusion. There is no pm_idle handler installed
> (or uninstalled) anymore.
>
> --
> Thanks,
> -Meraj
>
>
>
>
>
>
> On Tue, Jul 22, 2014 at 4:54 PM, Mulyadi Santosa
> <mulyadi.santosa@gmail.com> wrote:
>> On Fri, Jul 18, 2014 at 11:07 PM, Mohammad Merajul Islam Molla
>> <meraj.enigma@gmail.com> wrote:
>>> Hello,
>>>
>>> In drivers/cpuidle/cpuidle.c, there are two functions
>>> cpuidle_install_idle_handler & cpuidle_uninstall_idle_handler. The
>>> names seem confusing to me as they don't install any handler, rather
>>> set 'initialized' variable to 1/0.
>>>
>>> In v3.0 kernel, these functions used to look as below where they
>>> installed and uninstalled some handler function -
>>>
>>> void cpuidle_install_idle_handler(void)
>>> 123 {
>>> 124 if (enabled_devices && (pm_idle != cpuidle_idle_call)) {
>>> 125 /* Make sure all changes finished before we switch
>>> to new idle */
>>> 126 smp_wmb();
>>> 127 pm_idle = cpuidle_idle_call;
>> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>> pm_idle is the callback handler, and it is assigned to
>> cpuidle_idle_call. I think this what it means by "install"
>>
>>
>> --
>> regards,
>>
>> Mulyadi Santosa
>> Freelance Linux trainer and consultant
>>
>> blog: the-hydra.blogspot.com
>> training: mulyaditraining.blogspot.com
>
> _______________________________________________
> Kernelnewbies mailing list
> Kernelnewbies at kernelnewbies.org
> http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Confused by function names cpuidle_install_idle_handler/cpuidle_uninstall_idle_handler
2014-07-22 15:07 ` Mohammad Merajul Islam Molla
@ 2014-07-22 15:43 ` Anand Moon
0 siblings, 0 replies; 6+ messages in thread
From: Anand Moon @ 2014-07-22 15:43 UTC (permalink / raw)
To: kernelnewbies
Hi Meraj,
Look at the condition where they are called,
If they justify the meaning of the functions
it's ok to keep the name as it is.
Other wise if you have alternate name for these functions,
just send a patch to the maintainers of the code changes.
-Anand Moon
On Tuesday, July 22, 2014 8:38 PM, Mohammad Merajul Islam Molla <meraj.enigma@gmail.com> wrote:
Hi Anand,
Thanks for your reply. I looked up the patch.
Question:
? ? ? ? ? ? ? ?? Should these functions be renamed now that they don't
do what they used to do earlier?
--
Thanks,
-Meraj
On Tue, Jul 22, 2014 at 8:43 PM, Anand Moon <moon.linux@yahoo.com> wrote:
> Hi Meraj,
>
> Please look at the following commit this will help clear you confusion
>
> git commit id : a0bfa1373859e9d11dc92561a8667588803e42d8
>
>
> patch :0001-cpuidle-stop-depending-on-pm_idle.patch
>
> To generate following patch:
>
> #git format-patch -1 a0bfa1373859e9d11dc92561a8667588803e42d8
>
> 0001-cpuidle-stop-depending-on-pm_idle.patch
>
>
> -Anand Moon
>
>
> On Tuesday, July 22, 2014 7:28 PM, Mohammad Merajul Islam Molla <meraj.enigma@gmail.com> wrote:
> Hello,
>
> As mentioned, the code excerpt was from kernel version 3.0.
>
> In recent kernel (version 3.16.0-rc4), the code for the two mentioned
> functions looks as below -
>
> /**
> * cpuidle_install_idle_handler - installs the cpuidle idle loop handler
> */
> void cpuidle_install_idle_handler(void)
> {
>? ? ? ?? if (enabled_devices) {
>? ? ? ? ? ? ? ?? /* Make sure all changes finished before we switch to
> new idle */
>? ? ? ? ? ? ? ?? smp_wmb();
>? ? ? ? ? ? ? ?? initialized = 1;
>? ? ? ?? }
> }
>
> /**
> * cpuidle_uninstall_idle_handler - uninstalls the cpuidle idle loop
> handler
> */
> void cpuidle_uninstall_idle_handler(void)
> {
>? ? ? ?? if (enabled_devices) {
>? ? ? ? ? ? ? ?? initialized = 0;
>? ? ? ? ? ? ? ?? kick_all_cpus_sync();
>? ? ? ?? }
> }
>
>
> That is the source of confusion. There is no pm_idle handler installed
> (or uninstalled) anymore.
>
> --
> Thanks,
> -Meraj
>
>
>
>
>
>
> On Tue, Jul 22, 2014 at 4:54 PM, Mulyadi Santosa
> <mulyadi.santosa@gmail.com> wrote:
>> On Fri, Jul 18, 2014 at 11:07 PM, Mohammad Merajul Islam Molla
>> <meraj.enigma@gmail.com> wrote:
>>> Hello,
>>>
>>> In drivers/cpuidle/cpuidle.c, there are two functions
>>> cpuidle_install_idle_handler & cpuidle_uninstall_idle_handler. The
>>> names seem confusing to me as they don't install any handler, rather
>>> set 'initialized'? variable to 1/0.
>>>
>>> In v3.0 kernel, these functions used to look as below where they
>>> installed and uninstalled some handler function? -
>>>
>>> void cpuidle_install_idle_handler(void)
>>> 123 {
>>> 124? ? ? ?? if (enabled_devices && (pm_idle != cpuidle_idle_call)) {
>>> 125? ? ? ? ? ? ? ?? /* Make sure all changes finished before we switch
>>> to new idle */
>>> 126? ? ? ? ? ? ? ?? smp_wmb();
>>> 127? ? ? ? ? ? ? ?? pm_idle = cpuidle_idle_call;
>> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>> pm_idle is the callback handler, and it is assigned to
>> cpuidle_idle_call. I think this what it means by "install"
>>
>>
>> --
>> regards,
>>
>> Mulyadi Santosa
>> Freelance Linux trainer and consultant
>>
>> blog: the-hydra.blogspot.com
>> training: mulyaditraining.blogspot.com
>
> _______________________________________________
> Kernelnewbies mailing list
> Kernelnewbies at kernelnewbies.org
> http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-22 15:43 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-18 16:07 Confused by function names cpuidle_install_idle_handler/cpuidle_uninstall_idle_handler Mohammad Merajul Islam Molla
2014-07-22 10:54 ` Mulyadi Santosa
2014-07-22 13:56 ` Mohammad Merajul Islam Molla
2014-07-22 14:43 ` Anand Moon
2014-07-22 15:07 ` Mohammad Merajul Islam Molla
2014-07-22 15:43 ` Anand Moon
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.