I made the changes to the code and below is the API detail. Please take a look, and if there is no issues, please update the document accordingly. Note that the PSARC materials need to be ready today.
- A datalink_mediareq_t type is added. The dladm_walk_datalink_id() function will take a datalink_mediareq_t argument, in order to indicate that it walks either "ANY" media types, or "GLDv3" media types or a specific media type. - Other functions like dladm_create_datalink_id(), dladm_create_conf(), dladm_name2info() or dladm_datalink_id2info() will also take an additional argument. It is either a "uint32_t" or a "uint32_t *" argument, which carries the media type as either input or output. If you are interested in the details, please see the webrev in /net/aquila.prc/export/home/cathy/clearview/webrev_media_type. Take a look at the datalink_mediareq_t definition in mac.h, and the API changes in libdllink.h. If you have a better name for the macro DATALINK_MEDIAREQ_GLDV3, that would be better. The macro refers to all media types that are unified to GLDv3 (which therefore has the vanity naming support, for example). Thanks - Cathy > Are we going to add another datalink_media_t type? Whether we should just > use the dlpi media type macros? > > I was discussing about this with Meem yesterday. We might need to define a > simple structure type which contains the dlpi media type, and a flag to > indicate DL_ANY (any media is ok) and DL_SUPPORTED (for media types that is > supported by GLDv3). > > Thanks > - Cathy >> Cathy Zhou wrote: >>> Dan, >>> >>> I didn't find the media_type changes in the document. Please be noted >>> that if you add a media_type field. You need to update several functions: >>> >>> dladm_create_datalink_id(), dladm_name2info(), dladm_datalink_id2info(), >>> dladm_create_conf(). >>> >>> They all need to add a media_type argument. >>> >> I think I uploaded the wrong version by mistake. >> >> I tried uploading a fixed version, but there was an error trying to >> upload it to the site. I'll try again when the problem is fixed. >> >> In the meantime, those internal can take a look at: >> >> http://atlantic.east/~dg199075/link_id_management.pdf >> >> if you're interested in looking at it now. Apologies to the external >> folks, hopefully the problem will be fixed soon. >> >> Dan >> >> _________________________________ >> clearview-discuss mailing list >> clearview-discuss at opensolaris.org > > > _________________________________ > clearview-discuss mailing list > clearview-discuss at opensolaris.org
