From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53672) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WwYVE-00031S-Up for qemu-devel@nongnu.org; Mon, 16 Jun 2014 11:07:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WwYV6-0006pH-Qp for qemu-devel@nongnu.org; Mon, 16 Jun 2014 11:07:20 -0400 Received: from e06smtp16.uk.ibm.com ([195.75.94.112]:45198) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WwYV6-0006p4-FP for qemu-devel@nongnu.org; Mon, 16 Jun 2014 11:07:12 -0400 Received: from /spool/local by e06smtp16.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 16 Jun 2014 16:07:10 +0100 Received: from b06cxnps3075.portsmouth.uk.ibm.com (d06relay10.portsmouth.uk.ibm.com [9.149.109.195]) by d06dlp03.portsmouth.uk.ibm.com (Postfix) with ESMTP id 695491B08074 for ; Mon, 16 Jun 2014 16:07:36 +0100 (BST) Received: from d06av01.portsmouth.uk.ibm.com (d06av01.portsmouth.uk.ibm.com [9.149.37.212]) by b06cxnps3075.portsmouth.uk.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id s5GF77Na31588372 for ; Mon, 16 Jun 2014 15:07:07 GMT Received: from d06av01.portsmouth.uk.ibm.com (localhost [127.0.0.1]) by d06av01.portsmouth.uk.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id s5GF76MX026529 for ; Mon, 16 Jun 2014 09:07:07 -0600 Date: Mon, 16 Jun 2014 17:07:01 +0200 From: Greg Kurz Message-ID: <20140616170701.018bbc69@bahia.local> In-Reply-To: <539AE6DD.8010608@suse.de> References: <20140613111703.22108.14322.stgit@bahia.local> <539AE6DD.8010608@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: Kevin Wolf , Peter Maydell , Stefan Hajnoczi , "Michael S. Tsirkin" , Rusty Russell , qemu-devel@nongnu.org, Juan Quintela , aneesh.kumar@linux.vnet.ibm.com, Anthony Liguori , Amit Shah , Paolo Bonzini , Andreas =?UTF-8?B?RsOkcmJlcg==?= On Fri, 13 Jun 2014 13:56:13 +0200 Alexander Graf wrote: > > On 13.06.14 13:18, Greg Kurz wrote: > > Hi, > > > > This version merges the changes requested during the v7 review, remarks from > > ppc64 dump support review (yes, we talked about virtio there) and the work on > > virtio subsections migration. Also two new patches have been added: > > - patch #1 is a preliminary fix for virtio-serial posted by Alexander Graf > > - patch #9 prepares the work on the virtio_is_big_endian() helper > > > > The most significant changes are: > > - introduction of a new CPU method for virtio > > - endianness is taken from CPU that resets the device > > - fastpath virtio memory accessors for fixed endian targets > > - VMState based virtio subsections (compatibility friendly) > > > > You'll find more detailed changelog in each patch. > > > > Please comment and hopefully apply. > > > > Thanks ! > > Reviewed-by: Alexander Graf > > > Alex > Hi, I've reworked the patch set according to Alex's last remarks. Do any of the other maintainers have pending remarks ? Should I wait or send a v9 now ? Thanks. -- Gregory Kurz kurzgreg@fr.ibm.com gkurz@linux.vnet.ibm.com Software Engineer @ IBM/Meiosys http://www.ibm.com Tel +33 (0)562 165 496 "Anarchy is about taking complete responsibility for yourself." Alan Moore.