On 12/01/2015 11:45 AM, Dmitry Vyukov wrote: > On Tue, Dec 1, 2015 at 7:52 PM, Mike Kravetz <mike.kravetz@xxxxxxxxxx> wrote: >> On 12/01/2015 06:04 AM, Dmitry Vyukov wrote: >>> There seems to be another leak if nrg is not NULL on this path, but >>> it's not what happens in my case since the WARNING does not fire. >> >> If nrg is not NULL, then it was added to the resv map and 'should' be >> free'ed when the map is free'ed. This is not optimal, but I do not >> think it would lead to a leak. I'll take a close look at this code >> with an emphasis on the leak you discovered. > > > Hi Mike, > > Note that it's not just a leak report, it is an actual leak. You > should be able to reproduce it. > OK, finally found the bug which is in region_del(). It does not correctly handle a "placeholder" region descriptor that is left after an aborted operation when the start of the region to be deleted is for the same page. I will have a patch shortly, after some testing. -- Mike Kravetz -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>