cleanup on do_pages_move()
Status from this message: [merged] mm-migratec-unify-not-queued-for-migration-handling-in-do_pages_move.patch removed from -mm tree
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v5 Latest4 patches 1 replyApplied 4/4
- v44 patches 0 repliesSuperseded
- v34 patches 2 repliesSuperseded
Earlier versions · 1
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v2 | 4 patches | 11 replies | Superseded |
Changes in v5 from the cover letter
: rebase on top of latest upstream, "linux/pipe_fs_i.h: fix kernel-doc
warnings after @wait was split"
Patches in v5 All 4 applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/4 | mm/migrate.c: no need to check for i > start in do_pages_move()mm/ | +3 | 0 replies | Applied |
| 2/4 | mm/migrate.c: wrap do_move_pages_to_node() and store_status()mm/ | +29 | 0 replies | Applied |
| 3/4 | mm/migrate.c: check pagelist in move_pages_and_store_status()mm/ | +3 | 0 replies | Applied |
| 4/4 | mm/migrate.c: unify "not queued for migration" handling in do_pages_move()mm/ | +6 | 0 replies | Applied |
Reviews trailers per patch
All 4 patches have a Reviewed-by or Acked-by.
| Patch | Michal Hocko | David Hildenbrand |
|---|---|---|
| 1/4 | A-b | R-b |
| 2/4 | A-b | R-b |
| 3/4 | A-b | R-b |
| 4/4 | A-b | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [Patch v5 0/4] cleanup on do_pages_move()
The logic in do_pages_move() is a little mess for audience to read and has some potential error on handling the return value. Especially there are three calls on do_move_pages_to_node() and store_status() with almost the same form.
This patch set tries to make the code a little friendly for audience by consolidate the calls.
v5:
: rebase on top of latest upstream, "linux/pipe_fs_i.h: fix kernel-doc
warnings after @wait was split"
v4:
* rephrase changelog based on suggestion from David Hildenbrand
* trivial change on code style and comment
v3:
* rebase on top of Yang Shi's fix "mm: move_pages: report the number of non-attempted pages"
v2:
* remove some unnecessary cleanup
Wei Yang (4):
mm/migrate.c: no need to check for i > start in do_pages_move() mm/migrate.c: wrap do_move_pages_to_node() and store_status() mm/migrate.c: check pagelist in move_pages_and_store_status() mm/migrate.c: unify "not queued for migration" handling in
do_pages_move()
mm/migrate.c | 88 +++++++++++++++++++++++----------------------------- 1 file changed, 39 insertions(+), 49 deletions(-)
Signature · 1 line
2.17.1