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?
>
Assuming that identifier length isn't a significant constraint, you
could spell out "transient" instead of using "np." But I think Jim's
suggestion above, which doesn't require any suffix, is a more natural
way to express this idea.
Scott
--
Scott Rotondo
Principal Engineer, Solaris Security Technologies
President, Trusted Computing Group
Phone/FAX: +1 408 850 3655 (Internal x68278)
