From: Cezary Rojewski <cezary.rojewski@intel.com>
To: broonie@kernel.org
Cc: tiwai@suse.com, perex@perex.cz, amade@asmblr.net,
linux-sound@vger.kernel.org, andriy.shevchenko@linux.intel.com,
krzysztof.hejmowski@intel.com,
Cezary Rojewski <cezary.rojewski@intel.com>
Subject: [PATCH v2 0/3] ASoC: Intel: catpt: Error handling and debug improvements
Date: Thu, 28 May 2026 10:34:41 +0200 [thread overview]
Message-ID: <20260528083444.1439233-1-cezary.rojewski@intel.com> (raw)
Outcome of a long debug to solve one, long-standing bug ocurring very
rarely on Haswell/Broadwell machines during the boot procedure of the
AudioDSP firmware. Clever/unfortunate user can increase the
reproduction rare to 100%.
The bug: an exception occurring early during FW boot (firmware side, not
the software one) leaves the firmware hanging and the existing software
code is incappable of recognizing such problem. The only solution a
user currently has is: rmmod and then modprobe the driver.
Recently, together with Krzysztof from the firmware team decided to take
it up and clear the dashboard.
The exception handling takes just a few lines of code (all part of the
first patch), everything else that this patchset is composed of improves
the debugability and logging. If anything similar pops up, the
developers can see what's going on.
Changes in v2:
- moved the COREDUMP_RELEASE operation from dsp.c to ipc.c, right after
catpt_coredump() to ensure it's always executed, even if the dump
itself is not written due to -ENOMEM, suggested by Mark
- spelling fixes in the patch 1/3 message, s/COREDYMP/COREDUMP/
Cezary Rojewski (3):
ASoC: Intel: catpt: Complete coredump handling
ASoC: Intel: catpt: Add pretty-trace for large IPC payloads
ASoC: Intel: catpt: Print error code if board-registration fails
sound/soc/intel/catpt/Makefile | 3 ++-
sound/soc/intel/catpt/device.c | 5 +----
sound/soc/intel/catpt/ipc.c | 8 ++++++++
sound/soc/intel/catpt/loader.c | 3 +++
sound/soc/intel/catpt/registers.h | 12 ++++++++++++
sound/soc/intel/catpt/trace.c | 27 +++++++++++++++++++++++++++
sound/soc/intel/catpt/trace.h | 22 +++++++++++++++-------
7 files changed, 68 insertions(+), 12 deletions(-)
create mode 100644 sound/soc/intel/catpt/trace.c
--
2.34.1
next reply other threads:[~2026-05-28 8:33 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-28 8:34 Cezary Rojewski [this message]
2026-05-28 8:34 ` [PATCH v2 1/3] ASoC: Intel: catpt: Complete coredump handling Cezary Rojewski
2026-05-28 8:34 ` [PATCH v2 2/3] ASoC: Intel: catpt: Add pretty-trace for large IPC payloads Cezary Rojewski
2026-05-28 8:34 ` [PATCH v2 3/3] ASoC: Intel: catpt: Print error code if board-registration fails Cezary Rojewski
2026-05-28 18:12 ` [PATCH v2 0/3] ASoC: Intel: catpt: Error handling and debug improvements Mark Brown
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=20260528083444.1439233-1-cezary.rojewski@intel.com \
--to=cezary.rojewski@intel.com \
--cc=amade@asmblr.net \
--cc=andriy.shevchenko@linux.intel.com \
--cc=broonie@kernel.org \
--cc=krzysztof.hejmowski@intel.com \
--cc=linux-sound@vger.kernel.org \
--cc=perex@perex.cz \
--cc=tiwai@suse.com \
/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.