Peter Memishian wrote: > Looks good. A couple more minor nits. > > > 1 Introduction > > ============== > > > > This case proposes two changes related to the dlmgmtd daemon: > > > > 1) Change the dlmgmtd daemon to be run by the "dladm" user, and no > extra > > privileges are required. > > What is meant by "... and no extra privileges are required"? Extra > compared to what?
I meant that no extra privilege than "dladm" already has are needed. I can certainly remove that. > > > 2) Introduce the /etc/svc/volatile/dladm directory and change the > dlmgmtd > > s/Introduce the/Create a/ Okay. > > > Several issues are found related to the above implementation though: > > s/are/were/ > Okay. - Cathy
