From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43309) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Whx9S-00058A-Qi for qemu-devel@nongnu.org; Wed, 07 May 2014 04:24:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Whx9M-0002MY-Ct for qemu-devel@nongnu.org; Wed, 07 May 2014 04:24:30 -0400 Received: from mx1.redhat.com ([209.132.183.28]:31929) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Whx9M-0002MK-4c for qemu-devel@nongnu.org; Wed, 07 May 2014 04:24:24 -0400 Date: Wed, 7 May 2014 10:24:21 +0200 From: Kevin Wolf Message-ID: <20140507082421.GB4045@noname.str.redhat.com> References: <1399371550-5212-1-git-send-email-kwolf@redhat.com> <5368C332.3040302@web.de> <53694E1F.4090708@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <53694E1F.4090708@redhat.com> Subject: Re: [Qemu-devel] [PATCH] block: Fix open flags with BDRV_O_SNAPSHOT List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Reitz Cc: Jan Kiszka , qemu-devel@nongnu.org, stefanha@redhat.com Am 06.05.2014 um 23:03 hat Max Reitz geschrieben: > On 06.05.2014 13:10, Jan Kiszka wrote: > >On 2014-05-06 12:19, Kevin Wolf wrote: > >>The immediately visible effect of this patch is that it fixes committing > >>a temporary snapshot to its backing file. Previously, it would fail with > >>a "permission denied" error because bdrv_inherited_flags() forced the > >>backing file to be read-only, ignoring the r/w reopen of bdrv_commit(). > >> > >>The bigger problem this releaved is that the original open flags must > >>actually only be applied to the temporary snapshot, and the original > >>image file must be treated as a backing file of the temporary snapshot > >>and get the right flags for that. > >> > >>Reported-by: Jan Kiszka > >>Signed-off-by: Kevin Wolf > >>--- > >> block.c | 34 +++++++++++++++++++--------------- > >> include/block/block.h | 2 +- > >> tests/qemu-iotests/051 | 4 ++++ > >> tests/qemu-iotests/051.out | 10 ++++++++++ > >> 4 files changed, 34 insertions(+), 16 deletions(-) > >Works fine here! > > > >(For unknown reason, applying the iotest hunk didn't work for me, though.) > > The lines are too long and therefore split in this mail, they need > to be joined manually before applying the patch. Perhaps the monitor should be changed to avoid printing so many useless control characters, then we'd hit the limit less often... Stefan, didn't you plan to do something like this? Or was it unrelated? Kevin