Use zfsctl_snapshot_hold() wrapper

zfs_refcount_*() are to be wrapped by zfsctl_snapshot_*() in this file.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-by: Tony Hutter <hutter2@llnl.gov>
Signed-off-by: Tomohiro Kusumi <kusumi.tomohiro@gmail.com>
Closes #9039
This commit is contained in:
Tomohiro Kusumi 2019-07-18 01:07:53 +09:00 committed by Brian Behlendorf
parent 8062b7686a
commit c8802ba08d
1 changed files with 3 additions and 3 deletions

View File

@ -192,7 +192,7 @@ static void
zfsctl_snapshot_add(zfs_snapentry_t *se) zfsctl_snapshot_add(zfs_snapentry_t *se)
{ {
ASSERT(RW_WRITE_HELD(&zfs_snapshot_lock)); ASSERT(RW_WRITE_HELD(&zfs_snapshot_lock));
zfs_refcount_add(&se->se_refcount, NULL); zfsctl_snapshot_hold(se);
avl_add(&zfs_snapshots_by_name, se); avl_add(&zfs_snapshots_by_name, se);
avl_add(&zfs_snapshots_by_objsetid, se); avl_add(&zfs_snapshots_by_objsetid, se);
} }
@ -269,7 +269,7 @@ zfsctl_snapshot_find_by_name(char *snapname)
search.se_name = snapname; search.se_name = snapname;
se = avl_find(&zfs_snapshots_by_name, &search, NULL); se = avl_find(&zfs_snapshots_by_name, &search, NULL);
if (se) if (se)
zfs_refcount_add(&se->se_refcount, NULL); zfsctl_snapshot_hold(se);
return (se); return (se);
} }
@ -290,7 +290,7 @@ zfsctl_snapshot_find_by_objsetid(spa_t *spa, uint64_t objsetid)
search.se_objsetid = objsetid; search.se_objsetid = objsetid;
se = avl_find(&zfs_snapshots_by_objsetid, &search, NULL); se = avl_find(&zfs_snapshots_by_objsetid, &search, NULL);
if (se) if (se)
zfs_refcount_add(&se->se_refcount, NULL); zfsctl_snapshot_hold(se);
return (se); return (se);
} }