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