Commit 6e7ba8b5 authored by Liam Howlett's avatar Liam Howlett Committed by Andrew Morton

maple_tree: mte_set_full() and mte_clear_full() clang-analyzer clean up

mte_set_full() and mte_clear_full() were incorrectly setting a pointer to
a value without returning a result.  Fix this by returning the modified
pointer to be use as necessary.  Also add a third function to return if
the bit is set or not.

Link: https://lore.kernel.org/lkml/20221026120029.12555-1-lukas.bulwahn@gmail.com/
Link: https://lkml.kernel.org/r/20221028144520.2776767-1-Liam.Howlett@oracle.comSigned-off-by: default avatarLiam R. Howlett <Liam.Howlett@oracle.com>
Suggested-by: default avatarLukas Bulwahn <lukas.bulwahn@gmail.com>
Suggested-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 57e9cc50
......@@ -323,14 +323,19 @@ static inline void *mte_safe_root(const struct maple_enode *node)
return (void *)((unsigned long)node & ~MAPLE_ROOT_NODE);
}
static inline void mte_set_full(const struct maple_enode *node)
static inline void *mte_set_full(const struct maple_enode *node)
{
node = (void *)((unsigned long)node & ~MAPLE_ENODE_NULL);
return (void *)((unsigned long)node & ~MAPLE_ENODE_NULL);
}
static inline void mte_clear_full(const struct maple_enode *node)
static inline void *mte_clear_full(const struct maple_enode *node)
{
node = (void *)((unsigned long)node | MAPLE_ENODE_NULL);
return (void *)((unsigned long)node | MAPLE_ENODE_NULL);
}
static inline bool mte_has_null(const struct maple_enode *node)
{
return (unsigned long)node & MAPLE_ENODE_NULL;
}
static inline bool ma_is_root(struct maple_node *node)
......
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