summaryrefslogtreecommitdiffstats
path: root/mm/migrate.c
diff options
context:
space:
mode:
authorMinchan Kim <minchan.kim@gmail.com>2011-01-25 15:07:26 -0800
committerLinus Torvalds <torvalds@linux-foundation.org>2011-01-26 10:50:02 +1000
commit28bd65781c848d95ba6a7f58b5c4b8265a804ec6 (patch)
tree158a70f9b08644e1c3ca958dc376a853759e3745 /mm/migrate.c
parent33a938774fdb9933e9c77504b035f4f87c0859df (diff)
downloadop-kernel-dev-28bd65781c848d95ba6a7f58b5c4b8265a804ec6.zip
op-kernel-dev-28bd65781c848d95ba6a7f58b5c4b8265a804ec6.tar.gz
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.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/migrate.c b/mm/migrate.c
index 46fe8cc..9f29a3b 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.
*/
OpenPOWER on IntegriCloud