From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:33120) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TEXEk-000608-B4 for qemu-devel@nongnu.org; Wed, 19 Sep 2012 23:15:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TEXEi-0004ss-Px for qemu-devel@nongnu.org; Wed, 19 Sep 2012 23:15:34 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36687) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TEXEi-0004rv-H4 for qemu-devel@nongnu.org; Wed, 19 Sep 2012 23:15:32 -0400 Message-ID: <505A8A4E.9080402@redhat.com> Date: Thu, 20 Sep 2012 11:15:26 +0800 From: Amos Kong MIME-Version: 1.0 References: <50482351.6050302@redhat.com> <1346987463-13666-1-git-send-email-akong@redhat.com> In-Reply-To: <1346987463-13666-1-git-send-email-akong@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2] add a boot parameter to set reboot timeout List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Amos Kong Cc: aliguori@us.ibm.com, seabios@seabios.org, qemu-devel@nongnu.org, gleb@redhat.com, kevin@koconnor.net On 07/09/12 11:11, Amos Kong wrote: > Added an option to let qemu transfer a configuration file to bios, > "etc/boot-fail-wait", which could be specified by command > -boot reboot-timeout=T > T have a max value of 0xffff, unit is ms. > > With this option, guest will wait for a given time if not find > bootabled device, then reboot. If reboot-timeout is '-1', guest > will not reboot, qemu passes '-1' to bios by default. > > This feature need the new seabios's support. > > Seabios pulls the value from the fwcfg "file" interface, this > interface is used because SeaBIOS needs a reliable way of > obtaining a name, value size, and value. It in no way requires > that there be a real file on the user's host machine. > > Signed-off-by: Amos Kong > --- > v2: qemu passes '-1' to bios, guest will not reboot Gleb, Anthony, any comments? > --- > hw/fw_cfg.c | 25 +++++++++++++++++++++++++ > qemu-config.c | 3 +++ > qemu-options.hx | 12 +++++++++--- > vl.c | 3 ++- > 4 files changed, 39 insertions(+), 4 deletions(-) > > diff --git a/hw/fw_cfg.c b/hw/fw_cfg.c > index 7b3b576..dcde1a9 100644 > --- a/hw/fw_cfg.c > +++ b/hw/fw_cfg.c > @@ -183,6 +183,30 @@ static void fw_cfg_bootsplash(FWCfgState *s) > } > } > > +static void fw_cfg_reboot(FWCfgState *s) > +{ > + int reboot_timeout = -1; > + char *p; > + const char *temp; > + > + /* get user configuration */ > + QemuOptsList *plist = qemu_find_opts("boot-opts"); > + QemuOpts *opts = QTAILQ_FIRST(&plist->head); > + if (opts != NULL) { > + temp = qemu_opt_get(opts, "reboot-timeout"); > + if (temp != NULL) { > + p = (char *)temp; > + reboot_timeout = strtol(p, (char **)&p, 10); > + } > + } > + /* validate the input */ > + if (reboot_timeout > 0xffff) { > + error_report("reboot timeout is larger than 65535, force it to 65535."); > + reboot_timeout = 0xffff; > + } > + fw_cfg_add_file(s, "etc/boot-fail-wait", g_memdup(&reboot_timeout, 4), 4); > +} > + > static void fw_cfg_write(FWCfgState *s, uint8_t value) > { > int arch = !!(s->cur_entry & FW_CFG_ARCH_LOCAL); > @@ -497,6 +521,7 @@ FWCfgState *fw_cfg_init(uint32_t ctl_port, uint32_t data_port, > fw_cfg_add_i16(s, FW_CFG_MAX_CPUS, (uint16_t)max_cpus); > fw_cfg_add_i16(s, FW_CFG_BOOT_MENU, (uint16_t)boot_menu); > fw_cfg_bootsplash(s); > + fw_cfg_reboot(s); > > s->machine_ready.notify = fw_cfg_machine_ready; > qemu_add_machine_init_done_notifier(&s->machine_ready); > diff --git a/qemu-config.c b/qemu-config.c > index c05ffbc..b9f9e0f 100644 > --- a/qemu-config.c > +++ b/qemu-config.c > @@ -638,6 +638,9 @@ QemuOptsList qemu_boot_opts = { > }, { > .name = "splash-time", > .type = QEMU_OPT_STRING, > + }, { > + .name = "reboot-timeout", > + .type = QEMU_OPT_STRING, > }, > { /*End of list */ } > }, > diff --git a/qemu-options.hx b/qemu-options.hx > index 3c411c4..0249a60 100644 > --- a/qemu-options.hx > +++ b/qemu-options.hx > @@ -339,13 +339,14 @@ ETEXI > > DEF("boot", HAS_ARG, QEMU_OPTION_boot, > "-boot [order=drives][,once=drives][,menu=on|off]\n" > - " [,splash=sp_name][,splash-time=sp_time]\n" > + " [,splash=sp_name][,splash-time=sp_time][,reboot-timeout=rb_time]\n" > " 'drives': floppy (a), hard disk (c), CD-ROM (d), network (n)\n" > " 'sp_name': the file's name that would be passed to bios as logo picture, if menu=on\n" > - " 'sp_time': the period that splash picture last if menu=on, unit is ms\n", > + " 'sp_time': the period that splash picture last if menu=on, unit is ms\n" > + " 'rb_timeout': the timeout before guest reboot when boot failed, unit is ms\n", > QEMU_ARCH_ALL) > STEXI > -@item -boot [order=@var{drives}][,once=@var{drives}][,menu=on|off][,splash=@var{sp_name}][,splash-time=@var{sp_time}] > +@item -boot [order=@var{drives}][,once=@var{drives}][,menu=on|off][,splash=@var{sp_name}][,splash-time=@var{sp_time}][,reboot-timeout=@var{rb_timeout}] > @findex -boot > Specify boot order @var{drives} as a string of drive letters. Valid > drive letters depend on the target achitecture. The x86 PC uses: a, b > @@ -364,6 +365,11 @@ limitation: The splash file could be a jpeg file or a BMP file in 24 BPP > format(true color). The resolution should be supported by the SVGA mode, so > the recommended is 320x240, 640x480, 800x640. > > +A timeout could be passed to bios, guest will pause for @var{rb_timeout} ms > +when boot failed, then reboot. If @var{rb_timeout} is '-1', guest will not > +reboot, qemu passes '-1' to bios by default. Currently Seabios for X86 > +system support it. > + > @example > # try to boot from network first, then from hard disk > qemu-system-i386 -boot order=nc > diff --git a/vl.c b/vl.c > index 7c577fa..1bd9931 100644 > --- a/vl.c > +++ b/vl.c > @@ -2622,7 +2622,8 @@ int main(int argc, char **argv, char **envp) > { > static const char * const params[] = { > "order", "once", "menu", > - "splash", "splash-time", NULL > + "splash", "splash-time", > + "reboot-timeout", NULL > }; > char buf[sizeof(boot_devices)]; > char *standard_boot_devices; > -- Amos.