hashtable: add get_size method

Right now there's no easy way to know if a hash table is empty,
which I'm going to need in a future commit.

This commit adds a get_size method to return the number of items in
the hash table.

terminal: add new get_name() method

The name is passed in at construct time, but it gets canonicalized.

This commit adds api to get the final name. The api is also useful,
since it prevents callers that need the mapping from having to maintain
it separately, which will be good for a future clean up.

renderer: use enum for selecting renderer type, not plugin path

ply_renderer_new takes a path to a renderer plugin, or NULL to
try each one in turn.

It's cleaner to abstract the path behind an enum type, so this
commit makes that change.

Now it will be possible to instantiate specific renderers without
hardcoding the paths to plugins in more than one place.

main: drop check_for_consoles call at startup

It was only used to make kernel_console_tty get set,
which we no longer have.

main: drop kernel_console_tty

We aren't actually really using it anymore.  It gets checked
in on place in the debugging code, but only before it's set.

This commit drops it.

main: drop parameters for check_for_consoles function

both the default tty and whether or not displays should be added
are available in the global state object (as state->default_tty
and state->is_shown respectively), so they don't need to be parameters.

The latter one was a boolean, so having it as a parameter was confusing
at the call sites.

This commit drops the parameters.

drm: simplify device open

libdrm's open functions are just wrappers around open(), and our
detection of which parameter to pass to drmOpen() is not 100%
reliable.

Simplify the code and just call open() directly.

Fixes https://bugs.freedesktop.org/show_bug.cgi?id=71590

systemd-units: don't timeout the plymouth waiting

A timeout causes things to fail, so we shouldn't timeout.

two-step: fix unlock screen

The previous commit introduced a bug where the unlock screen won't
get shown if it's requested within the first 5 seconds of startup.

This commit fixes that by forcing a redraw if the state switches from
NORMAL.

two-step: introduce delayed startup

Many machines these days can boot in 5 seconds or less.
In those cases, there's little point in showing a boot splash.

This commit introduces a StartupDelay option to the two step
plugin to prevent it from displaying anything for a few seconds.

two-step: add support for watermark

The watermark is overlaid on top of the background but below all the
other content.

two-step: delete extraneous log spew

two-step: add support for tiled background image

If there's a file named background-tile.png in the theme
then it will get used as the background behind the other
content in place of the background gradient.

ply-image: add tiling support

This follow up commit adds tiling support to images, so
that splash screens can have been background patterns instead
of just background colors.

pixel-buffer: add tiling function

It can be useful to tile a group of pixels across the screen
(for e.g. backgrounds).

This commit adds API to pixel buffer to do that tiling.

A follow up commit will add support into ply-image so images can
be loaded from disk and then tiled on screen.

configure: add back --enable-tracing

It was accidentally removed, when dropping device specific drm support.

docs: add docbook based man pages

This commit adds man pages for the plymouthd and plymouth client
commands.

scripts: make plymouth-update-initrd somewhat more useful

plymouth-update-initrd is a script that should probably be eventually
dropped.  It was originally created to unpack and initrd and graft
plymouth in.  Later, it just became a thin wrapper around mkinitrd.

These days, very few (any?) distros use mkinitrd.  Change it to call
dracut instead.

Relevant irc log:

<newfo> this is SO WRONG:
http://cgit.freedesktop.org/plymouth/tree/scripts/plymouth-update-initrd
<newfo> it won't work on any distro
<newfo> would you mind updating that please?
<newfo> replace the mkinitrd line in
/usr/libexec/plymouth/plymouth-update-initrd to  dracut -f
/boot/initramfs-$(uname -r).img $(uname -r)
<-- newfo has quit (Client Quit)

main: force tty to /dev/tty if $DISPLAY is set

the X11 renderer is useful for testing themes without rebooting,
but it really only functions if the user puts --tty=/dev/tty or
similar.  Furthermore, if a user neglects to do that really bad
stuff happens since plymouth will try to take over VT1.

This commit makes --tty=/dev/tty implied if $DISPLAY is set.

Make contact the default text plugin

And rename the old text plugin "tribar"

https://bugs.freedesktop.org/show_bug.cgi?id=55286

Add contact plugin

https://bugs.freedesktop.org/show_bug.cgi?id=55286

configure: don't bother with rhgb-compat-link anymore

it's been N years since rhgb, let's toggle the default to no.

drm: drop driver specific renderers

All the major modesetting drivers support the generic interface now, so
there's no reason to have driver specific code.

This commit drops all that extra code and hard codes the generic interface

systemd units: halt/reboot/kexec/power systemd unit have changed names

Ensure we use the new 'systemd-' prefixed names now.

systemd units: Replace prefdm.service with display-manager.service

The former is dead, the latter is the new name.  See:

