main-loop: drop spin_counter
commit21891a5a3011608845b5d7f1f9cce60cdc2bcc62
authorStefan Hajnoczi <stefanha@redhat.com>
Wed, 30 May 2018 19:42:38 +0000 (30 20:42 +0100)
committerStefan Hajnoczi <stefanha@redhat.com>
Fri, 1 Jun 2018 15:01:29 +0000 (1 16:01 +0100)
tree0139d82b77231d52d622848849f7697e021f6533
parentee5306d0923377439776e8a30b9fd2de34b5cbfb
main-loop: drop spin_counter

Commit d759c951f3287fad04210a52f2dc93f94cf58c7f ("replay: push
replay_mutex_lock up the call tree") removed the !timeout lock
optimization in the main loop.

The idea of the optimization was to avoid ping-pongs between threads by
keeping the Big QEMU Lock held across non-blocking (!timeout) main loop
iterations.

A warning is printed when the main loop spins without releasing BQL for
long periods of time.  These warnings were supposed to aid debugging but
in practice they just alarm users.  They are considered noise because
the cause of spinning is not shown and is hard to find.

Now that the lock optimization has been removed, there is no danger of
hogging the BQL.  Drop the spin counter and the infamous warning.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Jeff Cody <jcody@redhat.com>
tests/qemu-iotests/common.filter
util/main-loop.c