From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:58359) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UhezD-00025Q-6z for qemu-devel@nongnu.org; Wed, 29 May 2013 07:56:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UhezB-0000Qu-Jh for qemu-devel@nongnu.org; Wed, 29 May 2013 07:56:11 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38904) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UhezB-0000Qq-CW for qemu-devel@nongnu.org; Wed, 29 May 2013 07:56:09 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r4TBu8JJ022619 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 29 May 2013 07:56:08 -0400 Date: Wed, 29 May 2013 13:56:06 +0200 From: Kevin Wolf Message-ID: <20130529115606.GG3521@dhcp-200-207.str.redhat.com> References: <1369827340-32063-1-git-send-email-famz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1369827340-32063-1-git-send-email-famz@redhat.com> Subject: Re: [Qemu-devel] [PATCH v3] block: add block driver read only whitelist List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng Cc: qemu-devel@nongnu.org, stefanha@redhat.com Am 29.05.2013 um 13:35 hat Fam Zheng geschrieben: > We may want to include a driver in the whitelist for read only tasks > such as diagnosing or exporting guest data (with libguestfs as a good > example). This patch introduces a readonly whitelist option, and for > backward compatibility, the old configure option --block-drv-whitelist > is now an alias to rw whitelist. > > Drivers in readonly list is only permitted to open file readonly, and > returns -ENOTSUP for RW opening. > > E.g. To include vmdk readonly, and others read+write: > ./configure --target-list=x86_64-softmmu \ > --block-drv-rw-whitelist=qcow2,raw,file,qed \ > --block-drv-ro-whitelist=vmdk > > v3: > Fix hw/block/xen_disk.c. > > v2: > Use a separate configure option, no more magic prefix. > > Signed-off-by: Fam Zheng > --- > block.c | 43 +++++++++++++++++++++++++++---------------- > blockdev.c | 4 ++-- > configure | 20 +++++++++++++++----- > hw/block/xen_disk.c | 8 ++++++-- > include/block/block.h | 3 ++- > scripts/create_config | 11 +++++++++-- > 6 files changed, 61 insertions(+), 28 deletions(-) Thanks, applied to the block branch. For future patches, please make sure to include the version history of the patch below the "---" line so that git am automatically drops it. Kevin