qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: Alexey Kardashevskiy <aik@ozlabs.ru>
Cc: Mike Day <ncmike@ncultra.org>, Gerd Hoffmann <kraxel@gmail.com>,
	qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] usb-ohci: add vmstate descriptor
Date: Sat, 22 Mar 2014 21:18:57 +0100	[thread overview]
Message-ID: <532DF031.40209@suse.de> (raw)
In-Reply-To: <53228F11.1020801@ozlabs.ru>

Am 14.03.2014 06:09, schrieb Alexey Kardashevskiy:
> On 03/07/2014 12:57 AM, Mike Day wrote:
>>
>> Alexey Kardashevskiy <aik@ozlabs.ru> writes:
>>
>>> This adds migration support for OHCI.
>>>
>>> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
>>
>> Reviewed-by: Mike Day <ncmike@ncultra.org>
> 
> Thanks!
> 
> What is next?

The USB maintainer (that you've now CC'ed) needs to review and pick it
up. It may help to understand what went wrong without it. Because AFAIU
migration is possible without VMSD, just not with VMSD that sets
.unmigratable = 1. Understanding what this patch is about may also help
in determining whether this needs to be fixed in 2.0 or not.

That said, the patch does look okay to me on brief sight.

Regards,
Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

  reply	other threads:[~2014-03-22 20:19 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-06  6:43 [Qemu-devel] [PATCH] usb-ohci: add vmstate descriptor Alexey Kardashevskiy
2014-03-06 13:57 ` Mike Day
2014-03-14  5:09   ` Alexey Kardashevskiy
2014-03-22 20:18     ` Andreas Färber [this message]
2014-03-22 20:54       ` Peter Maydell
2014-03-22 21:04         ` Andreas Färber
2014-03-22 21:23           ` Peter Maydell
2014-03-24  5:53             ` Alexey Kardashevskiy
2014-03-24  9:26               ` Peter Maydell
2014-03-24 10:26                 ` Gerd Hoffmann
2014-03-22 21:50           ` Alexey Kardashevskiy

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=532DF031.40209@suse.de \
    --to=afaerber@suse.de \
    --cc=aik@ozlabs.ru \
    --cc=kraxel@gmail.com \
    --cc=ncmike@ncultra.org \
    --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 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).