> I am thinking to add a function > > int mac_open_by_linkid(dladm_linkid_t linkid, mac_handle_t *mhp);
A nit, but the idea of having the parameter be a "dladm_linkid_t" feels strange. I think of that as a userland datatype; shouldn't there be a dls_linkid_t for kernel use? -- meem