http://cgit.freedesktop.org/systemd/systemd/commit/?id=b2c9cbafb8b1a2742978e1faf37fbb3cf4fb5d44
https://fedoraproject.org/wiki/Features/DisplayManagerRework

key-file: ignore leading spaces and blank lines

leading spaces and blank lines currently cause parse errors.
This commit fixes the parser to be more lenient.

https://bugs.freedesktop.org/show_bug.cgi?id=67006

Miscellaneous fixes for compiler warnings

Fix various warnings turned up with -Wall.  After fixing these remaining
issues, plymouth now builds successfully with gcc 4.8 using
"-Werror -Wno-error=unused-result -Wno-error=sign-compare".

script: include head with prototype of ply_get_timestamp()

ply_get_timestamp() returns a double, but the compiler assumes it will
return an int if ply-utils.h isn't #included.  This disparity between
what gets returned and what the compiler is expecting will be returned
causes astonishing knock-on effects (the label plugin fails to display
any text). See:

https://bugs.launchpad.net/ubuntu/+source/plymouth/+bug/1187318

and

http://lists.freedesktop.org/archives/plymouth/2013-June/000733.html

main: redirect standard io to /dev/null

do not redirect standard io to console tty anymore, it can cause issue
with Xorg grabbing tty
(https://bugzilla.novell.com/show_bug.cgi?id=811185).

Initial patch by Ray Strode

systemd: make sure to ignore all exit codes in ExecStartPost

terminal-session: don't close ptmx unless it's opened

This prevents a scary close(-1) in straces.

terminal-session: plug /dev/console fd leak

When we stop redirecting console messages, we temporarly open
/dev/console.  We don't ever close it.

This commit fixes that.

logger: don't close already logger if already closed

This prevents close(-1) from showing up in strace

plymouth: Add line numbers to tracing output

 The plymouth tracing output lists the function but there are many
 cases in which the same message is issued from multiple places in the same
 function, which makes debugging more difficult. This patch adds the line
 number to the output to uniquely identify each site.

Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Signed-off-by: Cristian Rodríguez <crrodriguez@opensuse.org>

Lookup the location of udevadm

It is not always in /bin , but /usr/...

configure: Use pkgconfig to determine the systemd unit dir location

Don't assume it's in /lib/systemd/system

systemd: Look up systemd-tty-ask-password-agent location

Don't assume it's in /bin.

drm: inform kernel about recently flushed areas

Some drivers (QXL in particular) require to be notified when a part
of the mapped frame buffer memory is changed.

This commit adds a drmModeDirtyFB call after flushing changes to the
frame buffer to accomplish that.

systemd: add Wants: systemd-vconsole-setup.service

Harald says we need it.

systemd: add plymouth-start.service to initrd-switch-root.target

<haraldh> halfline, can you add plymouth-start.service to
plymouth-populate-initrd ??
<halfline> haraldh: sure, can you give me details on the bug its fixing?
<haraldh> except, if you put yourself out of the plymouth cgroup
<haraldh> and the "@" in argv[0][0]
<haraldh> better put plymouth-start.service in the
initrd-switch-root.target
<haraldh> easier
<halfline> well we already do the "@" in argv[0][0]
<halfline> and we already put KillMode=none SendSIGKILL=no in the
service file
<halfline> but will add the change
<haraldh> hmm, ok

animation,throbber: go back to frame dropping

Right now we figure out which animation frame to use
based on a static counter variable.  Since it's static
instead of per-object, if there are multiple instances
it ends up counting up too fast.

We could:
1) make it per-object state
2) drop it and use the ifdef'd out alternative implementation
that potentially drops frames if the machine is sluggish

This commit chooses 2, but we may end up going to one if the
frame dropping turns out to be problematic.

Based on deductive work from Kevin Murphy.

main: use correct cache file

Now that we allow switching into the initrd we shouldn't try
to load the boot cache file when shutting down.

main: all update-root-fs for shutdown

Some distros now switch root back to the initrd at shutdown.

This commit allows plymouth to go along for the ride.

generate-initrd: Use pigz if available.

gitignore: Ignore systemd service and path files.

utils: don't create pipes non-blocking

The daemonizing code depends on its pipe io being
blocking.  The other user of ply_open_unidirectional_pipe
will work with blocking or non blocking io.

