search for: end_head

...obably missing "content-length: 0" header in the response. If we don't close the connection and don't send content-length the client cannot do much but wait :-) > > + def do_OPTIONS(self): > > > > + self.send_response(200) > > > + self.send_header("Content-type", "application/json; > > > charset=UTF-8") > > > + self.end_headers() > > > + # Advertize only zero support. > > > + self.wfile.write(b'''{ "features": [ "zero" ] }''...

...will close the connection after every request. This is not a good implementation of the imageio server, and also hides bugs in this code. Should be fixed by adding: protocol_version = "HTTP/1.1" > + def do_OPTIONS(self): + self.send_response(200) > + self.send_header("Content-type", "application/json; > charset=UTF-8") > + self.end_headers() > + # Advertize only zero support. > + self.wfile.write(b'''{ "features": [ "zero" ] }''') > + > + # eg. zero reque...

...24 +116,44 @@ from http.server import HTTPServer, BaseHTTPRequestHandler > import threading > > class RequestHandler(BaseHTTPRequestHandler): > + protocol_version = 'HTTP/1.1' > + > def do_OPTIONS(self): > - self.send_response(200) > - self.send_header("Content-type", "application/json; charset=UTF-8") > - self.end_headers() > + self.discard_request() > + > # Advertize only zero support. > - self.wfile.write(b'''{ "features": [ "zero" ] }''&...

...also > hides bugs in this code. > > Should be fixed by adding: > > protocol_version = "HTTP/1.1" I tried the attached patch, but for some reason it just hangs. > > + def do_OPTIONS(self): > > + self.send_response(200) > > + self.send_header("Content-type", "application/json; > > charset=UTF-8") > > + self.end_headers() > > + # Advertize only zero support. > > + self.wfile.write(b'''{ "features": [ "zero" ] }''') > > + &...

...color #008000 \end_index \secnumdepth 3 \tocdepth 3 \paragraph_separation indent \paragraph_indentation default \quotes_language english \papercolumns 1 \papersides 1 \paperpagestyle default \tracking_changes false \output_changes false \html_math_output 0 \html_css_as_file 0 \html_be_strict false \end_header \begin_body \begin_layout Standard Antoine ? bella. \end_layout \end_body \end_document

...diff --git a/virtio-0.9.4.lyx b/virtio-0.9.4.lyx index 6c7bab1..ef3951c 100644 --- a/virtio-0.9.4.lyx +++ b/virtio-0.9.4.lyx @@ -58,6 +58,7 @@ \html_be_strict false \author -608949062 "Rusty Russell,,," \author 1531152142 "pbonzini" +\author 2090695081 "Jason" \end_header \begin_body @@ -4012,8 +4013,19 @@ configuration layout Two configuration fields are currently defined. The mac address field always exists (though is only valid if VIRTIO_NET_F_MAC is set), and the status field only exists if VIRTIO_NET_F_STATUS is set. + +\change_inserted 2090695081 133...

...diff --git a/virtio-0.9.4.lyx b/virtio-0.9.4.lyx index 6c7bab1..ef3951c 100644 --- a/virtio-0.9.4.lyx +++ b/virtio-0.9.4.lyx @@ -58,6 +58,7 @@ \html_be_strict false \author -608949062 "Rusty Russell,,," \author 1531152142 "pbonzini" +\author 2090695081 "Jason" \end_header \begin_body @@ -4012,8 +4013,19 @@ configuration layout Two configuration fields are currently defined. The mac address field always exists (though is only valid if VIRTIO_NET_F_MAC is set), and the status field only exists if VIRTIO_NET_F_STATUS is set. + +\change_inserted 2090695081 133...

...import HTTPServer, BaseHTTPRequestHandler +import random +import threading + +# Choose a random port number in range [50000,59999] +imageio_port = random.randint(50000,60000) + +class RequestHandler(BaseHTTPRequestHandler): + def do_OPTIONS(self): + self.send_response(200) + self.send_header("Content-type", "application/json; charset=UTF-8") + self.end_headers() + # Advertize only zero support. + self.wfile.write(b'''{ "features": [ "zero" ] }''') + + # eg. zero request. Just ignore it. + def...

v1 was here: https://www.redhat.com/archives/libguestfs/2018-September/msg00121.html v2: - Rewrote patch 2 from scratch so it incorporates Nir's suggestions. - Add fake module to EXTRA_DIST. - Retested. Unfortunately I am no longer able to test the ordinary conversion path because ovirtsdk4 is incompatible with Fedora 29 / Python 3.7:

This adds a test of -o rhv-upload. Obviously for an upstream test we cannot require a working oVirt server. This test works by faking the ovirtsdk4 Python module, setting PYTHONPATH so that the fake module is picked up instead of the real module (if installed). However it's more complex than that because the nbdkit plugin also expects to talk to a working imageio HTTPS server. Therefore

...hv-upload-module/ovirtsdk4/__init__.py > index 84b9d56aa..8d1058d67 100644 > --- a/v2v/test-v2v-o-rhv-upload-module/ovirtsdk4/__init__.py > +++ b/v2v/test-v2v-o-rhv-upload-module/ovirtsdk4/__init__.py > @@ -146,7 +146,7 @@ class RequestHandler(BaseHTTPRequestHandler): > self.end_headers() > > def discard_request(self): > - length = self.headers['Content-Length'] > + length = self.headers.get('Content-Length') > if length: > length = int(length) > content = self.rfile.read(length) > -...

