Re: [Qemu-devel] [RFC PATCH 6/8] quorum: Avoid bdrv_aio_writev() for rewrites

2016-11-21 Thread Kevin Wolf
Am 18.11.2016 um 22:11 hat Eric Blake geschrieben: > On 11/18/2016 06:21 AM, Kevin Wolf wrote: > > >>> +ret = bdrv_co_pwritev(s->children[co->i], > >>> + acb->sector_num * BDRV_SECTOR_SIZE, > >>> + acb->nb_sectors * BDRV_SECTOR_SIZE, > >>> +

Re: [Qemu-devel] [RFC PATCH 6/8] quorum: Avoid bdrv_aio_writev() for rewrites

2016-11-18 Thread Eric Blake
On 11/18/2016 06:21 AM, Kevin Wolf wrote: >>> +ret = bdrv_co_pwritev(s->children[co->i], >>> + acb->sector_num * BDRV_SECTOR_SIZE, >>> + acb->nb_sectors * BDRV_SECTOR_SIZE, >>> + acb->qiov, 0); >>> +(void) ret; >> >

Re: [Qemu-devel] [RFC PATCH 6/8] quorum: Avoid bdrv_aio_writev() for rewrites

2016-11-18 Thread Alberto Garcia
On Fri 18 Nov 2016 01:21:12 PM CET, Kevin Wolf wrote: >> > +/* one less rewrite to do */ >> > +acb->rewrite_count--; >> > +qemu_coroutine_enter_if_inactive(acb->co); >> >> I think you should only enter acb->co when acb->rewrite_count reaches >> zero. >> >> In all other cases the main

Re: [Qemu-devel] [RFC PATCH 6/8] quorum: Avoid bdrv_aio_writev() for rewrites

2016-11-18 Thread Kevin Wolf
Am 17.11.2016 um 15:54 hat Alberto Garcia geschrieben: > On Thu 10 Nov 2016 06:19:07 PM CET, Kevin Wolf wrote: > > +static void quorum_rewrite_entry(void *opaque) > > +{ > > +QuorumCo *co = opaque; > > +QuorumAIOCB *acb = co->acb; > > +BDRVQuorumState *s = acb->bs->opaque; > > +int

Re: [Qemu-devel] [RFC PATCH 6/8] quorum: Avoid bdrv_aio_writev() for rewrites

2016-11-17 Thread Alberto Garcia
On Thu 10 Nov 2016 06:19:07 PM CET, Kevin Wolf wrote: > Replacing it with bdrv_co_pwritev() prepares us for byte granularity > requests and gets us rid of the last bdrv_aio_*() user in quorum. > > Signed-off-by: Kevin Wolf > --- > block/quorum.c | 52 +-

Re: [Qemu-devel] [RFC PATCH 6/8] quorum: Avoid bdrv_aio_writev() for rewrites

2016-11-10 Thread Eric Blake
On 11/10/2016 11:19 AM, Kevin Wolf wrote: > Replacing it with bdrv_co_pwritev() prepares us for byte granularity > requests and gets us rid of the last bdrv_aio_*() user in quorum. > > Signed-off-by: Kevin Wolf > --- > block/quorum.c | 52 +--- > 1

[Qemu-devel] [RFC PATCH 6/8] quorum: Avoid bdrv_aio_writev() for rewrites

2016-11-10 Thread Kevin Wolf
Replacing it with bdrv_co_pwritev() prepares us for byte granularity requests and gets us rid of the last bdrv_aio_*() user in quorum. Signed-off-by: Kevin Wolf --- block/quorum.c | 52 +--- 1 file changed, 33 insertions(+), 19 deletions(-) diff -