mirror of
https://github.com/clearlinux/kvmtool.git
synced 2026-10-03 23:38:35 +00:00
kvm tools: remove max_high field in rb_int_node structure
Since nothing depends on the max_high field values anymore, we can just remove the field and the code that was used to maintain it. Signed-off-by: Michel Lespinasse <walken@google.com> Signed-off-by: Pekka Enberg <penberg@kernel.org>
This commit is contained in:
committed by
Will Deacon
parent
ffc83de357
commit
a83f8ed142
@@ -1,20 +1,17 @@
|
||||
#ifndef KVM__INTERVAL_RBTREE_H
|
||||
#define KVM__INTERVAL_RBTREE_H
|
||||
|
||||
#include <linux/rbtree_augmented.h>
|
||||
#include <linux/rbtree.h>
|
||||
#include <linux/types.h>
|
||||
|
||||
#define RB_INT_INIT(l, h) \
|
||||
(struct rb_int_node){.low = l, .high = h, .max_high = h}
|
||||
(struct rb_int_node){.low = l, .high = h}
|
||||
#define rb_int(n) rb_entry(n, struct rb_int_node, node)
|
||||
|
||||
struct rb_int_node {
|
||||
struct rb_node node;
|
||||
u64 low;
|
||||
u64 high;
|
||||
|
||||
/* max_high will store the highest high of it's 2 children. */
|
||||
u64 max_high;
|
||||
};
|
||||
|
||||
/* Return the rb_int_node interval in which 'point' is located. */
|
||||
@@ -24,6 +21,10 @@ struct rb_int_node *rb_int_search_single(struct rb_root *root, u64 point);
|
||||
struct rb_int_node *rb_int_search_range(struct rb_root *root, u64 low, u64 high);
|
||||
|
||||
int rb_int_insert(struct rb_root *root, struct rb_int_node *data);
|
||||
void rb_int_erase(struct rb_root *root, struct rb_int_node *node);
|
||||
|
||||
static inline void rb_int_erase(struct rb_root *root, struct rb_int_node *node)
|
||||
{
|
||||
rb_erase(&node->node, root);
|
||||
}
|
||||
|
||||
#endif
|
||||
|
||||
+1
-57
@@ -35,57 +35,6 @@ struct rb_int_node *rb_int_search_range(struct rb_root *root, u64 low, u64 high)
|
||||
return range;
|
||||
}
|
||||
|
||||
/*
|
||||
* Update a node after it has been linked into the tree:
|
||||
*/
|
||||
static void propagate_callback(struct rb_node *node, struct rb_node *stop)
|
||||
{
|
||||
struct rb_int_node *i_node;
|
||||
|
||||
if (node == stop)
|
||||
return;
|
||||
|
||||
i_node = rb_int(node);
|
||||
i_node->max_high = i_node->high;
|
||||
|
||||
if (node->rb_left)
|
||||
i_node->max_high = max(i_node->max_high, rb_int(node->rb_left)->max_high);
|
||||
if (node->rb_right)
|
||||
i_node->max_high = max(i_node->max_high, rb_int(node->rb_right)->max_high);
|
||||
}
|
||||
|
||||
/*
|
||||
* Copy the extra data to a new node:
|
||||
*/
|
||||
static void copy_callback(struct rb_node *node_old, struct rb_node *node_new)
|
||||
{
|
||||
struct rb_int_node *i_node_old = rb_int(node_old);
|
||||
struct rb_int_node *i_node_new = rb_int(node_new);
|
||||
|
||||
i_node_new->low = i_node_old->low;
|
||||
i_node_new->high = i_node_old->high;
|
||||
|
||||
i_node_new->max_high = i_node_old->max_high;
|
||||
}
|
||||
|
||||
/*
|
||||
* Update after tree rotation:
|
||||
*/
|
||||
static void rotate_callback(struct rb_node *node_old, struct rb_node *node_new)
|
||||
{
|
||||
propagate_callback(node_old, NULL);
|
||||
propagate_callback(node_new, NULL);
|
||||
}
|
||||
|
||||
/*
|
||||
* All augmented rbtree callbacks:
|
||||
*/
|
||||
struct rb_augment_callbacks callbacks = {
|
||||
.propagate = propagate_callback,
|
||||
.copy = copy_callback,
|
||||
.rotate = rotate_callback,
|
||||
};
|
||||
|
||||
int rb_int_insert(struct rb_root *root, struct rb_int_node *i_node)
|
||||
{
|
||||
struct rb_node **node = &root->rb_node, *parent = NULL;
|
||||
@@ -103,12 +52,7 @@ int rb_int_insert(struct rb_root *root, struct rb_int_node *i_node)
|
||||
}
|
||||
|
||||
rb_link_node(&i_node->node, parent, node);
|
||||
rb_insert_augmented(&i_node->node, root, &callbacks);
|
||||
rb_insert_color(&i_node->node, root);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
void rb_int_erase(struct rb_root *root, struct rb_int_node *node)
|
||||
{
|
||||
rb_erase_augmented(&node->node, root, &callbacks);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user