Hi,
I think it's time to start planning the next seabios release. First, because a bunch of changes have piled up in master. And second, because of the smbios changes. They are a step forward in making seabios less dependent on qemu internals and I want have that in qemu soonish.
So, how about this plan:
(1) merge smbios patches in qemu (patchset is close to final now). (2) merge smbios patch in seabios (3) start freeze (4) tag release candidate (early may is realistic for that I think). (5) pull release candidate into qemu for wider testing (6) merge bugfix patches if needed (7) tag final release 2-3 weeks after release candidate (end of may probably). (8) pull final release into qemu.
Comments?
cheers, Gerd
Check fw_cfg for the presence of an aggregate set of smbios tables (etc/smbios/smbios-tables) and an entry point structure (etc/smbios/smbios-anchor), and, if found, use them instead of generating entries locally.
We ensure the presence of a type 0 (bios information) structure by generating it locally if necessary, which is expected to be the common case.
Signed-off-by: Gabriel L. Somlo somlo@cmu.edu ---
Only difference in this version (aside from the slightly reworded commit blurb) is a modified comment:
- /* Enable targeted content distribution (needed for SVVP, per SeaBIOS) */ + /* Enable targeted content distribution (needed for SVVP) */
which I cut'n'pasted (back) from QEMU for generating the local SeaBIOS type 0 table. I figure saying "per SeaBIOS" when we're *in* SeaBIOS would be a bit silly... :)
Thanks, Gabriel
src/fw/biostables.c | 135 ++++++++++++++++++++++++++++++++++++++++++++++++++++ src/fw/smbios.c | 2 +- src/util.h | 3 +- 3 files changed, 138 insertions(+), 2 deletions(-)
diff --git a/src/fw/biostables.c b/src/fw/biostables.c index 818a8fe..17bee8e 100644 --- a/src/fw/biostables.c +++ b/src/fw/biostables.c @@ -13,6 +13,7 @@ #include "std/mptable.h" // MPTABLE_SIGNATURE #include "std/pirtable.h" // struct pir_header #include "std/smbios.h" // struct smbios_entry_point +#include "romfile.h" #include "string.h" // memcpy #include "util.h" // copy_table #include "x86.h" // outb @@ -314,6 +315,140 @@ display_uuid(void) } }
+#define set_str_field_or_skip(type, field, value) \ + do { \ + int size = (value != NULL) ? strlen(value) + 1 : 0; \ + if (size > 1) { \ + memcpy(end, value, size); \ + end += size; \ + p->field = ++str_index; \ + } else { \ + p->field = 0; \ + } \ + } while (0) + +static void * +smbios_new_type_0(void *start, + const char *vendor, const char *version, const char *date) +{ + struct smbios_type_0 *p = (struct smbios_type_0 *)start; + char *end = (char *)start + sizeof(struct smbios_type_0); + int str_index = 0; + + p->header.type = 0; + p->header.length = sizeof(struct smbios_type_0); + p->header.handle = 0; + + set_str_field_or_skip(0, vendor_str, vendor); + set_str_field_or_skip(0, bios_version_str, version); + p->bios_starting_address_segment = 0xe800; + set_str_field_or_skip(0, bios_release_date_str, date); + + p->bios_rom_size = 0; /* FIXME */ + + /* BIOS characteristics not supported */ + memset(p->bios_characteristics, 0, 8); + p->bios_characteristics[0] = 0x08; + + /* Enable targeted content distribution (needed for SVVP) */ + p->bios_characteristics_extension_bytes[0] = 0; + p->bios_characteristics_extension_bytes[1] = 4; + + p->system_bios_major_release = 0; + p->system_bios_minor_release = 0; + p->embedded_controller_major_release = 0xFF; + p->embedded_controller_minor_release = 0xFF; + + *end = 0; + end++; + if (!str_index) { + *end = 0; + end++; + } + + return end; +} + +#define BIOS_NAME "SeaBIOS" +#define BIOS_DATE "04/01/2014" + +static int +smbios_romfile_setup(void) +{ + struct romfile_s *f_anchor = romfile_find("etc/smbios/smbios-anchor"); + struct romfile_s *f_tables = romfile_find("etc/smbios/smbios-tables"); + struct smbios_entry_point ep; + struct smbios_type_0 *t0; + u16 qtables_len, need_t0 = 1; + u8 *qtables, *tables; + + if (!f_anchor || !f_tables || f_anchor->size != sizeof(ep)) + return 0; + + f_anchor->copy(f_anchor, &ep, f_anchor->size); + + if (f_tables->size != ep.structure_table_length) + return 0; + + qtables = malloc_tmphigh(f_tables->size); + if (!qtables) { + warn_noalloc(); + return 0; + } + f_tables->copy(f_tables, qtables, f_tables->size); + ep.structure_table_address = (u32)qtables; /* for smbios_next(), below */ + + /* did we get a type 0 structure ? */ + for (t0 = smbios_next(&ep, NULL); t0; t0 = smbios_next(&ep, t0)) + if (t0->header.type == 0) { + need_t0 = 0; + break; + } + + qtables_len = ep.structure_table_length; + if (need_t0) { + /* common case: add our own type 0, with 3 strings and 4 '\0's */ + u16 t0_len = sizeof(struct smbios_type_0) + strlen(BIOS_NAME) + + strlen(VERSION) + strlen(BIOS_DATE) + 4; + ep.structure_table_length += t0_len; + if (t0_len > ep.max_structure_size) + ep.max_structure_size = t0_len; + ep.number_of_structures++; + } + + /* allocate final blob and record its address in the entry point */ + if (ep.structure_table_length > BUILD_MAX_SMBIOS_FSEG) + tables = malloc_high(ep.structure_table_length); + else + tables = malloc_fseg(ep.structure_table_length); + if (!tables) { + warn_noalloc(); + free(qtables); + return 0; + } + ep.structure_table_address = (u32)tables; + + /* populate final blob */ + if (need_t0) + tables = smbios_new_type_0(tables, BIOS_NAME, VERSION, BIOS_DATE); + memcpy(tables, qtables, qtables_len); + free(qtables); + + /* finalize entry point */ + ep.checksum -= checksum(&ep, 0x10); + ep.intermediate_checksum -= checksum((void *)&ep + 0x10, ep.length - 0x10); + + copy_smbios(&ep); + return 1; +} + +void +smbios_setup(void) +{ + if (smbios_romfile_setup()) + return; + smbios_legacy_setup(); +}
void copy_table(void *pos) diff --git a/src/fw/smbios.c b/src/fw/smbios.c index 0ac9ff5..dba0541 100644 --- a/src/fw/smbios.c +++ b/src/fw/smbios.c @@ -504,7 +504,7 @@ smbios_init_type_127(void *start) #define TEMPSMBIOSSIZE (32 * 1024)
void -smbios_setup(void) +smbios_legacy_setup(void) { if (! CONFIG_SMBIOS) return; diff --git a/src/util.h b/src/util.h index 4f242bf..9557581 100644 --- a/src/util.h +++ b/src/util.h @@ -78,6 +78,7 @@ extern struct smbios_entry_point *SMBiosAddr; void copy_smbios(void *pos); void display_uuid(void); void copy_table(void *pos); +void smbios_setup(void);
// fw/coreboot.c extern const char *CBvendor, *CBpart; @@ -117,7 +118,7 @@ void make_bios_readonly(void); void qemu_prep_reset(void);
// fw/smbios.c -void smbios_setup(void); +void smbios_legacy_setup(void);
// fw/smm.c void smm_device_setup(void);
On Wed, Apr 23, 2014 at 04:41:32PM +0200, Gerd Hoffmann wrote:
Hi,
I think it's time to start planning the next seabios release. First, because a bunch of changes have piled up in master. And second, because of the smbios changes. They are a step forward in making seabios less dependent on qemu internals and I want have that in qemu soonish.
So, how about this plan:
(1) merge smbios patches in qemu (patchset is close to final now). (2) merge smbios patch in seabios (3) start freeze (4) tag release candidate (early may is realistic for that I think). (5) pull release candidate into qemu for wider testing (6) merge bugfix patches if needed (7) tag final release 2-3 weeks after release candidate (end of may probably). (8) pull final release into qemu.
Comments?
A release around the end of May sounds good to me. I think we should aim for a feature freeze of no more than two weeks - if any problems or defects are found then we can extend the release.
I have some ideas for possible code changes after the next release. I'll send a separate email.
-Kevin
On Mi, 2014-04-23 at 15:04 -0400, Gabriel L. Somlo wrote:
Check fw_cfg for the presence of an aggregate set of smbios tables (etc/smbios/smbios-tables) and an entry point structure (etc/smbios/smbios-anchor), and, if found, use them instead of generating entries locally.
We ensure the presence of a type 0 (bios information) structure by generating it locally if necessary, which is expected to be the common case.
Qemu smbios patch series is merged upstream. Pushed seabios patch too.
cheers, Gerd
On 05/06/14 15:57, Gerd Hoffmann wrote:
On Mi, 2014-04-23 at 15:04 -0400, Gabriel L. Somlo wrote:
Check fw_cfg for the presence of an aggregate set of smbios tables (etc/smbios/smbios-tables) and an entry point structure (etc/smbios/smbios-anchor), and, if found, use them instead of generating entries locally.
We ensure the presence of a type 0 (bios information) structure by generating it locally if necessary, which is expected to be the common case.
Qemu smbios patch series is merged upstream. Pushed seabios patch too.
Thank you all. OVMF client code is on my todo list. I guess I'll get there sometime.
Laszlo
On Mi, 2014-04-23 at 16:41 +0200, Gerd Hoffmann wrote:
Hi,
I think it's time to start planning the next seabios release. First, because a bunch of changes have piled up in master. And second, because of the smbios changes. They are a step forward in making seabios less dependent on qemu internals and I want have that in qemu soonish.
So, how about this plan:
(1) merge smbios patches in qemu (patchset is close to final now). (2) merge smbios patch in seabios
We are here now ...
(3) start freeze (4) tag release candidate (early may is realistic for that I think).
... so it's time kick freeze + release candidate I guess ;)
Are there any pending patches for the next release which should better make it into the release candidate?
cheers, Gerd
On Wed, May 07, 2014 at 12:33:35PM +0200, Gerd Hoffmann wrote:
On Mi, 2014-04-23 at 16:41 +0200, Gerd Hoffmann wrote:
Hi,
I think it's time to start planning the next seabios release. First, because a bunch of changes have piled up in master. And second, because of the smbios changes. They are a step forward in making seabios less dependent on qemu internals and I want have that in qemu soonish.
So, how about this plan:
(1) merge smbios patches in qemu (patchset is close to final now). (2) merge smbios patch in seabios
We are here now ...
(3) start freeze (4) tag release candidate (early may is realistic for that I think).
... so it's time kick freeze + release candidate I guess ;)
Are there any pending patches for the next release which should better make it into the release candidate?
I just sent a few fixes to the mailing list (xhci msleep, romlayout.o rebuild, int 1589 fix). I think these should go into the final release. Unless there are comments on the above patches I'll push them on Monday.
I think we can then also start the feature freeze on Monday.
-Kevin
Hi,
... so it's time kick freeze + release candidate I guess ;)
Are there any pending patches for the next release which should better make it into the release candidate?
I just sent a few fixes to the mailing list (xhci msleep, romlayout.o rebuild, int 1589 fix). I think these should go into the final release. Unless there are comments on the above patches I'll push them on Monday.
I think we can then also start the feature freeze on Monday.
Sounds good (and the patches look sane to me too btw). So lets tag the release candidate next monday, after applying the patches.
cheers, Gerd
On Fri, May 09, 2014 at 04:12:32PM +0200, Gerd Hoffmann wrote:
Hi,
... so it's time kick freeze + release candidate I guess ;)
Are there any pending patches for the next release which should better make it into the release candidate?
I just sent a few fixes to the mailing list (xhci msleep, romlayout.o rebuild, int 1589 fix). I think these should go into the final release. Unless there are comments on the above patches I'll push them on Monday.
I think we can then also start the feature freeze on Monday.
Sounds good (and the patches look sane to me too btw). So lets tag the release candidate next monday, after applying the patches.
FYI, I tagged "rel-1.7.5-rc1".
-Kevin
-
Gabriel L. Somlo -
Gerd Hoffmann -
Kevin O'Connor -
Laszlo Ersek