linux-kernel - Re: [PATCH] PM / domains: Kconfig: always enable PM

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <5229811.LXthdvUIfI@vostro.rjw.lan>
Date:	Fri, 14 Nov 2014 23:41:15 +0100
From:	"Rafael J. Wysocki" <rjw@...ysocki.net>
To:	Kevin Hilman <khilman@...nel.org>
Cc:	Geert Uytterhoeven <geert@...ux-m68k.org>,
	Ulf Hansson <ulf.hansson@...aro.org>,
	Geert Uytterhoeven <geert+renesas@...der.be>,
	Linux PM list <linux-pm@...r.kernel.org>,
	linaro-kernel <linaro-kernel@...ts.linaro.org>,
	Grygorii Strashko <grygorii.strashko@...com>,
	Len Brown <len.brown@...el.com>, Pavel Machek <pavel@....cz>,
	open list <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] PM / domains: Kconfig: always enable PM_RUNTIME when genpd enabled

On Friday, November 14, 2014 09:36:17 AM Kevin Hilman wrote:
> Geert Uytterhoeven <geert@...ux-m68k.org> writes:
> 
> > Hi Kevin,
> >
> > On Thu, Nov 13, 2014 at 11:28 PM, Kevin Hilman <khilman@...nel.org> wrote:
> >> It makes little sense to use generic power domains without runtime PM.
> >
> > Does it?
> > It still powers down the PM domains on system suspend (at least on my
> > boards ;-)
> 
> Sure, but your devices are also using runtime PM, so I'm not sure how
> does that change my statement above?

Questions here are (1) how many users will actually want to disable PM_RUNTIME
for systems using genpd (my sort of educated guess is "none") and (2) whether
or not we need to power up stuff dynamically during initialization on systems
using genpd (and several recent discussions indicate that the answer here is
"yes, we need to do that").

> >> Also, since the complexities of handling the !PM_RUNTIME case are
> >> causing more trouble and confusion than they're worth, let's simplify
> >> the world by making genpd always enable runtime PM.
> >
> > What do other people think?
> 
> Yes, I'm curious also.  This patch was supposed to be marked RFC.

I'm totally for making this change and I'm going to queue up this patch for
3.19, so if anyone sees a really good reason for not doing this, please let
me know what that reason is as soon as you can.

Rafael

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/