Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] dataplane: simplify indirect descriptor read

2015-11-03 Thread Igor Mammedov
On Mon, 2 Nov 2015 17:43:16 + Stefan Hajnoczi wrote: > On Wed, Oct 28, 2015 at 05:48:02PM +0200, Michael S. Tsirkin wrote: > > Use address_space_read to make sure we handle the case of an indirect > > descriptor crossing DIMM boundary correctly. > > > > Signed-off-by: Michael S. Tsirkin > >

Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] dataplane: simplify indirect descriptor read

2015-10-30 Thread Stefan Hajnoczi
On Thu, Oct 29, 2015 at 11:28:05AM +0100, Igor Mammedov wrote: > On Wed, 28 Oct 2015 17:48:02 +0200 > "Michael S. Tsirkin" wrote: > > -struct vring_desc *desc_ptr; > > -MemoryRegion *mr; > > - > > /* Translate indirect descriptor */ > > -desc_ptr = vring_map(&mr, >

Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] dataplane: simplify indirect descriptor read

2015-10-29 Thread Igor Mammedov
On Wed, 28 Oct 2015 17:48:02 +0200 "Michael S. Tsirkin" wrote: > Use address_space_read to make sure we handle the case of an indirect > descriptor crossing DIMM boundary correctly. > > Signed-off-by: Michael S. Tsirkin Tested-by: Igor Mammedov > --- > > Warning: compile-tested only. > > h

Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] dataplane: simplify indirect descriptor read

2015-10-29 Thread Igor Mammedov
On Wed, 28 Oct 2015 17:48:02 +0200 "Michael S. Tsirkin" wrote: > Use address_space_read to make sure we handle the case of an indirect > descriptor crossing DIMM boundary correctly. > > Signed-off-by: Michael S. Tsirkin > --- > > Warning: compile-tested only. > > hw/virtio/dataplane/vring.c