qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] migration: remove subsections in fdc and rtl813


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH] migration: remove subsections in fdc and rtl8139 and bump versions
Date: Thu, 04 Aug 2011 08:20:32 -0500
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110516 Lightning/1.0b2 Thunderbird/3.1.10

On 08/04/2011 02:22 AM, Paolo Bonzini wrote:
On 08/03/2011 11:42 PM, Juan Quintela wrote:
> I can certainly limit the change to IDE if we think machine, floppy,
> and rtl8139 are safe.

Ok, only IDE is broken, something done if we are not reverting the
others.

Floppy is broken too, and has the problem that the subsection is sent
almost always. On the other hand IDE is only broken if we have
subsections, and that is a rare case.

So perhaps the best course of action is to only fix floppy, either as in
your patch or by reverting 7d905f71. Let's keep IDE as it is broken as
0.14 was, and apply my patch to master.


Hrm, so the idea is to keep IDE broken but not worry because it is so rare? Why not just force the subsection to be sent?

I'm strongly in favor of bumping the version number of any broken device for 0.15, applying Paolo's patches to master, converting to Visitors, and then implementing a new protocol with better (perhaps ASN.1).

I think we can do this without touching very much code at all and in such a way that we don't have to worry about major regressions.

Regards,

Anthony Liguori

 I'll rebase as soon as there is
agreement this is the way to go.

Paolo





reply via email to

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