All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: Eduardo Habkost <ehabkost@redhat.com>,
	Michael Roth <mdroth@linux.vnet.ibm.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>,
	Igor Mammedov <imammedo@redhat.com>,
	Marcelo Tosatti <mtosatti@redhat.com>,
	qemu-devel@nongnu.org, qemu-stable@nongnu.org
Subject: Re: [Qemu-devel] [Qemu-stable] [PATCH] target-i386: Support migratable=no properly
Date: Thu, 04 Sep 2014 17:50:25 +0200	[thread overview]
Message-ID: <54088A41.9090307@suse.de> (raw)
In-Reply-To: <20140904151934.GA8481@thinpad.lan.raisama.net>

Am 04.09.2014 17:19, schrieb Eduardo Habkost:
> On Tue, Sep 02, 2014 at 10:08:56AM -0500, Michael Roth wrote:
>> Quoting Eduardo Habkost (2014-08-21 15:14:38)
>>> Forgot to add qemu-devel to "To:". Adding it.
>>>
>>> On Wed, Aug 20, 2014 at 05:30:12PM -0300, Eduardo Habkost wrote:
>>>> When the "migratable" property was implemented, the behavior was tested
>>>> by changing the default on the code, but actually using the option on
>>>> the command-line (e.g. "-cpu host,migratable=false") doesn't work as
>>>> expected. This is a regression for a common use case of "-cpu host",
>>>> which is to enable features that are supported by the host CPU + kernel
>>>> before feature-specific code is added to QEMU.
>>>>
>>>> Fix this by initializing the feature words for "-cpu host" on
>>>> x86_cpu_parse_featurestr(), right after parsing the CPU options.
>>>>
>>>> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
>>
>> Ping for stable 2.1.1, freeze is Wednesday.
>>
>> Reviewed-by: Michael Roth <mdroth@linux.vnet.ibm.com>
> 
> Ping? Who can help me get this applied?
> 
> I was expecting to need to argue to get this considered as -stable
> material, but not to have trouble getting it applied to master...

Thanks, applied to qom-cpu:
https://github.com/afaerber/qemu-cpu/commits/qom-cpu

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-09-04 15:50 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-20 20:30 [Qemu-devel] [PATCH] target-i386: Support migratable=no properly Eduardo Habkost
2014-08-21 20:14 ` Eduardo Habkost
2014-09-02 15:08   ` [Qemu-devel] [Qemu-stable] " Michael Roth
2014-09-04 15:19     ` Eduardo Habkost
2014-09-04 15:50       ` Andreas Färber [this message]

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=54088A41.9090307@suse.de \
    --to=afaerber@suse.de \
    --cc=ehabkost@redhat.com \
    --cc=imammedo@redhat.com \
    --cc=mdroth@linux.vnet.ibm.com \
    --cc=mtosatti@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-stable@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.