[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH v13 27/31] block/curl: Harmonize option defaults
From: |
Max Reitz |
Subject: |
[Qemu-block] [PATCH v13 27/31] block/curl: Harmonize option defaults |
Date: |
Fri, 1 Feb 2019 20:29:31 +0100 |
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 1c9e4f6a64..2c07a694d8 100644
--- a/block/curl.c
+++ b/block/curl.c
@@ -61,8 +61,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"
@@ -76,6 +74,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;
@@ -696,7 +698,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);
@@ -704,13 +706,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.20.1
- [Qemu-block] [PATCH v13 19/31] block: Use bdrv_dirname() for relative filenames, (continued)
- [Qemu-block] [PATCH v13 19/31] block: Use bdrv_dirname() for relative filenames, Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 22/31] block: Add BlockDriver.bdrv_gather_child_options, Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 21/31] block: Add strong_runtime_opts to BlockDriver, Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 28/31] block/curl: Implement bdrv_refresh_filename(), Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 29/31] block/null: Generate filename even with latency-ns, Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 26/31] block/nvme: Fix bdrv_refresh_filename(), Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 25/31] block: Do not copy exact_filename from format file, Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 31/31] iotests: Test json:{} filenames of internal BDSs, Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 23/31] block: Generically refresh runtime options, Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 30/31] block: BDS options may lack the "driver" option, Max Reitz, 2019/02/01
- [Qemu-block] [PATCH v13 27/31] block/curl: Harmonize option defaults,
Max Reitz <=
- [Qemu-block] [PATCH v13 24/31] block: Purify .bdrv_refresh_filename(), Max Reitz, 2019/02/01
- Re: [Qemu-block] [Qemu-devel] [PATCH v13 00/31] Fix some filename generation issues, no-reply, 2019/02/03
- Re: [Qemu-block] [PATCH v13 00/31] Fix some filename generation issues, Max Reitz, 2019/02/10