Commit e23ca00b authored by Christoph Lameter's avatar Christoph Lameter Committed by Linus Torvalds

[PATCH] Some page migration fixups

- Remove sparse comment

- Remove duplicated include

- Return the correct error condition in migrate_page_remove_references().
Signed-off-by: default avatarChristoph Lameter <clameter@sgi.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 1e624196
...@@ -16,8 +16,7 @@ ...@@ -16,8 +16,7 @@
#include <linux/module.h> #include <linux/module.h>
#include <linux/swap.h> #include <linux/swap.h>
#include <linux/pagemap.h> #include <linux/pagemap.h>
#include <linux/buffer_head.h> /* for try_to_release_page(), #include <linux/buffer_head.h>
buffer_heads_over_limit */
#include <linux/mm_inline.h> #include <linux/mm_inline.h>
#include <linux/pagevec.h> #include <linux/pagevec.h>
#include <linux/rmap.h> #include <linux/rmap.h>
...@@ -28,8 +27,6 @@ ...@@ -28,8 +27,6 @@
#include "internal.h" #include "internal.h"
#include "internal.h"
/* The maximum number of pages to take off the LRU for migration */ /* The maximum number of pages to take off the LRU for migration */
#define MIGRATE_CHUNK_SIZE 256 #define MIGRATE_CHUNK_SIZE 256
...@@ -234,7 +231,7 @@ int migrate_page_remove_references(struct page *newpage, ...@@ -234,7 +231,7 @@ int migrate_page_remove_references(struct page *newpage,
if (!page_mapping(page) || page_count(page) != nr_refs || if (!page_mapping(page) || page_count(page) != nr_refs ||
*radix_pointer != page) { *radix_pointer != page) {
write_unlock_irq(&mapping->tree_lock); write_unlock_irq(&mapping->tree_lock);
return 1; return -EAGAIN;
} }
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment