From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56439) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gXmy4-0006gC-PA for qemu-devel@nongnu.org; Fri, 14 Dec 2018 07:53:25 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gXmy1-0002qv-Cc for qemu-devel@nongnu.org; Fri, 14 Dec 2018 07:53:24 -0500 Received: from mx1.redhat.com ([209.132.183.28]:57632) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gXmy1-0002qQ-4P for qemu-devel@nongnu.org; Fri, 14 Dec 2018 07:53:21 -0500 Date: Fri, 14 Dec 2018 07:53:15 -0500 From: "Michael S. Tsirkin" Message-ID: <20181214075249-mutt-send-email-mst@kernel.org> References: <20181213210057.3676-1-quintela@redhat.com> <33183CC9F5247A488A2544077AF19020DB1BECBE@dggeml511-mbx.china.huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <33183CC9F5247A488A2544077AF19020DB1BECBE@dggeml511-mbx.china.huawei.com> Subject: Re: [Qemu-devel] [PATCH v3 00/16] Virtio devices split from virtio-pci List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Gonglei (Arei)" Cc: Juan Quintela , "qemu-devel@nongnu.org" , Thomas Huth , Gerd Hoffmann On Fri, Dec 14, 2018 at 07:07:44AM +0000, Gonglei (Arei) wrote: > > > -----Original Message----- > > From: Juan Quintela [mailto:quintela@redhat.com] > > Sent: Friday, December 14, 2018 5:01 AM > > To: qemu-devel@nongnu.org > > Cc: Michael S. Tsirkin ; Thomas Huth ; > > Gerd Hoffmann ; Gonglei (Arei) > > ; Juan Quintela > > Subject: [PATCH v3 00/16] Virtio devices split from virtio-pci > > > > Hi > > > > v3: > > - rebase to master > > - only compile them if CONFIG_PCI is set (thomas) > > > > Please review. > > > > Later, Juan. > > > > V2: > > > > - Rebase on top of master > > > > Please review. > > > > Later, Juan. > > > > [v1] > > From previous verision (in the middle of make check tests): > > - split also the bits of virtio-pci.h (mst suggestion) > > - add gpu, crypt and gpg bits > > - more cleanups > > - fix all the copyrights (the ones not changed have been there > > foverever) > > - be consistent with naming, vhost-* or virtio-* > > > > Please review, Juan. > > > > Juan Quintela (16): > > virtio: split vhost vsock bits from virtio-pci > > virtio: split virtio input host bits from virtio-pci > > virtio: split virtio input bits from virtio-pci > > virtio: split virtio rng bits from virtio-pci > > virtio: split virtio balloon bits from virtio-pci > > virtio: split virtio 9p bits from virtio-pci > > virtio: split vhost user blk bits from virtio-pci > > virtio: split vhost user scsi bits from virtio-pci > > virtio: split vhost scsi bits from virtio-pci > > virtio: split virtio scsi bits from virtio-pci > > virtio: split virtio blk bits rom virtio-pci > > virtio: split virtio net bits rom virtio-pci > > virtio: split virtio serial bits rom virtio-pci > > virtio: split virtio gpu bits rom virtio-pci.h > > virtio: split virtio crypto bits rom virtio-pci.h > > virtio: virtio 9p really requires CONFIG_VIRTFS to work > > > > default-configs/virtio.mak | 3 +- > > hw/display/virtio-gpu-pci.c | 14 + > > hw/display/virtio-vga.c | 1 + > > hw/virtio/Makefile.objs | 15 + > > hw/virtio/vhost-scsi-pci.c | 95 ++++ > > hw/virtio/vhost-user-blk-pci.c | 101 ++++ > > hw/virtio/vhost-user-scsi-pci.c | 101 ++++ > > hw/virtio/vhost-vsock-pci.c | 82 ++++ > > hw/virtio/virtio-9p-pci.c | 86 ++++ > > hw/virtio/virtio-balloon-pci.c | 94 ++++ > > hw/virtio/virtio-blk-pci.c | 97 ++++ > > hw/virtio/virtio-crypto-pci.c | 14 + > > hw/virtio/virtio-input-host-pci.c | 45 ++ > > hw/virtio/virtio-input-pci.c | 154 ++++++ > > hw/virtio/virtio-net-pci.c | 96 ++++ > > hw/virtio/virtio-pci.c | 783 ------------------------------ > > hw/virtio/virtio-pci.h | 234 --------- > > hw/virtio/virtio-rng-pci.c | 86 ++++ > > hw/virtio/virtio-scsi-pci.c | 106 ++++ > > hw/virtio/virtio-serial-pci.c | 112 +++++ > > tests/Makefile.include | 20 +- > > 21 files changed, 1311 insertions(+), 1028 deletions(-) > > create mode 100644 hw/virtio/vhost-scsi-pci.c > > create mode 100644 hw/virtio/vhost-user-blk-pci.c > > create mode 100644 hw/virtio/vhost-user-scsi-pci.c > > create mode 100644 hw/virtio/vhost-vsock-pci.c > > create mode 100644 hw/virtio/virtio-9p-pci.c > > create mode 100644 hw/virtio/virtio-balloon-pci.c > > create mode 100644 hw/virtio/virtio-blk-pci.c > > create mode 100644 hw/virtio/virtio-input-host-pci.c > > create mode 100644 hw/virtio/virtio-input-pci.c > > create mode 100644 hw/virtio/virtio-net-pci.c > > create mode 100644 hw/virtio/virtio-rng-pci.c > > create mode 100644 hw/virtio/virtio-scsi-pci.c > > create mode 100644 hw/virtio/virtio-serial-pci.c > > > > -- > > 2.19.2 > > For series: > Reviewed-by: Gonglei > > > Thanks, > -Gonglei Thanks! Can you pls align Reviewed-by: tag at the 1st column in the future? Makes it easier to apply the tag.