From: Juan Quintela <quintela@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: [Qemu-devel] Re: [PATCH 5/5] ide: fix migration in the middle of a bmdma transfer
Date: Wed, 16 Jun 2010 15:02:20 +0200 [thread overview]
Message-ID: <m3aaqvnnab.fsf@trasno.mitica> (raw)
In-Reply-To: <4C189162.4040701@redhat.com> (Paolo Bonzini's message of "Wed, 16 Jun 2010 10:54:58 +0200")
Paolo Bonzini <pbonzini@redhat.com> wrote:
> On 06/15/2010 03:31 PM, Juan Quintela wrote:
>> Signed-off-by: Juan Quintela<quintela@redhat.com>
>
> Sorry if this has been discussed to death before (if so I must have
> missed it...).
>
> With subsections available, what about taking advantage of the new
> protocol extension and add to the subsection info about the size of
> the subsection?
Not trivial with current infrastructure :(
> Also, with the size information, would it make sense to specify
> optional subsections that the receiver could choose to ignore?
We agreed that this was going to be forbidden. If sender send data ->
it needs to be received. Sender can decide to not send a subsection if
its data is not needed.
> Mandatory subsections are something such that round-trip A->B->A fail
> unless B understands the subsection, while optional subsections are
> such that A can provide a default. IDE subsections would be optional,
> for example.
This is the whole point of the .needed() function. if
neeeded(foo_subsection) returns false -> subsection is not needed.
If it returns true -> it is needed, destination has to understand it.
Later, Juan.
next prev parent reply other threads:[~2010-06-16 13:02 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-15 13:31 [Qemu-devel] [PATCH 0/5] Migration subsections (and ide as example) Juan Quintela
2010-06-15 13:31 ` [Qemu-devel] [PATCH 1/5] Revert "ide save/restore pio/atapi cmd transfer fields and io buffer" Juan Quintela
2010-06-15 13:51 ` Avi Kivity
2010-06-15 13:31 ` [Qemu-devel] [PATCH 2/5] Revert "ide save/restore current transfer fields" Juan Quintela
2010-06-15 13:31 ` [Qemu-devel] [PATCH 3/5] vmstate: add subsections code Juan Quintela
2010-06-15 13:31 ` [Qemu-devel] [PATCH 4/5] ide: fix migration in the middle of pio operation Juan Quintela
2010-06-15 13:31 ` [Qemu-devel] [PATCH 5/5] ide: fix migration in the middle of a bmdma transfer Juan Quintela
2010-06-15 13:59 ` Avi Kivity
2010-06-15 14:05 ` [Qemu-devel] " Juan Quintela
2010-06-16 8:54 ` Paolo Bonzini
2010-06-16 13:02 ` Juan Quintela [this message]
2010-06-15 14:09 ` [Qemu-devel] [PATCH 0/5] Migration subsections (and ide as example) Anthony Liguori
2010-06-15 14:19 ` [Qemu-devel] " Juan Quintela
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=m3aaqvnnab.fsf@trasno.mitica \
--to=quintela@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.