From patchwork Thu Jan 17 15:34:00 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alberto Garcia X-Patchwork-Id: 10768461 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id B3FE8139A for ; Thu, 17 Jan 2019 15:49:28 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A2A0A304A8 for ; Thu, 17 Jan 2019 15:49:28 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 96C62304D4; Thu, 17 Jan 2019 15:49:28 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.7 required=2.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,MAILING_LIST_MULTI autolearn=ham version=3.3.1 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id D04B2304A8 for ; Thu, 17 Jan 2019 15:49:27 +0000 (UTC) Received: from localhost ([127.0.0.1]:46921 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gk9v4-00069k-Uk for patchwork-qemu-devel@patchwork.kernel.org; Thu, 17 Jan 2019 10:49:27 -0500 Received: from eggs.gnu.org ([209.51.188.92]:39360) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gk9h1-0002DY-U8 for qemu-devel@nongnu.org; Thu, 17 Jan 2019 10:34:57 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gk9h0-00007o-Iu for qemu-devel@nongnu.org; Thu, 17 Jan 2019 10:34:55 -0500 Received: from fanzine.igalia.com ([91.117.99.155]:59819) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gk9gz-0007Ip-IF; Thu, 17 Jan 2019 10:34:53 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=igalia.com; s=20170329; h=References:In-Reply-To:References:In-Reply-To:Message-Id:Date:Subject:Cc:To:From; bh=ZgMfyo0Hd69QWlq1Qihp88O5I5f5/7De7Nxyww223Sg=; b=nlHA9N/68yhesZz/fUwwKs8LZQBCi3B1dofVNGaIwnPxnZ4M9DOeYBJhvIwyEK8NbzSdqrTMpALACCn2FzPZqFLTETG00mKwSjTSx/alSXP2lHeEPw5zVSW0aDFuDsZANFGFztMgG2HeA4B2L9uBhxItgPXyptHSAfrAuSZgorAuBFMXIw0aNrlMOBMuaY6ceZJx393S+WT4tUdGIm8OLr+DWRrE3TqLaOhgr40Ea/d6o4SHBUG9HObIhQu5byTwnmj8+16eYEFpVLWx8ET9XjOVyCL54ZFP+I2K+8UnwY1oQNEEgqpmL2ANpFymEX4e+FW1jHq+3dULlAR4/Qr4tQ==; Received: from [194.100.51.2] (helo=perseus.local) by fanzine.igalia.com with esmtpsa (Cipher TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim) id 1gk9gY-0003be-Ht; Thu, 17 Jan 2019 16:34:26 +0100 Received: from berto by perseus.local with local (Exim 4.89) (envelope-from ) id 1gk9gJ-0007RS-1d; Thu, 17 Jan 2019 17:34:11 +0200 From: Alberto Garcia To: qemu-devel@nongnu.org Date: Thu, 17 Jan 2019 17:34:00 +0200 Message-Id: <61bbfa3721938326edb03db58a6a246b384f3260.1547739122.git.berto@igalia.com> X-Mailer: git-send-email 2.11.0 In-Reply-To: References: In-Reply-To: References: X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x (no timestamps) [generic] [fuzzy] X-Received-From: 91.117.99.155 Subject: [Qemu-devel] [PATCH 09/13] block: Add 'runtime_opts' and 'mutable_opts' fields to BlockDriver X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Wolf , Alberto Garcia , qemu-block@nongnu.org, Max Reitz Errors-To: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Sender: "Qemu-devel" X-Virus-Scanned: ClamAV using ClamSMTP This patch adds two new fields to BlockDriver: - runtime_opts: list of runtime options for a particular block driver. We'll use this list later to detect what options are missing when we try to reopen a block device. - mutable_opts: names of the runtime options that can be reset to their default value after a block device has been added. This way an option can not be reset by omitting it during reopen unless we explicitly allow it. This also sets runtime_opts (and mutable_opts where appropriate) in all drivers that allow reopening. Most of those drivers don't actually support changing any of their options. If the user specifies a new value for an option that can't be changed then we already detect that and forbid it (in bdrv_reopen_prepare()). But if the user omits an option in order to try to reset it to its default value we need to detect that, so we'll use these two new fields for that. Signed-off-by: Alberto Garcia --- block/blkdebug.c | 1 + block/crypto.c | 1 + block/file-posix.c | 10 ++++++++++ block/iscsi.c | 2 ++ block/null.c | 2 ++ block/nvme.c | 1 + block/qcow.c | 1 + block/rbd.c | 1 + block/sheepdog.c | 2 ++ block/vpc.c | 1 + include/block/block_int.h | 7 +++++++ 11 files changed, 29 insertions(+) diff --git a/block/blkdebug.c b/block/blkdebug.c index 0759452925..bba7645e09 100644 --- a/block/blkdebug.c +++ b/block/blkdebug.c @@ -893,6 +893,7 @@ static BlockDriver bdrv_blkdebug = { .protocol_name = "blkdebug", .instance_size = sizeof(BDRVBlkdebugState), .is_filter = true, + .runtime_opts = &runtime_opts, .bdrv_parse_filename = blkdebug_parse_filename, .bdrv_file_open = blkdebug_open, diff --git a/block/crypto.c b/block/crypto.c index f0a5f6b987..ba4da53191 100644 --- a/block/crypto.c +++ b/block/crypto.c @@ -635,6 +635,7 @@ BlockDriver bdrv_crypto_luks = { .bdrv_co_create_opts = block_crypto_co_create_opts_luks, .bdrv_co_truncate = block_crypto_co_truncate, .create_opts = &block_crypto_create_opts_luks, + .runtime_opts = &block_crypto_runtime_opts_luks, .bdrv_reopen_prepare = block_crypto_reopen_prepare, .bdrv_refresh_limits = block_crypto_refresh_limits, diff --git a/block/file-posix.c b/block/file-posix.c index 8aee7a3fb8..d08233ece3 100644 --- a/block/file-posix.c +++ b/block/file-posix.c @@ -443,6 +443,8 @@ static QemuOptsList raw_runtime_opts = { }, }; +static const char *const mutable_opts[] = { "x-check-cache-dropped", NULL }; + static int raw_open_common(BlockDriverState *bs, QDict *options, int bdrv_flags, int open_flags, bool device, Error **errp) @@ -2743,6 +2745,8 @@ BlockDriver bdrv_file = { .format_name = "file", .protocol_name = "file", .instance_size = sizeof(BDRVRawState), + .runtime_opts = &raw_runtime_opts, + .mutable_opts = mutable_opts, .bdrv_needs_filename = true, .bdrv_probe = NULL, /* no probe for protocols */ .bdrv_parse_filename = raw_parse_filename, @@ -3220,6 +3224,8 @@ static BlockDriver bdrv_host_device = { .format_name = "host_device", .protocol_name = "host_device", .instance_size = sizeof(BDRVRawState), + .runtime_opts = &raw_runtime_opts, + .mutable_opts = mutable_opts, .bdrv_needs_filename = true, .bdrv_probe_device = hdev_probe_device, .bdrv_parse_filename = hdev_parse_filename, @@ -3346,6 +3352,8 @@ static BlockDriver bdrv_host_cdrom = { .format_name = "host_cdrom", .protocol_name = "host_cdrom", .instance_size = sizeof(BDRVRawState), + .runtime_opts = &raw_runtime_opts, + .mutable_opts = mutable_opts, .bdrv_needs_filename = true, .bdrv_probe_device = cdrom_probe_device, .bdrv_parse_filename = cdrom_parse_filename, @@ -3479,6 +3487,8 @@ static BlockDriver bdrv_host_cdrom = { .format_name = "host_cdrom", .protocol_name = "host_cdrom", .instance_size = sizeof(BDRVRawState), + .runtime_opts = &raw_runtime_opts, + .mutable_opts = mutable_opts, .bdrv_needs_filename = true, .bdrv_probe_device = cdrom_probe_device, .bdrv_parse_filename = cdrom_parse_filename, diff --git a/block/iscsi.c b/block/iscsi.c index a7e8c1ffaf..70f07b2105 100644 --- a/block/iscsi.c +++ b/block/iscsi.c @@ -2456,6 +2456,7 @@ static BlockDriver bdrv_iscsi = { .bdrv_close = iscsi_close, .bdrv_co_create_opts = iscsi_co_create_opts, .create_opts = &iscsi_create_opts, + .runtime_opts = &runtime_opts, .bdrv_reopen_prepare = iscsi_reopen_prepare, .bdrv_reopen_commit = iscsi_reopen_commit, .bdrv_co_invalidate_cache = iscsi_co_invalidate_cache, @@ -2493,6 +2494,7 @@ static BlockDriver bdrv_iser = { .bdrv_close = iscsi_close, .bdrv_co_create_opts = iscsi_co_create_opts, .create_opts = &iscsi_create_opts, + .runtime_opts = &runtime_opts, .bdrv_reopen_prepare = iscsi_reopen_prepare, .bdrv_reopen_commit = iscsi_reopen_commit, .bdrv_co_invalidate_cache = iscsi_co_invalidate_cache, diff --git a/block/null.c b/block/null.c index d442d3e901..5df4b46723 100644 --- a/block/null.c +++ b/block/null.c @@ -256,6 +256,7 @@ static BlockDriver bdrv_null_co = { .format_name = "null-co", .protocol_name = "null-co", .instance_size = sizeof(BDRVNullState), + .runtime_opts = &runtime_opts, .bdrv_file_open = null_file_open, .bdrv_parse_filename = null_co_parse_filename, @@ -275,6 +276,7 @@ static BlockDriver bdrv_null_aio = { .format_name = "null-aio", .protocol_name = "null-aio", .instance_size = sizeof(BDRVNullState), + .runtime_opts = &runtime_opts, .bdrv_file_open = null_file_open, .bdrv_parse_filename = null_aio_parse_filename, diff --git a/block/nvme.c b/block/nvme.c index 982097b5b1..7c71fba057 100644 --- a/block/nvme.c +++ b/block/nvme.c @@ -1139,6 +1139,7 @@ static BlockDriver bdrv_nvme = { .format_name = "nvme", .protocol_name = "nvme", .instance_size = sizeof(BDRVNVMeState), + .runtime_opts = &runtime_opts, .bdrv_parse_filename = nvme_parse_filename, .bdrv_file_open = nvme_file_open, diff --git a/block/qcow.c b/block/qcow.c index 0a235bf393..bffece818a 100644 --- a/block/qcow.c +++ b/block/qcow.c @@ -1186,6 +1186,7 @@ static QemuOptsList qcow_create_opts = { static BlockDriver bdrv_qcow = { .format_name = "qcow", .instance_size = sizeof(BDRVQcowState), + .runtime_opts = &qcow_runtime_opts, .bdrv_probe = qcow_probe, .bdrv_open = qcow_open, .bdrv_close = qcow_close, diff --git a/block/rbd.c b/block/rbd.c index 8a1a9f4b6e..6de6112ce8 100644 --- a/block/rbd.c +++ b/block/rbd.c @@ -1231,6 +1231,7 @@ static QemuOptsList qemu_rbd_create_opts = { static BlockDriver bdrv_rbd = { .format_name = "rbd", .instance_size = sizeof(BDRVRBDState), + .runtime_opts = &runtime_opts, .bdrv_parse_filename = qemu_rbd_parse_filename, .bdrv_refresh_limits = qemu_rbd_refresh_limits, .bdrv_file_open = qemu_rbd_open, diff --git a/block/sheepdog.c b/block/sheepdog.c index 90ab43baa4..6dd66d0b99 100644 --- a/block/sheepdog.c +++ b/block/sheepdog.c @@ -3288,6 +3288,7 @@ static BlockDriver bdrv_sheepdog_tcp = { .bdrv_attach_aio_context = sd_attach_aio_context, .create_opts = &sd_create_opts, + .runtime_opts = &runtime_opts, }; static BlockDriver bdrv_sheepdog_unix = { @@ -3325,6 +3326,7 @@ static BlockDriver bdrv_sheepdog_unix = { .bdrv_attach_aio_context = sd_attach_aio_context, .create_opts = &sd_create_opts, + .runtime_opts = &runtime_opts, }; static void bdrv_sheepdog_init(void) diff --git a/block/vpc.c b/block/vpc.c index d886465b7e..640c956d0d 100644 --- a/block/vpc.c +++ b/block/vpc.c @@ -1235,6 +1235,7 @@ static BlockDriver bdrv_vpc = { .bdrv_get_info = vpc_get_info, .create_opts = &vpc_create_opts, + .runtime_opts = &vpc_runtime_opts, .bdrv_has_zero_init = vpc_has_zero_init, }; diff --git a/include/block/block_int.h b/include/block/block_int.h index fd0e88d17a..e680dda86b 100644 --- a/include/block/block_int.h +++ b/include/block/block_int.h @@ -345,6 +345,13 @@ struct BlockDriver { /* List of options for creating images, terminated by name == NULL */ QemuOptsList *create_opts; + /* Runtime options for a block device, terminated by name == NULL */ + QemuOptsList *runtime_opts; + /* + * Names of the runtime options that can be reset by omitting + * their value on reopen, NULL-terminated. + */ + const char *const *mutable_opts; /* * Returns 0 for completed check, -errno for internal errors.