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..
