qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Avi Kivity <avi@redhat.com>
Cc: Stefan Hajnoczi <stefanha@gmail.com>, Dor Laor <dlaor@redhat.com>,
	qemu-devel Developers <qemu-devel@nongnu.org>,
	libvirt-list@redhat.com, Blue Swirl <blauwirbel@gmail.com>,
	"Shribman, Aidan" <aidan.shribman@sap.com>
Subject: Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps
Date: Mon, 08 Aug 2011 10:08:20 -0500	[thread overview]
Message-ID: <4E3FFBE4.9060206@codemonkey.ws> (raw)
In-Reply-To: <4E3FF524.7070509@redhat.com>

On 08/08/2011 09:39 AM, Avi Kivity wrote:
>> The other option is to allow 1-off compression algorithms in the form
>> of plugins. I think in this case, plugins are a pretty good compromise
>> in terms of isolating complexity while allowing something that at
>> least works very well for one particular type of workload.
>
> I think you underestimate the generality of XBZRLE (or maybe I'm
> overestimating it?).

This is really my fundamental concern.  When it comes to something that 
we have to support for a very long time, no one should be estimating 
anything.  We should make these decisions based on an awful lot of 
analysis on a wide variety of workloads.

It's hard to do this in QEMU today because we don't have a module 
mechanism to make it easy for users to try out new things without fully 
committing to including something in the tree.

But I don't think that's the root of the problem I have.  I really am 
just extremely reluctant to commit to something that we have to support 
forever.

Thinking more about it though, I think there can be another 
solution--feature negotiation.

I view adding feature negotiation as a pre-requisite to adding any type 
of transport compression such as XBZRLE.  That will let us support 
migration to older QEMUs and also to eventually remove XBZRLE if we 
decide it doesn't make sense anymore.

Regards,

Anthony Liguori

> It's not reasonable to ask users to match a
> compression algorithm to their workload; most times they won't be
> interacting with the host at all. We need compression to be enabled at
> all time, turning itself off if it finds it isn't effective so it can
> consume less cpu.
>

  reply	other threads:[~2011-08-08 15:08 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-08  8:42 [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps Shribman, Aidan
2011-08-08 13:29 ` Anthony Liguori
2011-08-08 13:41   ` Alexander Graf
2011-08-08 13:46     ` Anthony Liguori
2011-08-08 13:49     ` Avi Kivity
2011-08-08 13:51   ` Avi Kivity
2011-08-08 14:15     ` Anthony Liguori
2011-08-08 14:23       ` Avi Kivity
2011-08-08 14:33         ` Anthony Liguori
2011-08-08 14:39           ` Avi Kivity
2011-08-08 15:08             ` Anthony Liguori [this message]
2011-08-08 14:04   ` [Qemu-devel] [libvirt] " Daniel P. Berrange
2011-08-08 14:42     ` Avi Kivity
2011-08-08 14:46 ` [Qemu-devel] " Avi Kivity
2011-08-08 14:47   ` Avi Kivity
2011-08-08 14:56     ` Stefan Hajnoczi
2011-08-08 15:01       ` Avi Kivity
2011-08-08 15:10     ` Anthony Liguori
2011-08-08 15:15       ` Avi Kivity
2011-08-08 16:19         ` Anthony Liguori
2011-08-08 16:53           ` Avi Kivity
2011-08-08 16:55             ` Anthony Liguori
2011-08-10 15:07               ` Shribman, Aidan
2011-08-10 15:12                 ` Avi Kivity
2011-08-10 15:58                   ` Anthony Liguori
2011-08-10 16:08                     ` Avi Kivity
2011-08-10 16:23                       ` Anthony Liguori
2011-08-10 16:40                         ` Avi Kivity
2011-08-10 19:27                           ` Anthony Liguori
2011-08-11  8:03                             ` Shribman, Aidan
2011-08-11 13:00                               ` Anthony Liguori
2011-08-11  8:17                             ` Avi Kivity
2011-08-11  9:16                               ` [Qemu-devel] [libvirt] " Daniel P. Berrange
2011-08-11  9:20                                 ` Avi Kivity
2011-08-11 13:03                               ` [Qemu-devel] " Anthony Liguori
2011-08-11  9:24                             ` [Qemu-devel] [libvirt] " Daniel P. Berrange

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E3FFBE4.9060206@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=aidan.shribman@sap.com \
    --cc=avi@redhat.com \
    --cc=blauwirbel@gmail.com \
    --cc=dlaor@redhat.com \
    --cc=libvirt-list@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).