From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34488) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bh5dN-0002LZ-KD for qemu-devel@nongnu.org; Mon, 05 Sep 2016 21:57:10 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bh5dM-0006Ns-RB for qemu-devel@nongnu.org; Mon, 05 Sep 2016 21:57:09 -0400 Date: Tue, 6 Sep 2016 09:49:50 +0800 From: Fam Zheng Message-ID: <20160906014950.GD653@al.usersys.redhat.com> References: <1470662013-19785-1-git-send-email-famz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1470662013-19785-1-git-send-email-famz@redhat.com> Subject: Re: [Qemu-devel] [PATCH v7 00/20] block: Image locking series for 2.8 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , qemu-block@nongnu.org, Jeff Cody , rjones@redhat.com, Markus Armbruster , stefanha@redhat.com, pbonzini@redhat.com, den@openvz.org, Max Reitz , John Snow On Mon, 08/08 21:13, Fam Zheng wrote: > v7: - Rebase. > - Address comments from Kevin and Max. > - Option rename: "exclusive" -> "auto". [Kevin] > - Option placement: "root BDS" -> "node". > It's still a bit controversy where the option should go, but so far it > seems per node is the safest. > - Assert in bdrv_open_common that lock flags don't conflict. [Kevin] > - Skip qemu_dup patches as they are already merged. > - Fix a wrong assertion in raw_reopen_to_unlock. > - Put "static int" to the same line in raw-posix.c. [Kevin] > - Remove superfluous blank lines. > - Add comment in raw posix's sophisticated reopen code. [Kevin] > - .bdrv_lockf is kept on the contrary to separate .bdrv_lock and > .bdrv_unlock, because having the BdrvLockfCmd in the interface makes > implementing the reopen logic as is done in this series much easier. > - Drop the unnecessary mirror target locking patch. [Max] > - Add a patch to let 130 work with image locking. Ping?