linux-kernel - Re: [PATCH v3 1/4] Documentation: Add AMD Zen debugging document

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

Message-ID: <Z_nP3FGEZzvRf26g@archie.me>
Date: Sat, 12 Apr 2025 09:28:44 +0700
From: Bagas Sanjaya <bagasdotme@...il.com>
To: Mario Limonciello <superm1@...nel.org>, Borislav Petkov <bp@...en8.de>,
	Jean Delvare <jdelvare@...e.com>,
	Andi Shyti <andi.shyti@...nel.org>,
	Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
Cc: Jonathan Corbet <corbet@....net>,
	Mario Limonciello <mario.limonciello@....com>,
	Yazen Ghannam <yazen.ghannam@....com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Ingo Molnar <mingo@...hat.com>,
	Dave Hansen <dave.hansen@...ux.intel.com>,
	"maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT)" <x86@...nel.org>,
	"H . Peter Anvin" <hpa@...or.com>,
	Shyam Sundar S K <Shyam-sundar.S-k@....com>,
	Hans de Goede <hdegoede@...hat.com>,
	"open list:DOCUMENTATION" <linux-doc@...r.kernel.org>,
	open list <linux-kernel@...r.kernel.org>,
	"open list:I2C/SMBUS CONTROLLER DRIVERS FOR PC" <linux-i2c@...r.kernel.org>,
	"open list:AMD PMC DRIVER" <platform-driver-x86@...r.kernel.org>,
	"Gautham R . Shenoy" <gautham.shenoy@....com>
Subject: Re: [PATCH v3 1/4] Documentation: Add AMD Zen debugging document

On Fri, Apr 11, 2025 at 09:19:09PM -0500, Mario Limonciello wrote:
> 
> 
> On 4/11/25 21:12, Bagas Sanjaya wrote:
> > On Thu, Apr 10, 2025 at 03:01:59PM -0500, Mario Limonciello wrote:
> > > v3:
> > >   * Move debugging.rst to index.rst
> > 
> > Do you plan to add more AMD-specific admin docs in the future?
> 
> I don't have any others planned right now.  That move was because I noticed
> a toc warning with how I had it structured before.
> 
> > (BTW, I don't
> > follow v2 discussions.)
> 
> Don't worry; documentation hasn't been talked about in v2, it's all been
> discussion on the S5_RESET_STATUS.

Understand.

> 
> > 
> > > +As there are a lot of places that problems can occur, a debugging script has
> > > +been created that can help test for common problems and offer suggestions.
> > > +
> > > +https://git.kernel.org/pub/scm/linux/kernel/git/superm1/amd-debug-tools.git/tree/amd_s2idle.py
> > > +
> > > +If you have an s2idle issue, it's best to start with this and follow instructions
> > > +from its findings.  If you continue to have an issue, raise a bug with the
> > > +report generated from this script.
> > 
> > To mailing list following Documentation/admin-guide/reporting-issues.rst?
> 
> Actually I prefer them to drm/amd with the s2idle bug template.  I'll add
> this detail.
> 
> > 
> > > +First convert the GPIO number into hex. ::
> > > +
> > > +  $ python3 -c "print(hex(59))"
> > > +  0x3b
> > > +
> > > +Next determine which ACPI table has the ``_EVT`` entry. For example: ::
> > > +
> > > +  $ sudo grep EVT /sys/firmware/acpi/tables/SSDT*
> > > +  grep: /sys/firmware/acpi/tables/SSDT27: binary file matches
> > > +
> > > +Decode this table:::
> > > +
> > > +  $ sudo cp /sys/firmware/acpi/tables/SSDT27 .
> > > +  $ sudo iasl -d SSDT27
> > 
> > Nit: two colons are sufficient for literal code-block.
> 
> Thanks.
> 
> > 
> > > +To activate PM debugging, use the kernel command line option: ``pm_debug_messages``.
> > > +
> > > +Or enable the feature using the sysfs file: ``/sys/power/pm_debug_messages``
> > > +Constraints that are not met will be displayed in the kernel log and can be
> > > +viewed using anything that processes the kernel ring buffer such as ``dmesg``` or
> > > +``journalctl``.
> > 
> > "To activate PM debugging, either specify ``pm_debug_messagess`` kernel
> > command-line option at boot or write to ``/sys/power/pm_debug_messages``.
> > Unmet constraints will be displayed in the kernel log and can be
> > viewed by logging tools that process kernel ring buffer like dmesg or
> > journalctl."
> > 
> > > +`patch <https://lore.kernel.org/amd-gfx/20250305051402.1550046-3-chiahsuan.chung@amd.com/T/#u>`_
> > 
> > What about that patchset status? It was not reviewed by upstream maintainers,
> > right?
> > 
> 
> At the time I wrote this document it wasn't yet merged.  It's been since
> merged, I will update to the commit details.
> 

OK, thanks!

-- 
An old man doll... just what I always wanted! - Clara

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)