* [Qemu-devel] [PATCH 0/1] 2.10-stable only: disable ais facility
@ 2017-09-21 15:14 Christian Borntraeger
2017-09-21 15:14 ` [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: " Christian Borntraeger
0 siblings, 1 reply; 8+ messages in thread
From: Christian Borntraeger @ 2017-09-21 15:14 UTC (permalink / raw)
To: Cornelia Huck
Cc: Michael Roth, Richard Henderson, Alexander Graf, Yi Min Zhao,
Halil Pasic, qemu-devel, David Hildenbrand,
Christian Borntraeger
now with qemu-devel on cc..
Cornelia, David and myself had a lengthy discussion about
ais migration and its issues. As the solution turns out to
be more complex, we will disable the ais facility in 2.10
and fix the issue properly in 2.11.
Christian Borntraeger (1):
s390x/ais: for 2.10 stable: disable ais facility
target/s390x/kvm.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
--
2.9.4
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: disable ais facility
2017-09-21 15:14 [Qemu-devel] [PATCH 0/1] 2.10-stable only: disable ais facility Christian Borntraeger
@ 2017-09-21 15:14 ` Christian Borntraeger
2017-09-21 15:18 ` David Hildenbrand
2017-09-21 15:21 ` Cornelia Huck
0 siblings, 2 replies; 8+ messages in thread
From: Christian Borntraeger @ 2017-09-21 15:14 UTC (permalink / raw)
To: Cornelia Huck
Cc: Michael Roth, Richard Henderson, Alexander Graf, Yi Min Zhao,
Halil Pasic, qemu-devel, David Hildenbrand,
Christian Borntraeger
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.
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
---
target/s390x/kvm.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
index c4c5791..eb0dbb3 100644
--- a/target/s390x/kvm.c
+++ b/target/s390x/kvm.c
@@ -308,8 +308,13 @@ int kvm_arch_init(MachineState *ms, KVMState *s)
}
}
- /* Try to enable AIS facility */
- kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0);
+ /*
+ * 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 let's disable ais in the 2.10
+ * machine.
+ */
+ /* kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0); */
qemu_mutex_init(&qemu_sigp_mutex);
--
2.9.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: disable ais facility
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
1 sibling, 0 replies; 8+ messages in thread
From: David Hildenbrand @ 2017-09-21 15:18 UTC (permalink / raw)
To: Christian Borntraeger, Cornelia Huck
Cc: Michael Roth, Richard Henderson, Alexander Graf, Yi Min Zhao,
Halil Pasic, qemu-devel
On 21.09.2017 17:14, Christian Borntraeger 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.
>
> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> ---
> target/s390x/kvm.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
> index c4c5791..eb0dbb3 100644
> --- a/target/s390x/kvm.c
> +++ b/target/s390x/kvm.c
> @@ -308,8 +308,13 @@ int kvm_arch_init(MachineState *ms, KVMState *s)
> }
> }
>
> - /* Try to enable AIS facility */
> - kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0);
> + /*
> + * 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 let's disable ais in the 2.10
> + * machine.
> + */
> + /* kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0); */
>
> qemu_mutex_init(&qemu_sigp_mutex);
>
>
Reviewed-by: David Hildenbrand <david@redhat.com>
--
Thanks,
David
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: disable ais facility
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
1 sibling, 1 reply; 8+ messages in thread
From: Cornelia Huck @ 2017-09-21 15:21 UTC (permalink / raw)
To: Christian Borntraeger
Cc: Michael Roth, Richard Henderson, Alexander Graf, Yi Min Zhao,
Halil Pasic, qemu-devel, David Hildenbrand
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?
>
> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
> Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> ---
> target/s390x/kvm.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
> index c4c5791..eb0dbb3 100644
> --- a/target/s390x/kvm.c
> +++ b/target/s390x/kvm.c
> @@ -308,8 +308,13 @@ int kvm_arch_init(MachineState *ms, KVMState *s)
> }
> }
>
> - /* Try to enable AIS facility */
> - kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0);
> + /*
> + * 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 let's disable ais in the 2.10
> + * machine.
> + */
> + /* kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0); */
>
> qemu_mutex_init(&qemu_sigp_mutex);
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: disable ais facility
2017-09-21 15:21 ` Cornelia Huck
@ 2017-09-21 16:50 ` Michael Roth
2017-09-21 20:26 ` Michael Roth
0 siblings, 1 reply; 8+ messages in thread
From: Michael Roth @ 2017-09-21 16:50 UTC (permalink / raw)
To: Christian Borntraeger, Cornelia Huck
Cc: Richard Henderson, Alexander Graf, Yi Min Zhao, Halil Pasic,
qemu-devel, David Hildenbrand
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.
>
> >
> > Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
> > Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> > ---
> > target/s390x/kvm.c | 9 +++++++--
> > 1 file changed, 7 insertions(+), 2 deletions(-)
> >
> > diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
> > index c4c5791..eb0dbb3 100644
> > --- a/target/s390x/kvm.c
> > +++ b/target/s390x/kvm.c
> > @@ -308,8 +308,13 @@ int kvm_arch_init(MachineState *ms, KVMState *s)
> > }
> > }
> >
> > - /* Try to enable AIS facility */
> > - kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0);
> > + /*
> > + * 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 let's disable ais in the 2.10
> > + * machine.
> > + */
> > + /* kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0); */
> >
> > qemu_mutex_init(&qemu_sigp_mutex);
> >
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: disable ais facility
2017-09-21 16:50 ` Michael Roth
@ 2017-09-21 20:26 ` Michael Roth
2017-09-22 6:47 ` Christian Borntraeger
0 siblings, 1 reply; 8+ messages in thread
From: Michael Roth @ 2017-09-21 20:26 UTC (permalink / raw)
To: Christian Borntraeger, Cornelia Huck
Cc: Halil Pasic, Yi Min Zhao, David Hildenbrand, qemu-devel,
Alexander Graf, Richard Henderson
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.
>
> >
> > >
> > > Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
> > > Reviewed-by: Cornelia Huck <cohuck@redhat.com>
> > > ---
> > > target/s390x/kvm.c | 9 +++++++--
> > > 1 file changed, 7 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/target/s390x/kvm.c b/target/s390x/kvm.c
> > > index c4c5791..eb0dbb3 100644
> > > --- a/target/s390x/kvm.c
> > > +++ b/target/s390x/kvm.c
> > > @@ -308,8 +308,13 @@ int kvm_arch_init(MachineState *ms, KVMState *s)
> > > }
> > > }
> > >
> > > - /* Try to enable AIS facility */
> > > - kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0);
> > > + /*
> > > + * 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 let's disable ais in the 2.10
> > > + * machine.
> > > + */
> > > + /* kvm_vm_enable_cap(s, KVM_CAP_S390_AIS, 0); */
> > >
> > > qemu_mutex_init(&qemu_sigp_mutex);
> > >
> >
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: disable ais facility
2017-09-21 20:26 ` Michael Roth
@ 2017-09-22 6:47 ` Christian Borntraeger
2017-09-22 7:24 ` Cornelia Huck
0 siblings, 1 reply; 8+ messages in thread
From: Christian Borntraeger @ 2017-09-22 6:47 UTC (permalink / raw)
To: Michael Roth, Cornelia Huck
Cc: Halil Pasic, Yi Min Zhao, David Hildenbrand, qemu-devel,
Alexander Graf, Richard Henderson
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.
Christian
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] s390x/ais: for 2.10 stable: disable ais facility
2017-09-22 6:47 ` Christian Borntraeger
@ 2017-09-22 7:24 ` Cornelia Huck
0 siblings, 0 replies; 8+ messages in thread
From: Cornelia Huck @ 2017-09-22 7:24 UTC (permalink / raw)
To: Christian Borntraeger
Cc: Michael Roth, Halil Pasic, Yi Min Zhao, David Hildenbrand,
qemu-devel, Alexander Graf, Richard Henderson
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.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-09-22 7:24 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
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 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.