* [PATCH v3] s390/virtio: add BSD license to virtio-ccw
@ 2017-11-28 13:17 Michael S. Tsirkin
2017-11-28 13:51 ` Cornelia Huck
0 siblings, 1 reply; 3+ messages in thread
From: Michael S. Tsirkin @ 2017-11-28 13:17 UTC (permalink / raw)
To: linux-kernel
Cc: Christian Borntraeger, Cornelia Huck, Martin Schwidefsky,
Heiko Carstens, linux-s390, virtualization
The original intent of the virtio header relicensing
from 2008 was to make sure anyone can implement compatible
devices/drivers. The virtio-ccw was omitted by mistake.
We have an ack from the only contributor as well as the
maintainer from IBM, so it's not too late to fix that.
Make it dual-licensed with GPLv2, as the whole kernel is GPL2.
Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
Acked-by: Cornelia Huck <cohuck@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
arch/s390/include/uapi/asm/virtio-ccw.h | 6 +-----
arch/s390/kernel/setup.c | 1 -
2 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/arch/s390/include/uapi/asm/virtio-ccw.h b/arch/s390/include/uapi/asm/virtio-ccw.h
index 967aad3..2b605f7 100644
--- a/arch/s390/include/uapi/asm/virtio-ccw.h
+++ b/arch/s390/include/uapi/asm/virtio-ccw.h
@@ -1,13 +1,9 @@
-/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
+/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */
/*
* Definitions for virtio-ccw devices.
*
* Copyright IBM Corp. 2013
*
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License (version 2 only)
- * as published by the Free Software Foundation.
- *
* Author(s): Cornelia Huck <cornelia.huck@de.ibm.com>
*/
#ifndef __KVM_VIRTIO_CCW_H
diff --git a/arch/s390/kernel/setup.c b/arch/s390/kernel/setup.c
index 164a1e1..c022007 100644
--- a/arch/s390/kernel/setup.c
+++ b/arch/s390/kernel/setup.c
@@ -60,7 +60,6 @@
#include <asm/ptrace.h>
#include <asm/sections.h>
#include <asm/ebcdic.h>
-#include <asm/kvm_virtio.h>
#include <asm/diag.h>
#include <asm/os_info.h>
#include <asm/sclp.h>
--
MST
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3] s390/virtio: add BSD license to virtio-ccw
2017-11-28 13:17 [PATCH v3] s390/virtio: add BSD license to virtio-ccw Michael S. Tsirkin
@ 2017-11-28 13:51 ` Cornelia Huck
2017-11-29 9:22 ` Heiko Carstens
0 siblings, 1 reply; 3+ messages in thread
From: Cornelia Huck @ 2017-11-28 13:51 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: linux-kernel, Christian Borntraeger, Martin Schwidefsky,
Heiko Carstens, linux-s390, virtualization
On Tue, 28 Nov 2017 15:17:52 +0200
"Michael S. Tsirkin" <mst@redhat.com> wrote:
> The original intent of the virtio header relicensing
> from 2008 was to make sure anyone can implement compatible
> devices/drivers. The virtio-ccw was omitted by mistake.
>
> We have an ack from the only contributor as well as the
> maintainer from IBM, so it's not too late to fix that.
>
> Make it dual-licensed with GPLv2, as the whole kernel is GPL2.
>
> Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
> Acked-by: Cornelia Huck <cohuck@redhat.com>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> ---
> arch/s390/include/uapi/asm/virtio-ccw.h | 6 +-----
> arch/s390/kernel/setup.c | 1 -
> 2 files changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/arch/s390/include/uapi/asm/virtio-ccw.h b/arch/s390/include/uapi/asm/virtio-ccw.h
> index 967aad3..2b605f7 100644
> --- a/arch/s390/include/uapi/asm/virtio-ccw.h
> +++ b/arch/s390/include/uapi/asm/virtio-ccw.h
> @@ -1,13 +1,9 @@
> -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
> +/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */
> /*
> * Definitions for virtio-ccw devices.
> *
> * Copyright IBM Corp. 2013
> *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License (version 2 only)
> - * as published by the Free Software Foundation.
> - *
> * Author(s): Cornelia Huck <cornelia.huck@de.ibm.com>
> */
> #ifndef __KVM_VIRTIO_CCW_H
> diff --git a/arch/s390/kernel/setup.c b/arch/s390/kernel/setup.c
> index 164a1e1..c022007 100644
> --- a/arch/s390/kernel/setup.c
> +++ b/arch/s390/kernel/setup.c
> @@ -60,7 +60,6 @@
> #include <asm/ptrace.h>
> #include <asm/sections.h>
> #include <asm/ebcdic.h>
> -#include <asm/kvm_virtio.h>
> #include <asm/diag.h>
> #include <asm/os_info.h>
> #include <asm/sclp.h>
Shouldn't the second hunk already be in master?
Still ack otherwise.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3] s390/virtio: add BSD license to virtio-ccw
2017-11-28 13:51 ` Cornelia Huck
@ 2017-11-29 9:22 ` Heiko Carstens
0 siblings, 0 replies; 3+ messages in thread
From: Heiko Carstens @ 2017-11-29 9:22 UTC (permalink / raw)
To: Cornelia Huck
Cc: Michael S. Tsirkin, linux-kernel, Christian Borntraeger,
Martin Schwidefsky, linux-s390, virtualization
On Tue, Nov 28, 2017 at 02:51:19PM +0100, Cornelia Huck wrote:
> On Tue, 28 Nov 2017 15:17:52 +0200
> "Michael S. Tsirkin" <mst@redhat.com> wrote:
>
> > The original intent of the virtio header relicensing
> > from 2008 was to make sure anyone can implement compatible
> > devices/drivers. The virtio-ccw was omitted by mistake.
> >
> > We have an ack from the only contributor as well as the
> > maintainer from IBM, so it's not too late to fix that.
> >
> > Make it dual-licensed with GPLv2, as the whole kernel is GPL2.
> >
> > Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>
> > Acked-by: Cornelia Huck <cohuck@redhat.com>
> > Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> > ---
> > arch/s390/include/uapi/asm/virtio-ccw.h | 6 +-----
> > arch/s390/kernel/setup.c | 1 -
> > 2 files changed, 1 insertion(+), 6 deletions(-)
> >
> > diff --git a/arch/s390/include/uapi/asm/virtio-ccw.h b/arch/s390/include/uapi/asm/virtio-ccw.h
> > index 967aad3..2b605f7 100644
> > --- a/arch/s390/include/uapi/asm/virtio-ccw.h
> > +++ b/arch/s390/include/uapi/asm/virtio-ccw.h
> > @@ -1,13 +1,9 @@
> > -/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
> > +/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */
> > /*
> > * Definitions for virtio-ccw devices.
> > *
> > * Copyright IBM Corp. 2013
> > *
> > - * This program is free software; you can redistribute it and/or modify
> > - * it under the terms of the GNU General Public License (version 2 only)
> > - * as published by the Free Software Foundation.
> > - *
> > * Author(s): Cornelia Huck <cornelia.huck@de.ibm.com>
> > */
> > #ifndef __KVM_VIRTIO_CCW_H
> > diff --git a/arch/s390/kernel/setup.c b/arch/s390/kernel/setup.c
> > index 164a1e1..c022007 100644
> > --- a/arch/s390/kernel/setup.c
> > +++ b/arch/s390/kernel/setup.c
> > @@ -60,7 +60,6 @@
> > #include <asm/ptrace.h>
> > #include <asm/sections.h>
> > #include <asm/ebcdic.h>
> > -#include <asm/kvm_virtio.h>
> > #include <asm/diag.h>
> > #include <asm/os_info.h>
> > #include <asm/sclp.h>
>
> Shouldn't the second hunk already be in master?
>
> Still ack otherwise.
The removal of the license text is also already in linux-next, however no
need to send an updated version. I'll fix it up ;)
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-11-29 9:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-11-28 13:17 [PATCH v3] s390/virtio: add BSD license to virtio-ccw Michael S. Tsirkin
2017-11-28 13:51 ` Cornelia Huck
2017-11-29 9:22 ` Heiko Carstens
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).