summary refs log tree commit diff stats
path: root/tests/tcg/multiarch/linux/linux-test.c
diff options
context:
space:
mode:
authorAlex Bennée <alex.bennee@linaro.org>2022-12-21 09:04:09 +0000
committerAlex Bennée <alex.bennee@linaro.org>2022-12-23 15:15:57 +0000
commit2bc6c79417b89c3306b724577e775f03fe61fb2e (patch)
treef4f8eadd5c78fbca7a7f4180b9417f8cc48b173d /tests/tcg/multiarch/linux/linux-test.c
parent7f788779458e0ab8ffeecab42e9b269e98ec9a86 (diff)
downloadfocaccia-qemu-2bc6c79417b89c3306b724577e775f03fe61fb2e.tar.gz
focaccia-qemu-2bc6c79417b89c3306b724577e775f03fe61fb2e.zip
tests/tcg: fix unused variable in linux-test
The latest hexagon compiler picks up that we never consume wcount.
Given the name of the #define that rcount checks against is WCOUNT_MAX
I figured the check just got missed.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20221221090411.1995037-5-alex.bennee@linaro.org>
Diffstat (limited to 'tests/tcg/multiarch/linux/linux-test.c')
-rw-r--r--tests/tcg/multiarch/linux/linux-test.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/tests/tcg/multiarch/linux/linux-test.c b/tests/tcg/multiarch/linux/linux-test.c
index 5a2a4f2258..64f57cb287 100644
--- a/tests/tcg/multiarch/linux/linux-test.c
+++ b/tests/tcg/multiarch/linux/linux-test.c
@@ -354,13 +354,17 @@ static void test_pipe(void)
             if (FD_ISSET(fds[0], &rfds)) {
                 chk_error(read(fds[0], &ch, 1));
                 rcount++;
-                if (rcount >= WCOUNT_MAX)
+                if (rcount >= WCOUNT_MAX) {
                     break;
+                }
             }
             if (FD_ISSET(fds[1], &wfds)) {
                 ch = 'a';
                 chk_error(write(fds[1], &ch, 1));
                 wcount++;
+                if (wcount >= WCOUNT_MAX) {
+                    break;
+                }
             }
         }
     }