parsers: change parser.parse to accept a buffer

Rather than having each of the parsers do their own open(), read(), etc,
use the registered filenames array to find & open parser conf files.

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

parsers: Add filenames to struct parser

In preparation of moving file handling to the discover core (rather than
the parsers), include the conf file names in struct parser.

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

lib/url: Add pb_join_url

Add a a function to join a string to a base URL

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

device-handler: Don't unmount non-mounted devices

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

test/lib: Hook into `make check`

This change hooks the new list tests into 'make check'. To do this, we
need to fix the return code of the list-test program.

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

discover: Fix potentially-unused var warning

discover/boot.c: In function ‘boot’:
discover/boot.c:153:13: error: ‘local_initrd’ may be used uninitialized in this function [-Werror=maybe-uninitialized]

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

event: Make strings const

The processing of events does not modify the event strings, so change the
strings in struct event to const (read-only).

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

test: Rename test/list to test/lib

Having a test/list directory is a little too specific, so rename
the test/list directory to test/lib as a place for any lib tests.

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

list: Fix seg fault with list_for_each_entry_safe

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

test: Add list test

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

discover: Add missing udev_destroy call

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

discover: Separate temporary and permanent device data

At present, we keep both permanent (eg links/n_links) and temporary
(event) data in struct discover_context.

This change makes discover_context a temporary structure, just used
during actual device discovery. Once discovery is complete, the
permanent data (discover_device) is "committed" to the device handler.

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

lib/list: Add list_for_each_entry_safe

Add a list iterator that is safe against deletion of the current
element.

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

discover: Consolidate user events by device ID

Currently, we assume all user events are for a new device. This means
that we can never add boot options to an existing device.

This change tries to find an existing (matching by ID) device before
creating a new one in the user event add path.

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

protocol: Separate device add from boot-option add messages

We want to cater for situations where boot options may be discovered
some time after we get notificiation about devices. For instance,
discovering boot options from DHCP configuration parameters. In this
case, we'll need to notify UIs of boot options appear some time after
the device (and/or other boot options on the same device) has appeared.

This change adds a new protocol message type,
PB_PROTOCOL_ACTION_BOOT_OPTION_ADD. We also rename
PB_PROTOCOL_ACTION_ADD to make it clear that it is just for devices.

The discover server is updated to send boot option add events at device
discover time, but we are now able to decouple this later.

We also update the clients to handle the boot option add events
separately.

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

pb-protocol: Don't allocate in deserialise functions

Curently, the protocol deserialise functions are allocating device and
boot_command structures. This (implicitly) makes them responsible for
initialisation of these structures too.

Rather that making the protocol responsible for initialising the devices
and boot commands, this change gives the deserialise functions an
argument to an already-instanciated structure. This means that the
creation is no longer implied by the deserialise.

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

ncurses: Fix build warning

Add missing header include.  Fixes this warning:

  ui/ncurses/nc-cui.c: warning: implicit declaration of function 'pb_protocol_device_cmp'

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

discover: Fix boot with initrd error

Fix typo in boot() routine.

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

Move --dry-run option to discover server

Now that the server does the booting, we should move the --dry-run
argument to the server.

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

Move boot to discover server

This change moves the boot-via-kexec functionality from the UIs to the
discover server.

On the UI side: rather than run kexec directly, we just send a message
to the discover server. Because this is generic discover client
functionality, we no longer need the boot callbacks in the twin- and
ncurses-specific code.

We also remove the kexec and URL-loading code from the UIs, and add it
to the discover server code, in paths.c. We expose this to the server
though a new function:

  load_path(void *, const char *, unsigned int *);

On the server side, we simply move hook up the boot() function to use
the load_file and kexec calls.

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

discover: Implement device handler boot path

This change adds a funtion, device_handler_boot, which processes the
boot command message from the discover server.

We add a new file, discover/boot.c (and a corresponding header) with a
skeleton for the final kexec code.

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

discover: parse boot message from incoming ACTION_BOOT messages

Add a function in the protocol code to deserialise a boot message, and
use it to extract a boot_command in the discover server.

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

