From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:36945) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RSr2S-0007k3-EZ for qemu-devel@nongnu.org; Tue, 22 Nov 2011 09:09:38 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RSr2O-0004Wm-Fl for qemu-devel@nongnu.org; Tue, 22 Nov 2011 09:09:32 -0500 Received: from mx1.redhat.com ([209.132.183.28]:50150) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RSr2O-0004WV-8n for qemu-devel@nongnu.org; Tue, 22 Nov 2011 09:09:28 -0500 Message-ID: <4ECBADD3.90404@redhat.com> Date: Tue, 22 Nov 2011 15:12:35 +0100 From: Kevin Wolf MIME-Version: 1.0 References: <1321640945-9827-1-git-send-email-kwolf@redhat.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/9] qcow2: Fix error paths for internal snapshots List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: qemu-devel@nongnu.org Am 21.11.2011 17:58, schrieb Stefan Hajnoczi: > On Fri, Nov 18, 2011 at 6:28 PM, Kevin Wolf wrote: >> Kevin Wolf (9): >> qcow2: Return real error code in qcow2_read_snapshots >> qcow2: Return real error code in qcow2_write_snapshots >> qcow2: Update snapshot table information at once >> qcow2: Cleanups and memleak fix in qcow2_snapshot_create >> qcow2: Rework qcow2_snapshot_create error handling >> qcow2: Return real error in qcow2_snapshot_goto >> qcow2: Fix order of refcount updates in qcow2_snapshot_goto >> qcow2: Fix order in qcow2_snapshot_delete >> qcow2: Fix error path in qcow2_snapshot_load_tmp >> >> block/qcow2-refcount.c | 7 +- >> block/qcow2-snapshot.c | 330 +++++++++++++++++++++++++++++++++++------------- >> block/qcow2.c | 5 +- >> 3 files changed, 248 insertions(+), 94 deletions(-) > > Reviewed-by: Stefan Hajnoczi Applied to the block branch (for 1.1) Do you think any of the patches could/should be merged for 1.0? Kevin