Merge tag 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost
Pull virtio fixes from Michael Tsirkin: "Very late in the cycle but both risky if left unfixed and more or less obvious.." * tag 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost: vdpa/mlx5: Set err = -ENOMEM in case dma_map_sg_attrs fails vhost-vdpa: protect concurrent access to vhost device iotlb
This commit is contained in:
@@ -278,8 +278,10 @@ done:
|
|||||||
mr->log_size = log_entity_size;
|
mr->log_size = log_entity_size;
|
||||||
mr->nsg = nsg;
|
mr->nsg = nsg;
|
||||||
mr->nent = dma_map_sg_attrs(dma, mr->sg_head.sgl, mr->nsg, DMA_BIDIRECTIONAL, 0);
|
mr->nent = dma_map_sg_attrs(dma, mr->sg_head.sgl, mr->nsg, DMA_BIDIRECTIONAL, 0);
|
||||||
if (!mr->nent)
|
if (!mr->nent) {
|
||||||
|
err = -ENOMEM;
|
||||||
goto err_map;
|
goto err_map;
|
||||||
|
}
|
||||||
|
|
||||||
err = create_direct_mr(mvdev, mr);
|
err = create_direct_mr(mvdev, mr);
|
||||||
if (err)
|
if (err)
|
||||||
|
|||||||
@@ -745,9 +745,11 @@ static int vhost_vdpa_process_iotlb_msg(struct vhost_dev *dev,
|
|||||||
const struct vdpa_config_ops *ops = vdpa->config;
|
const struct vdpa_config_ops *ops = vdpa->config;
|
||||||
int r = 0;
|
int r = 0;
|
||||||
|
|
||||||
|
mutex_lock(&dev->mutex);
|
||||||
|
|
||||||
r = vhost_dev_check_owner(dev);
|
r = vhost_dev_check_owner(dev);
|
||||||
if (r)
|
if (r)
|
||||||
return r;
|
goto unlock;
|
||||||
|
|
||||||
switch (msg->type) {
|
switch (msg->type) {
|
||||||
case VHOST_IOTLB_UPDATE:
|
case VHOST_IOTLB_UPDATE:
|
||||||
@@ -768,6 +770,8 @@ static int vhost_vdpa_process_iotlb_msg(struct vhost_dev *dev,
|
|||||||
r = -EINVAL;
|
r = -EINVAL;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
unlock:
|
||||||
|
mutex_unlock(&dev->mutex);
|
||||||
|
|
||||||
return r;
|
return r;
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user