discover-server: Add reference to server in struct client

We'll need to reference the server when handling messages from clients.

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

lib/url: Move URL-handling code to lib

We'll need to use the URL handling code in the server, so move it to the
lib/ directory.

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

ui/ncurses: Add discover client to struct cui

.. we'll need it later.

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

discover-client: interact directly with waitset

Currently, clients need to mess with the discover client fd directly,
and manually register the waiter.

Instead, this change adds a waitset parameter to
discover_client_register, so that the discover client can register
itself, and call discover_client_process directly. This means no proxy
handlers, and no casts to waiter callbacks.

We can also get rid of discover_client_get_fd.

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

discover-client: Add discover_client_boot

This change implements the client side of the server-based boot
interface. We add a funcion, discover_client_boot, which serialises a
boot message, then sends it to the server.

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

discover: Allow discover server to receive boot messages

Currently, the petitboot socket is one way: messages are only sent from
server to client.

Beause we want the clients to trigger a boot, this change allows the
server to receive messages on the petitboot socket. This is just a
matter of adding a waiter to the client-specific socket, then handling
incoming messages.

At present we don't do anything with the messages, but we'll add a
handler later.

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

Add boot command structure to petitboot protocol description

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

ui: callback & boot actions: kexec -> boot

find ui/ -type f |
xargs sed -i -e s/kexec_cb/boot_cb/g \
-e s/on_kexec/on_boot/g \
-e s/run_kexec/boot/g

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

ui: pb_kexec_data -> pb_boot_data

find ui/ -type f |
xargs sed -i -e s/pb_kexec_data/pb_boot_data/g \
-e 's/\<kd\>/bd/g'

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

ui/ncurses: ked -> boot-editor

git mv ui/ncurses/nc-ked.c ui/ncurses/nc-boot-editor.c
git mv ui/ncurses/nc-ked.h ui/ncurses/nc-boot-editor.h
find ui/ncurses -type f |
xargs sed -i -e s/nc-ked\./nc-boot-editor./g \
             -e s/ked/boot_editor/g

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

pb-udhcpc: Fix generation of TFTP URLs

DHCP boot-file parameters may not contain a preceeding slash. In this
case, the current udhcp script will not form a correct URL.

This change adds the slash unconditionally; in the case that the
pathname already has a preceeding slash, the URL parser will handle this
correctly.

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

ui/common: Remove unused artwork

A few of the oxygen icons are no longer used, so remove them from the
artwork tree.

Coincidentally, this fixes and issue with `make dist`, as the
drive-removable-media-usb-pendrive.png file is too long (when used with
an autogenerated git version string) to be tar-ed up.

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

test/urls: Fix distcheck breakages

We need a couple of extra automake variables to get `make distcheck`
working.

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

waiter: Don't rely on global variables to keep waiter state

Rather than defining the set of waiters (and pollfds) in waiter.c, add a
struct waitset to contain these.  A waitset is created with
waitset_create, which is passed to the waiter_* functions.

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

lib/types: Create common file for type definitions

The device and boot_option types are defined in pb-protocol.h, but
aren't really specific to the procotol. This means a lot of
non-messaging-related files are #including the protocol definitions
unnecessarily.

This change separates the types out into lib/types/types.h.

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

test/url: Add http-simple.test

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

ui/common/url: Fix multiple-preceeding-slash on pathnames

Fix the double-slash URL test.

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

test: Add URL parser test infrastructure

This change adds some simple testing infrastrcture to the URL parser.
We use a small C binary (parse-url) to run the url parser on its
argument, and compare the output with an expected datafile.

An initial test is included, to check the behaviour of URLs with
multiple slashes between host and pathname. This test currently fails.

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

ui/common/url: Add pb_url_scheme_name

Add a function to lookup the name of a scheme, for more useful URL
output.

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

ui/common/url: remove scheme separator from URL scheme definitions

To make the scheme definitions more useful for other functions.

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

ui/common/url: move pb_scheme_info array to file scope

We'd like to use the scheme info array for other functions, so move it
to file scope, and rename to schemes.

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

