On Thu, 25 Jun 2020 17:28:41 +0530 Rahul Lakkireddy wrote:
> +     if (refcount_read(&pi->vi_mirror_refcnt) > 1) {
> +             refcount_dec(&pi->vi_mirror_refcnt);
> +             return;
> +     }

FWIW this looks very dodgy. If you know nothing changes the count
between the read and the dec here, you probably don't need atomic
refcounts at all..

Reply via email to