diff options
author | Ohad Ben-Cohen <ohad@wizery.com> | 2012-01-12 08:26:57 +0100 |
---|---|---|
committer | Ohad Ben-Cohen <ohad@wizery.com> | 2012-02-22 17:28:48 +0100 |
commit | dd6da1c5e99a3b736b52370c504a2b8843cbfda6 (patch) | |
tree | c6d08a5fb72f817d5071c1e9338e9fe5a98e5fad /drivers | |
parent | rpmsg: rename virtqueue_add_buf_gfp to virtqueue_add_buf (diff) | |
download | linux-dd6da1c5e99a3b736b52370c504a2b8843cbfda6.tar.xz linux-dd6da1c5e99a3b736b52370c504a2b8843cbfda6.zip |
remoteproc: don't use virtio's weak barriers
When creating a virtqueue for rpmsg, tell virtio we're not interested
in "weak" smp barriers, since we're talking to a real device.
On ARM, this means using a DSB instead of a DMB, which is needed
for platforms that kick the remote processor using some kind of
a mailbox device mapped to Device memory (otherwise the kick can
jump ahead and wake the remote processor before it has observed
the changes to the vrings).
Signed-off-by: Ohad Ben-Cohen <ohad@wizery.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/remoteproc/remoteproc_rpmsg.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/remoteproc/remoteproc_rpmsg.c b/drivers/remoteproc/remoteproc_rpmsg.c index f30be4c193ef..4f73e811bb80 100644 --- a/drivers/remoteproc/remoteproc_rpmsg.c +++ b/drivers/remoteproc/remoteproc_rpmsg.c @@ -96,7 +96,11 @@ static struct virtqueue *rp_find_vq(struct virtio_device *vdev, dev_dbg(rproc->dev, "vring%d: va %p qsz %d\n", id, addr, len); - vq = vring_new_virtqueue(len, AMP_VRING_ALIGN, vdev, addr, + /* + * Create the new vq, and tell virtio we're not interested in + * the 'weak' smp barriers, since we're talking with a real device. + */ + vq = vring_new_virtqueue(len, AMP_VRING_ALIGN, vdev, false, addr, rproc_virtio_notify, callback, name); if (!vq) { dev_err(rproc->dev, "vring_new_virtqueue %s failed\n", name); |