Allow host programs to be configured

Rather than hard-coding in lib/system/system.c, this change adds a set
of #defines for host programs, through the autoheader config.h

These host programs can then be set through configure:

 ./configure HOST_PROG_MOUNT=/usr/bin/mount

Because we need to define nine host programs, we add an autoconf macro
to avoid repeating the definition code.

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

discover: Remove {U,}MOUNT_BIN defines

Use pb_system_apps instead.

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

Add udevadm to pb_system_apps

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

discover: Use pb_run_cmd rather than calling system()

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

configure.ac: Add --enable-werror

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

discover: remove (unused) duplicate struct mount_map

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

discover: remove unused message.h file

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

parser: remove unused fields from struct parser

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

ui/twin: fix compiler warnings

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

lib/pb-protocol: suppress warning for unused buf_len arg

When assert is disabled, we end up with an warning for buf_len.

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

ui/ncurses: Fix warning from unnecessary if statement

 ui/ncurses/nc-menu.c: In function 'pmenu_process_key':
 ui/ncurses/nc-menu.c:207:33: error: suggest braces around empty body in an 'if' statement [-Werror=empty-body]

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

discover: simplify handler switch statement

Instead of using nested switch statements in device_handler_event,
define an array of handlers, and lookup instead.

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

nc: Only expand debug macros if DEBUG is defined

To reduce debugging output from the UI

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

Convert echo to printf in scripts

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

Fix sftp loader

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

Increase X screen size

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

Speed up --start-daemon option

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

Cleanup --dry-run option code

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

Add missing EXTRA_DIST files

Add UI option --dry-run

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

Add new macro DEFINE_LIST()

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

pb-discover: Ignore ram, loop and no-name devices

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

Fix device mount retry

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

Add more paths to grub2 parser

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

Log to stderr, allow --log=-

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

Zero automatic sockaddr

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

Add grub.cfg from opensuse install

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

Add DEVPATH to udev_print_event()

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

Add grub2 conf file parser

Add grub2 parser and sample config file.
Reorder parser priorities:
  From (yaboot -> kboot) to (kboot -> grub2 -> yaboot).

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

Parser test cleanup

Major cleanup of parser tests.

 * Change test data directory layout to ease test automation.
 * Remove some unneeded files.
 * Re-write test wraper script, rename to run-parser-tests.
 * Rework for autotools.

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

Minor comment fix

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

Add new member get_pair to conf parser context

To allow parsers more control over finding paramerter pairs
add a new member get_pair to the parser struct conf_context.
Initialize get_pair=conf_get_pair_equal for existing parsers.

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

Refactor parser routine conf_get_param_pair()

Refactor conf_get_param_pair() and rename to conf_get_pair().
Add two new wrapper routines conf_get_pair_equal() and
conf_get_pair_space().

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

Add parser routine conf_replace_char()

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

Convert test to automake

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

Fix build error in parser-test

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

Use static array for parsers

Change the parser structure array implementation from using an
array in a seperate parsers section to a static array of pointers
in parser.c.  Parser priority is now set by the position in the
new parsers array.

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

Fix seg fault with bare kboot.conf variables

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

Use define_parser() for native

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

Add logrotate.conf

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

Add time to version string

To comply with Debian's package version rules.

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

Fixup artwork licenses

Replace icon files with files from KDE 4.3 Oxygen.  Add artwork
license info.

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

Add --start-daemon option to ui programs

Add the option --start-daemon to automatically start
pb-discover if it is not already started.  For use
when running as a stand-alone app.

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

Log file cleanups

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

Rename pb-nc, pb-twin to petitboot-nc, petitboot-twin

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

Retry device mount without 'ro' option

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

Add '--action=add' to udevadm trigger command

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

Move installed programs from bin to sbin

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

Cleanup man pages

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

Update twin ui to use discover server

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

Remove union in pb_opt_data

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

Minor include path fix

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

Add usage comment to parse_user_event

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

Rename pb-nc-generic to pb-nc

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

Remove unused file pb-cui.c

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

Fixups for 'make dist'

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

Add artwork files network.png, system.png

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