Hi Andrew,

Today's linux-next merge of the akpm tree got conflicts in mm/migrate.c
and include/linux/migrate.h between commits 070059b28578 ("mm/compaction:
Move migration fail/success stats to migrate.c") and 2d85cca46951
("mm/migrate: Add a tracepoint for migrate_pages") from the tip tree and
commit "mm: adjust address_space_operations.migratepage() return code"
from the akpm tree.

I fixed it up (see below) and can carry the fix as necessary (no action
is required).

-- 
Cheers,
Stephen Rothwell                    s...@canb.auug.org.au

diff --cc include/linux/migrate.h
index bcb5ddc,50d2974..0000000
--- a/include/linux/migrate.h
+++ b/include/linux/migrate.h
@@@ -7,16 -7,13 +7,23 @@@
  
  typedef struct page *new_page_t(struct page *, unsigned long private, int **);
  
 +enum migrate_reason {
 +      MR_COMPACTION,
 +      MR_MEMORY_FAILURE,
 +      MR_MEMORY_HOTPLUG,
 +      MR_SYSCALL,             /* also applies to cpusets */
 +      MR_MEMPOLICY_MBIND,
 +      MR_NUMA_MISPLACED,
 +      MR_CMA
 +};
 +
+ /*
+  * Return values from addresss_space_operations.migratepage():
+  * - negative errno on page migration failure;
+  * - zero on page migration success;
+  */
+ #define MIGRATEPAGE_SUCCESS           0
+ 
  #ifdef CONFIG_MIGRATION
  
  extern void putback_lru_pages(struct list_head *l);
diff --cc mm/migrate.c
index c0afe60,c3724b6..0000000
--- a/mm/migrate.c
+++ b/mm/migrate.c
@@@ -981,8 -977,7 +981,8 @@@ int migrate_pages(struct list_head *fro
                        case -EAGAIN:
                                retry++;
                                break;
-                       case 0:
+                       case MIGRATEPAGE_SUCCESS:
 +                              nr_succeeded++;
                                break;
                        default:
                                /* Permanent failure */
@@@ -991,14 -986,8 +991,14 @@@
                        }
                }
        }
-       rc = 0;
+       rc = nr_failed + retry;
  out:
 +      if (nr_succeeded)
 +              count_vm_events(PGMIGRATE_SUCCESS, nr_succeeded);
 +      if (nr_failed)
 +              count_vm_events(PGMIGRATE_FAIL, nr_failed);
 +      trace_mm_migrate_pages(nr_succeeded, nr_failed, mode, reason);
 +
        if (!swapwrite)
                current->flags &= ~PF_SWAPWRITE;
  

Attachment: pgpWXbZWjNC0w.pgp
Description: PGP signature

Reply via email to