The BUG_ON() was misplaced when this function was introduced in commit
ec83f825 "mm.h: add helper function to test-and-clear _PGC_allocated".
It will fire incorrectly if _PGC_allocated is already clear on entry. Thus
it should be moved after the if statement.
Signed-off-by: Paul Durrant <paul.durr...@citrix.com>
---
Cc: Andrew Cooper <andrew.coop...@citrix.com>
Cc: George Dunlap <george.dun...@eu.citrix.com>
Cc: Ian Jackson <ian.jack...@eu.citrix.com>
Cc: Jan Beulich <jbeul...@suse.com>
Cc: Julien Grall <julien.gr...@arm.com>
Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
Cc: Stefano Stabellini <sstabell...@kernel.org>
Cc: Tim Deegan <t...@xen.org>
Cc: Wei Liu <w...@xen.org>
---
xen/include/xen/mm.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/xen/include/xen/mm.h b/xen/include/xen/mm.h
index 5d0c19f011..cfea38f801 100644
--- a/xen/include/xen/mm.h
+++ b/xen/include/xen/mm.h
@@ -667,9 +667,11 @@ static inline void put_page_alloc_ref(struct page_info
*page)
* the caller holding an additional reference. I.e. the allocation
* reference must never be the last reference held.
*/
- BUG_ON((page->count_info & PGC_count_mask) <= 1);
if ( test_and_clear_bit(_PGC_allocated, &page->count_info) )
+ {
+ BUG_ON((page->count_info & PGC_count_mask) <= 1);
put_page(page);
+ }
}
#endif /* __XEN_MM_H__ */
--
2.20.1.2.gb21ebb671
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
