Commit 4524051c authored by Gerd Hoffmann's avatar Gerd Hoffmann Committed by Anthony Liguori
Browse files

Add search path support for qemu data files. 



This patch allows to specify multiple directories where qemu should look
for data files.  To implement that the behavior of the -L switch is
slightly different now:  Instead of replacing the data directory the
path specified will be appended to the data directory list.  So when
specifiying -L multiple times all directories specified will be checked,
in the order they are specified on the command line, instead of just the
last one.

Additionally the default paths are always appended to the directory
data list.  This allows to specify a incomplete directory (such as the
seabios out/ directory) via -L.  Anything not found there will be loaded
from the default paths, so you don't have to create a symlink farm for
all the rom blobs.

For trouble-shooting a tracepoint has been added, logging which blob
has been loaded from which location.

Signed-off-by: default avatarGerd Hoffmann <kraxel@redhat.com>
Message-id: 1362739344-8068-1-git-send-email-kraxel@redhat.com
Signed-off-by: default avatarAnthony Liguori <aliguori@us.ibm.com>
parent 1272ec88

trace-events

+1 −0
Original line number Diff line number Diff line
@@ -473,6 +473,7 @@ scsi_request_sense(int target, int lun, int tag) "target %d lun %d tag %d" 


# vl.c

vm_state_notify(int running, int reason) "running %d reason %d"

load_file(const char *name, const char *path) "name %s location %s"



# block/qcow2.c

qcow2_writev_start_req(void *co, int64_t sector, int nb_sectors) "co %p sector %" PRIx64 " nb_sectors %d"

vl.c

+23 −13
Original line number Diff line number Diff line
@@ -179,7 +179,8 @@ int main(int argc, char **argv) 
#define MAX_VIRTIO_CONSOLES 1

#define MAX_SCLP_CONSOLES 1



static const char *data_dir;

static const char *data_dir[16];

static int data_dir_idx;

const char *bios_name = NULL;

enum vga_retrace_method vga_retrace_method = VGA_RETRACE_DUMB;

DisplayType display_type = DT_DEFAULT;
@@ -2276,14 +2277,16 @@ static int balloon_parse(const char *arg) 


char *qemu_find_file(int type, const char *name)

{

    int len;

    int i;

    const char *subdir;

    char *buf;



    /* Try the name as a straight path first */

    if (access(name, R_OK) == 0) {

        trace_load_file(name, name);

        return g_strdup(name);

    }



    switch (type) {

    case QEMU_FILE_TYPE_BIOS:

        subdir = "";
@@ -2294,14 +2297,16 @@ char *qemu_find_file(int type, const char *name) 
    default:

        abort();

    }

    len = strlen(data_dir) + strlen(name) + strlen(subdir) + 2;

    buf = g_malloc0(len);

    snprintf(buf, len, "%s/%s%s", data_dir, subdir, name);

    if (access(buf, R_OK)) {



    for (i = 0; i < data_dir_idx; i++) {

        buf = g_strdup_printf("%s/%s%s", data_dir[i], subdir, name);

        if (access(buf, R_OK) == 0) {

            trace_load_file(name, buf);

            return buf;

        }

        g_free(buf);

        return NULL;

    }

    return buf;

    return NULL;

}



static int device_help_func(QemuOpts *opts, void *opaque)
@@ -3285,7 +3290,9 @@ int main(int argc, char **argv, char **envp) 
                add_device_config(DEV_GDB, optarg);

                break;

            case QEMU_OPTION_L:

                data_dir = optarg;

                if (data_dir_idx < ARRAY_SIZE(data_dir)) {

                    data_dir[data_dir_idx++] = optarg;

                }

                break;

            case QEMU_OPTION_bios:

                bios_name = optarg;
@@ -3925,12 +3932,15 @@ int main(int argc, char **argv, char **envp) 


    /* If no data_dir is specified then try to find it relative to the

       executable path.  */

    if (!data_dir) {

        data_dir = os_find_datadir(argv[0]);

    if (data_dir_idx < ARRAY_SIZE(data_dir)) {

        data_dir[data_dir_idx] = os_find_datadir(argv[0]);

        if (data_dir[data_dir_idx] != NULL) {

            data_dir_idx++;

        }

    }

    /* If all else fails use the install path specified when building. */

    if (!data_dir) {

        data_dir = CONFIG_QEMU_DATADIR;

    if (data_dir_idx < ARRAY_SIZE(data_dir)) {

        data_dir[data_dir_idx++] = CONFIG_QEMU_DATADIR;

    }



    /*