Anurag S. Maskey wrote: > Code review for > > *Bug 10860* <http://defect.opensolaris.org/bz/show_bug.cgi?id=10860> - > system locations should be activated if no other condition are met > > http://defect.opensolaris.org/bz/show_bug.cgi?id=10860 > > at > > http://zhadum.east/export/ws/am223141/temp/nwam1-work/webrev/ > looks good to me. One thought I had wrt locations - though it's unrelated to these changes - is that when we refresh the location service, we should probably clear it too (via smf_restore_instance()), just in case it has gone into maintenance at some stage earlier on.
Alan
