buf_ring: Consistently use atomic_*_32
We are operating on uint32_t values, use uint32_t atomic functions. Reviewed by: alc, imp, kib, markj Sponsored by: Arm Ltd Differential Revision: https://reviews.freebsd.org/D46150
This commit is contained in:
+4
-4
@@ -94,7 +94,7 @@ buf_ring_enqueue(struct buf_ring *br, void *buf)
|
||||
}
|
||||
continue;
|
||||
}
|
||||
} while (!atomic_cmpset_acq_int(&br->br_prod_head, prod_head, prod_next));
|
||||
} while (!atomic_cmpset_acq_32(&br->br_prod_head, prod_head, prod_next));
|
||||
#ifdef DEBUG_BUFRING
|
||||
if (br->br_ring[prod_head] != NULL)
|
||||
panic("dangling value in enqueue");
|
||||
@@ -108,7 +108,7 @@ buf_ring_enqueue(struct buf_ring *br, void *buf)
|
||||
*/
|
||||
while (br->br_prod_tail != prod_head)
|
||||
cpu_spinwait();
|
||||
atomic_store_rel_int(&br->br_prod_tail, prod_next);
|
||||
atomic_store_rel_32(&br->br_prod_tail, prod_next);
|
||||
critical_exit();
|
||||
return (0);
|
||||
}
|
||||
@@ -132,7 +132,7 @@ buf_ring_dequeue_mc(struct buf_ring *br)
|
||||
critical_exit();
|
||||
return (NULL);
|
||||
}
|
||||
} while (!atomic_cmpset_acq_int(&br->br_cons_head, cons_head, cons_next));
|
||||
} while (!atomic_cmpset_acq_32(&br->br_cons_head, cons_head, cons_next));
|
||||
|
||||
buf = br->br_ring[cons_head];
|
||||
#ifdef DEBUG_BUFRING
|
||||
@@ -146,7 +146,7 @@ buf_ring_dequeue_mc(struct buf_ring *br)
|
||||
while (br->br_cons_tail != cons_head)
|
||||
cpu_spinwait();
|
||||
|
||||
atomic_store_rel_int(&br->br_cons_tail, cons_next);
|
||||
atomic_store_rel_32(&br->br_cons_tail, cons_next);
|
||||
critical_exit();
|
||||
|
||||
return (buf);
|
||||
|
||||
Reference in New Issue
Block a user