linux-kernel - Re: [PATCH v2 00/21] pmdomain: Add generic ->sync

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] [day] [month] [year] [list]

Message-ID: <fd4cfe7a-e29b-4237-b82f-48354deead3b@ideasonboard.com>
Date: Fri, 13 Jun 2025 13:33:16 +0300
From: Tomi Valkeinen <tomi.valkeinen@...asonboard.com>
To: Ulf Hansson <ulf.hansson@...aro.org>,
 Saravana Kannan <saravanak@...gle.com>, Stephen Boyd <sboyd@...nel.org>,
 linux-pm@...r.kernel.org
Cc: "Rafael J . Wysocki" <rafael@...nel.org>,
 Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
 Michael Grzeschik <m.grzeschik@...gutronix.de>,
 Bjorn Andersson <andersson@...nel.org>, Abel Vesa <abel.vesa@...aro.org>,
 Peng Fan <peng.fan@....nxp.com>, Johan Hovold <johan@...nel.org>,
 Maulik Shah <maulik.shah@....qualcomm.com>,
 Michal Simek <michal.simek@....com>, Konrad Dybcio <konradybcio@...nel.org>,
 Thierry Reding <thierry.reding@...il.com>,
 Jonathan Hunter <jonathanh@...dia.com>,
 linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
 Geert Uytterhoeven <geert+renesas@...der.be>
Subject: Re: [PATCH v2 00/21] pmdomain: Add generic ->sync_state() support to
 genpd

Hi Ulf,

On 23/05/2025 16:39, Ulf Hansson wrote:
> Changes in v2:
> 	- Well, quite a lot as I discovered various problems when doing
> 	additional testing of corner-case. I suggest re-review from scratch,
> 	even if I decided to keep some reviewed-by tags.
> 	- Added patches to allow some drivers that needs to align or opt-out
> 	from the new common behaviour in genpd.
> 
> If a PM domain (genpd) is powered-on during boot, there is probably a good
> reason for it. Therefore it's known to be a bad idea to allow such genpd to be
> powered-off before all of its consumer devices have been probed. This series
> intends to fix this problem.
> 
> We have been discussing these issues at LKML and at various Linux-conferences
> in the past. I have therefore tried to include the people I can recall being
> involved, but I may have forgotten some (my apologies), feel free to loop them
> in.
> 
> I have tested this with QEMU with a bunch of local test-drivers and DT nodes.
> Let me know if you want me to share this code too.
> 
> Please help review and test!

I tested this Renesas white-hawk board, and it hangs at boot. With
earlycon, I captured with/without boot logs, attached.

The hang case doesn't look very healthy with all these: "kobject:
'(null)' ((____ptrval____)): is not initialized, yet kobject_get() is
being called."

 Tomi

View attachment "minicom-hang.log" of type "text/x-log" (50527 bytes)

View attachment "minicom-ok.log" of type "text/x-log" (31105 bytes)