> Liang Li <liang.z...@intel.com> wrote:
> > Now, multiple thread compression can co-work with xbzrle. when xbzrle
> > is on, multiple thread compression will only work at the first round
> > of RAM data sync.
> >
> > Signed-off-by: Liang Li <liang.z...@intel.com>
> > Signed-off-by: Yang Zhang <yang.z.zh...@intel.com>
> > Reviewed-by: Dr.David Alan Gilbert <dgilb...@redhat.com>
> 
> Drop this patch and just give an error when trying to set xbzrle and
> compression?  User have to pick one and only one, no second guess him/her?
> 

Live migration can benefit from compression co-work with xbzrle. You know, 
xbzrle 
transfer the raw RAM pages to destination in the ram bulk stage, and after 
that, it transfers
the diff data. The ram bulk stage is where compression can do optimization, and 
beside 
the ram bulk stage, xbzrle may do better than compression  in some situation. So
compression and xbzrle are not in conflict but complementary.

I think it's a pity if we limit the use to select only one of them. If there is 
no strong reason, 
I don't agree to drop this patch.

Liang

Reply via email to