#
332755 |
|
19-Apr-2018 |
avg |
MFC r331666: ZFS vn_rele_async: catch up with the use of refcount(9) for the vnode use count
It's not sufficient nor required to use the vnode interlock when checking if we are going to drop the last use count as the code in vputx() uses refcount (atomic) operations for both checking and decrementing the use code. Apply the same method to vn_rele_async(). While here, remove vn_rele_inactive(), a wrapper around vrele() that didn't add any value.
Also, the change required making vfs_refcount_release_if_not_last() public. I've made vfs_refcount_acquire_if_not_zero() public as well. They are in sys/refcount.h now. While making the move I've dropped the vfs_ prefix.
Sponsored by: Panzura
|
#
331722 |
|
29-Mar-2018 |
eadler |
Revert r330897:
This was intended to be a non-functional change. It wasn't. The commit message was thus wrong. In addition it broke arm, and merged crypto related code.
Revert with prejudice.
This revert skips files touched in r316370 since that commit was since MFCed. This revert also skips files that require $FreeBSD$ property changes.
Thank you to those who helped me get out of this mess including but not limited to gonzo, kevans, rgrimes.
Requested by: gjb (re)
|
#
180762 |
|
23-Jul-2008 |
des |
pjd@'s r180759 was intended to revert r180755 due to ipfilter breakage, but removed too much, breaking the build in other places instead. Now that the ipfilter issue has been fixed (or hacked around), address the second issue by restoring r180755, with one small change. I don't feel comfortable using assert(3) in a header that will be included in userland code that may or may not already have an assertion mechanism in place, so KASSERT() evaluates to a no-op in the !_KERNEL case.
|