From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60259) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zz0cn-0006Op-RI for qemu-devel@nongnu.org; Wed, 18 Nov 2015 06:10:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zz0cj-0003kz-RQ for qemu-devel@nongnu.org; Wed, 18 Nov 2015 06:10:05 -0500 Received: from mx1.redhat.com ([209.132.183.28]:57604) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zz0cj-0003kX-Lv for qemu-devel@nongnu.org; Wed, 18 Nov 2015 06:10:01 -0500 From: Juan Quintela In-Reply-To: <1447751311-2317-7-git-send-email-den@openvz.org> (Denis V. Lunev's message of "Tue, 17 Nov 2015 12:08:26 +0300") References: <1447751311-2317-1-git-send-email-den@openvz.org> <1447751311-2317-7-git-send-email-den@openvz.org> Date: Wed, 18 Nov 2015 12:09:47 +0100 Message-ID: <87h9kjo96s.fsf@emacs.mitica> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PATCH 06/11] snapshot: create bdrv_all_find_snapshot helper Reply-To: quintela@redhat.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Denis V. Lunev" Cc: Kevin Wolf , qemu-devel@nongnu.org, stefanha@redhat.com "Denis V. Lunev" wrote: > to check that snapshot is available for all loaded block drivers. > The check bs != bs1 in hmp_info_snapshots is an optimization. The check > for availability of this snapshot will return always true as the list > of snapshots was collected from that image. > > The patch also ensures proper locking. > > Signed-off-by: Denis V. Lunev > CC: Juan Quintela > CC: Stefan Hajnoczi > CC: Kevin Wolf Reviewed-by: Juan Quintela