[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/6] virtio: handle non contigious s/g entries
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 0/6] virtio: handle non contigious s/g entries |
Date: |
Tue, 27 Oct 2015 16:24:15 +0000 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Tue, Oct 27, 2015 at 01:51:20PM +0200, Michael S. Tsirkin wrote:
> On Tue, Oct 27, 2015 at 10:47:54AM +0200, Michael S. Tsirkin wrote:
> > TL;DR:
> > This fixes virtio in a way transparent to guest.
> > We should now be able to revert commits aa8580cd and df0acded19ec which
> > worked
> > around it in a way that's not transparent.
>
> I didn't check dataplane BTW. Igor? Stefan?
It doesn't handle it:
/* TODO handle non-contiguous memory across region boundaries */
iov->iov_base = vring_map(&mr, desc->addr, desc->len,
desc->flags & VRING_DESC_F_WRITE);
if (!iov->iov_base) {
error_report("Failed to map descriptor addr %#" PRIx64 " len %u",
(uint64_t)desc->addr, desc->len);
return -EFAULT;
}
The assumption is that vring_map() can map the full desc->len bytes.
The same logic that you're trying to add in this series could be added to
vring_pop() but it makes the number of sg list entries accepted by QEMU
variable and dependent on DIMMs :(. Previously guests could be sure that 1024
sgs work.
- [Qemu-devel] [PATCH 4/6] virtio-serial: convert to virtio_map, (continued)
- [Qemu-devel] [PATCH 4/6] virtio-serial: convert to virtio_map, Michael S. Tsirkin, 2015/10/27
- [Qemu-devel] [PATCH 3/6] virtio-blk: convert to virtqueue_map, Michael S. Tsirkin, 2015/10/27
- [Qemu-devel] [PATCH 5/6] virtio-scsi: convert to virtqueue_map, Michael S. Tsirkin, 2015/10/27
- [Qemu-devel] [PATCH 6/6] virtio: drop virtqueue_map_sg, Michael S. Tsirkin, 2015/10/27
- Re: [Qemu-devel] [PATCH 0/6] virtio: handle non contigious s/g entries, Michael S. Tsirkin, 2015/10/27
- Re: [Qemu-devel] [PATCH 0/6] virtio: handle non contigious s/g entries, Michael S. Tsirkin, 2015/10/27
- Re: [Qemu-devel] [PATCH 0/6] virtio: handle non contigious s/g entries, Cornelia Huck, 2015/10/27