[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAPaKu7QVEXp5Zsp2mNW+_wnci4O_UAienba0gNvXK_8iykwZCg@mail.gmail.com>
Date: Sat, 19 Jul 2025 18:13:00 -0700
From: Chia-I Wu <olvaffe@...il.com>
To: Daniel Almeida <daniel.almeida@...labora.com>
Cc: Boris Brezillon <boris.brezillon@...labora.com>, Steven Price <steven.price@....com>,
Liviu Dudau <liviu.dudau@....com>, Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>, linux-kernel@...r.kernel.org,
dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH 0/9] drm/panthor: add devcoredump support
On Sat, Jul 19, 2025 at 5:41 PM Daniel Almeida
<daniel.almeida@...labora.com> wrote:
>
> Hi Chia-I Wu :)
>
> > On 19 Jul 2025, at 21:01, Chia-I Wu <olvaffe@...il.com> wrote:
> >
> > This series adds devcoredump support to panthor.
> >
> > This is written from scratch and is not based on the prior work[1]. The
> > main differences are
>
> I wonder why this was started from scratch? IIRC, that work stopped, among
> other things, because we were not sure about what exactly to include in the
> dump. I don't think it warranted a completely new implementation, IMHO.
As noted in the listed differences, this impl triggers coredumps in
more places (e.g., mmu faults), captures lower-level hw regs,
separates capturing and processing, and outputs in text format. It
turns out there is little code that can be inherited from the prior
work.
It also does not support dumping successful jobs.
>
> Do you plan to work on the userspace part as well?
Yes, there is a very early tool in
https://gitlab.freedesktop.org/panfrost/linux/-/issues/44. There is
also a sample dump that shows the raw dump, the decoded one, and the
decoded ringbufs / cmdbufs.
>
> -- Daniel
>
Powered by blists - more mailing lists