From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752405AbdKHSdy (ORCPT ); Wed, 8 Nov 2017 13:33:54 -0500 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:59146 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752196AbdKHSdv (ORCPT ); Wed, 8 Nov 2017 13:33:51 -0500 Subject: Re: [PATCH v2] s390/virtio: mark headers as BSD licensed To: Cornelia Huck Cc: "Michael S. Tsirkin" , linux-kernel@vger.kernel.org, Greg Kroah-Hartman , Martin Schwidefsky , Heiko Carstens , linux-s390@vger.kernel.org References: <1510160307-13235-1-git-send-email-mst@redhat.com> <9c2debae-5c00-75ad-bc4d-13af9c4f431d@de.ibm.com> <20171108191531.2d2f7210.cohuck@redhat.com> From: Christian Borntraeger Date: Wed, 8 Nov 2017 19:33:45 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <20171108191531.2d2f7210.cohuck@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-TM-AS-GCONF: 00 x-cbid: 17110818-0008-0000-0000-000004A91A7F X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 17110818-0009-0000-0000-00001E3BB8F8 Message-Id: <7ab1b0f9-6db2-1d49-3bc6-320e95c71d5b@de.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-11-08_03:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 impostorscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1707230000 definitions=main-1711080246 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/08/2017 07:15 PM, Cornelia Huck wrote: > On Wed, 8 Nov 2017 19:12:25 +0100 > Christian Borntraeger wrote: > >> On 11/08/2017 05:59 PM, Michael S. Tsirkin wrote: >>> Virtio UAPI headers aren't just for UAPI, it's for guests/hypervisors as >>> well. The s390 ones need to be BSD as well. >>> >>> Fixes: e2be04c7f995 ("License cleanup: add SPDX license identifier to uapi header files with a license") >>> Cc: Greg Kroah-Hartman >>> Signed-off-by: Michael S. Tsirkin >>> --- >>> >>> since v1: >>> drop an extra comment chunk, reported by Cornelia >>> >>> arch/s390/include/uapi/asm/kvm_virtio.h | 26 +++++++++++++++++++++++++- >> >> FWIW, this file will go away anyway. >> Looks like the following commit in the s390 feature branch >> https://git.kernel.org/pub/scm/linux/kernel/git/s390/linux.git/commit/drivers/s390/virtio/Makefile?h=features&id=7fb2b2d512448cf0e914c4647a1cf02b52263702 >> missed this file. > > Can we simply rip this out? > >> >> >> Regarding the virtio-ccw.h headers it is probably something that happened by accident. >> Back then Rusty (as an IBMer) proposed to change all virtio headers to BSD licence. > > ISTR that the relicensing already went through... is that not the case? The original kvm_virtio.h code was added in 2008 commit e976a2b997fc4ad70ccc53acfe62811c4aaec851 Author: Christian Borntraeger Date: Tue Mar 25 18:47:46 2008 +0100 s390: KVM guest: virtio device support, and kvm hypercalls The BSD re-licencing happened in 2008 commit 674bfc23c585b34c42263d73fb51710d49762a23 Author: Rusty Russell Date: Fri Jul 25 12:06:03 2008 -0500 virtio: clarify that ABI is usable by any implementations We want others to implement and use virtio, so it makes sense to BSD license the non-__KERNEL__ parts of the headers to make this crystal clear. and it seems that it missed arch/include/uapi/asm/kvm_virtio.h arch/include/uapi/asm/virtio_ccw.h probably just copied what kvm_virtio.h had in 2013. So its somewhat fair to assume that both files should be BSD, but they are not according to the licence statement. Gregs SPDX change just changed the GPL text in the header files to a SPDX statement. Now Michael wants to change this to BSD as it probably was intended but not written down. Sigh. > I thought this was just a fixup to align with reality? > >> >>> arch/s390/include/uapi/asm/virtio-ccw.h | 26 +++++++++++++++++++++++++- >>> 2 files changed, 50 insertions(+), 2 deletions(-) >