Matt, Yes, I'll move them.
Thanks, Jerry On Wed, Aug 17, 2016 at 2:30 PM, Matthew Ahrens <[email protected]> wrote: > This looks good, can you also move these readonly props to the section > above, labeled /* readonly number properties */. > > --matt > > On Tue, Aug 16, 2016 at 5:45 AM, Jerry Jelinek <[email protected]> > wrote: > >> Here is the fix for: >> 7304 zfs filesystem/snapshot counts should be read-only >> >> This issue was found by the folks at Delphix and the fix was suggested by >> Sara Hartse. >> >> Because this is so simple I have just included the diffs here: >> >> --- a/usr/src/common/zfs/zfs_prop.c >> +++ b/usr/src/common/zfs/zfs_prop.c >> @@ -405,10 +405,10 @@ zfs_prop_init(void) >> UINT64_MAX, PROP_DEFAULT, ZFS_TYPE_FILESYSTEM | >> ZFS_TYPE_VOLUME, >> "<count> | none", "SSLIMIT"); >> zprop_register_number(ZFS_PROP_FILESYSTEM_COUNT, >> "filesystem_count", >> - UINT64_MAX, PROP_DEFAULT, ZFS_TYPE_FILESYSTEM, >> + UINT64_MAX, PROP_READONLY, ZFS_TYPE_FILESYSTEM, >> "<count>", "FSCOUNT"); >> zprop_register_number(ZFS_PROP_SNAPSHOT_COUNT, "snapshot_count", >> - UINT64_MAX, PROP_DEFAULT, ZFS_TYPE_FILESYSTEM | >> ZFS_TYPE_VOLUME, >> + UINT64_MAX, PROP_READONLY, ZFS_TYPE_FILESYSTEM | >> ZFS_TYPE_VOLUME, >> "<count>", "SSCOUNT"); >> >> Thanks, >> Jerry >> >> > *illumos-developer* | Archives > <https://www.listbox.com/member/archive/182179/=now> > <https://www.listbox.com/member/archive/rss/182179/21175122-8b1c0701> | > Modify > <https://www.listbox.com/member/?&;> > Your Subscription <http://www.listbox.com> > ------------------------------------------- openzfs-developer Archives: https://www.listbox.com/member/archive/274414/=now RSS Feed: https://www.listbox.com/member/archive/rss/274414/28015062-cce53afa Modify Your Subscription: https://www.listbox.com/member/?member_id=28015062&id_secret=28015062-f966d51c Powered by Listbox: http://www.listbox.com
