Re: [PATCH 1/3] When migrate_pages returns 0, all pages must havebeen released

[Christoph Lameter]

On Thu, 20 Jan 2011, Andrea Arcangeli wrote:

> Which following putback_lru_page()?  You mean
> putback_lru_page(newpage)? That is for the newly allocated page
> (allocated at the very top, so always needed), it's not relevant to
> the page_count(page) = 1. The page_count 1 is hold by the caller, so
> it's leaking memory right now (for everything but compaction).

Ahh yes we removed the putback_lru_pages call from migrate_pages()
and broke the existing release logic. The caller has to call
putback_release_pages() as per commit
cf608ac19c95804dc2df43b1f4f9e068aa9034ab

If that is still the case then we still have the double free.

Could we please document the calling conventions exactly in the source?
Right now it says that the caller should call putback_lru_pages().


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/