[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 8/9] QMP commands changes
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v5 8/9] QMP commands changes |
Date: |
Tue, 3 Jan 2012 16:20:17 +0000 |
On Tue, Jan 3, 2012 at 3:57 PM, Orit Wasserman <address@hidden> wrote:
> On 01/03/2012 05:47 PM, Stefan Hajnoczi wrote:
>> On Tue, Jan 3, 2012 at 3:34 PM, Orit Wasserman <address@hidden> wrote:
>>> + .args_type = "detach:-d,blk:-b,inc:-i,xbrle:-x,uri:s",
>>> + .params = "[-d] [-b] [-i] [-x] uri",
>>> + .help = "migrate to URI"
>>> + "\n\t -d to not wait for completion"
>>> + "\n\t -b for migration without shared storage with"
>>> + " full copy of disk"
>>> + "\n\t -i for migration without"
>>> + " shared storage with incremental copy of disk"
>>> + " (base image shared between source and destination)"
>>> + "\n\t -x to use XBRLE page delta compression",
>>
>> It's too bad that this algorithm is user-visible and needs to be
>> expicitly enabled/disabled. I think it would be useful in a much
>> wider range of cases if the trade-offs were understood well enough so
>> that QEMU can include a threshold or heuristic which chooses the
>> migration algorithm behind the scenes.
>
> I agree that an automatic switching on/off XBRLE is very desirable.
> At the first phase we merge it as a user option, and the in the next phase
> will be adding
> the part that switch it on/off.
Okay, if there's an intent to do that next phase then great. I'll
take a look at this series and review it soon.
Stefan
- [Qemu-devel] [PATCH v5 3/9] Add save_block_hdr function, (continued)
- [Qemu-devel] [PATCH v5 6/9] Add xbrle parameters to MigrationState, Orit Wasserman, 2012/01/03
- [Qemu-devel] [PATCH v5 7/9] Add set_cachesize to change XBRLE cache size, Orit Wasserman, 2012/01/03
- Re: [Qemu-devel] [PATCH v5 0/9] XBZRLE delta for live migration of large memory apps, Anthony Liguori, 2012/01/03
- Re: [Qemu-devel] [PATCH v5 0/9] XBZRLE delta for live migration of large memory apps, Avi Kivity, 2012/01/04
- [Qemu-devel] [PATCH v5 0/9] XBZRLE delta for live migration of large memory apps, Orit Wasserman, 2012/01/03