[animation] Properly handle zero animation frames

In the event there are no animation frames, we should
fail to load the animation control.

[drm] free stored connector

For most connectors associating monitors with our virtual
'heads', we merely track their ids.  This means we don't have
to bother freeing any client side state at deallocation time.

There's one exception, though.  The main connector, connector0,
we keep an open reference to.  We do this, because it owns the
mode object we use in SetCrtc calls.

This commit ensures that connector0 for each head is properly freed
when that head is deallocated.

[drm] Allow multiple monitors to share one controller

Some underpowered video cards will have multiple connectors
tied to one controller.  In this case all connectors get
the same "cloned" output automatically.

This commit detects this situation and prevents plymouth from
wastefully trying to allocate several frame buffers.  This
commit also prevents plymouth from constantly trying to switch
between those allocated frame buffers every frame of the animation.

[array] Support integer elements

Right now all array elements must be pointer values, or
at least pointer sized values.

This commit allows 32-bit integers as elements as well.

Revert "[drm] Store color depth in buffers"

This reverts commit 4469b8719e32cb179a4ae792e4aaf1bcf7789618.

It was really only needed for commit
625b82f2c3a4f7aa1ea90133fbbda854bd3c2f8d which we've now
reverted, so revert this one, too.

Revert "[drm] bail if not in 24bpp color mode"

This reverts commit 625b82f2c3a4f7aa1ea90133fbbda854bd3c2f8d.

Even if the kernel fb console is 8bpp, we can (and indeed do)
create our own 24bpp buffer, so the commit wasn't right.

[drm] bail if not in 24bpp color mode

Some server hardware gets initialized into 8bpp mode to
conserve memory.  We can't work in that mode, so we now
check for it and back early.

[drm] Store color depth in buffers

We only really support full color layouts at the moment
for kernel modesetting drivers, but this commit adds a
"color_depth" member to the buffer structs so we can at
least recognize non-conforming buffers.

[main] Properly handle no serial consoles

The previous commit introduced an error where
console could potentially be accessed while
NULL.

This commit addresses that problem.

[main] Ignore duplicate serial consoles on kernel command line

Right now if a user erroneously puts, e.g., console=ttyS0 twice
on their kernel command line, plymouth will open that tty twice
and write to it twice, and read from it twice, etc.

This commit filters those duplicates out.

[boot-splash] Add built-in plugin

There are times when plymouth is running that the filesystem isn't
accessible.  For instance, if a user has /usr as a separate partition,
then when first leaving the initrd, plymouth won't have access to its
plugins.

In those cases we really need to survive if the user hits escape.

This commit compiles details into the binary.  In this way, if the
plugins aren't available, we still have something to fall back to.

[main] Parse args at end of /proc/cmdline properly

We weren't treating \n as an acceptable boundary

[terminal] fix debug statement

It was missing an argument.

[configure] Bump configure to 0.8.4

We aren't doing a release yet, this is just prep work
for an eventual release.

[main] only look for a shell in valid returned prefixed strings

Fixes a segfault if no "init=" is present

[main] Don't watch for keyboard input if no keyboard

In the event should_ignore_show_splash_calls () returns
true, we won't ever have a keyboard set.  This commit
make sure that we don't try to use the keyboard if its
unavailable.

[main] Try to be smarter about init=/foo on kernel command line

Right now, we put plymouth into a sort of degraded mode when we
find init=/anything on the kernel command line.  This is so if
the user does init=/bin/sh to get fix their system, we don't
get in the way.

This breaks plymouth for things like init=/sbin/bootchartd and even
init=/sbin/init.  We've previously had a plymouth.override-splash
kernel command line option to force plymouth on in those cases.

This commit flips things around a bit.  Now init=/sbin/bootchartd
will work by default and there's a new option
plymouth.ignore-show-splash which forces things back into degraded
mode.  We also will implicitly do the degraded mode for e.g.
init=/foo/barsh

[main] Use plymouth.arg instead of plymouth:arg for kernel cmdline

