Commit be59df79 authored by Peter Maydell's avatar Peter Maydell Committed by Paolo Bonzini
Browse files

tests/test-char.c: Don't use main_loop_wait()'s return value 



In QEMU's main_loop() we used to check whether we should do
a nonblocking call to main_loop(); this was deleted in commit e330c118,
because now that vCPUs always drop the I/O thread lock it is an unnecessary
optimization.

The loop in test-char.c copied the old QEMU main_loop() code, but
here the nonblocking check has never been necessary because this
standalone test case doesn't hold the I/O lock anyway. Remove it,
so we can drop the main_loop_wait() return value.

Signed-off-by: default avatarPeter Maydell <peter.maydell@linaro.org>
Message-Id: <1498584769-12439-2-git-send-email-peter.maydell@linaro.org>
Reviewed-by: default avatarMarc-André Lureau <marcandre.lureau@redhat.com>
Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
parent 0ec7b534

tests/test-char.c

+1 −5
Original line number Diff line number Diff line
@@ -20,13 +20,9 @@ typedef struct FeHandler { 


static void main_loop(void)

{

    bool nonblocking;

    int last_io = 0;



    quit = false;

    do {

        nonblocking = last_io > 0;

        last_io = main_loop_wait(nonblocking);

        main_loop_wait(false);

    } while (!quit);

}