[brussels-dev] PSARC 2008/171 mac_register_priv_prop()

[Garrett D'Amore]

sowmini.varadhan at sun.com wrote:
> On (03/19/08 14:14), Garrett D'Amore wrote:
>   
>> I prefer table based registration, for sure. But I actually have a  
>> different suggestion.
>>     
>
> mac_register_private_prop is also needed for making private
> props known to gldv3 for the purpose of dladm, as we discussed
> earlier.
>
>   
>> The only consumer for these at present is NDD. What I'd do is abandon  
>> attempts to support the outliers via NDD, unless there was enough users  
>> of the "outlier" to justify promotion to a "well-known" alias or  
>> somesuch (e.g. if all NSN drivers use a certain alternate spelling, then  
>> you should silently convert it in the framework, with no extra effort on  
>> the part of the driver.)
>>     
>
> Well the problem is that each driver has these variants for 
> flow-control. Some drivers have "specials" for bcopy/dma
> thresholds etc. Yes, these should be converted to dladm, but 
> we are trying to provide ndd compatibility, so I'm afraid we're
> stuck with supporting this with ndd for a while.
>   

*WHY?* I don't understand why ndd support is required for these 
specials. The only reason to sustain, IMO, ndd support, is for the 
programs that rely on it for MII support. (SunVTS, customer scripts.) I 
don't think it is that important to maintain this for "outliers".

-- Garrett
>
> --Sowmini
>
>