[Qemu-devel] [PATCH v2 0/3] qemu-img: rebase: Improve/optimize rebase operation

Sam Eiderman posted 3 patches 4 years, 12 months ago
Test docker-clang@ubuntu failed
Test docker-mingw@fedora failed
Test asan failed
Test checkpatch passed
Failed in applying to current master (apply log)
There is a newer version of this series
qemu-img.c | 85 ++++++++++++++++++++++++++++++++++++++++++--------------------
1 file changed, 58 insertions(+), 27 deletions(-)
[Qemu-devel] [PATCH v2 0/3] qemu-img: rebase: Improve/optimize rebase operation
Posted by Sam Eiderman 4 years, 12 months ago
This patch series aims to improve the speed of qemu-img rebase.

1. Mainly by removing unnecessary reads when rebasing on the same
   chain.
2. But also by minimizing the number of bdrv_open calls rebase
   requires.

v2:

 - Added missing g_free in
   "qemu-img: rebase: Reuse in-chain BlockDriverState"*

Sam Eiderman (3):
  qemu-img: rebase: Reuse parent BlockDriverState
  qemu-img: rebase: Reduce reads on in-chain rebase
  qemu-img: rebase: Reuse in-chain BlockDriverState

 qemu-img.c | 85 ++++++++++++++++++++++++++++++++++++++++++--------------------
 1 file changed, 58 insertions(+), 27 deletions(-)

-- 
2.13.3


Re: [Qemu-devel] [PATCH v2 0/3] qemu-img: rebase: Improve/optimize rebase operation
Posted by Eric Blake 4 years, 12 months ago
On 5/2/19 8:58 AM, Sam Eiderman wrote:
> This patch series aims to improve the speed of qemu-img rebase.
> 
> 1. Mainly by removing unnecessary reads when rebasing on the same
>    chain.
> 2. But also by minimizing the number of bdrv_open calls rebase
>    requires.
> 

When sending a v2 series, it's best to do so as a new top-level thread
rather than in-reply-to the v1 series, as our CI tools are more likely
to spot it that way.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] [PATCH v2 0/3] qemu-img: rebase: Improve/optimize rebase operation
Posted by Sam Eiderman 4 years, 11 months ago
Gentle ping.
Do you want me to resend this patch as a new top-level thread?

Thanks,
Sam

> On 2 May 2019, at 17:21, Eric Blake <eblake@redhat.com> wrote:
> 
> On 5/2/19 8:58 AM, Sam Eiderman wrote:
>> This patch series aims to improve the speed of qemu-img rebase.
>> 
>> 1. Mainly by removing unnecessary reads when rebasing on the same
>>   chain.
>> 2. But also by minimizing the number of bdrv_open calls rebase
>>   requires.
>> 
> 
> When sending a v2 series, it's best to do so as a new top-level thread
> rather than in-reply-to the v1 series, as our CI tools are more likely
> to spot it that way.
> 
> -- 
> Eric Blake, Principal Software Engineer
> Red Hat, Inc.           +1-919-301-3226
> Virtualization:  qemu.org | libvirt.org
> 


Re: [Qemu-devel] [PATCH v2 0/3] qemu-img: rebase: Improve/optimize rebase operation
Posted by Sam 4 years, 12 months ago
I see,

Thanks

> On 2 May 2019, at 17:21, Eric Blake <eblake@redhat.com> wrote:
> 
> On 5/2/19 8:58 AM, Sam Eiderman wrote:
>> This patch series aims to improve the speed of qemu-img rebase.
>> 
>> 1. Mainly by removing unnecessary reads when rebasing on the same
>>   chain.
>> 2. But also by minimizing the number of bdrv_open calls rebase
>>   requires.
>> 
> 
> When sending a v2 series, it's best to do so as a new top-level thread
> rather than in-reply-to the v1 series, as our CI tools are more likely
> to spot it that way.
> 
> -- 
> Eric Blake, Principal Software Engineer
> Red Hat, Inc.           +1-919-301-3226
> Virtualization:  qemu.org | libvirt.org
>