Plymouth has historically used "plymouth:" to identify arguments destined
for it on the kernel command line.  Most other things seem to the period
instead of colon, so this commit just makes plymouth conform.

For compatibility, we still support the old way.

[main] drop all the open strstr calls

Our kernel command line processing is rather ad hoc.
This commit tries to clean it up a small amount by
adding functions to hide the strstr details.

[populate-initrd] don't croak when encountering broken symlink

Because of an apparent think-o in the script, if a custom theme
has a broken symlink in it, plymouth-populate-initrd would silently
stop processing files and ship an incomplete set of data files.

This commit changes "break" to "continue" so that broken symlinks
are ignored, which was probably the original intent.

[set-default-theme] Properly quote arguments to basename

It's important to make sure the theme name is properly
quoted when passed to the basename command.  This
is because, if the theme name is empty we want the empty
string returned, not the suffix that would otherwise be
stripped off.

Some discussion here:
https://bugzilla.redhat.com/show_bug.cgi?id=606634

[details] detach from event loop when quitting

If the user has the details plugin configured, and the
system does plymouth quit --retain-splash, then their
may be a crash.

This is because the plugin event loop exit handler will called
after the plugin is destroyed.  This commit makes sure
the exit handler is removed before the plugin is freed.

[main] try more aggressively to find a terminal

Before we defaulted to tty1, but some systems just
don't have tty1.  Normally, those systems specify
an alternate console on the kernel command line,
but not always.

This commit tries to make things work in those cases
as well.

I'd like to find a more generic way to make this all
work.

[terminal] Add OPOST to tty attributes

ONLCR apparently requires OPOST to work.

This should prevent weird "stair stepping"
effects when non-plymouth things try to
use the terminal.

[terminal] Force enter to output newline

This matches the default unix behavior, and so we're going to be a lot
more robust against our terminal settings getting mucked with.

[client] Fix watch-keystroke cancellation

If someone calls --ignore-keystroke then
the watch-keystroke command shouldn't get
run and the exit code should be different
than if it does get run.

[viewer] make boot messages show up in monospace

They're sort of designed for it, and look weird in some cases,
otherwise.

[viewer] add window icon

Before it was showing up as an ugly generic icon.

[throbber] fix drawing artifact

The code was using the wrong x and y values, which meant initially
there is an opportunity to draw a frame in the wrong place.

[event-loop] Handle more than 8 events at once

epoll_wait was getting passed a pointer to 64 events,
but was only being told to use up to 8 of those events
at a time because it was using sizeof (pointer) for the
argument specifying the number of events instead of the
actual number of events.

[drm] Try to maintain current mode when overridden

plymouth has naïvely been treating the first available mode in
the connector object as the active mode. While this is true
most of the time, it doesn't hold true if the user overrides the
mode on the kernel command line.

This commit changes things to look up the actual active mode, and go
with that, which should help prevent flicker at start up in some cases.

This patch is based heavily on a patch from Forest Bond
<forest@alittletooquiet.net> here:

http://lists.freedesktop.org/archives/plymouth/2010-June/000369.html

and first mentioned in commit abfda7550ac2fca6d816862e271126c2f44fa3cc

[drm] Make find_controller return the crtc not crtc id

This will give us the ability to look at the currently active
mode and so paves the way toward cleaning up a recently added
FIXME in the code.

[drm] Store mode of each head as index

Previously, we would store the mode of each head as pointer in
the array of modes on the connector object.  Now we just store
the index into that array.

This is to make it clearer that we don't own the memory
associated with it directly, and that the memory associated with
it is automatically cleaned up when the connector is.

This also helps to highlight a problem mentioned by
Forest Bond <forest@alittletooquiet.net> here:

http://lists.freedesktop.org/archives/plymouth/2010-June/000369.html

Namely, we've been naïvely treating the first available mode in
the connector object as the active mode.  While this is true
most of the time, it doesn't hold true if the user overrides the
mode on the kernel command line.

[client] Add ignore-keystroke command

