Btw there is the easy ugly fix obviously, which would be this patch +
locking this function with the tree spin-lock(to avoid any race).
I'll check however if there is hope for a better possible design for
this function.
On 12/12/2022 16:00, Jason Gunthorpe wrote:
On Mon, Dec 12, 2022 at 03:55:37PM +0200, Patrisious Haddad wrote:
Agree, but changing the control flow of this function is really problematic
, it was even tried before if you remember commit "e4103312d7b7a" , it got
something to do with port allocation, I'll take another look over the code
to see what other options we have though.
Yes, we've changed this function many times because it is badly
mis-designed
Jason