Skip to content

plymouthd exists prematurely so boot splash doesn't work with 0.9.0

Submitted by Lukas Anzinger

Assigned to Ray Strode @halfline

Link to original bug (#80553)

Description

Created attachment 101794 plymouth debug log file for 0.9.0 (unpatched)

Hi,

I recently upgraded to plymouth 0.9.0 on Debian Sid and plymouth stopped working.

I'm using the following setup:

  • Linux 3.14.7 (linux-image-3.14-1-686-pae 3.14.7-1)
  • plymouth 0.9.0 (plymouth 0.9.0-3)
  • uvesafb with AMD Radeon E6760 Embedded (uvesafb mode_option=1280x800-32 scroll=ywrap)

$ cat /etc/plymouth/plymouthd.conf

Administrator customizations go in this file

[Daemon] Theme=fade-in ShowDelay=1

It's a clean installation with nothing more than a basic Debian Sid and plymouth on top. plymouth 0.8.8 worked but 0.9.0 doesn't. I've attached the log file that maybe could shed some light on the matter. It's rather short so I believe plymouthd exits prematurely.

I bisected the problem and I believe the following commit causes a regression on my setup:

commit 7e594f72 Author: Ray Strode rstrode@redhat.com Date: Thu Mar 6 14:42:16 2014 -0500

seat: make sure to open terminal when adding text displays

If we have a pixel display, the renderer will handle opening the
associated terminal. but if we don't have a pixel display, something
needs to open the terminal.

This commit adds code to do that.

diff --git a/src/libply-splash-core/ply-seat.c b/src/libply-splash-core/ply-seat.c index 541b29e..2ac8bf7 100644 --- a/src/libply-splash-core/ply-seat.c +++ b/src/libply-splash-core/ply-seat.c @@ -102,6 +102,19 @@ add_text_displays (ply_seat_t *seat) { ply_text_display_t *display;

  • if (!ply_terminal_is_open (seat->terminal))
  • {
  •  if (!ply_terminal_open (seat->terminal))
  •    {
  •      ply_trace ("could not add terminal %s: %m",
  •                 ply_terminal_get_name (seat->terminal));
  •      return;
  •    }
  • }
  • ply_trace ("adding text display for terminal %s",
  •         ply_terminal_get_name (seat->terminal));
  • display = ply_text_display_new (seat->terminal); ply_list_append_data (seat->text_displays, display); }

If I revert that commit on top of 0.9.0 plymouth works again.

Regards,

Lukas

Attachment 101794, "plymouth debug log file for 0.9.0 (unpatched)":
plymouth-0.9.0-debug.log

To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information