On Wed, Dec 03, 2014 at 06:54:08PM +0530, Chintan Pandya wrote: > Hi Kim, Hello, Chintan. > > This is really useful stuff that you are doing. And the runtime > allocation for storing page owner stack is a good call. > > Along with that, we also use extended version of the original > page_owner patch. The extension is, to store stack trace at the time > of freeing the page. That will indeed eat up space like anything > (just double of original page_owner) but it helps in debugging some > crucial issues. Like, illegitimate free, finding leaked pages (if we Sound really interesting. I hope to see it. > store their time stamps) etc. The same has been useful in finding > double-free cases in drivers. But we have never got a chance to > upstream that. Now that these patches are being discussed again, do > you think it would be good idea to integrate in the same league of > patches ? Good to hear. I think that you can upstream it separately. If you send the patch, I will review it and help to merge it. Thanks. -- 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>