[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v9 27/31] block/curl: Harmonize option defaults
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PATCH v9 27/31] block/curl: Harmonize option defaults |
Date: |
Thu, 28 Jun 2018 02:07:41 +0200 |
Both of the defaults we currently have in the curl driver are named
based on a slightly different schema, let's unify that and call both
CURL_BLOCK_OPT_${NAME}_DEFAULT.
While at it, we can add a macro for the third option for which a default
exists, namely "sslverify".
Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Alberto Garcia <address@hidden>
---
block/curl.c | 13 ++++++++-----
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/block/curl.c b/block/curl.c
index 8e4f17266b..f095fe4664 100644
--- a/block/curl.c
+++ b/block/curl.c
@@ -73,8 +73,6 @@ static CURLMcode __curl_multi_socket_action(CURLM
*multi_handle,
#define CURL_NUM_STATES 8
#define CURL_NUM_ACB 8
-#define READ_AHEAD_DEFAULT (256 * 1024)
-#define CURL_TIMEOUT_DEFAULT 5
#define CURL_TIMEOUT_MAX 10000
#define CURL_BLOCK_OPT_URL "url"
@@ -88,6 +86,10 @@ static CURLMcode __curl_multi_socket_action(CURLM
*multi_handle,
#define CURL_BLOCK_OPT_PROXY_USERNAME "proxy-username"
#define CURL_BLOCK_OPT_PROXY_PASSWORD_SECRET "proxy-password-secret"
+#define CURL_BLOCK_OPT_READAHEAD_DEFAULT (256 * 1024)
+#define CURL_BLOCK_OPT_SSLVERIFY_DEFAULT true
+#define CURL_BLOCK_OPT_TIMEOUT_DEFAULT 5
+
struct BDRVCURLState;
static bool libcurl_initialized;
@@ -706,7 +708,7 @@ static int curl_open(BlockDriverState *bs, QDict *options,
int flags,
}
s->readahead_size = qemu_opt_get_size(opts, CURL_BLOCK_OPT_READAHEAD,
- READ_AHEAD_DEFAULT);
+ CURL_BLOCK_OPT_READAHEAD_DEFAULT);
if ((s->readahead_size & 0x1ff) != 0) {
error_setg(errp, "HTTP_READAHEAD_SIZE %zd is not a multiple of 512",
s->readahead_size);
@@ -714,13 +716,14 @@ static int curl_open(BlockDriverState *bs, QDict
*options, int flags,
}
s->timeout = qemu_opt_get_number(opts, CURL_BLOCK_OPT_TIMEOUT,
- CURL_TIMEOUT_DEFAULT);
+ CURL_BLOCK_OPT_TIMEOUT_DEFAULT);
if (s->timeout > CURL_TIMEOUT_MAX) {
error_setg(errp, "timeout parameter is too large or negative");
goto out_noclean;
}
- s->sslverify = qemu_opt_get_bool(opts, CURL_BLOCK_OPT_SSLVERIFY, true);
+ s->sslverify = qemu_opt_get_bool(opts, CURL_BLOCK_OPT_SSLVERIFY,
+ CURL_BLOCK_OPT_SSLVERIFY_DEFAULT);
cookie = qemu_opt_get(opts, CURL_BLOCK_OPT_COOKIE);
cookie_secret = qemu_opt_get(opts, CURL_BLOCK_OPT_COOKIE_SECRET);
--
2.17.1
- [Qemu-devel] [PATCH v9 16/31] quorum: Make bdrv_dirname() return NULL, (continued)
- [Qemu-devel] [PATCH v9 16/31] quorum: Make bdrv_dirname() return NULL, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 14/31] block: Add bdrv_dirname(), Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 18/31] block/nfs: Implement bdrv_dirname(), Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 19/31] block: Use bdrv_dirname() for relative filenames, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 20/31] iotests: Add quorum case to test 110, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 22/31] block: Add BlockDriver.bdrv_gather_child_options, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 21/31] block: Add strong_runtime_opts to BlockDriver, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 23/31] block: Generically refresh runtime options, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 25/31] block: Do not copy exact_filename from format file, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 26/31] block/nvme: Fix bdrv_refresh_filename(), Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 27/31] block/curl: Harmonize option defaults,
Max Reitz <=
- [Qemu-devel] [PATCH v9 28/31] block/curl: Implement bdrv_refresh_filename(), Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 29/31] block/null: Generate filename even with latency-ns, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 24/31] block: Purify .bdrv_refresh_filename(), Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 30/31] block: BDS options may lack the "driver" option, Max Reitz, 2018/06/27
- [Qemu-devel] [PATCH v9 31/31] iotests: Test json:{} filenames of internal BDSs, Max Reitz, 2018/06/27