Peter Memishian wrote:
>  > 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?
> 
It certainly can be done. Do we need to ARC this though?

Reply via email to