Re: [Qemu-devel] [Qemu-block] [PATCH] Added iopmem device emulation

2016-11-09 Thread Stefan Hajnoczi
On Tue, Nov 08, 2016 at 09:46:47AM -0700, Logan Gunthorpe wrote: > On 08/11/16 08:58 AM, Stefan Hajnoczi wrote: > > My concern with the current implementation is that a PCI MMIO access > > invokes a synchronous blk_*() call. That can pause vcpu execution while > > I/O is happening and therefore le

Re: [Qemu-devel] [Qemu-block] [PATCH] Added iopmem device emulation

2016-11-08 Thread Logan Gunthorpe
Hey, On 08/11/16 08:58 AM, Stefan Hajnoczi wrote: > My concern with the current implementation is that a PCI MMIO access > invokes a synchronous blk_*() call. That can pause vcpu execution while > I/O is happening and therefore leads to unresponsive guests. QEMU's > monitor interface is also blo

Re: [Qemu-devel] [Qemu-block] [PATCH] Added iopmem device emulation

2016-11-08 Thread Stefan Hajnoczi
On Mon, Nov 07, 2016 at 10:09:29AM -0700, Logan Gunthorpe wrote: > On 07/11/16 03:28 AM, Stefan Hajnoczi wrote: > > It may be too early to merge this code into qemu.git if there is no > > hardware spec and this is a prototype device that is subject to change. > > Fair enough, though the interface

Re: [Qemu-devel] [Qemu-block] [PATCH] Added iopmem device emulation

2016-11-07 Thread Logan Gunthorpe
Hey, On 07/11/16 03:28 AM, Stefan Hajnoczi wrote: > It may be too early to merge this code into qemu.git if there is no > hardware spec and this is a prototype device that is subject to change. Fair enough, though the interface is so simple I don't know what could possibly change. > I'm wonderin

Re: [Qemu-devel] [Qemu-block] [PATCH] Added iopmem device emulation

2016-11-07 Thread Stefan Hajnoczi
On Fri, Nov 04, 2016 at 09:47:33AM -0600, Logan Gunthorpe wrote: > On 04/11/16 04:49 AM, Stefan Hajnoczi wrote: > > QEMU already has NVDIMM support (https://pmem.io/). It can be used both > > for passthrough and fake non-volatile memory: > > > > qemu-system-x86_64 \ > > -M pc,nvdimm=on \ >

Re: [Qemu-devel] [Qemu-block] [PATCH] Added iopmem device emulation

2016-11-04 Thread Logan Gunthorpe
Hi Stefan, On 04/11/16 04:49 AM, Stefan Hajnoczi wrote: > QEMU already has NVDIMM support (https://pmem.io/). It can be used both > for passthrough and fake non-volatile memory: > > qemu-system-x86_64 \ > -M pc,nvdimm=on \ > -m 1024,maxmem=$((4096 * 1024 * 1024)),slots=2 \ > -objec

Re: [Qemu-devel] [Qemu-block] [PATCH] Added iopmem device emulation

2016-11-04 Thread Stefan Hajnoczi
On Tue, Oct 18, 2016 at 03:55:11PM -0600, Logan Gunthorpe wrote: > An iopmem device is one which exposes volatile or non-volatile memory mapped > directly to a BAR region. One purpose is to provide buffers to do peer > to peer transfers on the bus. As such this device uses QEMU's drive > backing st