LGTM. I’m no ZFS expert, but this change looks trivial enough. 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 > > *openzfs-developer* | Archives > <https://www.listbox.com/member/archive/274414/=now> > <https://www.listbox.com/member/archive/rss/274414/28015165-f50abbe5> | > 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
