Commit 5950ada9 authored by Liam R. Howlett's avatar Liam R. Howlett Committed by Andrew Morton
Browse files

maple_tree: use MAS_BUG_ON() in mas_set_height() 

Use MAS_BUG_ON() instead of MT_BUG_ON() to get the maple state
information.  In the unlikely event of a tree height of > 31, try to
increase the probability of useful information being logged.

Link: https://lkml.kernel.org/r/20230518145544.1722059-12-Liam.Howlett@oracle.com


Signed-off-by: default avatarLiam R. Howlett <Liam.Howlett@oracle.com>
Cc: David Binderman <dcb314@hotmail.com>
Cc: Peng Zhang <zhangpeng.00@bytedance.com>
Cc: Sergey Senozhatsky <senozhatsky@chromium.org>
Cc: Vernon Yang <vernon2gm@gmail.com>
Cc: Wei Yang <richard.weiyang@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent bf96715e

lib/maple_tree.c

+1 −1
Original line number Diff line number Diff line
@@ -194,7 +194,7 @@ static void mas_set_height(struct ma_state *mas) 
	unsigned int new_flags = mas->tree->ma_flags;



	new_flags &= ~MT_FLAGS_HEIGHT_MASK;

	MT_BUG_ON(mas->tree, mas->depth > MAPLE_HEIGHT_MAX);

	MAS_BUG_ON(mas, mas->depth > MAPLE_HEIGHT_MAX);

	new_flags |= mas->depth << MT_FLAGS_HEIGHT_OFFSET;

	mas->tree->ma_flags = new_flags;

}