Discussion:
zfsctl_snapshot_inactive() vnode leak
k***@tvnetwork.hu
2013-12-25 20:33:40 UTC
Permalink
Dear Illumos team,

Please find a patch which would fix a vnode leak in zfsctl_snapshot_inactive()
some rare cases, if I am right.

Regards,

Kojedzinszky Richard
Matthew Ahrens
2013-12-29 17:45:56 UTC
Permalink
Looks good to me. I wonder if the check for v_count > 1 should be before
the call to gfs_dir_lookup()?

How can we reproduce the problem (and thus test the fix)?

Please test and submit an RTI.

--matt
Post by k***@tvnetwork.hu
Dear Illumos team,
Please find a patch which would fix a vnode leak in
zfsctl_snapshot_inactive() some rare cases, if I am right.
Regards,
Kojedzinszky Richard
Euronet Magyarorszag Informatikai Zrt.
-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182191/
21635000-ebd1d460
Modify Your Subscription: https://www.listbox.com/
member/?&
Powered by Listbox: http://www.listbox.com
-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182191/23047029-187a0c8d
Modify Your Subscription: https://www.listbox.com/member/?member_id=23047029&id_secret=23047029-2e85923f
Powered by Listbox: http://www.listbox.com

Loading...