> Here's the webrev: > > http://cr.opensolaris.org/~seb/webrev-smf/
As workarounds go, seems good. The formatting of 90-92 would be slightly preferable as: state=`svcprop -p restarter/state \ datalink-management:default 2>/dev/null` Also, the paranoid might prefer "network/datalink-management" to datalink-management. Are you confident that 5 seconds is enough? -- meem