Adds an ignore-keystroke command. This does the same thing as the
ignore-keystroke option, but is migrated to a command. The option still exists
for compatibility.

[terminal] Add some debug spew if terminal couldn't be opened

[keyboard] Don't blow an assertion if terminal is unavailable

Right now, we always assume the tty can be reopened when it
gets disconnected.  This isn't always true.  While plymouth
clearly won't function 100% properly without a tty, we shouldn't
blow an assertion.

[main] Be more forgiving of an absent boot splash

Mandriva has cases where they need to drop to details mode
before /usr is mounted.  /usr has the details splash plugin,
so this causes plymouth to explicitly exit.

This commit makes plymouth cope with that scenario slightly
better.  It takes out the exit(1) call which was clearly bogus,
and also makes parts of the code handle a NULL splash better.
This is not a real fix though.

Ultimately, we should make "details" be built-in as a nice failsafe.
There's no reason we should need to load it off the disk as a plugin.

[docs] Fix up encoding Kristian Høgsberg

[set-default-theme] More fixes to the fallback logic

It still wasn't properly falling back in the event the
configured theme was uninstalled.

[set-default-theme] Add more compat goo

If there's no configured theme installed, return "text"

[terminal] Don't stomp over original tty lock settings

We were repetedly saving over the original tty lock settings,
causing the tty to stay locked after boot up.

[configure] bump to 0.8.3

[two-step] Add optional header and throbber

These are just a few more bits for themes to optionally
make use of.  The throbber gets overlaid with the
progress animation, so they need to be concentric.
The header gets put on top.

[two-step] Add new ProgressFunction config option

We've historically used a expontial function for boot up,
to make it "feel" faster.  This equation was invented by
Will Woods.

Making progress linear with boot up is also useful though.

This commit makes it configurable.

[main] Save progress cache file earlier

The cache file record milestones during boot, so we should
write it out:

1) only after the root filesystem is mounted
2) as soon as we're no longer going to get updates about boot
   progress.

[boot-splash] Force progress to 1.0 when quitting

This potentially allows themes to move their progress
bars to the end before quiting.

[throbber] Change frame sequencer function

The throbber has a function for determining which frame
to show based on the current time.  This function was:

  ƒrame(t) = number_of_frames ∙ (⅟₂ sin(t) + ⅟₂)

Which basically oscillates between 0 and number_of_frames - 1,
over and over again.  There are two problems with this function;
  - after it runs through all the frames in order, it then procedes
    to run through them backward.
  - This function also starts in the middle of the set of throbber
    frames.

These problems don't matter for spinfinity, but will look wrong for most
other themes.

The new function is this:

  ƒrame(t) = number_of_frames ∙ (1⁄duration)(t mod duration)

This function solves both problems.  At time 0 it uses frame 0, and
after the last frame it jumps back to the first frame.

[throbber] set is_stopped after stopping

The throbber variable has an is_stopped state
variable that decides whether or not draws happen.

It starts off false, and gets toggled when started.

Unfortunately, it never gets reset to false
after be stopped.  This commit fixes that.

Handle NULL boot_buffer

boot_buffer may be NULL if plymouthd failed to attach to the console
session (e.g. if booting without an initramfs so /dev/pts isn't mounted
yet).  Handle this gracefully rather than segfaulting.

Handle plymouth:splash or plymouth:debug as last argument

If plymouth:splash or plymouth:debug is the last argument in
/proc/cmdline, then it will be terminated with '\n' rather than ' '.
Handle this.

Fix parsing of plymouth:debug=file:

Short-circuit evaluation meant that the path variable was never set if
plymouth:debug=file: was used.  Switch the ordering around to fix this.

[terminal] wait for vt switching to finish on quit

After changing the active vt during start up,
we get a signal when it finishes, because we're in
VT_PROCESS mode, so we don't need to explicitly block
waiting for it to finish.

During the quit path, though, we aren't in VT_PROCESS mode anymore,
won't get any signals, and still need to know that the VT switch
is finished before informing the client that the quit is
finished.

This commit forces ply_terminal_deactivate_vt to block until the
VT switch it initiates finishes.

