Re: [PATCH 05/11] Documentation: PM: amd-pstate: add debugging section for driver loading failure

From: Mario Limonciello
Date: Tue May 07 2024 - 11:01:56 EST


On 5/7/2024 02:15, Perry Yuan wrote:
To address issues with the loading of the amd-pstate driver on certain platforms,
It needs to enable dynamic debugging to capture debug messages during the driver
loading process. By adding "amd_pstate.dyndbg=+p cppc_acpi.dyndbg=+p loglevel=4 debug
amd_pstate=active" to the kernel command line, driver debug logging is enabled.

Here's an attempt at rewording this paragraph to consider.

For debugging issues reported by users with amd-pstate dynamic debugging output is often needed to aid in root cause. amd-pstate does use a variety of CPPC functions that also provide dynamic debugging output.

Add documentation showing how to use dynamic debug for both.

Signed-off-by: Perry Yuan <perry.yuan@xxxxxxx>
---
Documentation/admin-guide/pm/amd-pstate.rst | 13 +++++++++++++
1 file changed, 13 insertions(+)

diff --git a/Documentation/admin-guide/pm/amd-pstate.rst b/Documentation/admin-guide/pm/amd-pstate.rst
index 2695feec1baa..230e236796f7 100644
--- a/Documentation/admin-guide/pm/amd-pstate.rst
+++ b/Documentation/admin-guide/pm/amd-pstate.rst
@@ -476,6 +476,19 @@ operations for the new ``amd-pstate`` module with this tool. ::
Diagnostics and Tuning
=======================
+Debugging AMD P-State Driver Loading Issues
+------------------------------------------
+
+On some platforms, there may be issues with the loading of the amd-pstate driver.
+To capture debug messages for issue analysis, users can add below parameter,
+"amd_pstate.dyndbg=+p cppc_acpi.dyndbg=+p loglevel=4 debug amd_pstate=active"

Two things:

1. AFAICT you shouldn't need to set the loglevel, this should only affect console right? Most users will share a kernel ring buffer or journal.

2. I don't think the debugging suggestion documentation should include explicitly setting the mode to active mode. Active mode is the kernel default already. Users or their distro may have set the mode to a different mode by default and this may change things unintentionally for them.

+to the kernel command line. This will enable dynamic debugging and allow better
+analysis and troubleshooting of the driver loading process.
+
+Please note that adding this parameter will only enable debug logging during the
+driver loading phase and may affect system behavior. Use this option with caution
+and only for debugging purposes.
+
Trace Events
--------------