Commit 29b2681f authored by Peng Zhang's avatar Peng Zhang Committed by Andrew Morton

maple_tree: replace mas_logical_pivot() with mas_safe_pivot()

Replace mas_logical_pivot() with mas_safe_pivot() and drop
mas_logical_pivot() since it won't be used anymore.  We can do this since
now all nodes will have node limit pivot (if it is not full node).

Link: https://lkml.kernel.org/r/20230711035444.526-8-zhangpeng.00@bytedance.comSigned-off-by: default avatarPeng Zhang <zhangpeng.00@bytedance.com>
Reviewed-by: default avatarLiam R. Howlett <Liam.Howlett@oracle.com>
Tested-by: default avatarGeert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent a489539e
...@@ -728,33 +728,6 @@ mas_safe_min(struct ma_state *mas, unsigned long *pivots, unsigned char offset) ...@@ -728,33 +728,6 @@ mas_safe_min(struct ma_state *mas, unsigned long *pivots, unsigned char offset)
return mas->min; return mas->min;
} }
/*
* mas_logical_pivot() - Get the logical pivot of a given offset.
* @mas: The maple state
* @pivots: The pointer to the maple node pivots
* @offset: The offset into the pivot array
* @type: The maple node type
*
* When there is no value at a pivot (beyond the end of the data), then the
* pivot is actually @mas->max.
*
* Return: the logical pivot of a given @offset.
*/
static inline unsigned long
mas_logical_pivot(struct ma_state *mas, unsigned long *pivots,
unsigned char offset, enum maple_type type)
{
unsigned long lpiv = mas_safe_pivot(mas, pivots, offset, type);
if (likely(lpiv))
return lpiv;
if (likely(offset))
return mas->max;
return lpiv;
}
/* /*
* mte_set_pivot() - Set a pivot to a value in an encoded maple node. * mte_set_pivot() - Set a pivot to a value in an encoded maple node.
* @mn: The encoded maple node * @mn: The encoded maple node
...@@ -2202,7 +2175,7 @@ static noinline_for_kasan void mas_store_b_node(struct ma_wr_state *wr_mas, ...@@ -2202,7 +2175,7 @@ static noinline_for_kasan void mas_store_b_node(struct ma_wr_state *wr_mas,
goto b_end; goto b_end;
/* Handle new range ending before old range ends */ /* Handle new range ending before old range ends */
piv = mas_logical_pivot(mas, wr_mas->pivots, offset_end, wr_mas->type); piv = mas_safe_pivot(mas, wr_mas->pivots, offset_end, wr_mas->type);
if (piv > mas->last) { if (piv > mas->last) {
if (piv == ULONG_MAX) if (piv == ULONG_MAX)
mas_bulk_rebalance(mas, b_node->b_end, wr_mas->type); mas_bulk_rebalance(mas, b_node->b_end, wr_mas->type);
...@@ -4935,7 +4908,7 @@ static inline bool mas_anode_descend(struct ma_state *mas, unsigned long size) ...@@ -4935,7 +4908,7 @@ static inline bool mas_anode_descend(struct ma_state *mas, unsigned long size)
min = mas_safe_min(mas, pivots, offset); min = mas_safe_min(mas, pivots, offset);
data_end = ma_data_end(node, type, pivots, mas->max); data_end = ma_data_end(node, type, pivots, mas->max);
for (; offset <= data_end; offset++) { for (; offset <= data_end; offset++) {
pivot = mas_logical_pivot(mas, pivots, offset, type); pivot = mas_safe_pivot(mas, pivots, offset, type);
/* Not within lower bounds */ /* Not within lower bounds */
if (mas->index > pivot) if (mas->index > pivot)
...@@ -6981,7 +6954,7 @@ static void mas_validate_gaps(struct ma_state *mas) ...@@ -6981,7 +6954,7 @@ static void mas_validate_gaps(struct ma_state *mas)
gaps = ma_gaps(node, mt); gaps = ma_gaps(node, mt);
for (i = 0; i < mt_slot_count(mte); i++) { for (i = 0; i < mt_slot_count(mte); i++) {
p_end = mas_logical_pivot(mas, pivots, i, mt); p_end = mas_safe_pivot(mas, pivots, i, mt);
if (!gaps) { if (!gaps) {
if (!mas_get_slot(mas, i)) if (!mas_get_slot(mas, i))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment