[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v8 24/26] block/curl: Harmonize option defaults
From: |
Max Reitz |
Subject: |
[Qemu-devel] [PATCH v8 24/26] block/curl: Harmonize option defaults |
Date: |
Mon, 5 Feb 2018 16:18:33 +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>
---
block/curl.c | 13 ++++++++-----
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/block/curl.c b/block/curl.c
index 7c5418324f..4790c7dad5 100644
--- a/block/curl.c
+++ b/block/curl.c
@@ -72,8 +72,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"
@@ -87,6 +85,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;
@@ -714,7 +716,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);
@@ -722,13 +724,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.14.3
- Re: [Qemu-devel] [PATCH v8 20/26] block: Generically refresh runtime options, (continued)
[Qemu-devel] [PATCH v8 24/26] block/curl: Harmonize option defaults,
Max Reitz <=
[Qemu-devel] [PATCH v8 23/26] block: Fix FIXME from "Add BDS.backing_overridden", Max Reitz, 2018/02/05
[Qemu-devel] [PATCH v8 22/26] block: Do not copy exact_filename from format file, Max Reitz, 2018/02/05
[Qemu-devel] [PATCH v8 25/26] block/curl: Implement bdrv_refresh_filename(), Max Reitz, 2018/02/05
[Qemu-devel] [PATCH v8 26/26] block/null: Generate filename even with latency-ns, Max Reitz, 2018/02/05