* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] <1506350729-11311-1-git-send-email-thuth@redhat.com> @ 2017-09-25 17:54 ` Halil Pasic [not found] ` <3c2b8ae8-ca2a-7b26-8ebd-14f14c286529@linux.vnet.ibm.com> ` (2 subsequent siblings) 3 siblings, 0 replies; 12+ messages in thread From: Halil Pasic @ 2017-09-25 17:54 UTC (permalink / raw) To: Thomas Huth, Cornelia Huck; +Cc: linux-s390, kvm, virtualization On 09/25/2017 04:45 PM, Thomas Huth wrote: > There is no recent user space application available anymore which still > supports this old virtio transport, so let's disable this by default. > > Signed-off-by: Thomas Huth <thuth@redhat.com> I don't have any objections, but there may be something I'm not aware of. Let's see what Connie says. From my side it's ack. Via whom is this supposed to go in? Looking at the MAINTAINERS, I would say Martin or Heiko but I don't see them among the recipients. Regards, Halil > --- > arch/s390/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig > index 48af970..923bf04 100644 > --- a/arch/s390/Kconfig > +++ b/arch/s390/Kconfig > @@ -930,7 +930,7 @@ config S390_GUEST > the KVM hypervisor. > > config S390_GUEST_OLD_TRANSPORT > - def_bool y > + def_bool n > prompt "Guest support for old s390 virtio transport (DEPRECATED)" > depends on S390_GUEST > help > ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <3c2b8ae8-ca2a-7b26-8ebd-14f14c286529@linux.vnet.ibm.com>]
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] ` <3c2b8ae8-ca2a-7b26-8ebd-14f14c286529@linux.vnet.ibm.com> @ 2017-09-25 18:37 ` Christian Borntraeger [not found] ` <83cacb4a-94b1-c7c2-e50d-70afea5d3e9c@de.ibm.com> 1 sibling, 0 replies; 12+ messages in thread From: Christian Borntraeger @ 2017-09-25 18:37 UTC (permalink / raw) To: Halil Pasic, Thomas Huth, Cornelia Huck; +Cc: linux-s390, kvm, virtualization On 09/25/2017 07:54 PM, Halil Pasic wrote: > > > On 09/25/2017 04:45 PM, Thomas Huth wrote: >> There is no recent user space application available anymore which still >> supports this old virtio transport, so let's disable this by default. >> >> Signed-off-by: Thomas Huth <thuth@redhat.com> > > I don't have any objections, but there may be something I'm not aware of. > Let's see what Connie says. From my side it's ack. > > Via whom is this supposed to go in? Looking at the MAINTAINERS, I would > say Martin or Heiko but I don't see them among the recipients. FWIW as the original author of that transport Acked-by: Christian Borntraeger <borntraeger@de.ibm.com> I can pick this up for Martins/Heikos tree if you want. > > Regards, > Halil > >> --- >> arch/s390/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig >> index 48af970..923bf04 100644 >> --- a/arch/s390/Kconfig >> +++ b/arch/s390/Kconfig >> @@ -930,7 +930,7 @@ config S390_GUEST >> the KVM hypervisor. >> >> config S390_GUEST_OLD_TRANSPORT >> - def_bool y >> + def_bool n >> prompt "Guest support for old s390 virtio transport (DEPRECATED)" >> depends on S390_GUEST >> help >> ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <83cacb4a-94b1-c7c2-e50d-70afea5d3e9c@de.ibm.com>]
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] ` <83cacb4a-94b1-c7c2-e50d-70afea5d3e9c@de.ibm.com> @ 2017-09-26 2:44 ` Thomas Huth 2017-09-26 10:40 ` Heiko Carstens 1 sibling, 0 replies; 12+ messages in thread From: Thomas Huth @ 2017-09-26 2:44 UTC (permalink / raw) To: Christian Borntraeger, Halil Pasic, Cornelia Huck Cc: linux-s390, kvm, virtualization On 25.09.2017 20:37, Christian Borntraeger wrote: > > On 09/25/2017 07:54 PM, Halil Pasic wrote: >> >> >> On 09/25/2017 04:45 PM, Thomas Huth wrote: >>> There is no recent user space application available anymore which still >>> supports this old virtio transport, so let's disable this by default. >>> >>> Signed-off-by: Thomas Huth <thuth@redhat.com> >> >> I don't have any objections, but there may be something I'm not aware of. >> Let's see what Connie says. From my side it's ack. >> >> Via whom is this supposed to go in? Looking at the MAINTAINERS, I would >> say Martin or Heiko but I don't see them among the recipients. > > FWIW as the original author of that transport > Acked-by: Christian Borntraeger <borntraeger@de.ibm.com> > > I can pick this up for Martins/Heikos tree if you want. That would be great, yes, thanks! Thomas ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] ` <83cacb4a-94b1-c7c2-e50d-70afea5d3e9c@de.ibm.com> 2017-09-26 2:44 ` Thomas Huth @ 2017-09-26 10:40 ` Heiko Carstens 2017-09-26 10:41 ` Christian Borntraeger [not found] ` <376655d1-87e3-4187-d2d1-d1f23c02431a@de.ibm.com> 1 sibling, 2 replies; 12+ messages in thread From: Heiko Carstens @ 2017-09-26 10:40 UTC (permalink / raw) To: Christian Borntraeger Cc: linux-s390, kvm, Cornelia Huck, Thomas Huth, virtualization On Mon, Sep 25, 2017 at 08:37:36PM +0200, Christian Borntraeger wrote: > > On 09/25/2017 07:54 PM, Halil Pasic wrote: > > > > > > On 09/25/2017 04:45 PM, Thomas Huth wrote: > >> There is no recent user space application available anymore which still > >> supports this old virtio transport, so let's disable this by default. > >> > >> Signed-off-by: Thomas Huth <thuth@redhat.com> > > > > I don't have any objections, but there may be something I'm not aware of. > > Let's see what Connie says. From my side it's ack. > > > > Via whom is this supposed to go in? Looking at the MAINTAINERS, I would > > say Martin or Heiko but I don't see them among the recipients. > > FWIW as the original author of that transport > Acked-by: Christian Borntraeger <borntraeger@de.ibm.com> > > I can pick this up for Martins/Heikos tree if you want. When will this code be removed? When the config option was initially added Conny said it should survive "probably two kernel releases or so, depending on feedback". It was merged for v4.8. Now we are five kernel releases later... ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default 2017-09-26 10:40 ` Heiko Carstens @ 2017-09-26 10:41 ` Christian Borntraeger [not found] ` <376655d1-87e3-4187-d2d1-d1f23c02431a@de.ibm.com> 1 sibling, 0 replies; 12+ messages in thread From: Christian Borntraeger @ 2017-09-26 10:41 UTC (permalink / raw) To: Heiko Carstens Cc: linux-s390, kvm, Cornelia Huck, Thomas Huth, virtualization On 09/26/2017 12:40 PM, Heiko Carstens wrote: > On Mon, Sep 25, 2017 at 08:37:36PM +0200, Christian Borntraeger wrote: >> >> On 09/25/2017 07:54 PM, Halil Pasic wrote: >>> >>> >>> On 09/25/2017 04:45 PM, Thomas Huth wrote: >>>> There is no recent user space application available anymore which still >>>> supports this old virtio transport, so let's disable this by default. >>>> >>>> Signed-off-by: Thomas Huth <thuth@redhat.com> >>> >>> I don't have any objections, but there may be something I'm not aware of. >>> Let's see what Connie says. From my side it's ack. >>> >>> Via whom is this supposed to go in? Looking at the MAINTAINERS, I would >>> say Martin or Heiko but I don't see them among the recipients. >> >> FWIW as the original author of that transport >> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com> >> >> I can pick this up for Martins/Heikos tree if you want. > > When will this code be removed? > > When the config option was initially added Conny said it should survive > "probably two kernel releases or so, depending on feedback". > It was merged for v4.8. Now we are five kernel releases later... Lets wait for one release and then get rid of it? ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <376655d1-87e3-4187-d2d1-d1f23c02431a@de.ibm.com>]
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] ` <376655d1-87e3-4187-d2d1-d1f23c02431a@de.ibm.com> @ 2017-09-26 10:47 ` Heiko Carstens [not found] ` <20170926104752.GB3209@osiris> 1 sibling, 0 replies; 12+ messages in thread From: Heiko Carstens @ 2017-09-26 10:47 UTC (permalink / raw) To: Christian Borntraeger Cc: linux-s390, kvm, Cornelia Huck, Thomas Huth, virtualization On Tue, Sep 26, 2017 at 12:41:41PM +0200, Christian Borntraeger wrote: > > > On 09/26/2017 12:40 PM, Heiko Carstens wrote: > > On Mon, Sep 25, 2017 at 08:37:36PM +0200, Christian Borntraeger wrote: > >> > >> On 09/25/2017 07:54 PM, Halil Pasic wrote: > >>> > >>> > >>> On 09/25/2017 04:45 PM, Thomas Huth wrote: > >>>> There is no recent user space application available anymore which still > >>>> supports this old virtio transport, so let's disable this by default. > >>>> > >>>> Signed-off-by: Thomas Huth <thuth@redhat.com> > >>> > >>> I don't have any objections, but there may be something I'm not aware of. > >>> Let's see what Connie says. From my side it's ack. > >>> > >>> Via whom is this supposed to go in? Looking at the MAINTAINERS, I would > >>> say Martin or Heiko but I don't see them among the recipients. > >> > >> FWIW as the original author of that transport > >> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com> > >> > >> I can pick this up for Martins/Heikos tree if you want. > > > > When will this code be removed? > > > > When the config option was initially added Conny said it should survive > > "probably two kernel releases or so, depending on feedback". > > It was merged for v4.8. Now we are five kernel releases later... > > Lets wait for one release and then get rid of it? So it's going to be removed with the next merge window. Where is the patch? ;) ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <20170926104752.GB3209@osiris>]
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] ` <20170926104752.GB3209@osiris> @ 2017-09-26 10:57 ` Thomas Huth [not found] ` <ec6d80f2-17ca-4a5f-a357-5cc165b1acfd@redhat.com> 1 sibling, 0 replies; 12+ messages in thread From: Thomas Huth @ 2017-09-26 10:57 UTC (permalink / raw) To: Heiko Carstens, Christian Borntraeger Cc: linux-s390, Cornelia Huck, kvm, virtualization On 26.09.2017 12:47, Heiko Carstens wrote: > On Tue, Sep 26, 2017 at 12:41:41PM +0200, Christian Borntraeger wrote: >> >> >> On 09/26/2017 12:40 PM, Heiko Carstens wrote: >>> On Mon, Sep 25, 2017 at 08:37:36PM +0200, Christian Borntraeger wrote: >>>> >>>> On 09/25/2017 07:54 PM, Halil Pasic wrote: >>>>> >>>>> >>>>> On 09/25/2017 04:45 PM, Thomas Huth wrote: >>>>>> There is no recent user space application available anymore which still >>>>>> supports this old virtio transport, so let's disable this by default. >>>>>> >>>>>> Signed-off-by: Thomas Huth <thuth@redhat.com> >>>>> >>>>> I don't have any objections, but there may be something I'm not aware of. >>>>> Let's see what Connie says. From my side it's ack. >>>>> >>>>> Via whom is this supposed to go in? Looking at the MAINTAINERS, I would >>>>> say Martin or Heiko but I don't see them among the recipients. >>>> >>>> FWIW as the original author of that transport >>>> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com> >>>> >>>> I can pick this up for Martins/Heikos tree if you want. >>> >>> When will this code be removed? >>> >>> When the config option was initially added Conny said it should survive >>> "probably two kernel releases or so, depending on feedback". >>> It was merged for v4.8. Now we are five kernel releases later... >> >> Lets wait for one release and then get rid of it? > > So it's going to be removed with the next merge window. > Where is the patch? ;) Hmm, so far the code was always enabled by default, so in the unlikely case that somebody is still using it, they might not have noticed that this is marked as deprecated. So maybe it's better to keep the code for two more released, but disable it by default, so that people have time to realize that it is going away? ... just my 0.02 € ... but if you prefer, I can also write a patch that removes it immediately instead. Thomas _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <ec6d80f2-17ca-4a5f-a357-5cc165b1acfd@redhat.com>]
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] ` <ec6d80f2-17ca-4a5f-a357-5cc165b1acfd@redhat.com> @ 2017-09-26 11:04 ` Heiko Carstens [not found] ` <20170926110403.GC3209@osiris> 1 sibling, 0 replies; 12+ messages in thread From: Heiko Carstens @ 2017-09-26 11:04 UTC (permalink / raw) To: Thomas Huth; +Cc: linux-s390, kvm, Cornelia Huck, virtualization On Tue, Sep 26, 2017 at 12:57:26PM +0200, Thomas Huth wrote: > On 26.09.2017 12:47, Heiko Carstens wrote: > > So it's going to be removed with the next merge window. > > Where is the patch? ;) > > Hmm, so far the code was always enabled by default, so in the unlikely > case that somebody is still using it, they might not have noticed that > this is marked as deprecated. So maybe it's better to keep the code for > two more released, but disable it by default, so that people have time > to realize that it is going away? ... just my 0.02 € ... but if you > prefer, I can also write a patch that removes it immediately instead. Switching the default from yes to no won't prevent anybody from selecting it again. So I don't see any value in waiting even longer. Besides the original commit already printed a message to the console that the transport is deprecated. See commit 3b2fbb3f06ef ("virtio/s390: deprecate old transport"). _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <20170926110403.GC3209@osiris>]
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] ` <20170926110403.GC3209@osiris> @ 2017-09-26 11:10 ` Cornelia Huck 0 siblings, 0 replies; 12+ messages in thread From: Cornelia Huck @ 2017-09-26 11:10 UTC (permalink / raw) To: Heiko Carstens; +Cc: linux-s390, Thomas Huth, kvm, virtualization On Tue, 26 Sep 2017 13:04:03 +0200 Heiko Carstens <heiko.carstens@de.ibm.com> wrote: > On Tue, Sep 26, 2017 at 12:57:26PM +0200, Thomas Huth wrote: > > On 26.09.2017 12:47, Heiko Carstens wrote: > > > So it's going to be removed with the next merge window. > > > Where is the patch? ;) > > > > Hmm, so far the code was always enabled by default, so in the unlikely > > case that somebody is still using it, they might not have noticed that > > this is marked as deprecated. So maybe it's better to keep the code for > > two more released, but disable it by default, so that people have time > > to realize that it is going away? ... just my 0.02 € ... but if you > > prefer, I can also write a patch that removes it immediately instead. > > Switching the default from yes to no won't prevent anybody from selecting > it again. So I don't see any value in waiting even longer. > > Besides the original commit already printed a message to the console that > the transport is deprecated. > See commit 3b2fbb3f06ef ("virtio/s390: deprecate old transport"). > Would ack a removal as well. _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] <1506350729-11311-1-git-send-email-thuth@redhat.com> 2017-09-25 17:54 ` [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default Halil Pasic [not found] ` <3c2b8ae8-ca2a-7b26-8ebd-14f14c286529@linux.vnet.ibm.com> @ 2017-09-26 7:33 ` Christian Borntraeger 2017-09-26 7:53 ` Cornelia Huck 3 siblings, 0 replies; 12+ messages in thread From: Christian Borntraeger @ 2017-09-26 7:33 UTC (permalink / raw) To: Thomas Huth, Halil Pasic, Cornelia Huck; +Cc: linux-s390, kvm, virtualization On 09/25/2017 04:45 PM, Thomas Huth wrote: > There is no recent user space application available anymore which still > supports this old virtio transport, so let's disable this by default. > > Signed-off-by: Thomas Huth <thuth@redhat.com> thanks applied. > --- > arch/s390/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig > index 48af970..923bf04 100644 > --- a/arch/s390/Kconfig > +++ b/arch/s390/Kconfig > @@ -930,7 +930,7 @@ config S390_GUEST > the KVM hypervisor. > > config S390_GUEST_OLD_TRANSPORT > - def_bool y > + def_bool n > prompt "Guest support for old s390 virtio transport (DEPRECATED)" > depends on S390_GUEST > help > ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default [not found] <1506350729-11311-1-git-send-email-thuth@redhat.com> ` (2 preceding siblings ...) 2017-09-26 7:33 ` Christian Borntraeger @ 2017-09-26 7:53 ` Cornelia Huck 3 siblings, 0 replies; 12+ messages in thread From: Cornelia Huck @ 2017-09-26 7:53 UTC (permalink / raw) To: Thomas Huth; +Cc: linux-s390, kvm, virtualization On Mon, 25 Sep 2017 16:45:29 +0200 Thomas Huth <thuth@redhat.com> wrote: > There is no recent user space application available anymore which still > supports this old virtio transport, so let's disable this by default. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > arch/s390/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig > index 48af970..923bf04 100644 > --- a/arch/s390/Kconfig > +++ b/arch/s390/Kconfig > @@ -930,7 +930,7 @@ config S390_GUEST > the KVM hypervisor. > > config S390_GUEST_OLD_TRANSPORT > - def_bool y > + def_bool n > prompt "Guest support for old s390 virtio transport (DEPRECATED)" > depends on S390_GUEST > help Acked-by: Cornelia Huck <cohuck@redhat.com> ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default @ 2017-09-25 14:45 Thomas Huth 0 siblings, 0 replies; 12+ messages in thread From: Thomas Huth @ 2017-09-25 14:45 UTC (permalink / raw) To: Halil Pasic, Cornelia Huck; +Cc: linux-s390, kvm, virtualization There is no recent user space application available anymore which still supports this old virtio transport, so let's disable this by default. Signed-off-by: Thomas Huth <thuth@redhat.com> --- arch/s390/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig index 48af970..923bf04 100644 --- a/arch/s390/Kconfig +++ b/arch/s390/Kconfig @@ -930,7 +930,7 @@ config S390_GUEST the KVM hypervisor. config S390_GUEST_OLD_TRANSPORT - def_bool y + def_bool n prompt "Guest support for old s390 virtio transport (DEPRECATED)" depends on S390_GUEST help -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
end of thread, other threads:[~2017-09-26 11:10 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1506350729-11311-1-git-send-email-thuth@redhat.com>
2017-09-25 17:54 ` [PATCH] KVM: s390: Disable CONFIG_S390_GUEST_OLD_TRANSPORT by default Halil Pasic
[not found] ` <3c2b8ae8-ca2a-7b26-8ebd-14f14c286529@linux.vnet.ibm.com>
2017-09-25 18:37 ` Christian Borntraeger
[not found] ` <83cacb4a-94b1-c7c2-e50d-70afea5d3e9c@de.ibm.com>
2017-09-26 2:44 ` Thomas Huth
2017-09-26 10:40 ` Heiko Carstens
2017-09-26 10:41 ` Christian Borntraeger
[not found] ` <376655d1-87e3-4187-d2d1-d1f23c02431a@de.ibm.com>
2017-09-26 10:47 ` Heiko Carstens
[not found] ` <20170926104752.GB3209@osiris>
2017-09-26 10:57 ` Thomas Huth
[not found] ` <ec6d80f2-17ca-4a5f-a357-5cc165b1acfd@redhat.com>
2017-09-26 11:04 ` Heiko Carstens
[not found] ` <20170926110403.GC3209@osiris>
2017-09-26 11:10 ` Cornelia Huck
2017-09-26 7:33 ` Christian Borntraeger
2017-09-26 7:53 ` Cornelia Huck
2017-09-25 14:45 Thomas Huth
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox