On 09/29/2017 02:34 PM, Yuanhan Liu wrote:
On Sun, Sep 24, 2017 at 06:19:19PM +0200, Maxime Coquelin wrote:
  static int
-vhost_user_iotlb_msg(struct virtio_net *dev, struct VhostUserMsg *msg)
+is_vring_iotlb_update(struct vhost_virtqueue *vq, struct vhost_iotlb_msg *imsg)
  {
+       struct vhost_vring_addr *ra;
+       uint64_t start, end;
+
+       start = imsg->iova;
+       end = start + imsg->size;
+
+       ra = &vq->ring_addrs;
+       if (ra->desc_user_addr >= start && ra->desc_user_addr < end)
+               return 1;
+       if (ra->avail_user_addr >= start && ra->avail_user_addr < end)
+               return 1;
+       if (ra->used_user_addr >= start && ra->used_user_addr < end)
+               return 1;
+
+       return -1;

It should be "return 0" here.

Oops, thanks for spotting this.

Maxime

        --yliu
+}

Reply via email to