[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAG-rBigASZpsxEpjUnCWYpfmjuyJTQ44AXBr90fYbQWvit_YXA@mail.gmail.com>
Date: Wed, 21 Feb 2024 10:37:53 -0500
From: Sven van Ashbrook <svenva@...omium.org>
To: Takashi Iwai <tiwai@...e.de>
Cc: Vlastimil Babka <vbabka@...e.cz>, Karthikeyan Ramasubramanian <kramasub@...omium.org>,
LKML <linux-kernel@...r.kernel.org>, Brian Geffon <bgeffon@...gle.com>,
stable@...r.kernel.org, Curtis Malainey <cujomalainey@...omium.org>,
Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>, linux-sound@...r.kernel.org,
linux-mm@...ck.org
Subject: Re: Stall at page allocations with __GFP_RETRY_MAYFAIL (Re: [PATCH
v1] ALSA: memalloc: Fix indefinite hang in non-iommu case)
Hi Takashi,
On Wed, Feb 21, 2024 at 4:21 AM Takashi Iwai <tiwai@...e.de> wrote:
>
> Both look like the code path via async PM resume.
> Were both from the runtime PM resume? Or the system resume?
The large firmware allocation that triggers the stall happens in
runtime resume.
This means runtime resume and system resume are affected.
Due to the way runtime PM works, the system suspend path is also affected.
Because system suspend first wakes up any runtime suspended devices,
before executing their system suspend callback.
So we get:
system active, SOF runtime suspended (audio not active)
-> system suspend request
-> calls SOF runtime resume callback (stall happens here)
-> calls SOF system suspend callback
Powered by blists - more mailing lists