From: "Srivatsa S. Bhat" <srivatsa.bhat@linux.vnet.ibm.com>
To: Viresh Kumar <viresh.kumar@linaro.org>
Cc: "Jarzmik, Robert" <robert.jarzmik@intel.com>,
"Rafael J. Wysocki" <rjw@sisk.pl>,
"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>
Subject: Re: S3, SMP non boot cpus and /sys/devices/system/cpu[1-9]/cpufreq/scaling_max_freq
Date: Tue, 14 May 2013 15:57:39 +0530 [thread overview]
Message-ID: <5192119B.4020009@linux.vnet.ibm.com> (raw)
In-Reply-To: <CAKohponfQ11Ak9Q-HmSK83g1XB5_KF0UDHaSQh_jiEZBNtSUMw@mail.gmail.com>
On 05/14/2013 03:52 PM, Viresh Kumar wrote:
> On 14 May 2013 15:39, Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com> wrote:
>> And losing permissions isn't specific to suspend-to-ram right?
>> You said that yourself, that on regular cpu online/offline you lose
>> the changed permissions. So just write a resume hook to restore whatever
>> permissions you want after suspend-resume. See /usr/lib64/pm-utils/sleep.d
>> for some sample resume scripts.
>
> This is what i feel about losing chages on cpu hotplug/unplug:
> - On normal online/offline, we aren't required to preserve earlier settings.
>From Robert's descriptions, we _are_ required to preserve the settings. But
this is somewhat acceptable because the user _knows_ he is doing hotplug.
> - But on suspend/resume, user shouldn't know that cpus have been
> removed/added and so permissions should stay as is.
>
Hmm, you are right. It is wrong for us to expect the user to use custom scripts
to preserve permissions across S3, since he is supposed to remain ignorant of
CPU hotplug in this path.
> What do you say?
>
I agree, we should try to fix it for the S3/S4 case. Robert, I'm sorry, you were
right - if S3 is supposed to hide CPU hotplug, it should hide it completely or
not at all. And since we go with the former option, a fix for the permissions
is in order, IMHO.
Regards,
Srivatsa S. Bhat
next prev parent reply other threads:[~2013-05-14 10:30 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-13 13:01 S3, SMP non boot cpus and /sys/devices/system/cpu[1-9]/cpufreq/scaling_max_freq Jarzmik, Robert
2013-05-13 20:40 ` Srivatsa S. Bhat
2013-05-13 23:32 ` Rafael J. Wysocki
2013-05-14 9:06 ` Jarzmik, Robert
2013-05-14 10:09 ` Srivatsa S. Bhat
2013-05-14 10:22 ` Viresh Kumar
2013-05-14 10:27 ` Srivatsa S. Bhat [this message]
2013-05-14 11:22 ` Rafael J. Wysocki
2013-05-14 11:20 ` R, Durgadoss
2013-05-14 11:33 ` Rafael J. Wysocki
2013-05-14 11:36 ` R, Durgadoss
2013-05-14 11:54 ` Jarzmik, Robert
2013-05-14 12:34 ` Srivatsa S. Bhat
2013-05-14 13:00 ` Rafael J. Wysocki
2013-05-14 13:54 ` Srivatsa S. Bhat
2013-05-14 20:22 ` Rafael J. Wysocki
2013-05-15 8:24 ` Jarzmik, Robert
2013-05-15 8:37 ` R, Durgadoss
2013-05-15 6:16 ` Viresh Kumar
2013-05-15 6:30 ` Srivatsa S. Bhat
2013-05-15 6:45 ` Viresh Kumar
2013-05-15 7:33 ` Srivatsa S. Bhat
2013-05-15 7:44 ` Viresh Kumar
2013-05-15 8:18 ` Srivatsa S. Bhat
2013-05-15 20:50 ` Rafael J. Wysocki
2013-05-14 12:58 ` Rafael J. Wysocki
2013-05-14 14:01 ` Jarzmik, Robert
2013-05-14 14:16 ` Srivatsa S. Bhat
2013-05-14 14:05 ` Alan Stern
2013-05-15 9:20 ` Jarzmik, Robert
2013-05-15 14:15 ` Alan Stern
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5192119B.4020009@linux.vnet.ibm.com \
--to=srivatsa.bhat@linux.vnet.ibm.com \
--cc=linux-pm@vger.kernel.org \
--cc=rjw@sisk.pl \
--cc=robert.jarzmik@intel.com \
--cc=viresh.kumar@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox