From: Mason <slash.tmp@free.fr> To: Pavel Machek <pavel@ucw.cz> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>, Kevin Hilman <khilman@kernel.org>, Ulf Hansson <ulf.hansson@linaro.org>, Daniel Lezcano <daniel.lezcano@linaro.org>, linux-pm <linux-pm@vger.kernel.org>, Linux ARM <linux-arm-kernel@lists.infradead.org>, Thibaud Cornic <thibaud_cornic@sigmadesigns.com>, JB <jb_lescher@sigmadesigns.com> Subject: Re: Drivers taking different actions depending on sleep state Date: Sat, 10 Jun 2017 11:16:48 +0200 [thread overview] Message-ID: <7b4435e5-5903-2678-fddb-34e884f5d53f@free.fr> (raw) In-Reply-To: <20170609213049.GA28596@amd> On 09/06/2017 23:30, Pavel Machek wrote: > On Fri 2017-06-09 18:27:45, Mason wrote: >> On 09/06/2017 17:20, Mason wrote: >> >>> Currently my platform's "mem" is a true suspend-to-RAM trigger, >>> where drivers are supposed to save their state (register values >>> will be lost), then Linux hands control over to firmware which >>> enables RAM self-refresh and powers the chip down. When the system >>> resumes, drivers restore their state from their copy in memory. >>> >>> One driver is responsible for loading/unloading microcode running >>> on the DSPs. This operation is required only when powering down >>> the chip, but it should be avoided for "low-latency" sleeps. >>> >>> The problem is that, if I understand correctly, drivers have no way >>> of knowing which sleep state is being entered/exited? >>> >>> How can I have the microcode driver take different decisions >>> based on the sleep state? > > Well... question "does my chip lose state during standby/mem on _this_ > machine" is more complex then "is it standby or mem", right? I think it's binary... If power to the DSPs is cut, then they lose state. If the DSPs remain powered, then they maintain state. "mem" powers the entire chip down, including the DSPs (by implementation's choice) but we are investigating a lower-latency sleep state that wouldn't cut power. > You should really ask the regulator framework, not core code. The issue is that power cutting is not handled in Linux, it is done by firmware. So I'm not sure what there is to ask to the regulator framework? >> Mason385 javier__: there's some authentication required when S2R is involved (from the firmware) >> javier__ Mason385: ah, Ok. I just asked because if it was the latter, the regulator subsystem has infrastructure to keep the regulators on during S2R >> Mason385 javier__: OK so there's two issues. We are required to >> re-authenticate microcode when resuming from S2R (because someone >> "may" have tampered with the contents) and on suspend, power is cut >> to the DSPs and they lose context > > I'm not sure what you are developing. Someone also "may" have modified > the microcode while you were running. Someone also "may" have modified > the kernel in RAM. Not sure what you are developing, but protecting > against attacker with direct hardware access is impossible and not > welcome. There is no point in discussing the technical relevance of these requirements, because they are *mandatory* for certification. No certification, no customer. So the feature must be implemented, whether it increases "security" or not. FTR, what is being bitterly defended is Hollywood's pixels. Regards.
WARNING: multiple messages have this Message-ID (diff)
From: slash.tmp@free.fr (Mason) To: linux-arm-kernel@lists.infradead.org Subject: Drivers taking different actions depending on sleep state Date: Sat, 10 Jun 2017 11:16:48 +0200 [thread overview] Message-ID: <7b4435e5-5903-2678-fddb-34e884f5d53f@free.fr> (raw) In-Reply-To: <20170609213049.GA28596@amd> On 09/06/2017 23:30, Pavel Machek wrote: > On Fri 2017-06-09 18:27:45, Mason wrote: >> On 09/06/2017 17:20, Mason wrote: >> >>> Currently my platform's "mem" is a true suspend-to-RAM trigger, >>> where drivers are supposed to save their state (register values >>> will be lost), then Linux hands control over to firmware which >>> enables RAM self-refresh and powers the chip down. When the system >>> resumes, drivers restore their state from their copy in memory. >>> >>> One driver is responsible for loading/unloading microcode running >>> on the DSPs. This operation is required only when powering down >>> the chip, but it should be avoided for "low-latency" sleeps. >>> >>> The problem is that, if I understand correctly, drivers have no way >>> of knowing which sleep state is being entered/exited? >>> >>> How can I have the microcode driver take different decisions >>> based on the sleep state? > > Well... question "does my chip lose state during standby/mem on _this_ > machine" is more complex then "is it standby or mem", right? I think it's binary... If power to the DSPs is cut, then they lose state. If the DSPs remain powered, then they maintain state. "mem" powers the entire chip down, including the DSPs (by implementation's choice) but we are investigating a lower-latency sleep state that wouldn't cut power. > You should really ask the regulator framework, not core code. The issue is that power cutting is not handled in Linux, it is done by firmware. So I'm not sure what there is to ask to the regulator framework? >> Mason385 javier__: there's some authentication required when S2R is involved (from the firmware) >> javier__ Mason385: ah, Ok. I just asked because if it was the latter, the regulator subsystem has infrastructure to keep the regulators on during S2R >> Mason385 javier__: OK so there's two issues. We are required to >> re-authenticate microcode when resuming from S2R (because someone >> "may" have tampered with the contents) and on suspend, power is cut >> to the DSPs and they lose context > > I'm not sure what you are developing. Someone also "may" have modified > the microcode while you were running. Someone also "may" have modified > the kernel in RAM. Not sure what you are developing, but protecting > against attacker with direct hardware access is impossible and not > welcome. There is no point in discussing the technical relevance of these requirements, because they are *mandatory* for certification. No certification, no customer. So the feature must be implemented, whether it increases "security" or not. FTR, what is being bitterly defended is Hollywood's pixels. Regards.
next prev parent reply other threads:[~2017-06-10 9:17 UTC|newest] Thread overview: 118+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-06-09 15:20 Drivers taking different actions depending on sleep state Mason 2017-06-09 15:20 ` Mason 2017-06-09 16:27 ` Mason 2017-06-09 16:27 ` Mason 2017-06-09 21:30 ` Pavel Machek 2017-06-09 21:30 ` Pavel Machek 2017-06-10 9:16 ` Mason [this message] 2017-06-10 9:16 ` Mason 2017-06-09 22:05 ` Florian Fainelli 2017-06-09 22:05 ` Florian Fainelli 2017-06-09 22:53 ` Rafael J. Wysocki 2017-06-09 22:53 ` Rafael J. Wysocki 2017-06-21 21:16 ` Florian Fainelli 2017-06-21 21:16 ` Florian Fainelli 2017-06-21 21:59 ` Rafael J. Wysocki 2017-06-21 21:59 ` Rafael J. Wysocki 2017-06-21 22:48 ` Florian Fainelli 2017-06-21 22:48 ` Florian Fainelli 2017-06-21 22:57 ` Rafael J. Wysocki 2017-06-21 22:57 ` Rafael J. Wysocki 2017-06-21 23:55 ` Florian Fainelli 2017-06-21 23:55 ` Florian Fainelli 2017-06-22 0:03 ` Rafael J. Wysocki 2017-06-22 0:03 ` Rafael J. Wysocki 2017-06-22 15:18 ` Florian Fainelli 2017-06-22 15:18 ` Florian Fainelli 2017-06-22 16:09 ` Rafael J. Wysocki 2017-06-22 16:09 ` Rafael J. Wysocki 2017-06-22 8:51 ` Alexandre Belloni 2017-06-22 8:51 ` Alexandre Belloni 2017-06-22 16:00 ` Rafael J. Wysocki 2017-06-22 16:00 ` Rafael J. Wysocki 2017-06-23 1:08 ` [RFC 0/2] PM / suspend: Add platform_suspend_target_state() Florian Fainelli 2017-06-23 1:08 ` Florian Fainelli 2017-06-23 1:08 ` [RFC 1/2] " Florian Fainelli 2017-06-23 1:08 ` Florian Fainelli 2017-06-29 23:00 ` Rafael J. Wysocki 2017-06-29 23:00 ` Rafael J. Wysocki 2017-07-12 18:08 ` Florian Fainelli 2017-07-12 18:08 ` Florian Fainelli 2017-07-14 22:16 ` Rafael J. Wysocki 2017-07-14 22:16 ` Rafael J. Wysocki 2017-07-15 6:28 ` Pavel Machek 2017-07-15 6:28 ` Pavel Machek 2017-07-15 12:17 ` Rafael J. Wysocki 2017-07-15 12:17 ` Rafael J. Wysocki 2017-07-15 16:46 ` Pavel Machek 2017-07-15 16:46 ` Pavel Machek 2017-07-15 17:20 ` Florian Fainelli 2017-07-15 17:20 ` Florian Fainelli 2017-07-15 18:33 ` Alexandre Belloni 2017-07-15 18:33 ` Alexandre Belloni 2017-07-06 3:18 ` Pavel Machek 2017-07-06 3:18 ` Pavel Machek 2017-07-16 13:41 ` Alexandre Belloni 2017-07-16 13:41 ` Alexandre Belloni 2017-07-16 15:35 ` Florian Fainelli 2017-07-16 15:35 ` Florian Fainelli 2017-07-15 23:24 ` Rafael J. Wysocki 2017-07-15 23:24 ` Rafael J. Wysocki 2017-07-15 23:34 ` Mason 2017-07-15 23:34 ` Mason 2017-07-15 23:38 ` Rafael J. Wysocki 2017-07-15 23:38 ` Rafael J. Wysocki 2017-07-16 2:36 ` Florian Fainelli 2017-07-16 2:36 ` Florian Fainelli 2017-07-16 10:22 ` Rafael J. Wysocki 2017-07-16 10:22 ` Rafael J. Wysocki 2017-07-16 13:38 ` Alexandre Belloni 2017-07-16 13:38 ` Alexandre Belloni 2017-07-16 18:24 ` Pavel Machek 2017-07-16 18:24 ` Pavel Machek 2017-07-16 15:41 ` Florian Fainelli 2017-07-16 15:41 ` Florian Fainelli 2017-07-15 23:29 ` Rafael J. Wysocki 2017-07-15 23:29 ` Rafael J. Wysocki 2017-07-06 3:17 ` Pavel Machek 2017-07-06 3:17 ` Pavel Machek 2017-07-16 10:28 ` Rafael J. Wysocki 2017-07-16 10:28 ` Rafael J. Wysocki 2017-07-16 18:22 ` Pavel Machek 2017-07-16 18:22 ` Pavel Machek 2017-06-23 1:08 ` [RFC 2/2] soc: bcm: brcmstb: PM: Implement target_state callback Florian Fainelli 2017-06-23 1:08 ` Florian Fainelli 2017-06-29 23:04 ` Rafael J. Wysocki 2017-06-29 23:04 ` Rafael J. Wysocki 2017-07-16 2:36 ` [PATCH 0/2] PM / suspend: Add platform_suspend_target_state() Florian Fainelli 2017-07-16 2:36 ` Florian Fainelli 2017-07-16 2:36 ` [PATCH 1/2] " Florian Fainelli 2017-07-16 2:36 ` Florian Fainelli 2017-07-06 3:18 ` Pavel Machek 2017-07-06 3:18 ` Pavel Machek 2017-07-16 15:41 ` Florian Fainelli 2017-07-16 15:41 ` Florian Fainelli 2017-07-16 10:30 ` Rafael J. Wysocki 2017-07-16 10:30 ` Rafael J. Wysocki 2017-07-16 2:36 ` [PATCH 2/2] soc: bcm: brcmstb: PM: Implement target_state callback Florian Fainelli 2017-07-16 2:36 ` Florian Fainelli 2017-07-17 20:06 ` [PATCH v2] PM / suspend: Add suspend_target_state() Florian Fainelli 2017-07-17 20:06 ` Florian Fainelli 2017-07-17 20:16 ` Pavel Machek 2017-07-17 20:16 ` Pavel Machek 2017-07-17 21:03 ` Rafael J. Wysocki 2017-07-17 21:03 ` Rafael J. Wysocki 2017-07-17 21:21 ` Florian Fainelli 2017-07-17 21:21 ` Florian Fainelli 2017-07-20 8:03 ` Pavel Machek 2017-07-20 8:03 ` Pavel Machek 2017-07-17 22:10 ` [PATCH v3] PM / suspend: Export pm_suspend_target_state Florian Fainelli 2017-07-17 22:10 ` Florian Fainelli 2017-07-17 23:24 ` Rafael J. Wysocki 2017-07-17 23:24 ` Rafael J. Wysocki 2017-07-18 0:19 ` [PATCH v4] " Florian Fainelli 2017-07-18 0:19 ` Florian Fainelli 2017-07-24 20:55 ` Rafael J. Wysocki 2017-07-24 20:55 ` Rafael J. Wysocki 2017-07-13 12:03 ` Drivers taking different actions depending on sleep state Pavel Machek 2017-07-13 12:03 ` Pavel Machek
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=7b4435e5-5903-2678-fddb-34e884f5d53f@free.fr \ --to=slash.tmp@free.fr \ --cc=daniel.lezcano@linaro.org \ --cc=jb_lescher@sigmadesigns.com \ --cc=khilman@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-pm@vger.kernel.org \ --cc=pavel@ucw.cz \ --cc=rjw@rjwysocki.net \ --cc=thibaud_cornic@sigmadesigns.com \ --cc=ulf.hansson@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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.