qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Kevin Wolf <kwolf@redhat.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
	qemu-devel@nongnu.org, quintela@redhat.com
Subject: Re: [Qemu-devel] [PATCH] migration: remove subsections in fdc and rtl8139 and bump versions
Date: Thu, 04 Aug 2011 16:09:47 +0200	[thread overview]
Message-ID: <4E3AA82B.2030300@redhat.com> (raw)
In-Reply-To: <4E3A9CA0.9090706@codemonkey.ws>

Am 04.08.2011 15:20, schrieb Anthony Liguori:
> 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?

Wouldn't that only introduce more broken cases in which the destination
thinks it has successfully read the whole VM state when in fact it has
only been confused by a subsection?

Kevin

  reply	other threads:[~2011-08-04 14:07 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-02 23:08 [Qemu-devel] [PATCH] migration: remove subsections in fdc and rtl8139 and bump versions Anthony Liguori
2011-08-02 23:17 ` Anthony Liguori
2011-08-02 23:25 ` Juan Quintela
2011-08-03  0:12   ` Anthony Liguori
2011-08-03  6:44     ` Paolo Bonzini
2011-08-03 17:44       ` Anthony Liguori
2011-08-04  7:17         ` Paolo Bonzini
2011-08-03  9:00     ` Juan Quintela
2011-08-03 17:49       ` Anthony Liguori
2011-08-03 21:42         ` Juan Quintela
2011-08-04  7:22           ` Paolo Bonzini
2011-08-04 13:20             ` Anthony Liguori
2011-08-04 14:09               ` Kevin Wolf [this message]
2011-08-04 14:30                 ` Anthony Liguori
2011-08-04 15:07                   ` Paolo Bonzini
2011-08-04 20:31             ` Anthony Liguori
2011-08-04 12:59           ` Kevin Wolf
2011-08-04 13:12             ` Anthony Liguori
2011-08-04 14:36             ` Paolo Bonzini

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=4E3AA82B.2030300@redhat.com \
    --to=kwolf@redhat.com \
    --cc=anthony@codemonkey.ws \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.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).