On Tue, 30 Jun 2020 04:19:51 +0530 Rahul Lakkireddy wrote:
> + if (refcount_read(&pi->vi_mirror_refcnt) > 1) {
> + refcount_dec(&pi->vi_mirror_refcnt);
> + goto out_unlock;
> + }Please remove the use of refcount_t if you're doing this, otherwise a person looking at this code will waste time trying to validate if this clearly bogus use of refcount_t is harmless or not.
