[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v3 01/04] HVM firmware passthrough HVM defs header

On Wed, 2012-05-23 at 15:37 +0100, Ross Philipson wrote:
> +/* Up to 100 OEM strings can be set in xenstore using values of the
> form
> + * below. These strings will be loaded into the SMBIOS type 11
> structure.
> + */
> +#define HVM_XS_OEM_STRINGS             "bios-strings/oem-XX" 

I think this is actually pre-existing but we have snprintf in hvmloader
now so this could probably become "bios-strings/oem-%02d" (or whichever
format is really correct) and replace the manual munging in the code.
It's not important right now though.

I wouldn't say I've done a full review but I skimmed the rest and it
looked good.

We've been feature frozen for 4.2 for a while now, are you proposing
this for 4.3 rather than asking for a freeze exception?


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.