Re: [Qemu-devel] [PATCH] block: add watermark event

2014-08-08 Thread Eric Blake
On 08/08/2014 02:01 AM, Francesco Romani wrote: So let the block-set-watermark caller pass an absolute sector number instead. It keeps things simple for both QEMU and thin provisioning manager. >>> >>> No sector numbers in external interfaces, please. These units of 512 >>> bytes a

Re: [Qemu-devel] [PATCH] block: add watermark event

2014-08-08 Thread Francesco Romani
- Original Message - > From: "Stefan Hajnoczi" > To: "Kevin Wolf" > Cc: mdr...@linux.vnet.ibm.com, "Francesco Romani" , > qemu-devel@nongnu.org, lcapitul...@redhat.com > Sent: Tuesday, August 5, 2014 3:08:46 PM > Subject: Re: [Qemu-devel

Re: [Qemu-devel] [PATCH] block: add watermark event

2014-08-05 Thread Stefan Hajnoczi
On Tue, Aug 05, 2014 at 10:47:57AM +0200, Kevin Wolf wrote: > Am 01.08.2014 um 13:39 hat Stefan Hajnoczi geschrieben: > > On Tue, Jul 08, 2014 at 04:49:24PM +0200, Francesco Romani wrote: > > > @@ -5813,3 +5815,57 @@ void bdrv_flush_io_queue(BlockDriverState *bs) > > > bdrv_flush_io_queue(

Re: [Qemu-devel] [PATCH] block: add watermark event

2014-08-05 Thread Kevin Wolf
Am 01.08.2014 um 13:39 hat Stefan Hajnoczi geschrieben: > On Tue, Jul 08, 2014 at 04:49:24PM +0200, Francesco Romani wrote: > > @@ -5813,3 +5815,57 @@ void bdrv_flush_io_queue(BlockDriverState *bs) > > bdrv_flush_io_queue(bs->file); > > } > > } > > + > > +static bool watermark_exceed

Re: [Qemu-devel] [PATCH] block: add watermark event

2014-08-01 Thread Stefan Hajnoczi
On Tue, Jul 08, 2014 at 04:49:24PM +0200, Francesco Romani wrote: > @@ -5813,3 +5815,57 @@ void bdrv_flush_io_queue(BlockDriverState *bs) > bdrv_flush_io_queue(bs->file); > } > } > + > +static bool watermark_exceeded(BlockDriverState *bs, > + int64_t sec

Re: [Qemu-devel] [PATCH] block: add watermark event

2014-07-08 Thread Eric Blake
On 07/08/2014 08:49 AM, Francesco Romani wrote: > Managing applications, like oVirt (http://www.ovirt.org), make extensive > use of thin-provisioned disk images. > In order to let the guest run flawlessly and be not unnecessarily > paused, oVirt sets a watermark based on the percentage occupation o

[Qemu-devel] [PATCH] block: add watermark event

2014-07-08 Thread Francesco Romani
Managing applications, like oVirt (http://www.ovirt.org), make extensive use of thin-provisioned disk images. In order to let the guest run flawlessly and be not unnecessarily paused, oVirt sets a watermark based on the percentage occupation of the device against the advertised size, and automatica