On Wed, Feb 02, 2011 at 04:08:07PM +0100, Markus Armbruster wrote: > Gleb Natapov <gleb@xxxxxxxxxx> 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 <gleb@xxxxxxxxxx> > > 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. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html