[Differential] [Commented On] D1833: Add memory barriers to buf_ring

kmacy (Kip Macy) phabric-noreply at FreeBSD.org
Mon Feb 23 00:13:58 UTC 2015


kmacy added a comment.

Reading this again, cons_head is used only by dequeue and is thus protected by the lock or an atomic update. Hence the atomic_load_acq_32 isn't needed for it. On the other hand, there are other places where prod_tail needs to be read with a memory barrier.

REVISION DETAIL
  https://reviews.freebsd.org/D1833

To: zbb, kmacy, rpaulo, imp
Cc: meloun-miracle-cz, onwahe-gmail-com, andrew, ian, adrian, freebsd-arm


More information about the freebsd-arm mailing list