Print this page
5045 use atomic_{inc,dec}_* instead of atomic_add_*

@@ -5974,15 +5974,15 @@
                         if (pp == NULL) {
                                 mutex_exit(&ulp->ul_lock);
                         }
                         return (vmpss ? EIO : EINVAL);
                 }
-                atomic_add_long(&ulp->ul_vnops_cnt, 1);
+                atomic_inc_ulong(&ulp->ul_vnops_cnt);
                 if (pp == NULL)
                         mutex_exit(&ulp->ul_lock);
                 if (ufs_quiesce_pend) {
-                        if (!atomic_add_long_nv(&ulp->ul_vnops_cnt, -1))
+                        if (!atomic_dec_ulong_nv(&ulp->ul_vnops_cnt))
                                 cv_broadcast(&ulp->ul_cv);
                         return (vmpss ? EIO : EINVAL);
                 }
         }
 

@@ -5997,11 +5997,11 @@
                  * Segvn will retry via VOP_GETPAGE() if VOP_PAGEIO() fails.
                  */
                 if (!vmpss) {
                         rw_enter(&ip->i_contents, RW_READER);
                 } else if (!rw_tryenter(&ip->i_contents, RW_READER)) {
-                        if (!atomic_add_long_nv(&ulp->ul_vnops_cnt, -1))
+                        if (!atomic_dec_ulong_nv(&ulp->ul_vnops_cnt))
                                 cv_broadcast(&ulp->ul_cv);
                         return (EDEADLK);
                 }
         }
 

@@ -6010,11 +6010,11 @@
          * PAGESIZE rounded EOF.
          */
         if (vmpss && btopr(io_off + io_len) > btopr(ip->i_size)) {
                 if (dolock)
                         rw_exit(&ip->i_contents);
-                if (!atomic_add_long_nv(&ulp->ul_vnops_cnt, -1))
+                if (!atomic_dec_ulong_nv(&ulp->ul_vnops_cnt))
                         cv_broadcast(&ulp->ul_cv);
                 return (EFAULT);
         }
 
         if (pp == NULL) {

@@ -6023,11 +6023,11 @@
                 } else {
                         err = EINVAL;
                 }
                 if (dolock)
                         rw_exit(&ip->i_contents);
-                if (!atomic_add_long_nv(&ulp->ul_vnops_cnt, -1))
+                if (!atomic_dec_ulong_nv(&ulp->ul_vnops_cnt))
                         cv_broadcast(&ulp->ul_cv);
                 return (err);
         }
 
         /*

@@ -6130,11 +6130,11 @@
                 mutex_exit(&ip->i_tlock);
         }
 
         if (dolock)
                 rw_exit(&ip->i_contents);
-        if (vmpss && !atomic_add_long_nv(&ulp->ul_vnops_cnt, -1))
+        if (vmpss && !atomic_dec_ulong_nv(&ulp->ul_vnops_cnt))
                 cv_broadcast(&ulp->ul_cv);
         return (err);
 }
 
 /*