You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
JSL:BUG: DSP boot failed after system resume due to memory alloc failed : -12 #5161
I am not sure what you would expect from the audio driver @Vamshigopal. The system has a problem or some sort of memory leak. There's not much we can do here.
IIRC JSL does not rely on the IMR boot, maybe that's part of the problem. The firmware would be too old to make use of enhanced capabilities.
Yes @plbossart i can understand that there not much we can do if there is no memory left , but here if DSP boot fails and when system has as sufficient memory, we cant recover DSP without rebooting the whole system. This experience to user seems problematic. Espically in the sytems with 4gb RAM this issue occurs more frequently.
What Customer is looking from audio driver is is there a way we can upgrade the flags passed into the allocation such that it never fails ? I can understand if we try too hard we get watchdog and giving up is breaking audio.
Yes IMR is supported from CAVS2_5-001-drop-stable branch , so we cant support it for JSL platform.
This is 6.1 kernel, right? I think there were some fixes, improvements to make this allocation failure more rare.
It is also possible that some backported patch broke the allocation logic (afaik we had that in the past).
If we cannot allocate memory for the firmware to download then it is least of the problem in the system, which will fail in all sorts of way humanly possible.
commit a61c7d88d38cf3b9c88cf667c4f8a389a57744d4
Author: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Date: Fri Sep 23 18:35:01 2022 +0300
ALSA: memalloc: use __GFP_RETRY_MAYFAIL for DMA mem allocs
to fix the case. There is a risk the issue can come back as the solution done to try harder in audio driver, seems to trigger issues in other cases (in low-memory conditions).
This is 6.1 kernel, right? I think there were some fixes, improvements to make this allocation failure more rare. It is also possible that some backported patch broke the allocation logic (afaik we had that in the past).
@Vamshigopal I think this is same issue as we had with ADL-N -> #3915 and #3844
Yes @kv2019i we have similar issues in ADL-N and ADL , but one difference i would see here in JSL we dont have IMR.
@Vamshigopal, can we close this issue? The patches are in upstream for quite long time now and they should be fixing the problem once for all (famous last words).
Activity
Vamshigopal commentedon Aug 30, 2024
@plbossart @ujfalusi @kv2019i
plbossart commentedon Aug 30, 2024
I am not sure what you would expect from the audio driver @Vamshigopal. The system has a problem or some sort of memory leak. There's not much we can do here.
IIRC JSL does not rely on the IMR boot, maybe that's part of the problem. The firmware would be too old to make use of enhanced capabilities.
Vamshigopal commentedon Aug 30, 2024
Yes @plbossart i can understand that there not much we can do if there is no memory left , but here if DSP boot fails and when system has as sufficient memory, we cant recover DSP without rebooting the whole system. This experience to user seems problematic. Espically in the sytems with 4gb RAM this issue occurs more frequently.
What Customer is looking from audio driver is is there a way we can upgrade the flags passed into the allocation such that it never fails ? I can understand if we try too hard we get watchdog and giving up is breaking audio.
Yes IMR is supported from CAVS2_5-001-drop-stable branch , so we cant support it for JSL platform.
ujfalusi commentedon Aug 30, 2024
This is 6.1 kernel, right? I think there were some fixes, improvements to make this allocation failure more rare.
It is also possible that some backported patch broke the allocation logic (afaik we had that in the past).
If we cannot allocate memory for the firmware to download then it is least of the problem in the system, which will fail in all sorts of way humanly possible.
kv2019i commentedon Aug 30, 2024
@Vamshigopal I think this is same issue as we had with ADL-N -> #3915 and #3844
For latter, we submitted
to fix the case. There is a risk the issue can come back as the solution done to try harder in audio driver, seems to trigger issues in other cases (in low-memory conditions).
Vamshigopal commentedon Aug 30, 2024
yes 6.1 chrome kernel (https://chromium.googlesource.com/chromiumos/third_party/kernel/+/refs/tags/v6.1.105). If you know any fixes, please do share it i can check if its part of the kernel.
ujfalusi commentedon Aug 30, 2024
@Vamshigopal, do you have timeline when the reports started to com in or this has always been there since launch?
Vamshigopal commentedon Aug 30, 2024
As per customer , they started seeing this issue once they migrated the kernel from v5.4 to v6.1.
v5.4 https://chromium.googlesource.com/chromiumos/third_party/kernel/+/refs/heads/chromeos-5.4
v6.1 https://chromium.googlesource.com/chromiumos/third_party/kernel/+/refs/heads/chromeos-6.1
In v5.4 kernel they didnt see this issue, only in v6.1 kernel they saw this issue.
Vamshigopal commentedon Aug 30, 2024
kv2019i commentedon Aug 30, 2024
Ack, @Vamshigopal not using IMR makes this condition easier to hit. tiwai@a61c7d8 is worth a try on top of v6.1.
ujfalusi commentedon Jan 29, 2025
@Vamshigopal, can we close this issue? The patches are in upstream for quite long time now and they should be fixing the problem once for all (famous last words).
ujfalusi commentedon Feb 28, 2025
As we have now robust ways to overcome this issue in multiple levels, I will close the issue.