All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ross Philipson <ross.philipson@citrix.com>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: "Zhang, Eniac" <eniac-xw.zhang@hp.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: passing smbios table from qemu
Date: Thu, 9 Jan 2014 09:46:59 -0500	[thread overview]
Message-ID: <52CEB663.4070609@citrix.com> (raw)
In-Reply-To: <1389261984.27473.46.camel@kazak.uk.xensource.com>

On 01/09/2014 05:06 AM, Ian Campbell wrote:
> On Wed, 2014-01-08 at 18:29 +0000, Ross Philipson wrote:
>>> -----Original Message-----
>>> From: xen-devel-bounces@lists.xen.org [mailto:xen-devel-
>>> bounces@lists.xen.org] On Behalf Of Ian Campbell
>>> Sent: Tuesday, January 07, 2014 8:56 AM
>>> To: Zhang, Eniac
>>> Cc: xen-devel@lists.xen.org
>>> Subject: Re: [Xen-devel] passing smbios table from qemu
>>>
>>> On Mon, 2014-01-06 at 21:01 +0000, Zhang, Eniac wrote:
>>>
>>>> Question, am I missing anything, or this feature (passing smbios) is
>>>> still work in progress?
>>>
>>> Under Xen smbios tables are supplied via hvmloader, not via qemu.
>>>
>>> What tables and or values do you want to override/supply?
>>>
>>> I believe that libxc supports passing in extra smbios tables when
>>> building the guest (via struct xc_hvm_build_args.smbios_module) but
>>> nothing has been plumbed in to make use of this.
>>>
>>> I'm not aware of any on going work to plumb that stuff further up, e.g.
>>> to libxl and xl or other toolstacks. (I think the libxc functionality is
>>> only consumed by the XenClient toolstack).
>>
>> Just FYI, I did go back and add the support (and docs) for it in
>> libxl. I did this after the first set of patches went in per someone's
>> request (can't recall who it was at the moment).
>
> Ah yes, here it is:
>         smbios_firmware="STRING"
>             Specify a path to a file that contains extra SMBIOS firmware
>             structures to pass in to a guest. The file can contain a set DMTF
>             predefined structures which will override the internal defaults.
>             Not all predefined structures can be overridden, only the
>             following types: 0, 1, 2, 3, 11, 22, 39. The file can also contain
>             any number of vendor defined SMBIOS structures (type 128 - 255).
>             Since SMBIOS structures do not present their overall size, each
>             entry in the file must be preceded by a 32b integer indicating the
>             size of the next structure.
>
> Did you not have a tool/library for helping to create such blobs
> somewhere? Or is my memory playing tricks?

Your memory is intact; I did provide a helper library. I posted it as a 
tarball since I could not figure out where such a thing might live in 
the xen tree. I posted it twice - the second time with some fixes:

http://lists.xen.org/archives/html/xen-devel/2013-03/msg01850.html


>
> Ian.
>
>
> -----
> No virus found in this message.
> Checked by AVG - www.avg.com
> Version: 2014.0.4259 / Virus Database: 3658/6986 - Release Date: 01/08/14
>

  reply	other threads:[~2014-01-09 14:46 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-06 21:01 passing smbios table from qemu Zhang, Eniac
2014-01-07 13:56 ` Ian Campbell
     [not found]   ` <3B22ECA2D19A3D408C83F4F15A9CB7D45076233C@G4W3221.americas.hpqcorp.net>
     [not found]     ` <DF407A7CCC374747A5BE5B7739FDEB181EFEE57D@G2W2433.americas.hpqcorp.net>
     [not found]       ` <3B22ECA2D19A3D408C83F4F15A9CB7D450762410@G4W3221.americas.hpqcorp.net>
     [not found]         ` <DF407A7CCC374747A5BE5B7739FDEB181EFEE61D@G2W2433.americas.hpqcorp.net>
     [not found]           ` <2559B97E2BCECA4BA2AF63DA8CF5C7200D095CD7@G1W3640.americas.hpqcorp.net>
2014-01-08 16:58             ` Zhang, Eniac
2014-01-08 17:51               ` Ian Campbell
2014-01-08 18:29   ` Ross Philipson
2014-01-09 10:06     ` Ian Campbell
2014-01-09 14:46       ` Ross Philipson [this message]
2014-01-09 14:56         ` Ian Campbell
2014-01-09 19:00         ` Konrad Rzeszutek Wilk
2014-01-10 10:20           ` Ian Campbell
2014-01-10 14:25             ` Ross Philipson
2014-01-10 15:03               ` Ian Campbell
2014-01-10 20:32                 ` Ross Philipson
2014-01-13 10:25                   ` Ian Campbell
2014-01-13 11:31                     ` Ian Jackson
2014-01-14 15:21                       ` Ross Philipson
2014-01-14 15:23                         ` Ian Campbell
2014-01-14 15:39                           ` Ross Philipson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52CEB663.4070609@citrix.com \
    --to=ross.philipson@citrix.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=eniac-xw.zhang@hp.com \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.