Re: [dmidecode] [PATCH] dmidecode: additions from smbios 2.6.1 spec update

[Jean Delvare]

On Fri, 28 Aug 2009 14:57:50 -0400, Jarod Wilson wrote:
> On Friday 28 August 2009 13:03:08 Jean Delvare wrote:
> > On Fri, 28 Aug 2009 11:23:35 -0400, Jarod Wilson wrote:
> > > On Friday 28 August 2009 11:05:06 Jean Delvare wrote:
> > > > Hi Jarod,
> > > > 
> > > > On Thu, 27 Aug 2009 09:25:40 -0400, Jarod Wilson wrote:
> > > > > dmidecode: additions from smbios 2.6.1 spec update
> ...
> > I didn't check, did you add full support for SMBIOS 2.6.1 already?
> 
> Yeah, I *think* I got everything. Just double-checked the changelog
> and the processor list, everything seems to be there now.
> 
> > If
> > you did, then you probably want to update the specification reference
> > at the top of the source file.
> 
> Are you referring to the SUPPORTED_SMBIOS_VER define at the top of
> dmidecode.c? If so, it looks like there's some work to do to properly
> support version checking with the third revision level... The SMBIOS
> structure table (section 2.1.1) doesn't seem to provide support for
> anything more than major/minor either.

You're right about the third revision level, although I don't think
this should be an issue in practice (when only the third revision level
changes, there shouldn't be any incompatible changes, only
clarifications and new enumerated values.) But this isn't what I meant.
What I was referring to was:

 * Unless specified otherwise, all references are aimed at the "System
 * Management BIOS Reference Specification, Version 2.6" document,
 * available from http://www.dmtf.org/standards/smbios/.

where 2.6 would become 2.6.1.

Thanks,
-- 
Jean Delvare