From 42f96162c3d648ea4302dbb4c28b1edf1da3651b Mon Sep 17 00:00:00 2001 From: Konstantin Belousov Date: Thu, 8 Oct 2020 22:46:15 +0000 Subject: [PATCH] vm_page_dump_index_to_pa(): Add braces to the expression involving + and &. The precedence of the '&' operator is less than of '+'. Added braces do change the order of evaluation into the natural one, in my opinion. On the other hand, the value of the expression should not change since all elements should have page-aligned values. This fixes a gcc warning reported. Reported by: adrian Sponsored by: The FreeBSD Foundation MFC after: 1 week --- sys/vm/vm_page.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys/vm/vm_page.h b/sys/vm/vm_page.h index 915be661699..ff8bd0a9f21 100644 --- a/sys/vm/vm_page.h +++ b/sys/vm/vm_page.h @@ -639,7 +639,7 @@ vm_page_dump_index_to_pa(int bit) dump_avail[i] / PAGE_SIZE; if (bit < tot) return ((vm_paddr_t)bit * PAGE_SIZE + - dump_avail[i] & ~PAGE_MASK); + (dump_avail[i] & ~PAGE_MASK)); bit -= tot; } return ((vm_paddr_t)NULL);