On 9/11/20 2:37 PM, Florian Fainelli wrote:
I am by no means an authoritative CMA person but this behavior does not seem acceptable, there is no doubt the existing one is sub-optimal under specific circumstances, but an indefinite retry, as well as a 100ms sleep appear to be arbitrary at best. How about you introduce a parameter that allows the tuning of the number of retries and/or delay between retries?
Also:
You should send your patch to linux-mm@xxxxxxxxx -- that's where
most memory management type patches are reviewed.
You should also send your patch to someone who could actually merge
it into the kernel source tree -- assuming that's what you want to happen.
Try scripts/get_mainttainer.pl on your patch to see what it says.
And if you are going to use a "cover letter" or "introductory email" before
the actual patch, the second (patch(es)) should be sent chained to the
first email. git send-email should do this for you.