On 11/13/2013 08:37 AM, Bret Ketchum wrote: > > Don't know if it matters much but this patch cannot be applied > without the prototype definition in 16/16.
Thanks, I'll fix this. -corey > > > On Tue, Nov 12, 2013 at 10:33 AM, Corey Minyard <miny...@acm.org > <mailto:miny...@acm.org>> wrote: > > There was no way to directly add a table entry to the SMBIOS table, > even though the BIOS supports this. So add a function to do this. > This is in preparation for the IPMI handler adding it's SMBIOS table > entry. > > Signed-off-by: Corey Minyard <cminy...@mvista.com > <mailto:cminy...@mvista.com>> > --- > hw/i386/smbios.c | 27 +++++++++++++++++++++++++++ > 1 file changed, 27 insertions(+) > > diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c > index d3f1ee6..9c53131 100644 > --- a/hw/i386/smbios.c > +++ b/hw/i386/smbios.c > @@ -277,6 +277,33 @@ static void save_opt(const char **dest, > QemuOpts *opts, const char *name) > } > } > > +int smbios_table_entry_add(struct smbios_structure_header *entry) > +{ > + struct smbios_table *table; > + struct smbios_structure_header *header; > + unsigned int size = entry->length; > + > + if (!smbios_entries) { > + smbios_entries_len = sizeof(uint16_t); > + smbios_entries = g_malloc0(smbios_entries_len); > + } > + smbios_entries = g_realloc(smbios_entries, smbios_entries_len + > + sizeof(*table) + size); > + table = (struct smbios_table *)(smbios_entries + > smbios_entries_len); > + table->header.type = SMBIOS_TABLE_ENTRY; > + table->header.length = cpu_to_le16(sizeof(*table) + size); > + > + header = (struct smbios_structure_header *)(table->data); > + memcpy(header, entry, size); > + > + smbios_check_collision(header->type, SMBIOS_TABLE_ENTRY); > + > + smbios_entries_len += sizeof(*table) + size; > + (*(uint16_t *)smbios_entries) = > + cpu_to_le16(le16_to_cpu(*(uint16_t *)smbios_entries) + 1); > + return 0; > +} > + > void smbios_entry_add(QemuOpts *opts) > { > Error *local_err = NULL; > -- > 1.8.3.1 > >