[terminal] Add some debug spew

[splash] export config and policy directories

Export configuration and policy directories used by plymouth.

[main] do not set state->keyboard twice

state->keyboard is set already in set_keyboard, there is no need to do
the work twice.

[script] Remove debugging trace line

[script] Allow passing of values from the .plymouth file to the script

An example in the script.plymouth shows a value being set. These values are
then present at the first execution of the script set in the global scope.

[key-file] Add foreach function to access all stored values

Supplying a callback function, you can get each group/key/value triple. Allows
accessing values in the key file for which you may not know the keys.

[script] Default to text color alpha of 1 if nothing was passed

If NULL or no variable was passed, the aplha variable would be read as NaN
which gets clamped to 0 so the text would be completely transparent.

[main] Create a list of messages and when showing splash display them

The system now keeps a list of messages so they can be shown again when the
user flips themes or if the messages were passed before the splash was shown.

This also enables the splashes to show multiple messages (should they choose to
do so). There will later be a way of undisplaying a message.

[progress-animation] Typo bug in calculating max height

Was looking at the width rather than the height of the second image

[progress-animation] Remove unnecessary draw area

Must have been left over from some debugging

[progress-animation] Remove some now unnecessary area calculations

[progress-animation] Convert more functions to use buffer fill with buffer

[image] Give get buffer function more sensible name

There was a "_convert_" in the middle of the function name from a clumsy copy
paste.

[progress-animation] Use buffer fill with buffer instead of with raw data

[entry] Allow setting of text color

Default color is black.

[entry] Fix ask question entry text

The text was not being shown because show was not called. Also the default
colour is white which was not showing up on white background.

[entry] Convert from drawing image raw data to using pixel buffers

[image] Add asserts to make sure NULL is not passed as the image

[image] Allow access to the internal image buffer

Adds ply_image_convert_get_buffer to get the image buffer without destroying
the image. Forgot to add the .c.

[image] Allow access to the internal image buffer

Adds ply_image_convert_get_buffer to get the image buffer without destroying
the image.

[script] Simplify sprite area calculation

The sprite area no longer needs to be calculated as the buffer holds that
information.

[script] Use fill with buffer function

[animation] Use fill with buffer function

[throbber] Use fill with buffer function

[pixel-buffer] Add fill buffer with buffer functions

These draw the content of one buffer into another. This is cleaner than using
the raw data and passing the width and height of the data separately.

[animation] Move frames from image to pixel-buffers

The frames are now stored as pixel-buffers which will be cleaner to draw from
in the future.

[throbber] Move frames from image to pixel-buffers

The frames are now stored as pixel-buffers which will be cleaner to draw from
in the future.

[pixel-buffer] Add get width and height functions

These were accessible by the get_size function, but individual functions are
more convenient.

[main] Ignore keyboard changes when deactivating

There is a small window after plymouth is told
to quit or deactivate before it actually does.

During this window, if the user happens to hit
escape, bad things could happen.

We really don't want to mess with, e.g., the tty
settings when we're about to exit.

This commit puts plymouth in a sort of "degraded"
mode while it's deactivating.  During this time
frame, user input is ignored.

[terminal] Lock terminal settings

From time to time, various external programs
will muck with the tty we're using and make
the users password for encrypted disks show
up, make the enter key not work, etc.

We used to work around this by resetting the
tty the way we like it everytime we write the
screen.

We no longer do that after commit

e9a22723da7c9400d25aeff2625651b3d03be43f

Instead of changing it every time, it's probably
better to just prevent other programs from messing
up the settings in the first place.

This commit locks the terminal so if those programs
try to change the settings, they fail.

A better long term solution might be to get user input
/dev/input instead of the tty

[pixel-buffer] Remove offsets from buffet fill with data functions

The x and y offsets were supposed to allow only a part of the image data to be
drawn, but this didn't work correctly. Instead the clip region can be used to
emulate the same effect. So the x and y paramiters in these functions are now
dropped. The only instance that used these now uses a clip. All other calls had
x and y set to zero.