...org/ +#LyX 1.6.4 created this file. For more info see http://www.lyx.org/ \lyxformat 345 \begin_document \begin_header @@ -36,7 +36,7 @@ \paperpagestyle default \tracking_changes true \output_changes true -\author "" +\author "Michael S. Tsirkin" \author "" \end_header @@ -1971,8 +1971,13 @@ Notifying The Device \begin_layout Standard Device notification occurs by writing the 16-bit virtqueue index of this - virtqueue to the Queue Notify field of the virtio header in the PCI configurati -on space. + virtqueue to the Queue Notify field of the virtio header...

...org/ +#LyX 1.6.4 created this file. For more info see http://www.lyx.org/ \lyxformat 345 \begin_document \begin_header @@ -36,7 +36,7 @@ \paperpagestyle default \tracking_changes true \output_changes true -\author "" +\author "Michael S. Tsirkin" \author "" \end_header @@ -1971,8 +1971,13 @@ Notifying The Device \begin_layout Standard Device notification occurs by writing the 16-bit virtqueue index of this - virtqueue to the Queue Notify field of the virtio header in the PCI configurati -on space. + virtqueue to the Queue Notify field of the virtio header...

...c4 100644 --- a/virtio-spec.lyx +++ b/virtio-spec.lyx @@ -59,6 +59,7 @@ \author -608949062 "Rusty Russell,,," \author -385801441 "Cornelia Huck" cornelia.huck at de.ibm.com \author 1531152142 "Paolo Bonzini,,," +\author 1986246365 "Michael S. Tsirkin" \end_header \begin_body @@ -4170,9 +4171,46 @@ ID 1 \end_layout \begin_layout Description -Virtqueues 0:receiveq. - 1:transmitq. - 2:controlq +Virtqueues 0:receiveq +\change_inserted 1986246365 1352742829 +0 +\change_unchanged +. + 1:transmitq +\change_inserted 1986246365 1352742832 +0 +\change_delete...

...c4 100644 --- a/virtio-spec.lyx +++ b/virtio-spec.lyx @@ -59,6 +59,7 @@ \author -608949062 "Rusty Russell,,," \author -385801441 "Cornelia Huck" cornelia.huck at de.ibm.com \author 1531152142 "Paolo Bonzini,,," +\author 1986246365 "Michael S. Tsirkin" \end_header \begin_body @@ -4170,9 +4171,46 @@ ID 1 \end_layout \begin_layout Description -Virtqueues 0:receiveq. - 1:transmitq. - 2:controlq +Virtqueues 0:receiveq +\change_inserted 1986246365 1352742829 +0 +\change_unchanged +. + 1:transmitq +\change_inserted 1986246365 1352742832 +0 +\change_delete...

...d this file. For more info see http://www.lyx.org/ \lyxformat 345 \begin_document \begin_header @@ -36,8 +36,7 @@ \paperpagestyle default \tracking_changes true \output_changes true -\author "" -\author "" +\author "Michael S. Tsirkin" \author "" \end_header @@ -1441,7 +1440,28 @@ struct vring_desc { \begin_layout Plain Layout -#define VRING_DESC_F_WRITE 2 +#define VRING_DESC_F_WRITE 2 +\change_deleted 0 1265802057 + +\change_inserted 0 1265802048 + +\end_layout + +\begin_layout Plain Layout + +\change_inserted 0 1265802054 + +/* Thi...

...d this file. For more info see http://www.lyx.org/ \lyxformat 345 \begin_document \begin_header @@ -36,8 +36,7 @@ \paperpagestyle default \tracking_changes true \output_changes true -\author "" -\author "" +\author "Michael S. Tsirkin" \author "" \end_header @@ -1441,7 +1440,28 @@ struct vring_desc { \begin_layout Plain Layout -#define VRING_DESC_F_WRITE 2 +#define VRING_DESC_F_WRITE 2 +\change_deleted 0 1265802057 + +\change_inserted 0 1265802048 + +\end_layout + +\begin_layout Plain Layout + +\change_inserted 0 1265802054 + +/* Thi...

.../ if ( partinfo ) { /* Actually read the boot sector */ /* Pick the first buffer that isn't already in use */ boot_sector = (char *)(((unsigned long)partinfo + 511) & ~511); sprintf(logs, "partinfo: active_flag %d, start_head %d, start_sect %d, start_cyl %d, ostype %d, end_head %d, end_sect %d, end_cyl %d, start_lba %d, length %d, boot_sector %s\n", partinfo->active_flag, partinfo->start_head, partinfo->start_sect, partinfo->start_cyl, partinfo->ostype, partinfo->end_head, partinfo->end_sect, partinfo->end_cyl, partinfo->start_lba, partinfo...

...ted this file. For more info see http://www.lyx.org/ \lyxformat 345 \begin_document \begin_header @@ -36,7 +36,8 @@ \paperpagestyle default \tracking_changes true \output_changes true -\author "Rusty Russell,,," +\author "" +\author "" \author "" \end_header @@ -1784,6 +1785,9 @@ \begin_layout Standard The number of descriptors in the table is specified by the Queue Size field for this virtqueue. +\end_layout + +\begin_layout Subsection \begin_inset CommandInset label LatexCommand label name "sub:Indirect-Descriptors" @@ -2076,8 +2...

...ted this file. For more info see http://www.lyx.org/ \lyxformat 345 \begin_document \begin_header @@ -36,7 +36,8 @@ \paperpagestyle default \tracking_changes true \output_changes true -\author "Rusty Russell,,," +\author "" +\author "" \author "" \end_header @@ -1784,6 +1785,9 @@ \begin_layout Standard The number of descriptors in the table is specified by the Queue Size field for this virtqueue. +\end_layout + +\begin_layout Subsection \begin_inset CommandInset label LatexCommand label name "sub:Indirect-Descriptors" @@ -2076,8 +2...