On 23.04.23 08:42, Richard Guo wrote: > Thanks for the suggestion. I've split the patch into two as attached. > 0001 is just a minor simplification by replacing lfirst(list_head(list)) > with linitial(list). 0002 introduces new functions to reduce the > movement of list elements in several places so as to gain performance > improvement and benefit future callers.
These look sensible to me. If you could show some numbers that support the claim that there is a performance advantage, it would be even more convincing.
Thanks Peter for looking at those patches. I tried to devise a query to show performance gain but did not succeed :-(. So I begin to wonder if 0002 is worthwhile to do, as it seems that it does not solve any real problem.