Hi James,

config_ovr/fastreboot_default will certainly work.  Recalling my
conversations with the SMF team I suspect that's what they wanted me to
do as well and I simply misunderstood.  I will clarify with the SMF
team and post a summary.

I would be happy to rename the function to
scf_fastreboot_disable_transient().  Would that work?

Thanks,
Sherry

On Thu, Jun 04, 2009 at 08:56:13AM -0400, James Carlson wrote:
> Sherry Moore writes:
> >     To enable the above usage models, a new non-persistent property
> >     group "fastreboot_disable_np" will be introduced to
> >     svc:/system/boot-config:default disable the Fast Reboot by
> >     Default feature for the next reboot only.
> 
> It seems a bit awkward to have to use a property group itself as
> though it were a boolean property, rather than having an actual
> property.
> 
> Since SCF doesn't admit the possibility of temporary properties, and
> since having the system clear out a persistent property would probably
> be too hackish, did you consider creating a "config_ovr" temporary
> group with the new flag as "config_ovr/fastreboot_default"?  The
> design for the "general" property group seems to use this scheme:
> temporary settings go in "general_ovr" (a non-persistent property
> group) and permanent ones in "general."  Things that have to deal with
> those "general" properties look in both places.
> 
> One other very tiny nit: "_np" as a suffix has historically meant
> "non-POSIX."  Are we sure we want it to mean "non-persistent" as well?
> 
> -- 
> James Carlson, Solaris Networking              <james.d.carlson at sun.com>
> Sun Microsystems / 35 Network Drive        71.232W   Vox +1 781 442 2084
> MS UBUR02-212 / Burlington MA 01803-2757   42.496N   Fax +1 781 442 1677

-- 
Sherry Moore, Solaris Core Kernel       http://blogs.sun.com/sherrym

Reply via email to