qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] migration: add a MAINTAINERS entry for migratio


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH] migration: add a MAINTAINERS entry for migration
Date: Mon, 14 Nov 2011 15:08:25 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.21) Gecko/20110831 Lightning/1.0b2 Thunderbird/3.1.13

On 11/14/2011 11:40 AM, Juan Quintela wrote:
Anthony Liguori<address@hidden>  wrote:
I think this is an accurate reflection of the state of migration today.  This
is the second release in a row where we're scrambling to fix a critical issue
in migration.

We need to make our mind about it.

Ultimately, we need to make migration a priority. That's what I'm trying to do here.

The first step is to be open about the state of migration today. I personally don't have the bandwidth to invest a lot of effort in migration, but I can invest time in trying to find more people to work on migration, and help put together a proper roadmap.

We need to outline and document what we support and what we don't support. We need to invest in a test infrastructure. We need a roadmap that we can reasonably execute on. In short, we need to turn migration into a first class subsystem.

It's not about any single person or any single patch series. It's about deciding that migration is an important feature and deserves more focus and attention.

That's the conversation I'm trying to start with this patch.

Regards,

Anthony Liguori


A patch to do the reopen was posted long, long ago.  Code existed on
RHEL5 from 3 years ago.  Answer was that:
- we need to do it other way
- we need to change it inside qcow2

No suggestions for former, and internals of qcow2 are quite difficult
to grasp (at least for me).

Then my fault for not pushing more for the patches.

But then it happens again with migration with Huge Memory machines.
Series were rejected because it "only" fixed completely the stalls on
the iothread, and it don't fixed completely the problem on the vcpus.
And here we still are, we need to finish the migration thread to get
things included.

And then, we have problems with the format (that is not
comprehensible).  Took almost 2 years to convince you that we need a
"size", checksum, start/end markers.  And we got:
- on one hand, we need to have perfect solutions to get them integrated
   (huge memory patches)
- on the other hand, we can think about including patches that only fix
   one of the more minor points that we have (visitors).

So, the question is:

What we expect for migration?
- Backward compatibility: A must for corporate users ->  a burden for
   everybody else
- Testing: What is that?
- Format: it is a mess, but as Avi likes to point, we would have to
   "maintain" current one temporarily  (a.k.a. forever).

To make things more interesting, lots of changes on migration touch lot
of code (i.e. not only migration*/savevm.c), and getting that patches
accepted take forever.

The first step in fixing this problem is being up front about what the current
state of the subsystem is.  If we're going to treat migration as a release
blocking feature in the future, than we need to promote the migration subsystem
above 'Odd Fixes' status.

Later, Juan.

PD.  And yes, I agree that migration is in a very sad state today.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]