discover/event: Fix incorrect calculation of param data length

We're not accounting for the action in the event header (only the device
string) when we pass the length of param data to event_parse_params.
This means we walk past the end of the event data while parsing params.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

utils: Add busybox reboot script

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/boot: Add boot hooks

Add a method of running pre-boot hooks. Executable files in
/etc/petitboot/boot.d/ are run (in order) before we start the boot
process.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/boot: Put all boot params into a struct boot_task

This keeps all the boot-specific details in one place.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Add pkgsysconfdir

We'll need a sysconf dir to store the boot hooks.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Add initial dtb support

Updates & fixes by Jeremy Kerr <jk@ozlabs.org>.

Signed-off-by: Geoff Levand <geoff@infradead.org>
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: bring network interfaces down in network_shutdown

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Add network handling

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

lib: Add pb-config module

Add a library for (name, value) configuration.

Different storage backends are allowed (although currently hardcoded to
powerpc nvram), and config is read-only at present.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Add logging for default boot option behaviour

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/event: Allow empty param values

Useful for specifying defaults:

 pb-event add@defaults name='Netboot' \
   image=http://192.168.0.1/vmlinuz \
  default

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/ncurses: Add cancel-default reporting

When we first see key input, we can tell the discover server to cancel
the default boot.

Untested on ps3.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/common: Add code to send cancel-default messages

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui: implement timers with waitsets

A temporary change to the timers; we'll eventually remove these from the
ui code.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Add "cancel default boot" messages

Allow the default boot process to be cancelled, via a message with
action PB_PROTOCOL_ACTION_CANCEL_DEFAULT.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/event: Allow user events to be defaults

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Add countdown before default boot

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: implement default booting

When we see a boot option with is_default set, store it in the handler
and register a timeout waiter.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/boot: Allow null boot command

If we're booting from a default option, we don't have a boot command.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

lib/waiter: Add timeout waiters

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/grub2: Add default option parsing

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/test: print default status of boot options

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

types: Add is_default to struct boot_option

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

autotools: Fix make maintainer-clean

Fixes to make maintainer-clean work properly.

Signed-off-by: Geoff Levand <geoff@infradead.org>

test/parser: Remove gawk specifics

Remove the GNU awk specifics to allow the use of a standard
awk program.  Fixes build errors on older distros.

Signed-off-by: Geoff Levand <geoff@infradead.org>

discover: Send options to client in order

Send the boot options to the client in the order discovered.  This
change  is in follow up to inverting the order that the server
saves the options internally.

Signed-off-by: Geoff Levand <geoff@infradead.org>

discover/grub2: remove uuid log message

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/test: Add boot status messages to test client

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/ncurses: implement global keys

Add process_global_keys to handle global key events. Using this,
implement ctrl+l to refresh.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/ncurses: move getch() out of process_key callbacks

All process_key callbacks will want to query the key that was pressed,
so do the getch() once in cui_process_key, and pass the result to the
callback.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

lib/waiter: Ensure waiters are consistent during waiter_poll

We have a bug at the moment: if the waitset's->waiters array is updated
duing waiter_poll() (eg, a client connection is closed, and the client's
callback performs a waiter_remove()), then we may invoke callbacks for
incorrect waiters.

This change uses a consistent waiters array duing execution of
waiter_poll, so that any pollfds returned from poll() will result in
correct callback invocations.

This assumes that a waiter will only ever remove *itself* from the
waitset; otherwise, we may call a free()ed waiter.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

lib/waiter: allocate waiters separately from set->waiters

Since we reallocate set->waiters, we can't hand out pointers to within
that array.

Instead, this change allocates the 'struct waiter's separately from the
set->waiters array.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: fix error handling from failed client writes

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Fix accept() return value check

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: destroy client if message read failed

When a client disconnects, the read from the client's fd will return
EOF. We should destroy the client in this situation.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

utils/pb-console: use getty rather than exec

exec is a bit flaky for starting on consoles, so use getty instead.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

utils/pbconsole: Use here-document for usage text

Allows for cleaner usage message in the source.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: cleanup parser allocations on exit

We should talloc_free the parser list once we're done, enabling
valgrind's leak check. Before:

 [jk@pablo parser]$ valgrind ./test-null
 ...
 ==9330== HEAP SUMMARY:
 ==9330==     in use at exit: 288 bytes in 3 blocks
 ==9330==   total heap usage: 11 allocs, 8 frees, 1,177 bytes allocated

