[brussels-dev] [crossbow-discuss] code review Brussels persistence

Artem Kachitchkine Artem.Kachitchkin at Sun.COM
Wed May 14 12:29:02 PDT 2008


Michael Lim wrote:
>>
>> 	http://cr.opensolaris.org/~artem/pers-0421/
>>
> linkprop.c  471
> 
> Any reason why you don't add a flags argument to 
> dladm_datalink_id2info() instead
> of creating i_dladm_datalink_id2nfo()?

I did this to limit the scope of changes to linkprop.c. There are too 
many consumers of dladm_datalink_id2info().

The rest of the questions, please take a look at the latest webrev I'm 
going to send out shortly, where we moved property handling from MAC to 
DLS, and see if they still apply.

thanks,
-Artem


More information about the brussels-dev mailing list