All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Adrián Larumbe" <adrian.larumbe@collabora.com>
To: Christian Hewitt <christianshewitt@gmail.com>
Cc: Boris Brezillon <boris.brezillon@collabora.com>,
	 Rob Herring <robh@kernel.org>,
	Steven Price <steven.price@arm.com>,
	 Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	 Thomas Zimmermann <tzimmermann@suse.de>,
	David Airlie <airlied@gmail.com>,
	 Simona Vetter <daniel@ffwll.ch>,
	 AngeloGioacchino Del Regno
	<angelogioacchino.delregno@collabora.com>,
	dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] drm/panfrost: reset the Amlogic GXM GPU on runtime resume
Date: Wed, 14 Oct 2026 19:03:17 +0100	[thread overview]
Message-ID: <as_A5ygwMkEgGeP4@sobremesa> (raw)
In-Reply-To: <6B9C57A7-8BE5-461B-B44B-EF91FC2D2D93@gmail.com>

Hi Christian,

On 13.10.2026 09:02, Christian Hewitt wrote:
> > On 13 Oct 2026, at 1:25 am, Adrián Larumbe <adrian.larumbe@collabora.com> wrote:
> > 
> > On Wed, 30 Sep 2026 12:56:29 +0000, Christian Hewitt <christianshewitt@gmail.com> wrote:
> >> After its first runtime suspend, every power-off of the Mali-T820 in
> >> Amlogic GXM SoCs times out and floods the log with shader, tiler and L2
> >> power transition timeouts. Following a soft reset the cores report not
> >> ready but PWRTRANS never clears, however long the driver waits.
> > 
> > Would you mind building the driver from the latest drm-misc-next and testing
> > whether this is still the case? Today I merged a patch series with a fix
> > for a long-standing bug that prevented the device from going into suspension
> > because of mismanagement of the PM refcount. It might be that what you report
> > as an issue triggered by power transitions is something altogether different.
> 
> I’d tested with the v12 iteration of your series in my amlogic-7.3.y [0] tree
> before generating the patch I submitted. I’ve now tested with v15 (as merged)
> and other recent panfrost patches you submitted (which v15 depends upon) and
> with this patch reverted, and the timeout issue with T820 remains. I also saw
> a splat that I wasn’t seeing with the v12 iteration [1].
> 
> [0] https://github.com/chewitt/linux/commits/amlogic-7.3.y
> [1] https://paste.libreelec.tv/desertic-cataleya.log

Turns out I was wrong about having easy access to a Khadas VIM2, so whatever I try to
gather about the existing bug must come from these logs. I see one very interesting
warning in [1]:

WARNING: drivers/gpu/drm/panfrost/panfrost_device.c:490 at panfrost_exception_name+0x1c/0x30 [panfrost], CPU#0: swapper/0/0

Something's causing the GPU to report a fault whose status code is at
present not documented in the driver.

I think that, because this seems like an SoC integration issue, and because of the unknown
exception being reported by the GPU, we should have some further input from an Amlogic SoC
subsystem maintainer, and also Steven Price himself.

> After reinstating this patch on v15 + dependencies dmesg is clean again [2].
> 
> [2] https://paste.libreelec.tv/leptosomatic-cherlyn.log
> 
> >> The transitions complete normally when the GPU comes out of a hardware
> >> reset, as it does at probe. Use runtime PM with reset and clock control
> > 
> > panfrost_device_reset() is always called as part of the PM runtime resume
> > operation, so I'm assuming what you mean here is that reset control is
> > deasserted at runtime resume when the SoC supports it
> 
> I’m firmly at the hobbyist end of kernel contributors and I’m not able to
> technically explain the patch Claude created. It’s entirely possible that it
> pokes around in the right area of code but is not the correct fix, but I lack
> the knowledge to dig further without explicit guidance. I can also give you
> remote access to a Khadas VIM2 board if that would be easier or useful?
> 
> Christian

Adrian Larumbe

  reply	other threads:[~2026-10-14 18:03 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-30 12:56 [PATCH] drm/panfrost: reset the Amlogic GXM GPU on runtime resume Christian Hewitt
2026-10-12 21:25 ` Adrián Larumbe
2026-10-13  5:02   ` Christian Hewitt
2026-10-14 18:03     ` Adrián Larumbe [this message]
2026-10-14 18:05       ` Adrián Larumbe

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=as_A5ygwMkEgGeP4@sobremesa \
    --to=adrian.larumbe@collabora.com \
    --cc=airlied@gmail.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=boris.brezillon@collabora.com \
    --cc=christianshewitt@gmail.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mripard@kernel.org \
    --cc=robh@kernel.org \
    --cc=steven.price@arm.com \
    --cc=tzimmermann@suse.de \
    /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 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.