On Wed, Aug 23, 2017 at 12:52:49PM +0100, Roger Pau Monne wrote:
> On Wed, Aug 23, 2017 at 11:51:49AM +0100, Wei Liu wrote:
> > On Tue, Aug 22, 2017 at 01:50:23PM +0100, Roger Pau Monne wrote:
> > [...]
> > > diff --git a/tools/firmware/hvmloader/smbios_types.h
> > > b/tools/firmware/hvmloader/smbios_types.h
> > > index e924f819b3..2bf1645bfa 100644
> > > --- a/tools/firmware/hvmloader/smbios_types.h
> > > +++ b/tools/firmware/hvmloader/smbios_types.h
> > > @@ -104,6 +104,11 @@ struct smbios_type_3 {
> > > uint8_t power_supply_state;
> > > uint8_t thermal_state;
> > > uint8_t security_status;
> > > + uint32_t oem_specific;
> > > + uint8_t height;
> > > + uint8_t number_of_power_cords;
> > > + uint8_t contained_element_count;
> > > + uint8_t contained_element_length;
> >
> > Missing contained_elements ?
>
> contained_element_count is 0, so I don't think it's strictly needed.
> If you think it's required I can represent it as an empty array ([0]).
>
I think that would be better but it is up to Jan and Andrew to decide.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
