Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 465518 - sys-kernel/dracut expects /bin/plymouthd but recent sys-boot/plymouth installs /sbin/plymouthd
Summary: sys-kernel/dracut expects /bin/plymouthd but recent sys-boot/plymouth install...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Amadeusz Żołnowski (RETIRED)
URL: https://git.kernel.org/cgit/boot/drac...
Whiteboard:
Keywords: UPSTREAM
Depends on:
Blocks:
 
Reported: 2013-04-11 13:15 UTC by Andrius Štikonas
Modified: 2013-04-18 15:21 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
don't use absolute path to plymouth. should be upstream (dracut-plymouth.patch,6.14 KB, patch)
2013-04-11 14:17 UTC, Enrico Tagliavini
Details | Diff
don't use absolute path to plymouth. already upstream (dracut-plymouth.patch,6.14 KB, patch)
2013-04-11 18:27 UTC, Enrico Tagliavini
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrius Štikonas 2013-04-11 13:15:00 UTC
Dracut has a plymouth module which assumes that plymouthd is located in /bin/plymouthd.

However, recent versions of plymouth install plymouthd into /sbin/plymouthd. Therefore, dracut is unable to generate a consistent initramfs.

Some of the discussion happened in https://bugs.gentoo.org/show_bug.cgi?id=428146


Reproducible: Always

Steps to Reproduce:
1. Install plymouth and dracut
2. Generate initramfs

Actual Results:  
Plymouth starts very late in the boot process.

Expected Results:  
Plymouth stars immediately as initramfs gets loaded (and asks for LUKS passphrase).

plymouth versions up to 0.8.6.1 install plymouthd to /bin/plymouthd
Comment 1 Enrico Tagliavini 2013-04-11 13:36:28 UTC
The problem is not where the plymouth ebuild install the pluymouthd file. The problem is dracut. Dracut generate an inconsistend initramfs [for what I understand]:

looking at files here: /usr/lib/dracut/modules.d/50plymouth:

plymouth-populate-initrd.sh line 7: inst /sbin/plymouthd /bin/plymouthd

So the plymouthd should be installed in the /bin directory of the initramfs. I extraced mine and 

schroedingherscat initram # pwd
/boot/initram
schroedingherscat initram # ll *bin/plymouthd
-rwxr-xr-x 1 root root 48K Apr 11 14:02 sbin/plymouthd*

it is installed in sbin.

in plymouth-pretrigger.sh line 5:

if [ -x /bin/plymouthd -a -z "$DRACUT_SYSTEMD" ]; then

this result false since plymouthd is in /sbin in the initramfs

I'm still investigating why the inst function is failing, or if something moves plymouthd somewhere else.
Comment 2 Enrico Tagliavini 2013-04-11 13:52:05 UTC
Ok I got it: in gentoo dracut calls

/usr/libexec/plymouth/plymouth-populate-initrd

and guess waht: on line 375

inst ${PLYMOUTH_DAEMON_PATH} $INITRDDIR /sbin/plymouthd

Now..... who is in fault?
Comment 3 Enrico Tagliavini 2013-04-11 14:17:57 UTC
Created attachment 345224 [details, diff]
don't use absolute path to plymouth. should be upstream

I gone to #dracut IRC channel. Thank you very much to Harald Hoyer, he done the patch and pushed to git master.
Comment 4 Enrico Tagliavini 2013-04-11 18:27:54 UTC
Created attachment 345258 [details, diff]
don't use absolute path to plymouth. already upstream

There was a missing space in the previous patch. I tested this one. It works for me™®
Comment 5 Amadeusz Żołnowski (RETIRED) gentoo-dev 2013-04-15 07:23:04 UTC
Enrico, thank you! I'll apply the patch in 027-r1.
Comment 6 Amadeusz Żołnowski (RETIRED) gentoo-dev 2013-04-18 13:40:10 UTC
Fixed. Thank you!


*dracut-027-r1 (18 Apr 2013)

  18 Apr 2013; Amadeusz Żołnowski <aidecoe@gentoo.org>
  +files/027-0003-Do-not-call-plymouth-with-full-path.patch,
  +files/027-0004-plymouth-plymouth-pretrigger.sh-fixup-.patch,
  +dracut-027-r1.ebuild:
  Rev-bump. Fixes bug #465518.

  Thanks to Enrico Tagliavini <enrico.tagliavini@gmail.com> for report and
  handling it and to Harald Hoyer for fixing it.
Comment 7 Enrico Tagliavini 2013-04-18 15:13:13 UTC
Thank you very much for the backport
Comment 8 Enrico Tagliavini 2013-04-18 15:21:44 UTC
o.O sorry it seems I've changed the status. Returning to resolved