James Carlson wrote:
> Garrett D'Amore - sun microsystems wrote:
>   
>> This case offers a mostly drop-in compatible replacement (and upgrade) for
>> elxl.  However, there is some impact to driver.conf properties and automatic
>> media selection, as well as two new driver-specific dladm link properties,
>> so this case records these changes.  I believe these impacts are unlikely
>> to negatively affect anyone at all (are folks still using 10BASE2 or
>> 10BASE5?), so I'm filing as self-review.  If anyone wants this to be promoted
>> to a fast track please let me know.  Only minor binding is being sought at
>> this time.
>>     
>
> Thank you!  That looks like the right thing to do.
>
>   
>> _available_media     Uncommitted     Undocumented, expected to be removed.
>>     
>
> Undocumented == Private
>
> Other than that, this sounds great.
>   

I wasn't sure I could get away with that for a property that I was 
essentially exposing to users (although not documenting it to facilitate 
its later removal.)

If I can call this "Private", that works so much the better.

Btw, I need code reviewers for this as well... 
http://cr.opensolaris.org/~gdamore/elxl/


    - Garrett

Reply via email to