> The webrev is updated: > > http://cr.opensolaris.org/~yun/webrev_dlmgmtd/
Looks good. Regarding dlmgmt_main.c:103: I think it'd actually be clearer to just use "0644". Also, given that libdladm.c also has definitions for DLADM_DB_OWNER and DLADM_DB_GROUP, maybe we should put these in a shareable header, like sys/param.h? -- meem
