[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 15/16] smbios: Add a function to directly add an
From: |
Corey Minyard |
Subject: |
Re: [Qemu-devel] [PATCH 15/16] smbios: Add a function to directly add an entry |
Date: |
Wed, 13 Nov 2013 14:52:20 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 |
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 <address@hidden
> <mailto:address@hidden>> 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 <address@hidden
> <mailto:address@hidden>>
> ---
> 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
>
>
- [Qemu-devel] [PATCH 14/16] pc: Postpone adding ACPI and SMBIOS to fw_cfg, (continued)
[Qemu-devel] [PATCH 15/16] smbios: Add a function to directly add an entry, Corey Minyard, 2013/11/12
[Qemu-devel] [PATCH 16/16] ipmi: Add SMBIOS table entry, Corey Minyard, 2013/11/12
[Qemu-devel] [PATCH 09/16] ipmi: Add a local BMC simulation, Corey Minyard, 2013/11/12