On 21/09/21 03:01, Oliver Upton wrote:
Building demand_paging_test.c with clang throws the following warning:

demand_paging_test.c:182:7: error: logical not is only applied to the left hand 
side of this bitwise operator [-Werror,-Wlogical-not-parentheses]
                   if (!pollfd[0].revents & POLLIN)

Silence the warning by placing the bitwise operation within parentheses.

Signed-off-by: Oliver Upton <[email protected]>
---
  tools/testing/selftests/kvm/demand_paging_test.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/testing/selftests/kvm/demand_paging_test.c 
b/tools/testing/selftests/kvm/demand_paging_test.c
index e79c1b64977f..10edae425ab3 100644
--- a/tools/testing/selftests/kvm/demand_paging_test.c
+++ b/tools/testing/selftests/kvm/demand_paging_test.c
@@ -179,7 +179,7 @@ static void *uffd_handler_thread_fn(void *arg)
                        return NULL;
                }
- if (!pollfd[0].revents & POLLIN)
+               if (!(pollfd[0].revents & POLLIN))
                        continue;
r = read(uffd, &msg, sizeof(msg));


Queued (with small adjustments to the commit message and Cc: stable), thanks.

Paolo

_______________________________________________
kvmarm mailing list
[email protected]
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

Reply via email to