[v2,0/2] Reduce the space to be cleared for maple_big_node

Message

Wei Yang Sept. 8, 2024, 2:05 p.m. UTC

Found current code may clear maple_big_node redundantly.

First we define a filed parent, which is never used. After removing this, we
reduce the size of memory to be clear on memset.

Then mast_fill_bnode clear part of the structure twice, since slot and gap
shares some space. By clearing the whole structure, we can avoid it.

Wei Yang (2):
  maple_tree: remove maple_big_node.parent
  maple_tree: memset maple_big_node as a whole

 lib/maple_tree.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

Comments

Wei Yang Sept. 25, 2024, 2:45 a.m. UTC | #1

Hi, Andrew,

I don't see the notification on picking this in unstable/stable.

May I ask would it be picked up?

On Sun, Sep 08, 2024 at 02:05:52PM +0000, Wei Yang wrote:
>Found current code may clear maple_big_node redundantly.
>
>First we define a filed parent, which is never used. After removing this, we
>reduce the size of memory to be clear on memset.
>
>Then mast_fill_bnode clear part of the structure twice, since slot and gap
>shares some space. By clearing the whole structure, we can avoid it.
>
>Wei Yang (2):
>  maple_tree: remove maple_big_node.parent
>  maple_tree: memset maple_big_node as a whole
>
> lib/maple_tree.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
>-- 
>2.34.1