Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-30 Thread Peter Lieven
Am 30.05.2016 um 12:06 schrieb Kevin Wolf: Am 30.05.2016 um 11:53 hat Peter Lieven geschrieben: Am 30.05.2016 um 11:47 schrieb Kevin Wolf: Am 30.05.2016 um 11:30 hat Peter Lieven geschrieben: Am 30.05.2016 um 10:24 schrieb Kevin Wolf: Am 30.05.2016 um 08:25 hat Peter Lieven geschrieben: Am

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-30 Thread Kevin Wolf
Am 30.05.2016 um 11:53 hat Peter Lieven geschrieben: > Am 30.05.2016 um 11:47 schrieb Kevin Wolf: > >Am 30.05.2016 um 11:30 hat Peter Lieven geschrieben: > >>Am 30.05.2016 um 10:24 schrieb Kevin Wolf: > >>>Am 30.05.2016 um 08:25 hat Peter Lieven geschrieben: > Am 27.05.2016 um 10:55 schrieb

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-30 Thread Peter Lieven
Am 30.05.2016 um 11:47 schrieb Kevin Wolf: Am 30.05.2016 um 11:30 hat Peter Lieven geschrieben: Am 30.05.2016 um 10:24 schrieb Kevin Wolf: Am 30.05.2016 um 08:25 hat Peter Lieven geschrieben: Am 27.05.2016 um 10:55 schrieb Kevin Wolf: Am 27.05.2016 um 02:36 hat Fam Zheng geschrieben: On

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-30 Thread Peter Lieven
Am 30.05.2016 um 10:24 schrieb Kevin Wolf: Am 30.05.2016 um 08:25 hat Peter Lieven geschrieben: Am 27.05.2016 um 10:55 schrieb Kevin Wolf: Am 27.05.2016 um 02:36 hat Fam Zheng geschrieben: On Thu, 05/26 11:20, Paolo Bonzini wrote: On 26/05/2016 10:30, Fam Zheng wrote: This doesn't look too

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-30 Thread Kevin Wolf
Am 30.05.2016 um 11:30 hat Peter Lieven geschrieben: > Am 30.05.2016 um 10:24 schrieb Kevin Wolf: > >Am 30.05.2016 um 08:25 hat Peter Lieven geschrieben: > >>Am 27.05.2016 um 10:55 schrieb Kevin Wolf: > >>>Am 27.05.2016 um 02:36 hat Fam Zheng geschrieben: > On Thu, 05/26 11:20, Paolo Bonzini

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-30 Thread Kevin Wolf
Am 30.05.2016 um 08:25 hat Peter Lieven geschrieben: > Am 27.05.2016 um 10:55 schrieb Kevin Wolf: > >Am 27.05.2016 um 02:36 hat Fam Zheng geschrieben: > >>On Thu, 05/26 11:20, Paolo Bonzini wrote: > >>>On 26/05/2016 10:30, Fam Zheng wrote: > >>This doesn't look too wrong... Should the right

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-30 Thread Peter Lieven
Am 27.05.2016 um 10:55 schrieb Kevin Wolf: Am 27.05.2016 um 02:36 hat Fam Zheng geschrieben: On Thu, 05/26 11:20, Paolo Bonzini wrote: On 26/05/2016 10:30, Fam Zheng wrote: This doesn't look too wrong... Should the right sequence of events be head/after_head or head/after_tail? It's

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-27 Thread Kevin Wolf
Am 27.05.2016 um 02:36 hat Fam Zheng geschrieben: > On Thu, 05/26 11:20, Paolo Bonzini wrote: > > On 26/05/2016 10:30, Fam Zheng wrote: > > >> > > > >> > This doesn't look too wrong... Should the right sequence of events be > > >> > head/after_head or head/after_tail? It's probably simplest to

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-26 Thread Fam Zheng
On Thu, 05/26 11:20, Paolo Bonzini wrote: > > > On 26/05/2016 10:30, Fam Zheng wrote: > >> > > >> > This doesn't look too wrong... Should the right sequence of events be > >> > head/after_head or head/after_tail? It's probably simplest to just emit > >> > all four events. > > I've no idea.

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-26 Thread Paolo Bonzini
On 26/05/2016 10:30, Fam Zheng wrote: >> > >> > This doesn't look too wrong... Should the right sequence of events be >> > head/after_head or head/after_tail? It's probably simplest to just emit >> > all four events. > I've no idea. (That's why I leaned towards fixing the test case). Well,

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-26 Thread Fam Zheng
On Thu, 05/26 09:55, Paolo Bonzini wrote: > > > On 26/05/2016 09:10, Fam Zheng wrote: > > > > diff --git a/block/io.c b/block/io.c > > index d480097..a6523cf 100644 > > --- a/block/io.c > > +++ b/block/io.c > > @@ -1435,8 +1435,10 @@ int coroutine_fn bdrv_co_pwritev(BlockDriverState > > *bs, >

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-26 Thread Paolo Bonzini
On 26/05/2016 09:10, Fam Zheng wrote: > > diff --git a/block/io.c b/block/io.c > index d480097..a6523cf 100644 > --- a/block/io.c > +++ b/block/io.c > @@ -1435,8 +1435,10 @@ int coroutine_fn bdrv_co_pwritev(BlockDriverState *bs, > * than one aligned block. > */ > if

Re: [Qemu-block] [Qemu-devel] [PATCH V2] block/io: optimize bdrv_co_pwritev for small requests

2016-05-26 Thread Fam Zheng
On Thu, 05/26 14:50, Fam Zheng wrote: > On Tue, 05/24 16:30, Peter Lieven wrote: > > in a read-modify-write cycle a small request might cause > > head and tail to fall into the same aligned block. Currently > > QEMU reads the same block twice in this case which is > > not necessary. > > > >