From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47901) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZYz1g-0000p1-Ei for qemu-devel@nongnu.org; Mon, 07 Sep 2015 12:12:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZYz1f-0003LY-NQ for qemu-devel@nongnu.org; Mon, 07 Sep 2015 12:12:12 -0400 Date: Mon, 7 Sep 2015 18:12:02 +0200 From: Kevin Wolf Message-ID: <20150907161202.GF5529@noname.redhat.com> References: <1437414365-11881-1-git-send-email-mreitz@redhat.com> <1437414365-11881-3-git-send-email-mreitz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1437414365-11881-3-git-send-email-mreitz@redhat.com> Subject: Re: [Qemu-devel] [PATCH v4 02/38] blockdev: Allow creation of BDS trees without BB List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Reitz Cc: Alberto Garcia , qemu-block@nongnu.org, John Snow , qemu-devel@nongnu.org, Markus Armbruster , Stefan Hajnoczi Am 20.07.2015 um 19:45 hat Max Reitz geschrieben: > If the "id" field is missing from the options given to blockdev-add, > just omit the BlockBackend and create the BlockDriverState tree alone. > > However, if "id" is missing, "node-name" must be specified; otherwise, > the BDS tree would no longer be accessible. We can probably lift this restriction once Jeff's auto-generated ID patches are in. However, allowing additional things is easy, so no objection here. > Many BDS options which are not parsed by bdrv_open() (like caching) > cannot be specified for these BB-less BDS trees yet. A future patch will > remove this limitation. This makes the command mostly useless, but that's okay. We'll be working on converting flags to QDict options one by one and then it will start working. There is, however, one flag that doesn't correspond to an option or enable an additional feature that is simply missing until then. That one worries me a bit: BDRV_O_INCOMING. We should probably include it in this patch; or maybe better add another patch before this one which moves the setting of BDRV_O_INCOMING from blockdev_init() to bdrv_open_common(). > Signed-off-by: Max Reitz > Reviewed-by: Eric Blake > Reviewed-by: Alberto Garcia > --- > blockdev.c | 44 +++++++++++++++++++++++++++++++------------- > qapi/block-core.json | 13 +++++++++---- > tests/qemu-iotests/087 | 2 +- > tests/qemu-iotests/087.out | 4 ++-- > 4 files changed, 43 insertions(+), 20 deletions(-) Kevin