diff options
author | Michael S. Tsirkin <mst@redhat.com> | 2018-04-19 13:29:41 -0400 |
---|---|---|
committer | Michael S. Tsirkin <mst@redhat.com> | 2018-06-07 13:54:23 -0400 |
commit | 55e49dc43a835b19567e62142cb1c87dc7db7b3c (patch) | |
tree | b89327c3415a0b789c2532a8df9a938231d1c420 | |
parent | 29dcea88779c856c7dc92040a0c01233263101d4 (diff) |
virtio_ring: switch to dma_XX barriers for rpmsg
virtio is using barriers to order memory accesses, thus
dma_wmb/rmb is a good match.
Before
[mst@tuck linux]$ size drivers/virtio/virtio_ring.o
text data bss dec hex filename
11392 820 0 12212 2fb4 drivers/virtio/virtio_ring.o
After
mst@tuck linux]$ size drivers/virtio/virtio_ring.o
text data bss dec hex filename
11284 820 0 12104 2f48 drivers/virtio/virtio_ring.o
Cc: Ohad Ben-Cohen <ohad@wizery.com>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: linux-remoteproc@vger.kernel.org
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
-rw-r--r-- | include/linux/virtio_ring.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/virtio_ring.h b/include/linux/virtio_ring.h index bbf32524ab27..fab02133a919 100644 --- a/include/linux/virtio_ring.h +++ b/include/linux/virtio_ring.h | |||
@@ -35,7 +35,7 @@ static inline void virtio_rmb(bool weak_barriers) | |||
35 | if (weak_barriers) | 35 | if (weak_barriers) |
36 | virt_rmb(); | 36 | virt_rmb(); |
37 | else | 37 | else |
38 | rmb(); | 38 | dma_rmb(); |
39 | } | 39 | } |
40 | 40 | ||
41 | static inline void virtio_wmb(bool weak_barriers) | 41 | static inline void virtio_wmb(bool weak_barriers) |
@@ -43,7 +43,7 @@ static inline void virtio_wmb(bool weak_barriers) | |||
43 | if (weak_barriers) | 43 | if (weak_barriers) |
44 | virt_wmb(); | 44 | virt_wmb(); |
45 | else | 45 | else |
46 | wmb(); | 46 | dma_wmb(); |
47 | } | 47 | } |
48 | 48 | ||
49 | static inline void virtio_store_mb(bool weak_barriers, | 49 | static inline void virtio_store_mb(bool weak_barriers, |