This commit changes ply_open_unidirectional_pipe to create
blocking pipes.  This started causing failures with
commit 9ec69929 since it replaced broken code (passing
O_NONBLOCK to fcntl(fd, F_SETFD.. instead of F_SETFL)
with working code.

utils: Add strerror(errno) to error message

So it's easier to debug why it fails.

main: ensure add_consoles_from_file terminates buffer string

This isn't a real problem in practice, since the files getting
read will be much smaller than the buffer, but it looks bad.

This commit ensures the buffer always leaves room for the terminating
NUL.

main: fix small leak

text-progress-bar: fix os string parsing

If there's no newline the the end of string variable will be NULL
instead of at the end of string.

throbgress: add more debug spew

These were melded into place from two-step

populate-initrd: If PLYMOUTH_THEME_NAME is set, write it into plymouthd.conf

You can set PLYMOUTH_THEME_NAME when building initramfs to get a
different theme into initramfs, but this doesn't change the default
theme, so the resulting initramfs won't actually use the theme we
installed.

This patch makes plymouth-populate-initrd rewrite the 'Theme=XXX' line
in plymouthd.conf, so plymouth will use the theme we install.

key-file: use %m instead of %a in fscanf

It's meaning is less muzzled and this fixes a compiler warning.

Use fseeko and ftello just for the sake of LFS correctness

No bug involved.

Use O_CLOEXEC where adequate, simplify its use.

systemd: add WantedBy snippets

This fixes systemctl enable.

https://bugs.freedesktop.org/show_bug.cgi?id=38348

main: make plymouth show-splash idempotent

plymouth show-splash causes hairy things, that should only happen once,
like activating renderers to happen.

This commit makes subsequent show-splash calls be no-ops.

two-step: don't update progress when idle

We've already reach a state where we aren't drawing anymore, etc,
so don't update progress and potentially fire off animations
that won't be seen.

main: if deactivate when already deactivated return immediately

We were trying to ignore second deactivate requests, but
were instead crashing because we're trying to use a nullified
trigger.

This commit makes sure things don't go crashy when a user
does "plymouth deactivate" on an already deactivated plymouthd.

configure: bump to 0.8.8

x11: make window fullscreen by default

The x11 plugin currently does multi-head by default, which is
sometimes useful, but often just gets in the way.

This commit makes it use one, fullscreen head by default, and
falls back to the old behavior if the PLY_CREATE_FAKE_MULTI_HEAD_SETUP
environment variable is set.

x11: only create windows when necessary

right now, if a user does show-splash
more than once, the X11 plugin will keep recreating
windows.

This commit changes it to only create windows when necessary.

space-flares: reshow label after changing message

We should call ply_label_show any time we have a message
to show, so that if the root filesystem gets mounted, we
can try to load the label plugin again.

This should prevent weird situations where the text shows up
invisible unless the user hits escape twice.

http://bugs.freedesktop.org/show_bug.cgi?id=55669

fade-throbber: reshow label after changing message

We should call ply_label_show any time we have a message
to show, so that if the root filesystem gets mounted, we
can try to load the label plugin again.

This should prevent weird situations where the text shows up
invisible unless the user hits escape twice.

http://bugs.freedesktop.org/show_bug.cgi?id=55669

throbgress: reshow label after changing message

We should call ply_label_show any time we have a message
to show, so that if the root filesystem gets mounted, we
can try to load the label plugin again.

This should prevent weird situations where the text shows up
invisible unless the user hits escape twice.

http://bugs.freedesktop.org/show_bug.cgi?id=55669

two-step: reshow label after changing message

We should call ply_label_show any time we have a message
to show, so that if the root filesystem gets mounted, we
can try to load the label plugin again.

This should prevent weird situations where the text shows up
invisible unless the user hits escape twice.

http://bugs.freedesktop.org/show_bug.cgi?id=55669

boot-splash: avoid crash if plugin doesn't support system updates

Right now if the user hits escape a couple of times they'll get
a crash, since the details plugin doesn't support system updates.

This commit makes ply_boot_splash_system_update a no-op in the
event the splash doesn't support that feature.

throbber: stop throbber when freed

It's possible that a throbber could get freed while animating.
In that case, stop the animation right away, so we don't keep
running the animation timeout on freed memory.

animaton: stop animation when freed

It's possible that an animation could get freed before it finishes.
In that case, finish the animation right away, so we don't keep running the
animation timeout on freed memory.

throbgress: support system-update feature

This commit adds support for the system-update command
by updating the throbgress progress bar.

main: Quiet compiler warning about free(const char *)

Just make a duplicate variable.

plymouthd: set default theme to spinner

It's a nicer default than fade-in.

text: drop unused defines

service: start plymouthd unconditionally

It's possible to get in a state where plymouthd is
started in the initrd but not displayed.  In the
event this happens, we'll neglect to ever show it,
since it already has a pid file.

plymouthd is now hardened against getting called
multiple times, so we no longer need to protect it
at the systemd level.

This commit drops the

ConditionPathExists=!@plymouthruntimedir@/pid

line from the systemd service file, so we always
call plymouthd from the main fs and always call
plymouth show-splash.

main: silently succeed if plymouthd is started a second time

Make plymouthd idempotent to multiple calls.  This will make
it easier to integrate with boot scripts that start plymouthd
in the initrd and from the main fs.

main: Try to own socket before anything else

The first thing plymouthd should do is try to take control
over its server socket.  If it can't do that, then no plymouth
client will be able to communicate with it.  The socket is
also how we can detect if plymouthd is already running.

main: fix debug message

If plymouthd is already running, then a second invocation will
fail with the message:

could not log bootup

This message is misleading, so fix it.

configure: bump to 0.8.7

systemd: start after udev triggers

This fixes the race where udevadm settle finishes too early.

revert spurious broken changes in master commit 468c53a

https://bugs.freedesktop.org/show_bug.cgi?id=53440

script: Add alignment argument to Image.Text

Enables scripts to choose the text alignment they want with a seventh
argument to Image.Text API:

    new_image = Image.Text("Hello\n"
                           "Plymouth aligned world",
                            1, 1, 1, 1,
                           "DejaVu Bold,Italic 18",
                           "center");

This argument can be ignored, the default being left aligned.
Possible values are "left", "center", or "right".

http://lists.freedesktop.org/archives/plymouth/2012-August/000676.html

ply-text-progress-bar: strip quotes if present

The syntax for sysconfig-style files allows for quotes.

build: fix typo in configure.ac

s/reelase/release/

ply-terminal: remove pointless casts

The target variables are already of the desired type.

systemd: don't run read-write service from initrd

the initrd hits the local-fs.target as part of its normal
boot process.  We used to use local-fs.target as a way of
knowing the system / is read-write.  This no longer is a
valid mechanism.

This commit:

1) Stops installing plymouth-read-write service in the initrd
2) Makes it so if it does end up in the initrd it won't be
used

