From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
To: Reza Kurniawan <imkyufie@gmail.com>,
Mark Rutland <mark.rutland@arm.com>,
Lorenzo Pieralisi <lpieralisi@kernel.org>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>, Ulf Hansson <ulfh@kernel.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Daniel Lezcano <daniel.lezcano@kernel.org>,
Christian Loehle <christian.loehle@arm.com>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konradybcio@kernel.org>,
Abel Vesa <abelvesa@kernel.org>, Val Packett <val@packett.cool>
Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org,
linux-arm-msm@vger.kernel.org
Subject: Re: [PATCH v2 4/4] arm64: dts: qcom: msm8917: Use a shallower cluster state to fix unresponsive idle
Date: Wed, 21 Oct 2026 12:35:43 +0200 [thread overview]
Message-ID: <193e4885-9d0d-4f33-b2b1-650021413bec@oss.qualcomm.com> (raw)
In-Reply-To: <20261018-psci-fix-v2-4-35f65b59eb9a@gmail.com>
On 10/18/26 6:29 AM, Reza Kurniawan wrote:
> MSM8917 is a single cluster SoC, so powering down the cluster means
> powering down the whole system. This causes two problems:
>
> 1. The system becomes unresponsive, even to common interrupts. Waking it
> requires an interrupt from the MPM (MSM Power Manager) or scheduling
> the MPM to wake the system later. Linux currently has no way of doing
> the latter, so the cluster may never resume operation.
>
> 2. Entering power collapse blanks the screen. When the state is entered
> and exited repeatedly, this shows up as visible flicker.
>
> Even if MPM scheduling were implemented, the second problem would remain
> and it wouldn't fix the system being unresponsive at idle.
>
> Replace the idle state with PSCI mode 4 (dubbed GDHS downstream), which
> is less aggressive and can wake up from conventional interrupt sources.
>
> Signed-off-by: Reza Kurniawan <imkyufie@gmail.com>
> ---
For the sake of git bisect working, please order this patch before
patch 3.
Or alternatively, we can get rid of this one if the MPM rework
series lands first.
Konrad
prev parent reply other threads:[~2026-10-21 10:35 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-18 4:29 [PATCH v2 0/4] Fix PSCI on MSM8917 and MSM8937 Reza Kurniawan
2026-10-18 4:29 ` [PATCH v2 1/4] dt-bindings: arm: psci: Add skip-suspend-mode-switch property Reza Kurniawan
2026-10-18 4:29 ` [PATCH v2 2/4] cpuidle: psci: Allow skipping the suspend mode switch Reza Kurniawan
2026-10-18 4:29 ` [PATCH v2 3/4] arm64: dts: qcom: Skip suspend mode switch on MSM8917 and MSM8937 Reza Kurniawan
2026-10-21 10:35 ` Konrad Dybcio
2026-10-18 4:29 ` [PATCH v2 4/4] arm64: dts: qcom: msm8917: Use a shallower cluster state to fix unresponsive idle Reza Kurniawan
2026-10-21 10:35 ` Konrad Dybcio [this message]
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=193e4885-9d0d-4f33-b2b1-650021413bec@oss.qualcomm.com \
--to=konrad.dybcio@oss.qualcomm.com \
--cc=abelvesa@kernel.org \
--cc=andersson@kernel.org \
--cc=christian.loehle@arm.com \
--cc=conor+dt@kernel.org \
--cc=daniel.lezcano@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=imkyufie@gmail.com \
--cc=konradybcio@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=lpieralisi@kernel.org \
--cc=mark.rutland@arm.com \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=ulfh@kernel.org \
--cc=val@packett.cool \
/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
all inboxes | Powered by JetHome®