Hi Mike,
Thank you, comments below.
On 09/05/09 20:59, Mike Shapiro wrote:
On Wed, Sep 02, 2009 at 12:25:36PM -0400, Tom Pothier wrote:
The webrev is now also available on opensolaris.org:
http://cr.opensolaris.org/~pothier/onnv-x86gentopo-pb/
thx,
-t
I reviewed just the SMBIOS part. Here are the changes I'd like to see:
usr/src/common/smbios/smb_info.c
Line 275 needs to have this code:
if (isp->is_type == SMB_TYPE_EOT)
return (smb_set_errno(shp, ESMB_TYPE));
otherwise you return bogus data.
Added. Should this also be added to smbios_info_common() at line 207?
usr/src/common/smbios/smb_info.c
The interface to smb_info_contains() is not what I want. I don't
want to expose bcopy'ing out the implementation gunk with min/max.
Are you referencing the Type-3 elements min/max here?
I just want it to return an array of contained id_t's of other records.
The interface should be this:
int smbios_info_contains(smbios_hdl_t *shp, id_t id, uint_t idc, id_t *idv);
The caller passes an array of uninitialized id_t's 'idv', length 'idc'
The function decodes everything about record 'id', copies out the
internal record handles from whatever their encoding is (uint8/16)
to a set of id_t's with a for loop (not bcopy, since the size varies).
The function returns the actual count of contained things. If 'idc'
is smaller than that count, then idv only contains 'idc' elements.
i.e. you do not copy past the end of the caller's buffer.
The smb_infospecs table should contain fields to indicate (a) offset
of member with count of elements, (b) offset of member with contained
elements, (c) something to indicate format of those contained records.
For (c) either introduce flags or have a function pointer to handle
the different cases.
So I think what you want is in the case of Type-2 "Contained Object
Handles", return an array of these handles; in the case of Type-4
contained elements return an array of just the "Contained Element
Type"(s) - not the entire element.
If this is the case, I'll start making the changes and expect them in
the next webrev.
thx,
-t
The rest of smbios stuff looks good: thanks for working on this.
Please be sure to get a lot of help testing-- this wad is really
important but has a rather huge test matrix.
-Mike
_______________________________________________
fm-discuss mailing list
fm-discuss@opensolaris.org
