Bug 293978 - System doesn't shutdown/reboot anymore after systemd upgrade
Summary: System doesn't shutdown/reboot anymore after systemd upgrade
Status: RESOLVED UPSTREAM
Alias: None
Product: plasma4
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 294376 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-02-13 09:48 UTC by Andrea Scarpino
Modified: 2012-04-01 22:56 UTC (History)
14 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
fedora pam patch being tested (1.24 KB, patch)
2012-03-13 20:54 UTC, Rex Dieter
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrea Scarpino 2012-02-13 09:48:52 UTC
Version:           unspecified (using KDE 4.8.0) 
OS:                Linux

The shutdown/reboot options in the KMenu doesn't turn off/reboot the system anymore but exit the session and back to KDM.

This happens after the update to systemd version 41 and 42.

This is a KDE bug because the user which reported us the bug (see https://bugs.archlinux.org/task/28386) tried XFCE and everything is fine there.

I didn't test XFCE (I'll do), but I cannot shutdown/reboot my system anymore too.

Any info you need please ask.
Thanks.

Reproducible: Always

Steps to Reproduce:
Use systemd. Try to shutdown/reboot your system.

Actual Results:  
The session is closed and you are back to KDM.

Expected Results:  
The system turn off/reboot.

systemd 42
KDE 4.8.0
Comment 1 Jekyll Wu 2012-02-18 16:37:36 UTC
*** Bug 294376 has been marked as a duplicate of this bug. ***
Comment 2 Marius Cirsta 2012-02-20 20:51:45 UTC
I can confirm that this indeed happens with the new systemd and KDE 4.8.0
Comment 3 Rex Dieter 2012-03-13 20:52:41 UTC
Seems to be a pam issue (I'll attach a patch workaround we're testing in fedora)
Comment 4 Rex Dieter 2012-03-13 20:54:04 UTC
Created attachment 69593 [details]
fedora pam patch being tested

See also downstream bug http://bugzilla.redhat.com/796969
Comment 5 Luc Menut 2012-03-13 21:37:21 UTC
(In reply to comment #4)
> Created attachment 69593 [details]
> fedora pam patch being tested
> 
> See also downstream bug http://bugzilla.redhat.com/796969

All distribs that uses recent systemd have this bug.
It would be nice if this patch is incorporated in kdm, after a proper review by kdm devs.
Rex, please, could you open a review request for this patch to the kde review board?

regards,
Luc
Comment 6 Rex Dieter 2012-03-13 22:23:52 UTC
We will when it's gotten more testing and more confident that it's the right fix.  You're welcome to help by testing it out yourself, or help provide feedback (if you understand the code).
Comment 7 Martin Stolpe 2012-03-17 13:48:43 UTC
Hello,

I've tested the attached patch and it works for me. I haven't seen any side effects yet.

Regards
Martin
Comment 8 Colin Guthrie 2012-04-01 22:50:31 UTC
As my colleague Luc Menut pointed out on our downstream bug (https://bugs.mageia.org/show_bug.cgi?id=4605) this has now been fixed at the systemd side:

<quote>
This recent commit in systemd "logind: close FIFO before ending sessions
cleanly",
http://cgit.freedesktop.org/systemd/systemd/commit/?id=75c8e3cffd7da8eede614cf61384957af2c82a29
fixes this bug with KDE/KDM.

I tested our current systemd patched with the logind FIFO patch + our kdebase4-workspace without the fedora patch, and shutdown and reboot work fine.
</quote>

From my previous reading of his description, Rex's patch could very well still be valid, but it seems like it's not specifically needed for proper operation now.
Comment 9 Andrea Scarpino 2012-04-01 22:56:30 UTC
Fixed in systemd with this commit http://cgit.freedesktop.org/systemd/systemd/commit/?id=75c8e3cffd7da8eede614cf61384957af2c82a29