diff options
| author | Minchan Kim <minchan.kim@gmail.com> | 2011-01-25 15:07:26 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-01-26 10:50:02 +1000 |
| commit | 28bd65781c848d95ba6a7f58b5c4b8265a804ec6 (patch) | |
| tree | 158a70f9b08644e1c3ca958dc376a853759e3745 /mm/migrate.c | |
| parent | 33a938774fdb9933e9c77504b035f4f87c0859df (diff) | |
| download | lwn-28bd65781c848d95ba6a7f58b5c4b8265a804ec6.tar.gz lwn-28bd65781c848d95ba6a7f58b5c4b8265a804ec6.zip | |
mm: migration: clarify migrate_pages() comment
Callers of migrate_pages should putback_lru_pages to return pages
isolated to LRU or free list. Now comment is rather confusing. It says
caller always have to call it.
It is more clear to point out that the caller has to call it if
migrate_pages's return value isn't zero.
Signed-off-by: Minchan Kim <minchan.kim@gmail.com>
Cc: Christoph Lameter <cl@linux.com>
Cc: Andrea Arcangeli <aarcange@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/migrate.c')
| -rw-r--r-- | mm/migrate.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/migrate.c b/mm/migrate.c index 46fe8cc13d67..9f29a3b7aac2 100644 --- a/mm/migrate.c +++ b/mm/migrate.c @@ -888,7 +888,7 @@ out: * are movable anymore because to has become empty * or no retryable pages exist anymore. * Caller should call putback_lru_pages to return pages to the LRU - * or free list. + * or free list only if ret != 0. * * Return: Number of pages not migrated or error code. */ |