From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42673) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fq1x3-00009T-2j for qemu-devel@nongnu.org; Wed, 15 Aug 2018 15:59:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fpyVe-0006tf-Cl for qemu-devel@nongnu.org; Wed, 15 Aug 2018 12:19:32 -0400 Received: from mx3-rdu2.redhat.com ([66.187.233.73]:41218 helo=mx1.redhat.com) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1fpyVd-0006hA-AJ for qemu-devel@nongnu.org; Wed, 15 Aug 2018 12:18:57 -0400 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 916DD40216E8 for ; Wed, 15 Aug 2018 16:18:28 +0000 (UTC) References: <20180808114830.7169-1-quintela@redhat.com> <20180808114830.7169-18-quintela@redhat.com> From: Thomas Huth Message-ID: <159b7f67-2484-3339-48e5-b9cd944de579@redhat.com> Date: Wed, 15 Aug 2018 18:18:25 +0200 MIME-Version: 1.0 In-Reply-To: <20180808114830.7169-18-quintela@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 17/22] virtio: split vhost-user-scsi bits from virtio-pci List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Juan Quintela , qemu-devel@nongnu.org Cc: lvivier@redhat.com, dgilbert@redhat.com, peterx@redhat.com On 08/08/2018 01:48 PM, Juan Quintela wrote: > Signed-off-by: Juan Quintela > --- > hw/virtio/Makefile.objs | 1 + > hw/virtio/vhost-user-scsi-pci.c | 89 +++++++++++++++++++++++++++++++++ > hw/virtio/virtio-pci.c | 58 --------------------- > 3 files changed, 90 insertions(+), 58 deletions(-) > create mode 100644 hw/virtio/vhost-user-scsi-pci.c > > diff --git a/hw/virtio/Makefile.objs b/hw/virtio/Makefile.objs > index 2c95c55bd0..a5d2e38d3a 100644 > --- a/hw/virtio/Makefile.objs > +++ b/hw/virtio/Makefile.objs > @@ -12,6 +12,7 @@ obj-$(call land,$(CONFIG_VIRTIO_CRYPTO),$(CONFIG_VIRTIO_PCI)) += virtio-crypto-p > obj-$(CONFIG_LINUX) += vhost.o vhost-backend.o vhost-user.o > obj-$(CONFIG_VHOST_VSOCK) += vhost-vsock.o vhost-vsock-pci.o > obj-$(CONFIG_VHOST_USER_BLK) += vhost-user-blk-pci.o > +obj-$(CONFIG_VHOST_USER_SCSI) += vhost-user-scsi-pci.o > obj-$(CONFIG_VIRTIO_INPUT_HOST) += virtio-input-host-pci.o > obj-$(CONFIG_VIRTIO_INPUT) += virtio-input-pci.o > obj-$(CONFIG_VIRTIO_RNG) += virtio-rng-pci.o > diff --git a/hw/virtio/vhost-user-scsi-pci.c b/hw/virtio/vhost-user-scsi-pci.c > new file mode 100644 > index 0000000000..6f3a73c2a8 > --- /dev/null > +++ b/hw/virtio/vhost-user-scsi-pci.c > @@ -0,0 +1,89 @@ > +/* > + * Virtio PCI Bindings > + * > + * Copyright IBM, Corp. 2007 > + * Copyright (c) 2009 CodeSourcery > + * > + * Authors: > + * Anthony Liguori > + * Paul Brook > + * > + * This work is licensed under the terms of the GNU GPL, version 2. See > + * the COPYING file in the top-level directory. > + * > + * Contributions after 2012-01-13 are licensed under the terms of the > + * GNU GPL, version 2 or (at your option) any later version. > + */ The vhost-user-scsi code has been introduced in 2017 by Felipe Franciosi, so I think you could remove the above "Authors" lines, and adjust the GPL license statement to plain GPLv2+ here. Thomas