From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34570) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zlxbi-00035Z-LB for qemu-devel@nongnu.org; Tue, 13 Oct 2015 07:19:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zlxbh-0003lo-UJ for qemu-devel@nongnu.org; Tue, 13 Oct 2015 07:19:02 -0400 Date: Tue, 13 Oct 2015 13:18:57 +0200 From: Kevin Wolf Message-ID: <20151013111857.GH4906@noname.str.redhat.com> References: <1444392941-28704-1-git-send-email-kwolf@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1444392941-28704-1-git-send-email-kwolf@redhat.com> Subject: Re: [Qemu-devel] [PATCH v3 00/16] block: Get rid of bdrv_swap() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-block@nongnu.org Cc: berto@igalia.com, jcody@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, stefanha@redhat.com, famz@redhat.com, mreitz@redhat.com Am 09.10.2015 um 14:15 hat Kevin Wolf geschrieben: > bdrv_swap() has always been an ugly hack that we would rather have > avoided. When it was introduced, we simply didn't have the > infrastructure to update pointers instead of transplanting the contents > of BDS object, so we grudgingly added bdrv_swap() as a quick solution. > Meanwhile, most of the infrastructure exists and this series implements > the final step necessary to implement the required functionality in a > less adventurous way. Applied to the block branch. Thanks to everyone who reviewed this series. I think this is the best-reviewed series we've had in a long time in the block layer, with all patches having at least three and some up to five Reviewed-bys. Now we just need to do the same for all the other series on the list. :-) Kevin