After:

 [jk@pablo parser]$ valgrind ./test-null
 ...
 ==9940== HEAP SUMMARY:
 ==9940==     in use at exit: 0 bytes in 0 blocks
 ==9940==   total heap usage: 11 allocs, 11 frees, 1,177 bytes allocated
 ==9940==
 ==9940== All heap blocks were freed -- no leaks are possible

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

util/pb-console: Packaging updates

In preparation for packaging add a man page, a help option, and some
comments to the script.

Signed-off-by: Geoff Levand <geoff@infradead.org>

test/parser: Add check_not_present_resource

Add a new routine check_not_present_resource() to check
that a resource is not present.  This is typically used
to check that an initrd entry has not been found.

Also add any needed checks for no initrd to the existing
tests.

Signed-off-by: Geoff Levand <geoff@infradead.org>

parser: Use list to hold parsers

Now that we dynamically add parsers, we can use a list to
hold them.  Also simplifies the test_run_parser() routine.

Signed-off-by: Geoff Levand <geoff@infradead.org>

test/parser: Add ubuntu grub tests

Signed-off-by: Geoff Levand <geoff@infradead.org>

test/parser: Better message output

Signed-off-by: Geoff Levand <geoff@infradead.org>

parser/grub2: Add parse for linux16

Ubuntu uses the linux16 symbol in thier conf files for memory test entries.

Signed-off-by: Geoff Levand <geoff@infradead.org>

parser/grub2: Better menuentry parsing

Grub2 menuentry entry text can use double or single quotes.

Signed-off-by: Geoff Levand <geoff@infradead.org>

configure: Add check for libncurses

Signed-off-by: Geoff Levand <geoff@infradead.org>

test/parser: Add grub2 multiple resolution test

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/grub2: handle search commands to specify root filesystems

Add a resource type for grub, allowing us to parse search parameters.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/resource: Provide resolve_resource_against_device

Any other implementation of resources will need to resolve againsst
particular devices, so make resolve_devpath_against_device
publically-accessible, and rename to not be devpath-specific.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: log cleanup

Remove some of the more noisy log messages, and add some information
pertinent to device resolution events.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add yaboot test from a ppc64 RH8 machine

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add f18 grub config test

Add a real-world config file from a f18 ppc64 machine.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add simple GRUB2 parser test

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add simple kboot test

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add yaboot test for external devices

Add a yaboot test to add boot options for an external device, then
hotplug that device.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add simple yaboot test

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add test device hotplug functions

Add a function, test_hotplug_device(), to simulate the addition of
devices during tests. This should make it possible to observe boot
options' resources changing from unresolved state to resolved.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add resource check helpers

Add check_resolved_local_resource and check_unresolved_resource to
check the resources returned from parsers.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add check_name helper

Simple function to check a boot_option's name.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add check_args helper

Add a small helper to check boot option arguments.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Add check_boot_option_count helper & get_boot_option

Add a helper function to check the expected boot option counts, and
print the boot option details if the check fails.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Error on invalid parser name

We should error-out if we didn't find a valid parser, so it doesn't look
like we found no boot options.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Allow raw config data in test .c files

Add a little post-processing script (extract-config.awk) to allow us to
put the config data directly into the test .c files, without having to
quote the C string.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test/parser: Allow tests to provide pre-loaded config data

Some tests may want to use an existing buffer as config data, so add
test_read_config_data() to allow this.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

test: Replace parser tests with empty framework

This change is a major rework of the parser tests. Currently, the parser
tests just run the parsers on each subdir of data/, but we don't
actually check the results.

Rather than just running parsers and expecting a certain fixed output,
the new tests are arbitraty (short!) C programs. This allows us to set
up the device configuration / discover context etc in whatever manner
necessary, and check that the boot options that the parsers emit are
correct.

This change removes all of the old tests, and adds a new "null" test.
Actual tests will be added in subsequent changes.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Make device handler a little more testable

This change moves some of the device-handler code into an #ifdef-ed
section, so we can easily drop the stuff that's not required for
testing.

Although the change is quite large, most of it is moving entire
functions around.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

lib/url: Add pb_url_to_string

Add a function to unparse a URL, to make comparisons in tests easy.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Allow devices with no device_path

Devices that have been added via the user path may not have a device
path. In this case, don't segfault in device_match_path, and break out
of mount_device early.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Always associate resources with a boot option

