[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: |
Gleb Natapov |
Subject: |
Re: [Qemu-devel] [PATCHv8 12/16] Add bootindex parameter to net/block/fd device |
Date: |
Wed, 2 Feb 2011 17:10:04 +0200 |
On Wed, Feb 02, 2011 at 04:08:07PM +0100, Markus Armbruster wrote:
> 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()?
>
Yap. Will fix.
> > + 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];
--
Gleb.