Related to fedora bug 830482

throbgress: s/TRUE/true/

I introduced a compilation error in Ian's patch when changing it
slightly.

throgress: pull trigger right away if already idle

commit eed536ba544dee38aee3c697efd2e3980362954a introduced
a fix for hangs in the two-step plugin. This is a comparable
commit for throbgress.

https://bugzilla.redhat.com/show_bug.cgi?id=704658

configure: bump to 0.8.6.2

populate-initrd: copy service files to initrd

Since we're in charge of the systemd service files now,
we should probably be in charge of copying them to the initrd.

systemd: add plymouth-switch-root.service

This service file is needed by systemdized dracut to ensure we properly
transition from the initrd to the root file system

systemd: more updates to start service

The dashes next to the semi-colons seem to confuse systemd so drop them.

systemd: create missing links various targets

systemd: make sure plymouth-start never gets killed

systemd tries to bring down the world when going from
the initramfs to /

commit 9e5a276f322cfce46b5b2ed2125cb9ec67df7e9f tried
to prevent plymouthd from getting killed by settings
argv[0][0] to @

This isn't seemingly sufficient.  Throw some lines into
the plymouth-start service file that should hopefully help.

populate-initrd: put plymouthd in /sbin not /bin

This matches what we do post-boot better

configure: bump to 0.8.6.1

systemd: unfoul up Makefile

The previous systemd commit introduced a file named

systemd-ask-password-plymouth.path.in

The makefile was only looking for .service.in files
when stripping the .in suffix, so it got installed
incorrectly.

This commit fixes up the Makefile.

configure: bump to 0.8.6

systemd: add two more files

When plymouth service files were moved from systemd to plymouth
two files got lost in the shuffle.

This commit adds them.

http://bugzilla.freedesktop.org/51573

frame-buffer: fix flush_area fast-path

plymouth uses a shadow framebuffer to cache screen contents for
quick compositing.  This shadow framebuffer may or may not have
the same memory layout as the hardware framebuffer.  In cases
where the size and layout of pixels are the same between the shadow
framebuffer and the hardware framebuffer we can memcpy()
the pixels line-by-line to the hardware.  If the width of area being
flushed is the same number of bytes as the width of the hardware buffer,
then we can memcpy() the entire flush area in one call to memcpy.

The check for this latter fast-path has a miscalculation that tests
the number of pixels in the flush area width to number of bytes in the
buffer width. This commit adds the * 4 multiplier to correctly compare
bytes with bytes instead of pixels with bytes.

This commit also adds a sanity check to make sure the byte size of the
hardware framebuffer width is equal to the advertised row stride.

systemd: more udev clean ups

The udev trigger calls that are there, are actually
spurious.  udev will coldplug those subsystems at
start up anyway, so doing it explicitly is wrong.

This commit drops those trigger calls.

What does matter is the udevadm settle call. It's
what makes things block until the graphics driver
is loaded.

udevadm settle is a big sledgehammer, though. It blocks
until all the triggered events (even stuff unrelated to
graphics) are finished.

This commit adds a --exit-if-exists argument to udevadm settle,
so it will bail early as soon as the graphics devices
are up.