diff options
| author | Alex Bennée <alex.bennee@linaro.org> | 2022-12-21 09:04:09 +0000 |
|---|---|---|
| committer | Alex Bennée <alex.bennee@linaro.org> | 2022-12-23 15:15:57 +0000 |
| commit | 2bc6c79417b89c3306b724577e775f03fe61fb2e (patch) | |
| tree | f4f8eadd5c78fbca7a7f4180b9417f8cc48b173d /tests/tcg | |
| parent | 7f788779458e0ab8ffeecab42e9b269e98ec9a86 (diff) | |
| download | qemu-2bc6c79417b89c3306b724577e775f03fe61fb2e.zip qemu-2bc6c79417b89c3306b724577e775f03fe61fb2e.tar.gz qemu-2bc6c79417b89c3306b724577e775f03fe61fb2e.tar.bz2 | |
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')
| -rw-r--r-- | tests/tcg/multiarch/linux/linux-test.c | 6 |
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 5a2a4f2..64f57cb 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; + } } } } |