We should always be tallocing resources to a boot option context;
anything else (for example, the discover context) may have a different
lifetime.

In order to enforce this, we change the void *ctx argument to the
context_create functions to a struct discover_boot_option.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Don't access bytes before the start of empty strings

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Always add a NUL byte to config data

conf_get_pair will read one-byte past the end of the conf buffer, so
always NUL-terminate.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/yaboot: Fix boot option state

We're only picking up every second boot option.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/udev: use devname (not devpath) for device IDs

devnames are unique, and much shorter.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/grub2: Add grub2/grub.cfg config file

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

utils/Makefile: pb-console is dist_

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/nc: Don't segfault if we can't open log_file

Fallback to logging to /dev/null.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/nc: Handle cui_opt_data with no boot option more gracefully

We can currently segfault petitboot by escaping from the option editor
(before entering any details), then trying to boot the new, empty
option.

This change adds some sanity checks to prevent a segfault.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

ui/ncurses: Simplify menu item names

Just use the item name, rather than including boot option details.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

utils: Add pb-console util

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

pb-udhcpc: Look for more boot options

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

utils/pb-udhcpc: Use command-line format of pb-event

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

pb-event: move send code to a separate function

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

utils/pb-event: handle event data on command line

It'd be nice to avoid using printf just for the \0 characters, so allow
parsing events from the command line.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: handle boot_option->device_id entirely within handler

No need for parsers to populate (or forget to populate, in the case of
most parsers) opt->device_id, as we should do it on finalise.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/udev: Setup event params from udev properties

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/event: Add event_set_param

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/udev: Don't print properties of skipped devices

This cleans up the log output a little.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/grub2: Rework config state

Rather than creating boot options pre-emptively, in two paths, just do
it once when we see the menuentry option.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Don't invert option discovery order

Keep options in the order that we discovered them in; this makes testing
a little easier, as the options appear in the list in the same order
as the config file.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: Don't resend all options to all clients

Currently, when a new UI client connects, we send all boot options to
all clients. This results in existing clients getting duplicate add
events.

Instead, we only want to send existing boot options to the new client.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/event-parser: correctly populate boot option resources

The user event parser should be populating the discover_boot_option's
resources, not the strings in struct boot_option.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/resource.h: Add forward declarations

We need to define a few structs for the params of resource functions.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

Makefiles: remove -I$(includedir)

Currently, we include the system include dir in some of our makefiles;
this is causing build problems when cross-compiling, as the system
include dir may not contain files for the host.

The compiler should be searing in the proper system include dir, so just
remove the redundant -I.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

configure: Remove -Winline

I'm getting the following errors when compiling with --enable-werror:

 kboot-parser.c: In function 'kboot_process_pair':
 ./parser-conf.h:50:21: error: inlining failed in call to 'conf_get_pair_equal':
 call is unlikely and code size would grow [-Werror=inline]
 kboot-parser.c:65:7: error: called from here [-Werror=inline]

Since this is a header function, the inline isn't there for optimisation
reasons; the warning is just informing us of a code optimisation
decision. Drop -Winline to allow -Werror builds.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/user-event: better event debugging output

We don't currently handle conf events very well:

user_event remove event:
device: /net/eth0
name         => (null)
image        => (null)
args         => (null)

This change prints the event action properly, and prints all event
params.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/kboot: Fix boot_option initialisation

We're not populating d_opt->option, so are returning invalid boot
options from the kboot parser.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

lib/url: fix no-scheme URL parsing

We were incorrectly dropping the first strlen("file://") characters from
URLs with no scheme:

--- test/urls/data/localpath.test 2013-05-02 17:26:48.826359036 +0800
+++ /tmp/tmp.gn4JsWLw5o 2013-05-02 17:26:50.262364613 +0800
@@ -2,6 +2,6 @@
 scheme file
 host (null)
 port (null)
-path /test/path/to/local/file
-dir /test/path/to/local/
+path ath/to/local/file
+dir ath/to/local/
 file file

This change fixes the issue by indicating "no scheme found" by a NULL
return from pb_url_find_scheme, and hadling it appropriately. We add a
testcase too.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: log unresolved boot options

These are a bit of a special-case ("why isn't my boot option
appearing?"), so add a log message.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover: kexec harder

`kexec -e` will just call shutdown, which we've already tried, so it's
likely to fail. Add a further fallback to force a kexec with -e -f
options.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

discover/user-event: handle initrds

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>