* [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() @ 2019-08-22 18:31 Andrey Shinkevich 2019-08-22 18:55 ` Vladimir Sementsov-Ogievskiy 2019-08-22 21:09 ` Eric Blake 0 siblings, 2 replies; 8+ messages in thread From: Andrey Shinkevich @ 2019-08-22 18:31 UTC (permalink / raw) To: qemu-devel, qemu-block Cc: kwolf, fam, vsementsov, mreitz, stefanha, andrey.shinkevich, den Revert the commit 118f99442d 'block/io.c: fix for the allocation failure' and make better error handling for the file systems that do not support fallocate() for the unaligned byte range. Allow falling back to pwrite in case fallocate() returns EINVAL. Suggested-by: Kevin Wolf <kwolf@redhat.com> Suggested-by: Eric Blake <eblake@redhat.com> Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com> --- Discussed in email thread with the message ID <1554474244-553661-1-git-send-email-andrey.shinkevich@virtuozzo.com> block/file-posix.c | 7 +++++++ block/io.c | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/block/file-posix.c b/block/file-posix.c index fbeb006..2c254ff 100644 --- a/block/file-posix.c +++ b/block/file-posix.c @@ -1588,6 +1588,13 @@ static int handle_aiocb_write_zeroes(void *opaque) if (s->has_write_zeroes) { int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, aiocb->aio_offset, aiocb->aio_nbytes); + if (ret == -EINVAL) { + /* + * Allow falling back to pwrite for file systems that + * do not support fallocate() for unaligned byte range. + */ + return -ENOTSUP; + } if (ret == 0 || ret != -ENOTSUP) { return ret; } diff --git a/block/io.c b/block/io.c index 56bbf19..58f08cd 100644 --- a/block/io.c +++ b/block/io.c @@ -1558,7 +1558,7 @@ static int coroutine_fn bdrv_co_do_pwrite_zeroes(BlockDriverState *bs, assert(!bs->supported_zero_flags); } - if (ret < 0 && !(flags & BDRV_REQ_NO_FALLBACK)) { + if (ret == -ENOTSUP && !(flags & BDRV_REQ_NO_FALLBACK)) { /* Fall back to bounce buffer if write zeroes is unsupported */ BdrvRequestFlags write_flags = flags & ~BDRV_REQ_ZERO_WRITE; -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() 2019-08-22 18:31 [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() Andrey Shinkevich @ 2019-08-22 18:55 ` Vladimir Sementsov-Ogievskiy 2019-08-22 19:10 ` Vladimir Sementsov-Ogievskiy 2019-08-22 21:09 ` Eric Blake 1 sibling, 1 reply; 8+ messages in thread From: Vladimir Sementsov-Ogievskiy @ 2019-08-22 18:55 UTC (permalink / raw) To: Andrey Shinkevich, qemu-devel, qemu-block Cc: kwolf, fam, Denis Lunev, mreitz, stefanha 22.08.2019 21:31, Andrey Shinkevich wrote: > Revert the commit 118f99442d 'block/io.c: fix for the allocation failure' > and make better error handling for the file systems that do not support > fallocate() for the unaligned byte range. Allow falling back to pwrite > in case fallocate() returns EINVAL. > > Suggested-by: Kevin Wolf <kwolf@redhat.com> > Suggested-by: Eric Blake <eblake@redhat.com> > Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com> > --- > Discussed in email thread with the message ID > <1554474244-553661-1-git-send-email-andrey.shinkevich@virtuozzo.com> > > block/file-posix.c | 7 +++++++ > block/io.c | 2 +- > 2 files changed, 8 insertions(+), 1 deletion(-) > > diff --git a/block/file-posix.c b/block/file-posix.c > index fbeb006..2c254ff 100644 > --- a/block/file-posix.c > +++ b/block/file-posix.c > @@ -1588,6 +1588,13 @@ static int handle_aiocb_write_zeroes(void *opaque) > if (s->has_write_zeroes) { > int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, > aiocb->aio_offset, aiocb->aio_nbytes); > + if (ret == -EINVAL) { > + /* > + * Allow falling back to pwrite for file systems that > + * do not support fallocate() for unaligned byte range. > + */ > + return -ENOTSUP; > + } > if (ret == 0 || ret != -ENOTSUP) { > return ret; > } Hmm stop, you've done exactly what Den was afraid of: the next line s->has_write_zeroes = false; will disable write_zeroes forever. Something like --- a/block/file-posix.c +++ b/block/file-posix.c @@ -1588,10 +1588,12 @@ static int handle_aiocb_write_zeroes(void *opaque) if (s->has_write_zeroes) { int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, aiocb->aio_offset, aiocb->aio_nbytes); - if (ret == 0 || ret != -ENOTSUP) { + if (ret == 0 || (ret != -ENOTSUP && ret != -EINVAL)) { return ret; } - s->has_write_zeroes = false; + if (ret == -ENOTSUP) { + s->has_write_zeroes = false; + } } #endif will work better. So, handle ENOTSUP as "disable write_zeros forever", and EINVAL as "don't disable, but fallback to writing zeros". And we need same handling for following do_fallocate() calls too (otherwise they again fails with EINVAL which will break the whole thing). > diff --git a/block/io.c b/block/io.c > index 56bbf19..58f08cd 100644 > --- a/block/io.c > +++ b/block/io.c > @@ -1558,7 +1558,7 @@ static int coroutine_fn bdrv_co_do_pwrite_zeroes(BlockDriverState *bs, > assert(!bs->supported_zero_flags); > } > > - if (ret < 0 && !(flags & BDRV_REQ_NO_FALLBACK)) { > + if (ret == -ENOTSUP && !(flags & BDRV_REQ_NO_FALLBACK)) { > /* Fall back to bounce buffer if write zeroes is unsupported */ > BdrvRequestFlags write_flags = flags & ~BDRV_REQ_ZERO_WRITE; > > -- Best regards, Vladimir ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() 2019-08-22 18:55 ` Vladimir Sementsov-Ogievskiy @ 2019-08-22 19:10 ` Vladimir Sementsov-Ogievskiy 0 siblings, 0 replies; 8+ messages in thread From: Vladimir Sementsov-Ogievskiy @ 2019-08-22 19:10 UTC (permalink / raw) To: Andrey Shinkevich, qemu-devel, qemu-block Cc: kwolf, fam, Denis Lunev, mreitz, stefanha 22.08.2019 21:55, Vladimir Sementsov-Ogievskiy wrote: > 22.08.2019 21:31, Andrey Shinkevich wrote: >> Revert the commit 118f99442d 'block/io.c: fix for the allocation failure' >> and make better error handling for the file systems that do not support >> fallocate() for the unaligned byte range. Allow falling back to pwrite >> in case fallocate() returns EINVAL. >> >> Suggested-by: Kevin Wolf <kwolf@redhat.com> >> Suggested-by: Eric Blake <eblake@redhat.com> >> Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com> >> --- >> Discussed in email thread with the message ID >> <1554474244-553661-1-git-send-email-andrey.shinkevich@virtuozzo.com> >> >> block/file-posix.c | 7 +++++++ >> block/io.c | 2 +- >> 2 files changed, 8 insertions(+), 1 deletion(-) >> >> diff --git a/block/file-posix.c b/block/file-posix.c >> index fbeb006..2c254ff 100644 >> --- a/block/file-posix.c >> +++ b/block/file-posix.c >> @@ -1588,6 +1588,13 @@ static int j(void *opaque) >> if (s->has_write_zeroes) { >> int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, >> aiocb->aio_offset, aiocb->aio_nbytes); >> + if (ret == -EINVAL) { >> + /* >> + * Allow falling back to pwrite for file systems that >> + * do not support fallocate() for unaligned byte range. >> + */ >> + return -ENOTSUP; >> + } >> if (ret == 0 || ret != -ENOTSUP) { >> return ret; >> } > > Hmm stop, you've done exactly what Den was afraid of: > > the next line > s->has_write_zeroes = false; > > will disable write_zeroes forever. > > Something like > > --- a/block/file-posix.c > +++ b/block/file-posix.c > @@ -1588,10 +1588,12 @@ static int handle_aiocb_write_zeroes(void *opaque) > if (s->has_write_zeroes) { > int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, > aiocb->aio_offset, aiocb->aio_nbytes); > - if (ret == 0 || ret != -ENOTSUP) { > + if (ret == 0 || (ret != -ENOTSUP && ret != -EINVAL)) { > return ret; > } > - s->has_write_zeroes = false; > + if (ret == -ENOTSUP) { > + s->has_write_zeroes = false; > + } > } > #endif > > > will work better. So, handle ENOTSUP as "disable write_zeros forever", and EINVAL as > "don't disable, but fallback to writing zeros". And we need same handling for following do_fallocate() calls > too (otherwise they again fails with EINVAL which will break the whole thing). > Oops, sorry, I misread your patch, it's OK. Still we may want to handle other do_fallocate() calls in same manner, or may be just: @@ -1558,7 +1558,13 @@ static int coroutine_fn bdrv_co_do_pwrite_zeroes(BlockDriverState *bs, assert(!bs->supported_zero_flags); } - if (ret < 0 && !(flags & BDRV_REQ_NO_FALLBACK)) { + /* + * We are sure that our arguments make sense, so consider "invalid + * argument" in same manner as "not supported". + */ + if ((ret == -ENOTSUP || ret == -EINVAL) && + !(flags & BDRV_REQ_NO_FALLBACK)) + { /* Fall back to bounce buffer if write zeroes is unsupported */ BdrvRequestFlags write_flags = flags & ~BDRV_REQ_ZERO_WRITE; -- Best regards, Vladimir ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() 2019-08-22 18:31 [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() Andrey Shinkevich 2019-08-22 18:55 ` Vladimir Sementsov-Ogievskiy @ 2019-08-22 21:09 ` Eric Blake 2019-08-23 12:07 ` Andrey Shinkevich ` (2 more replies) 1 sibling, 3 replies; 8+ messages in thread From: Eric Blake @ 2019-08-22 21:09 UTC (permalink / raw) To: Andrey Shinkevich, qemu-devel, qemu-block Cc: kwolf, fam, vsementsov, mreitz, stefanha, den [-- Attachment #1.1: Type: text/plain, Size: 2286 bytes --] On 8/22/19 1:31 PM, Andrey Shinkevich wrote: > Revert the commit 118f99442d 'block/io.c: fix for the allocation failure' > and make better error handling for the file systems that do not support s/make/use/ > fallocate() for the unaligned byte range. Allow falling back to pwrite s/the/an/ > in case fallocate() returns EINVAL. > > Suggested-by: Kevin Wolf <kwolf@redhat.com> > Suggested-by: Eric Blake <eblake@redhat.com> > Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com> > --- > Discussed in email thread with the message ID > <1554474244-553661-1-git-send-email-andrey.shinkevich@virtuozzo.com> > > block/file-posix.c | 7 +++++++ > block/io.c | 2 +- > 2 files changed, 8 insertions(+), 1 deletion(-) > > diff --git a/block/file-posix.c b/block/file-posix.c > index fbeb006..2c254ff 100644 > --- a/block/file-posix.c > +++ b/block/file-posix.c > @@ -1588,6 +1588,13 @@ static int handle_aiocb_write_zeroes(void *opaque) > if (s->has_write_zeroes) { > int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, > aiocb->aio_offset, aiocb->aio_nbytes); > + if (ret == -EINVAL) { > + /* > + * Allow falling back to pwrite for file systems that > + * do not support fallocate() for unaligned byte range. s/for/for an/ > + */ > + return -ENOTSUP; > + } > if (ret == 0 || ret != -ENOTSUP) { > return ret; > } > diff --git a/block/io.c b/block/io.c > index 56bbf19..58f08cd 100644 > --- a/block/io.c > +++ b/block/io.c > @@ -1558,7 +1558,7 @@ static int coroutine_fn bdrv_co_do_pwrite_zeroes(BlockDriverState *bs, > assert(!bs->supported_zero_flags); > } > > - if (ret < 0 && !(flags & BDRV_REQ_NO_FALLBACK)) { > + if (ret == -ENOTSUP && !(flags & BDRV_REQ_NO_FALLBACK)) { > /* Fall back to bounce buffer if write zeroes is unsupported */ > BdrvRequestFlags write_flags = flags & ~BDRV_REQ_ZERO_WRITE; > > Reviewed-by: Eric Blake <eblake@redhat.com> -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() 2019-08-22 21:09 ` Eric Blake @ 2019-08-23 12:07 ` Andrey Shinkevich 2019-08-27 12:35 ` Denis V. Lunev 2019-08-27 12:39 ` Andrey Shinkevich 2 siblings, 0 replies; 8+ messages in thread From: Andrey Shinkevich @ 2019-08-23 12:07 UTC (permalink / raw) To: Eric Blake, qemu-devel, qemu-block Cc: kwolf, fam, Vladimir Sementsov-Ogievskiy, Denis Lunev, mreitz, stefanha On 23/08/2019 00:09, Eric Blake wrote: > On 8/22/19 1:31 PM, Andrey Shinkevich wrote: >> Revert the commit 118f99442d 'block/io.c: fix for the allocation failure' >> and make better error handling for the file systems that do not support > > s/make/use/ > >> fallocate() for the unaligned byte range. Allow falling back to pwrite > > s/the/an/ > >> in case fallocate() returns EINVAL. >> >> Suggested-by: Kevin Wolf <kwolf@redhat.com> >> Suggested-by: Eric Blake <eblake@redhat.com> >> Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com> >> --- >> Discussed in email thread with the message ID >> <1554474244-553661-1-git-send-email-andrey.shinkevich@virtuozzo.com> >> >> block/file-posix.c | 7 +++++++ >> block/io.c | 2 +- >> 2 files changed, 8 insertions(+), 1 deletion(-) >> >> diff --git a/block/file-posix.c b/block/file-posix.c >> index fbeb006..2c254ff 100644 >> --- a/block/file-posix.c >> +++ b/block/file-posix.c >> @@ -1588,6 +1588,13 @@ static int handle_aiocb_write_zeroes(void *opaque) >> if (s->has_write_zeroes) { >> int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, >> aiocb->aio_offset, aiocb->aio_nbytes); >> + if (ret == -EINVAL) { >> + /* >> + * Allow falling back to pwrite for file systems that >> + * do not support fallocate() for unaligned byte range. > > s/for/for an/ > >> + */ >> + return -ENOTSUP; >> + } >> if (ret == 0 || ret != -ENOTSUP) { >> return ret; >> } >> diff --git a/block/io.c b/block/io.c >> index 56bbf19..58f08cd 100644 >> --- a/block/io.c >> +++ b/block/io.c >> @@ -1558,7 +1558,7 @@ static int coroutine_fn bdrv_co_do_pwrite_zeroes(BlockDriverState *bs, >> assert(!bs->supported_zero_flags); >> } >> >> - if (ret < 0 && !(flags & BDRV_REQ_NO_FALLBACK)) { >> + if (ret == -ENOTSUP && !(flags & BDRV_REQ_NO_FALLBACK)) { >> /* Fall back to bounce buffer if write zeroes is unsupported */ >> BdrvRequestFlags write_flags = flags & ~BDRV_REQ_ZERO_WRITE; >> >> > > Reviewed-by: Eric Blake <eblake@redhat.com> > Thank you Eric! Also, I would prefer "make the finer error handling" because the word 'better' has a wide sense. I have prepared the v2 with your corrections and am waiting for Denis Lunev's response. He is coming back to work from his vacation on Tuesday. Andrey -- With the best regards, Andrey Shinkevich ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() 2019-08-22 21:09 ` Eric Blake 2019-08-23 12:07 ` Andrey Shinkevich @ 2019-08-27 12:35 ` Denis V. Lunev 2019-08-27 12:39 ` Andrey Shinkevich 2 siblings, 0 replies; 8+ messages in thread From: Denis V. Lunev @ 2019-08-27 12:35 UTC (permalink / raw) To: Eric Blake, Andrey Shinkevich, qemu-devel, qemu-block Cc: kwolf, fam, Vladimir Sementsov-Ogievskiy, stefanha, mreitz On 8/22/19 11:09 PM, Eric Blake wrote: > On 8/22/19 1:31 PM, Andrey Shinkevich wrote: >> Revert the commit 118f99442d 'block/io.c: fix for the allocation failure' >> and make better error handling for the file systems that do not support > s/make/use/ > >> fallocate() for the unaligned byte range. Allow falling back to pwrite > s/the/an/ > >> in case fallocate() returns EINVAL. >> >> Suggested-by: Kevin Wolf <kwolf@redhat.com> >> Suggested-by: Eric Blake <eblake@redhat.com> >> Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com> >> --- >> Discussed in email thread with the message ID >> <1554474244-553661-1-git-send-email-andrey.shinkevich@virtuozzo.com> >> >> block/file-posix.c | 7 +++++++ >> block/io.c | 2 +- >> 2 files changed, 8 insertions(+), 1 deletion(-) >> >> diff --git a/block/file-posix.c b/block/file-posix.c >> index fbeb006..2c254ff 100644 >> --- a/block/file-posix.c >> +++ b/block/file-posix.c >> @@ -1588,6 +1588,13 @@ static int handle_aiocb_write_zeroes(void *opaque) >> if (s->has_write_zeroes) { >> int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, >> aiocb->aio_offset, aiocb->aio_nbytes); >> + if (ret == -EINVAL) { >> + /* >> + * Allow falling back to pwrite for file systems that >> + * do not support fallocate() for unaligned byte range. > s/for/for an/ > >> + */ >> + return -ENOTSUP; >> + } >> if (ret == 0 || ret != -ENOTSUP) { >> return ret; >> } >> diff --git a/block/io.c b/block/io.c >> index 56bbf19..58f08cd 100644 >> --- a/block/io.c >> +++ b/block/io.c >> @@ -1558,7 +1558,7 @@ static int coroutine_fn bdrv_co_do_pwrite_zeroes(BlockDriverState *bs, >> assert(!bs->supported_zero_flags); >> } >> >> - if (ret < 0 && !(flags & BDRV_REQ_NO_FALLBACK)) { >> + if (ret == -ENOTSUP && !(flags & BDRV_REQ_NO_FALLBACK)) { >> /* Fall back to bounce buffer if write zeroes is unsupported */ >> BdrvRequestFlags write_flags = flags & ~BDRV_REQ_ZERO_WRITE; >> >> > Reviewed-by: Eric Blake <eblake@redhat.com> > Reviewed-by: Denis V. Lunev <den@openvz.org> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() 2019-08-22 21:09 ` Eric Blake 2019-08-23 12:07 ` Andrey Shinkevich 2019-08-27 12:35 ` Denis V. Lunev @ 2019-08-27 12:39 ` Andrey Shinkevich 2019-08-27 13:50 ` Eric Blake 2 siblings, 1 reply; 8+ messages in thread From: Andrey Shinkevich @ 2019-08-27 12:39 UTC (permalink / raw) To: Eric Blake, qemu-devel, qemu-block Cc: kwolf, fam, Vladimir Sementsov-Ogievskiy, Denis Lunev, mreitz, stefanha On 23/08/2019 00:09, Eric Blake wrote: > On 8/22/19 1:31 PM, Andrey Shinkevich wrote: >> Revert the commit 118f99442d 'block/io.c: fix for the allocation failure' >> and make better error handling for the file systems that do not support > > s/make/use/ > >> fallocate() for the unaligned byte range. Allow falling back to pwrite > > s/the/an/ > >> in case fallocate() returns EINVAL. >> >> Suggested-by: Kevin Wolf <kwolf@redhat.com> >> Suggested-by: Eric Blake <eblake@redhat.com> >> Signed-off-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com> >> --- >> Discussed in email thread with the message ID >> <1554474244-553661-1-git-send-email-andrey.shinkevich@virtuozzo.com> >> >> block/file-posix.c | 7 +++++++ >> block/io.c | 2 +- >> 2 files changed, 8 insertions(+), 1 deletion(-) >> >> diff --git a/block/file-posix.c b/block/file-posix.c >> index fbeb006..2c254ff 100644 >> --- a/block/file-posix.c >> +++ b/block/file-posix.c >> @@ -1588,6 +1588,13 @@ static int handle_aiocb_write_zeroes(void *opaque) >> if (s->has_write_zeroes) { >> int ret = do_fallocate(s->fd, FALLOC_FL_ZERO_RANGE, >> aiocb->aio_offset, aiocb->aio_nbytes); >> + if (ret == -EINVAL) { >> + /* >> + * Allow falling back to pwrite for file systems that >> + * do not support fallocate() for unaligned byte range. > > s/for/for an/ > >> + */ >> + return -ENOTSUP; >> + } >> if (ret == 0 || ret != -ENOTSUP) { >> return ret; >> } >> diff --git a/block/io.c b/block/io.c >> index 56bbf19..58f08cd 100644 >> --- a/block/io.c >> +++ b/block/io.c >> @@ -1558,7 +1558,7 @@ static int coroutine_fn bdrv_co_do_pwrite_zeroes(BlockDriverState *bs, >> assert(!bs->supported_zero_flags); >> } >> >> - if (ret < 0 && !(flags & BDRV_REQ_NO_FALLBACK)) { >> + if (ret == -ENOTSUP && !(flags & BDRV_REQ_NO_FALLBACK)) { >> /* Fall back to bounce buffer if write zeroes is unsupported */ >> BdrvRequestFlags write_flags = flags & ~BDRV_REQ_ZERO_WRITE; >> >> > > Reviewed-by: Eric Blake <eblake@redhat.com> > Eric, If you are good to take this patch to your branch with your corrections, please let us know. Otherwise, I will send the v2 with your corrections. -- With the best regards, Andrey Shinkevich ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() 2019-08-27 12:39 ` Andrey Shinkevich @ 2019-08-27 13:50 ` Eric Blake 0 siblings, 0 replies; 8+ messages in thread From: Eric Blake @ 2019-08-27 13:50 UTC (permalink / raw) To: Andrey Shinkevich, qemu-devel, qemu-block Cc: kwolf, fam, Vladimir Sementsov-Ogievskiy, Denis Lunev, mreitz, stefanha [-- Attachment #1.1: Type: text/plain, Size: 550 bytes --] On 8/27/19 7:39 AM, Andrey Shinkevich wrote: > > >>> >> >> Reviewed-by: Eric Blake <eblake@redhat.com> >> > Eric, > If you are good to take this patch to your branch with your corrections, > please let us know. Otherwise, I will send the v2 with your corrections. Since I based my fast-zero patches on top of this, I'm fine making the corrections locally and queuing this patch through my NBD tree. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-08-27 13:51 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-08-22 18:31 [Qemu-devel] [PATCH] block: workaround for unaligned byte range in fallocate() Andrey Shinkevich 2019-08-22 18:55 ` Vladimir Sementsov-Ogievskiy 2019-08-22 19:10 ` Vladimir Sementsov-Ogievskiy 2019-08-22 21:09 ` Eric Blake 2019-08-23 12:07 ` Andrey Shinkevich 2019-08-27 12:35 ` Denis V. Lunev 2019-08-27 12:39 ` Andrey Shinkevich 2019-08-27 13:50 ` Eric Blake
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.