[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv8 12/16] Add bootindex parameter to net/block/fd
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCHv8 12/16] Add bootindex parameter to net/block/fd device |
Date: |
Wed, 02 Feb 2011 16:08:07 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Gleb Natapov <address@hidden> writes:
> If bootindex is specified on command line a string that describes device
> in firmware readable way is added into sorted list. Later this list will
> be passed into firmware to control boot order.
>
> Signed-off-by: Gleb Natapov <address@hidden>
Just noticed something that slipped through review:
[...]
> diff --git a/vl.c b/vl.c
> index 2cd263e..dadc161 100644
> --- a/vl.c
> +++ b/vl.c
[...]
> @@ -693,6 +704,35 @@ static void restore_boot_devices(void *opaque)
> qemu_free(standard_boot_devices);
> }
>
> +void add_boot_device_path(int32_t bootindex, DeviceState *dev,
> + const char *suffix)
> +{
> + FWBootEntry *node, *i;
> +
> + if (bootindex < 0) {
> + return;
> + }
> +
> + assert(dev != NULL || suffix != NULL);
> +
> + node = qemu_mallocz(sizeof(FWBootEntry));
> + node->bootindex = bootindex;
> + node->suffix = strdup(suffix);
qemu_strdup()?
> + node->dev = dev;
> +
> + QTAILQ_FOREACH(i, &fw_boot_order, link) {
> + if (i->bootindex == bootindex) {
> + fprintf(stderr, "Two devices with same boot index %d\n",
> bootindex);
> + exit(1);
> + } else if (i->bootindex < bootindex) {
> + continue;
> + }
> + QTAILQ_INSERT_BEFORE(i, node, link);
> + return;
> + }
> + QTAILQ_INSERT_TAIL(&fw_boot_order, node, link);
> +}
> +
> static void numa_add(const char *optarg)
> {
> char option[128];
- Re: [Qemu-devel] [PATCHv8 12/16] Add bootindex parameter to net/block/fd device,
Markus Armbruster <=