Right now I'm using the generic property mechanism, but  have a special
case in dsl_prop_get_all() to ignore searching parents for this
particular property.  I'm not thrilled about it, but I only see two
other options:

1. Do not use the generic infrastructure.  This requires much more
   invasive changes that I'd rather avoid.

2. From the kernel's perspective have it be inheritable, but then fake
   up the non-inherited state in libzfs.  i.e. if the source is not
   ZFS_SRC_LOCAL, then pretend like it isn't set at all.

If the current hack is too offensive, moving it into libzfs seems like a
reasonable option.

- Eric

On Thu, Aug 10, 2006 at 10:40:49AM -0700, Matthew Ahrens wrote:
> On Thu, Aug 10, 2006 at 10:23:20AM -0700, Eric Schrock wrote:
> > A new option will be added, 'canmount', which specifies whether the
> > given filesystem can be mounted with 'zfs mount'.  This is a boolean
> > property, and is not inherited.
> 
> Cool, looks good.  Do you plan to implement this using the generic
> (inheritable) property infrastructure (eg. dsl_prop_set/get()), and just
> ignore the setting if it is inherited?
> 
> --matt

--
Eric Schrock, Solaris Kernel Development       http://blogs.sun.com/eschrock
_______________________________________________
zfs-discuss mailing list
zfs-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/zfs-discuss

Reply via email to