conf: Add support for setting timeout and readahead size for network disks

Some disk backends support configuring the readahead buffer or timeout
for requests. Add the knobs to the XML.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
index 9430ccdee192fe1879dd0981abc0878dcdd3bfa2..b6ddfa8e0cc7a64c73fe8f59ad41b5cc01d7ccbd 100644 (file)
--- a/docs/formatdomain.html.in
+++ b/docs/formatdomain.html.in
@@ -2852,6 +2852,8 @@
       <cookies>
         <cookie name="test">somevalue</cookie>
       </cookies>
+      <readahead size='65536'/>
+      <timeout seconds='6'/>
     </source>
     <target dev='hde' bus='ide' tray='open'/>
     <readonly/>
@@ -3402,6 +3404,20 @@
             must conform to the HTTP specification.
             <span class="since">Since 6.2.0</span>
           </dd>
+          <dt><code>readahead</code></dt>
+          <dd>
+            Specifies the size of the readahead buffer for protocols
+            which support it. (all 'curl' based drivers in qemu). The size
+            is in bytes. Note that '0' is considered as if the value is not
+            provided.
+            <span class="since">Since 6.2.0</span>
+          </dd>
+          <dt><code>timeout</code></dt>
+          <dd>
+            Specifies the connection timeout for protocols which support it.
+            Note that '0' is considered as if the value is not provided.
+            <span class="since">Since 6.2.0</span>
+          </dd>
         </dl>
 
         <p>

diff --git a/docs/schemas/domaincommon.rng b/docs/schemas/domaincommon.rng
index 85d6484dbdfa32100c5416739e25ca8d65592fe8..68054204519f0f4ccb8ebd1f312367359bb7ad2b 100644 (file)
--- a/docs/schemas/domaincommon.rng
+++ b/docs/schemas/domaincommon.rng
@@ -1808,6 +1808,25 @@
     </element>
   </define>
 
+  <define name="diskSourceNetworkProtocolPropsCommon">
+    <optional>
+      <element name="readahead">
+        <attribute name="size">
+          <ref name="positiveInteger"/>
+        </attribute>
+        <empty/>
+      </element>
+    </optional>
+    <optional>
+      <element name="timeout">
+        <attribute name="seconds">
+          <ref name="positiveInteger"/>
+        </attribute>
+        <empty/>
+      </element>
+    </optional>
+  </define>
+
   <define name="diskSourceNetworkProtocolSSLVerify">
     <element name="ssl">
       <attribute name="verify">
@@ -1854,6 +1873,7 @@
       <optional>
         <ref name="diskSourceNetworkProtocolHTTPCookies"/>
       </optional>
+      <ref name="diskSourceNetworkProtocolPropsCommon"/>
     </element>
   </define>
 
@@ -1873,6 +1893,7 @@
       <optional>
         <ref name="diskSourceNetworkProtocolHTTPCookies"/>
       </optional>
+      <ref name="diskSourceNetworkProtocolPropsCommon"/>
     </element>
   </define>
 
@@ -1892,6 +1913,7 @@
       <optional>
         <ref name="diskSourceNetworkProtocolSSLVerify"/>
       </optional>
+      <ref name="diskSourceNetworkProtocolPropsCommon"/>
     </element>
   </define>
 
@@ -1910,6 +1932,7 @@
       <optional>
         <ref name="encryption"/>
       </optional>
+      <ref name="diskSourceNetworkProtocolPropsCommon"/>
     </element>
   </define>
 

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index f3de76efe4814ad24c98a3f8528e6a66681fa998..71535f53f57a0721858b1ea983792c3ae63e75fe 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -9500,6 +9500,19 @@ virDomainDiskSourceNetworkParse(xmlNodePtr node,
             return -1;
     }
 
+    if (src->protocol == VIR_STORAGE_NET_PROTOCOL_HTTP ||
+        src->protocol == VIR_STORAGE_NET_PROTOCOL_HTTPS ||
+        src->protocol == VIR_STORAGE_NET_PROTOCOL_FTP ||
+        src->protocol == VIR_STORAGE_NET_PROTOCOL_FTPS) {
+
+        if (virXPathULongLong("string(./readahead/@size)", ctxt, &src->readahead) == -2 ||
+            virXPathULongLong("string(./timeout/@seconds)", ctxt, &src->timeout) == -2) {
+            virReportError(VIR_ERR_XML_ERROR, "%s",
+                          _("invalid readahead size or timeout"));
+            return -1;
+        }
+    }
+
     return 0;
 }
 
@@ -24632,6 +24645,12 @@ virDomainDiskSourceFormatNetwork(virBufferPtr attrBuf,
 
     virDomainDiskSourceFormatNetworkCookies(childBuf, src);
 
+    if (src->readahead)
+        virBufferAsprintf(childBuf, "<readahead size='%llu'/>\n", src->readahead);
+
+    if (src->timeout)
+        virBufferAsprintf(childBuf, "<timeout seconds='%llu'/>\n", src->timeout);
+
     return 0;
 }
 

diff --git a/src/util/virstoragefile.c b/src/util/virstoragefile.c
index 266e9c489e5a8e5edfb26c63fe20260dc1e71388..a80bffa015305093eb8f438bb7997b29f33639c3 100644 (file)
--- a/src/util/virstoragefile.c
+++ b/src/util/virstoragefile.c
@@ -2383,6 +2383,8 @@ virStorageSourceCopy(const virStorageSource *src,
     def->discard = src->discard;
     def->detect_zeroes = src->detect_zeroes;
     def->sslverify = src->sslverify;
+    def->readahead = src->readahead;
+    def->timeout = src->timeout;
 
     /* storage driver metadata are not copied */
     def->drv = NULL;

diff --git a/src/util/virstoragefile.h b/src/util/virstoragefile.h
index 95d9501dd8f2cd738622ac0dd5d13d2ec8e98036..dd2186c4ff4292e8b000cebec082b45c4994ce01 100644 (file)
--- a/src/util/virstoragefile.h
+++ b/src/util/virstoragefile.h
@@ -295,6 +295,9 @@ struct _virStorageSource {
     bool encryptionInherited;
     virStoragePRDefPtr pr;
     virTristateBool sslverify;
+    /* both values below have 0 as default value */
+    unsigned long long readahead; /* size of the readahead buffer in bytes */
+    unsigned long long timeout; /* connection timeout in seconds */
 
     virStorageSourceNVMeDefPtr nvme; /* type == VIR_STORAGE_TYPE_NVME */
 

diff --git a/tests/genericxml2xmlindata/disk-network-http.xml b/tests/genericxml2xmlindata/disk-network-http.xml
index bafb77c8ecd24322dec63ef2b1c5889441f72c46..a8430b83655d67472fa73dc6fe96fc5a52cd15b7 100644 (file)
--- a/tests/genericxml2xmlindata/disk-network-http.xml
+++ b/tests/genericxml2xmlindata/disk-network-http.xml
@@ -49,6 +49,8 @@
           <cookie name='test'>testcookievalue</cookie>
           <cookie name='test2'>blurb</cookie>
         </cookies>
+        <readahead size='65536'/>
+        <timeout seconds='10'/>
       </source>
       <target dev='vdd' bus='virtio'/>
     </disk>