From: Ilya Trukhanov <email@example.com> To: Ard Biesheuvel <firstname.lastname@example.org> Cc: "# 3.4.x" <email@example.com>, firstname.lastname@example.org, linux-efi <email@example.com>, Linux PM <firstname.lastname@example.org>, Javier Martinez Canillas <email@example.com>, Thomas Zimmermann <firstname.lastname@example.org>, "Rafael J. Wysocki" <email@example.com>, Len Brown <firstname.lastname@example.org>, email@example.com Subject: Re: [REGRESSION]: drivers/firmware: move x86 Generic System Framebuffers support Date: Thu, 11 Nov 2021 01:21:57 +0200 [thread overview] Message-ID: <20211110232157.xfeue3sbquyhtqmf@lahvuun> (raw) In-Reply-To: <CAMj1kXHuU8dFBUeM41bHu13nd2qQVPJizt8Epw596K89_samiQ@mail.gmail.com> On Wed, Nov 10, 2021 at 11:24:03PM +0100, Ard Biesheuvel wrote: > Hi Ilya, > > On Wed, 10 Nov 2021 at 21:02, Ilya Trukhanov <firstname.lastname@example.org> wrote: > > > > Suspend-to-RAM with elogind under Wayland stopped working in 5.15. > > > > This occurs with 5.15, 5.15.1 and latest master at > > 89d714ab6043bca7356b5c823f5335f5dce1f930. 5.14 and earlier releases work > > fine. > > > > git bisect gives d391c58271072d0b0fad93c82018d495b2633448. > > > > To reproduce: > > - Use elogind and Linux 5.15.1 with CONFIG_SYSFB_SIMPLEFB=n. > > - Start a Wayland session. I tested sway and weston, neither worked. > > - In a terminal emulator (I used alacritty) execute `loginctl suspend`. > > > > Normally after the last step the system would suspend, but it no longer > > does so after I upgraded to Linux 5.15. After running `loginctl suspend` > > in dmesg I get the following: > > [ 103.098782] elogind-daemon: Suspending system... > > [ 103.098794] PM: suspend entry (deep) > > [ 103.124621] Filesystems sync: 0.025 seconds > > > > But nothing happens afterwards. > > > > Suspend works as expected if I do any of the following: > > - Revert d391c58271072d0b0fad93c82018d495b2633448. > > - Build with CONFIG_SYSFB_SIMPLEFB=y. > > If this solves the issue, what else is there to discuss? Sorry, I'm not a kernel developer, but I was under the impression that this is a regression and should at least be brought to attention. I also think I'm probably not the last person to encounter this. I'm fortunate because I had the time to bisect and get the idea to try enabling that option, but others may not know how to fix it. The suspend not working is also not the only effect. After you execute `loginctl suspend`, for example, the compositor just hangs if you try to exit. Should you kill it with SysRq+I, the system suspends but after resume doesn't respond to anything and has to be hard reset. I think this is a pretty serious issue, even if it won't affect most users. Sorry if I wasn't meant to CC you. The issue reporting guide says that you should CC maintainers of affected subsystems. > > > > > - Suspend from tty, even if a Wayland session is running in parallel. > > - Suspend from under an X11 session. > > - Suspend with `echo mem > /sys/power/state`. > > > > If I attach strace to the elogind-daemon process after running > > `loginctl suspend` then the system immediately suspends. However, if > > I attach strace *prior* to running `loginctl suspend` then no suspend, > > and the process gets stuck on a write syscall to `/sys/power/state`. > > > > I "traced" a little bit with printk (sorry, I don't know of a better > > way) and the call chain is as follows: > > state_store -> pm_suspend -> enter_state -> suspend_prepare > > -> pm_prepare_console -> vt_move_to_console -> vt_waitactive > > -> __vt_event_wait > > > > __vt_event_wait just waits until wait_event_interruptible completes, but > > it never does (not until I attach to elogind-daemon with strace, at > > least). I did not follow the chain further. > > > > - Linux version 5.15.1 (lahvuun@lahvuun) (gcc (Gentoo 11.2.0 p1) 11.2.0, > > GNU ld (Gentoo 2.37_p1 p0) 2.37) #51 SMP PREEMPT Tue Nov 9 23:39:25 > > EET 2021 > > - Gentoo Linux 2.8 > > - x86_64 AuthenticAMD > > - dmesg: https://pastebin.com/duj33bY8 > > - .config: https://pastebin.com/7Hew1g0T
next prev parent reply other threads:[~2021-11-10 23:22 UTC|newest] Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-11-10 20:02 Ilya Trukhanov 2021-11-10 22:24 ` Ard Biesheuvel 2021-11-10 23:21 ` Ilya Trukhanov [this message] 2021-11-10 23:25 ` Ard Biesheuvel 2021-11-11 0:08 ` Ilya Trukhanov 2021-11-10 23:07 ` Javier Martinez Canillas 2021-11-11 0:45 ` Ilya Trukhanov 2021-11-11 7:31 ` Javier Martinez Canillas 2021-11-11 9:24 ` Javier Martinez Canillas 2021-11-11 10:52 ` Ilya Trukhanov 2021-11-11 11:13 ` Javier Martinez Canillas 2021-11-11 6:11 ` Thorsten Leemhuis
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20211110232157.xfeue3sbquyhtqmf@lahvuun \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [REGRESSION]: drivers/firmware: move x86 Generic System Framebuffers support' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).