Re: [PATCH] maple_tree: Make maple state reusable after mas_empty_area()

From: Michael Keyes
Date: Thu May 04 2023 - 17:59:40 EST


On 04.05.23 20:10, Edgecombe, Rick P wrote:
> On Thu, 2023-05-04 at 13:55 -0400, Liam R. Howlett wrote:
>> Do not update the min and max of the maple state to the slot of the
>> leaf
>> node.  Leaving the min and max to the node entry allows for the maple
>> state to be used in other operations.
>>
>> Users would get unexpected results from other operations on the maple
>> state after calling the affected function.
>>
>> Reported-by: "Edgecombe, Rick P" <rick.p.edgecombe@xxxxxxxxx>
>> Reported-by: Tad <support@xxxxxxxxx>
>> Reported-by: Michael Keyes <mgkeyes@xxxxxxxxxxxxxxxxxxxx>
>> Link:
>> https://lore.kernel.org/linux-mm/32f156ba80010fd97dbaf0a0cdfc84366608624d.camel@xxxxxxxxx/
>> Link:
>> https://lore.kernel.org/linux-mm/e6108286ac025c268964a7ead3aab9899f9bc6e9.camel@xxxxxxxxx/
>> Fixes: Fixes: 54a611b60590 ("Maple Tree: add new data structure")
>> Cc: <Stable@xxxxxxxxxxxxxxx>
>> Signed-off-by: Liam R. Howlett <Liam.Howlett@xxxxxxxxxx>
>> ---
> This fixes it for all the cases I encountered, thanks!
Me too. I had issues running an old version of LuaJIT, and it seems to
be running better than it had in a long time now! Thank you!

--
Michael