Closed ikrivosheev closed 2 years ago
@ikrivosheev thanks for the report, the DSP should be OFF for hibernate S4, but it looks like something is blocking it. @keqiaozhang @miRoox can someone try the same test on TGL. @ujfalusi @plbossart fyi.
Likely duplicate of #5892 ?
In S4 and S5 the IMR boot is not possible due to the fact that the backup sram loosing it's content. The Linux side fix is: https://github.com/thesofproject/linux/pull/3687
@ujfalusi yes! This is a duplicate, close
@ujfalusi how do changes come into linux main repository? I see that changes have been made in fork but I cannot understand when it comes into mainline.
@ikrivosheev, in a second look at the log fragment you have, this might not be a duplicate as this happens when trying to enter to suspend. The issue which is fixed by the linked PR was on the resume side of the hibernate (https://github.com/thesofproject/sof/issues/5892)
Let me re-open this and please add this file sof-dyndbg.conf.txt as /etc/modprobe.d/sof-dyndbg.conf (the extension matters), reboot and attach the kernel log gathered in this mode.
@plbossart, fyi, I can not recall seeing such error.
@ujfalusi thank you! I will add it today.
I'm having exactly the same issue when using sof-firmware v2.1.1 on Linux x 5.18.5-arch1-1
on my Dell XPS 13.
If I downgrade SOF to v2.0 then suspending works as expected.
Note: I see this issue with any suspend operation, not just suspend-then-hibernate.
Edited 16:09 UTC: Although suspending is fixed after downgrading v2.0, hibernation still doesn't seem to function. The machine wakes back up almost instantly. I'll see if it works by downgrading further and collect some logs, though I don't want to hijack this issue.
Agree @ujfalusi this is a different error - we should not have any IPC errors, ever. The firmware downgrade clearly hints at a firmware regression.
I also having sof-firmware v2.1.1-1 on Linux x 5.18.5-arch1-1 with platform Lenovo Thinkpad e15g2. The problem is relevant to me. My kernel log with /etc/modprobe.d/sof-dyndbg.conf: dmesg.log
@igor-winograd I don't see any issues in your log file?
@ujfalusi my dmesg logs
It's the same error actually, the IPC error is preceded by a boot failure. The second hibernate fails because the resume failed already. We should probably have a check that we don't try ANY IPC during the suspend operation if the DSP didn't boot...
@ujfalusi @plbossart close issue as duplicate?
And my question:
@ujfalusi how do changes come into linux main repository? I see that changes have been made in fork but I cannot understand when it comes into mainline.
@ikrivosheev, the patches are in linux-next, it depends when stable will pick them (the last patch out of the three will have conflict, but it is trivial). While waiting for them to be backported you can disable the IMR booting (it was introduced in 5.18 and it is a nice to have feature - for those who do not suspend to disk ;) ) by adding
options snd_sof sof_debug=128
to /etc/modprobe.d/sof.conf
for example (and remove other module parameter files adding options for SOF modules) and reboot.
This will make things work like they used to prior the IMR support regarding to resuming from suspend/hibernate.
Amazing, thanks! I was actually having troubles on suspend but only after either:
Laptop model: Thinkpad E14 (Gen 12). Kernel 5.18.0.
Controller: 00:1f.3 Multimedia audio controller: Intel Corporation Tiger Lake-LP Smart Sound Technology Audio Controller (rev 20)
@ujfalusi can we close now ? i.e. are all patches in -next now ?
@lgirdwood, yes all patches are in linux-next and we also have a workaround for 5.18 kernel (sof_debug=128
)
Closing.
Thank you all!
Describe the bug
To Reproduce
systemctl suspend-then-hibernate
Reproduction Rate Always
Expected behavior Computer go to hibernate
Environment 1) Branch name and commit hash of the 2 repositories: sof (firmware/topology) and linux (kernel driver).
9f4fda38f1c59f7a20a76eff48a0cb302cb0e8e55bda53ec0f1807e10dcdad3a
(arch linux)adcd22ba2f2aad911fdc53c478187bc6ddb9174fa52fa43ba07f6f0cc8d12a19
2) Name of the platform(s) on which the bug is observed.