Module Name: src Committed By: riastradh Date: Mon Feb 14 13:02:31 UTC 2022
Modified Files: src/sys/external/bsd/drm2/dist/drm: drm_mm.c Log Message: drm/mm: Assert invariant of rb lookup. Just to make sure I didn't get the sense of the lookup reversed, which is quite likely, because I've done it probably more than once in this code before... To generate a diff of this commit: cvs rdiff -u -r1.14 -r1.15 src/sys/external/bsd/drm2/dist/drm/drm_mm.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/external/bsd/drm2/dist/drm/drm_mm.c diff -u src/sys/external/bsd/drm2/dist/drm/drm_mm.c:1.14 src/sys/external/bsd/drm2/dist/drm/drm_mm.c:1.15 --- src/sys/external/bsd/drm2/dist/drm/drm_mm.c:1.14 Sun Dec 19 11:51:32 2021 +++ src/sys/external/bsd/drm2/dist/drm/drm_mm.c Mon Feb 14 13:02:31 2022 @@ -1,4 +1,4 @@ -/* $NetBSD: drm_mm.c,v 1.14 2021/12/19 11:51:32 riastradh Exp $ */ +/* $NetBSD: drm_mm.c,v 1.15 2022/02/14 13:02:31 riastradh Exp $ */ /************************************************************************** * @@ -45,7 +45,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: drm_mm.c,v 1.14 2021/12/19 11:51:32 riastradh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: drm_mm.c,v 1.15 2022/02/14 13:02:31 riastradh Exp $"); #include <linux/export.h> #include <linux/interval_tree_generic.h> @@ -401,7 +401,12 @@ static inline u64 rb_hole_size(struct rb static struct drm_mm_node *best_hole(struct drm_mm *mm, u64 size) { #ifdef __NetBSD__ - return rb_tree_find_node_leq(&mm->holes_size.rb_root.rbr_tree, &size); + struct drm_mm_node *best; + + best = rb_tree_find_node_leq(&mm->holes_size.rb_root.rbr_tree, &size); + KASSERT(best == NULL || size <= best->hole_size); + + return best; #else struct rb_node *rb = mm->holes_size.rb_root.rb_node; struct drm_mm_node *best = NULL;