[SeaBIOS] FirmwareTestSuiteLive

2012-09-19 Thread Fred . 
Use FirmwareTestSuiteLive to check how well SeaBIOS behaves.

https://wiki.ubuntu.com/HardwareEnablementTeam/Documentation/FirmwareTestSuiteLive
https://wiki.ubuntu.com/Kernel/Reference/fwts

___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios

[SeaBIOS] Removing "legacy option ROM" from boot menu

2012-09-19 Thread Privat 

Hello,

is it possible to remove the "legacy option ROM" entry from seabios' 
boot menu list? (I use QEMU).
Using make menuconfig I tried to disable "Boot Interfaces --> Option 
ROMS". After compiling and using the bios.bin I only receive a black 
screen. My vgabios.bin is in its proper location.


Note: This error only occurs when I disable the aforementioned function. 
Any clue how to solve this issue? I have seen screenshots where this 
entry doesn't appear so it seems it's possible to bypass this entry.


TIA,

Alex

___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios

Re: [SeaBIOS] Removing "legacy option ROM" from boot menu

2012-09-19 Thread Kevin O'Connor 
On Wed, Sep 19, 2012 at 03:12:19PM +0200, Privat wrote:
> Hello,
> 
> is it possible to remove the "legacy option ROM" entry from seabios'
> boot menu list? (I use QEMU).
> Using make menuconfig I tried to disable "Boot Interfaces --> Option
> ROMS". After compiling and using the bios.bin I only receive a black
> screen. My vgabios.bin is in its proper location.

The vgabios is an option rom, so disabling optionroms disables
execution of the vga bios.

> 
> Note: This error only occurs when I disable the aforementioned
> function. Any clue how to solve this issue? I have seen screenshots
> where this entry doesn't appear so it seems it's possible to bypass
> this entry.

Something in QEMU is passing an additional option rom to SeaBIOS.  Try
extracting the debug information from seabios:

qemu -chardev stdio,id=seabios -device 
isa-debugcon,iobase=0x402,chardev=seabios ...

-Kevin

___
SeaBIOS mailing list
SeaBIOS@seabios.org
http://www.seabios.org/mailman/listinfo/seabios

Re: [SeaBIOS] [Qemu-devel] [PATCH v2] add a boot parameter to set reboot timeout

2012-09-19 Thread Amos Kong 

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 0x, 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 > 0x) {
+error_report("reboot timeout is larger than 65535, force it to 
65535.");
+reboot_timeout = 0x;
+}
+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[] = {

Re: [SeaBIOS] [Qemu-devel] [PATCH v2] add a boot parameter to set reboot timeout

2012-09-19 Thread Gleb Natapov 
On Thu, Sep 20, 2012 at 11:15:26AM +0800, Amos Kong wrote:
> 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 0x, 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?
> 
Not from me. Looks good.

> >---
> >  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 > 0x) {
> >+error_report("reboot timeout is larger than 65535, force it to 
> >65535.");
> >+reboot_timeout = 0x;
> >+}
> >+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