summaryrefslogtreecommitdiffstats
path: root/autogen.sh
diff options
context:
space:
mode:
authorYonit Halperin <yhalperi@redhat.com>2013-08-05 12:10:15 -0400
committerYonit Halperin <yhalperi@redhat.com>2013-08-06 14:28:34 -0400
commit6ced0f698507de02a67cfcd25b7ab07e8c423fad (patch)
treec60015308e49d09aa351835776c5bb391ff6191b /autogen.sh
parentc2e46b926e0ee4226f0f93942e7fa2c5b409f73d (diff)
downloadspice-6ced0f698507de02a67cfcd25b7ab07e8c423fad.tar.gz
spice-6ced0f698507de02a67cfcd25b7ab07e8c423fad.tar.xz
spice-6ced0f698507de02a67cfcd25b7ab07e8c423fad.zip
red_worker: decrease the timeout when flushing commands and waiting for the client.
150 seconds is way too long period for holding the guest driver and waiting for a response for the client. This timeout was 15 seconds, but when off-screen surfaces ware introduced it was arbitrarily multiplied by 10. Other existing related bugs emphasize why it is important to decrease the timeout: (1) 994211 - the qxl driver waits for an async-io reponse for 60 seconds and after that, it switches to sync-io mode. Not only that the driver might use invalid data (since it didn't wait for the query to complete), falling back to sync-io mode introduces other errors. (2) 994175 - spice server sometimes doesn't recognize that the client has disconnected. (3) There might be cache inconsistency between the client and the server, and then the display channel waits indefinitely for a cache item (e.g., bug 977998) This patch changes the timeout to 30 seconds. I tested it under wifi +emulating 2.5Mbps network, together with playing video on the guest and changing resolutions in a loop. The timeout didn't expired during my tests. This bug is related to rhbz#964136 (but from rhbz#964136 info it is still not clear why the client wasn't responsive).
Diffstat (limited to 'autogen.sh')
0 files changed, 0 insertions, 0 deletions