From: Cornelia Huck <cohuck@redhat.com>
To: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: Michael Roth <mdroth@linux.vnet.ibm.com>,
Halil Pasic <pasic@linux.vnet.ibm.com>,
Yi Min Zhao <zyimin@linux.vnet.ibm.com>,
David Hildenbrand <david@redhat.com>,
qemu-devel <qemu-devel@nongnu.org>,
Alexander Graf <agraf@suse.de>,
Richard Henderson <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: disable ais facility
Date: Fri, 22 Sep 2017 09:24:26 +0200 [thread overview]
Message-ID: <20170922092426.53ca16ca.cohuck@redhat.com> (raw)
In-Reply-To: <31127d34-b117-59ec-ee8d-c4a7f8324173@de.ibm.com>
On Fri, 22 Sep 2017 08:47:41 +0200
Christian Borntraeger <borntraeger@de.ibm.com> wrote:
> On 09/21/2017 10:26 PM, Michael Roth wrote:
> > Quoting Michael Roth (2017-09-21 11:50:28)
> >> Quoting Cornelia Huck (2017-09-21 10:21:42)
> >>> On Thu, 21 Sep 2017 17:14:46 +0200
> >>> Christian Borntraeger <borntraeger@de.ibm.com> wrote:
> >>>
> >>>> The migration interface for ais was introduced with kernel 4.13
> >>>> but the capability itself had been active since 4.12. As migration
> >>>> support is considered necessary lets disable ais in the 2.10
> >>>> stable version. A proper fix and re-enablement will be done
> >>>> for qemu 2.11.
> >>>
> >>> Also now with qemu-devel on cc:
> >>>
> >>> So, should I apply to s390-next? Or can this be applied just to stable?
> >>
> >> Ideally we would get it in master or at least a branch that'll
> >> eventually get pulled so we can reference the upstream commit ID.
> >> It's not set in stone, but usually only patches specifically ported
> >> to stable are applied directly.
> >
> > Somehow I missed the "for 2.10 stable" in the subject line. I can apply
> > this directly if it comes to that, but it seems safer to apply this
> > upstream as well in the meantime, IMO. If the proper fix didn't make
> > 2.11 for whatever reason we'd still be okay with 2.10.1<->2.11, for
> > instance.
> >
>
> So what about the following. We will schedule this disable patch also for
> current master. Whenever we have a proper fix we can then revert/fixup the
> disable patch. This patch will then be backported to 2.10.1
> For 2.11 we then fixup the ais code and also add the "disable ais for compat
> machines" patch.
Sounds good. I'll queue this one and send a pull request, just to be on
the safe side.
prev parent reply other threads:[~2017-09-22 7:24 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-21 15:14 [Qemu-devel] [PATCH 0/1] 2.10-stable only: disable ais facility Christian Borntraeger
2017-09-21 15:14 ` [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: " Christian Borntraeger
2017-09-21 15:18 ` David Hildenbrand
2017-09-21 15:21 ` Cornelia Huck
2017-09-21 16:50 ` Michael Roth
2017-09-21 20:26 ` Michael Roth
2017-09-22 6:47 ` Christian Borntraeger
2017-09-22 7:24 ` Cornelia Huck [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=20170922092426.53ca16ca.cohuck@redhat.com \
--to=cohuck@redhat.com \
--cc=agraf@suse.de \
--cc=borntraeger@de.ibm.com \
--cc=david@redhat.com \
--cc=mdroth@linux.vnet.ibm.com \
--cc=pasic@linux.vnet.ibm.com \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
--cc=zyimin@linux.vnet.ibm.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 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.