Re: [PATCH 2/2] Documentation/arch/x86/amd-debugging: Add section about delay_suspend

[Mario Limonciello <superm1@kernel.org>]

On 8/5/26 20:41, Borislav Petkov wrote:
> On Thu, Aug 06, 2026 at 12:54:04AM +0200, Daniel Gibson wrote:
>> That document mainly talks about s2idle issues and lists issues that have
>> been quirked in the past, so IMHO it fits there?
> 
> Did you not read the first sentence of that document?
> 
>> Also, Mario Limonciello (of AMD) suggested documenting it there in:
>> https://lore.kernel.org/platform-driver-x86/f86d9af4-6415-427a-9f4e-639b2ae6203e@kernel.org/
> 
> Maybe he needs to read the first sentence of that document too.
> 
>> However, if there's a specific Wiki you think would be appropriate to
>> document this in, I should probably document it there as well.
> 
> You have a pretty good commit message in
> 
>    428b9fd2dce5 ("platform/x86/amd/pmc: Add delay_suspend module parameter")
> 
> You have a bugzilla entry, you have a mailing list thread, you have a module
> param documentation.
> 
> How much more do you need to document this? People should be able to use
> search engines or AI, right?
> 
> The problem with adding some hints as to how suspend can be fixed on some
> laptops is that we'll open the floodgates. Do you know how many laptops need
> quirks and fixes for suspend?
> 
> In any case, this doc is not for that.
> 
> Sorry.
> 

I guess I have a different opinion here.  If someone very technical 
comes to me and says I want to debug an issue with suspend or shutdown I 
want to give them a single link that has all the information they can 
use to learn.

I thought this could fit in well at 18.7, and then even if a search 
engine or AI tool stumbles on that page it can learn about the debugging 
parameter too.