[event-loop] Add reference count to event sources

This prevents the event loop from freeing sources
early in an iteration of the loop, and then dispatching
handlers for the source later in that same iteration.

[drm] Drop drm/ from include directives

One file was missing the fix.

[configure] bump to 0.8.1

[boot-splash] Remove displays before unloading plugin

If we don't do this, then the splash display handler will
still be set, causing all sorts of undefined and weird
behavior.

This makes hitting the escape key multiple times act more
reliable.

[main] Use updated terminal attrs on crash

In commit ea394383c54dda771dcf59e7ec11280fa945280b I changed
the terminal settings used by ply-terminal.  This commit puts
those same settings into effect in the crash path.

[main] add more debugging messages

[configure] Add some heuristics to find drm headers

On my system, some headers seem to be stuffed in /usr/include/libdrm
and other headers seem to be stuffed in /usr/include/drm .

I think the ultimate upstream goal is for everything to be in
/usr/include/libdrm but my system seems to be in some transition state.

My pkgconfig files only point me to one of the directories, so add some
heuristics to find the other one.

[client] Allow unlimited unlock attempts

The ask-for-password command has an option
to specify the maximum number of retries to
get the correct password.  The default number
of retries was 3.  This makes sense for /opt,
and to a lesser degree /home, but it doesn't
make sense at all for /, since failing means
the system won't boot.

The new default is "unlimited".  This fixes the
/ case, but has the downside that all integration
code that was depending on the 3 default before
will need to get updated.

[script] Reference this if function is called with current object as this

If calling a function which is connected to the current object, the object
operated on in the function call is the current object. This is rarely used and
somehow survived without causing crashes despite incorrectly not increasing the
refcount.

[drm] Conditionalize fb fallback with 1-head ttm

When using the old style transition code path where
plymouth quits before X starts, we can't use the drm
renderer for single head nouveau and readeon setups
because their kernel memory manager doesn't allow userspace
write access to the console fb and the old style
transition code needs to dump the last frame of the
plymouth animation to the console fb as intermediate step
of the transition process.

The newer plymouth deactivate mechanism doesn't need the
intermediate console fb copy, so we can stick with the
drm renderer all the time.

This commit conditionalizes the "bail to frame-buffer renderer"
check based on whether the build is configured to enable
the old style transition.

[script] Only attach to the keyboard if it has been set

Don't add an input handler to a NULL keyboard.

[script] Connect and disconnect the keyboard on start and stop

Previously we were connecting to the keyboard but not disconnecting from it as
unset_keyboard was not called before the plugin being destroyed. Now we connect
when on splash start and destroy which makes sure we disconnect before being
destroyed.

[drm] Drop drm/ from include directives

They've moved to a different directly with the latest
libdrm, and we should be relying on pkgconfig anyway.

[terminal] Do not discard the input when turning the terminal unbuffered

Drawing to renderers was resetting the terminal to unbuffered, but this was
also discarding any queued input keys. The fix should keep the input buffer
untouched.

[frame-buffer] Initialize backend->head.map_address

It's possible for activate to be called before map_to_device, for
example if a non-graphical splash plugin is in use.  Initialize
map_address to MAP_FAILED so that we won't try to redraw in this
situation.

[main] Watch keyboard input after the splash had a chance to quit

If the splash hasn't picked up any displays it will abort. We wait for the
possible abort before attaching the keyboard. This is a slight hack as now
add_displays_and_keyboard_to_boot_splash no longer adds the keyboards. Also the
splash plugin can access the keyboard before it is officially opened.

For now it just fixes the terminal fd < 0 assertion bug.

[main] watch keyboard input after displays are added

Since the boot splashes handle opening the terminal now,
we need to make sure that we don't try to watch for input on
the terminal until they've opened it.

This should fix some crashes users are reportedly seeing with
the details splash.

A potentially better fix would be to make watch_for_input defer
doing anything on its own using the terminal is opened.  That
would require a trigger and some new api to ply-terminal, I think.