mm/memory: improve insert_pages() error handling
Status from this message: [merged mm-stable] mm-memory-return-enomem-for-page-table-allocation-failure-in-insert_pages.patch removed from -mm tree
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v3 Latest2 patches 2 repliesApplied 2/2
- v23 patches 4 repliesSuperseded
Changes in v3 from the cover letter
* drop the walk_to_pmd() rename following review
* use the mm/memory: prefix consistently across the series
* carry review tags from v2
* rebase onto current mm-new
Patches in v3 All 2 applied
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/2 | mm/memory: simplify error handling in insert_pages()mm/ | +11 | 1 reply | Applied |
| 2/2 | mm/memory: return -ENOMEM for page-table allocation failure in insert_pages()mm/ | +1 | 0 replies | Applied |
Reviews trailers per patch
All 2 patches have a Reviewed-by or Acked-by.
| Patch | David Hildenbrand (Arm) | Lorenzo Stoakes (ARM) |
|---|---|---|
| 1/2 | A-b | R-b |
| 2/2 | 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 v3 0/2] mm/memory: improve insert_pages() error handling
Improve insert_pages() error handling.
The first patch simplifies error handling by initializing the error status to zero and assigning error codes at their respective failure sites.
The second patch returns -ENOMEM when walk_to_pmd() fails. A NULL return from walk_to_pmd() indicates failure to allocate an upper page-table level, so -ENOMEM is more appropriate than -EFAULT and is consistent with the subsequent pte_alloc() failure.
Changes in v3:
* drop the walk_to_pmd() rename following review
* use the mm/memory: prefix consistently across the series
* carry review tags from v2
* rebase onto current mm-new
Avi Weiss (2):
mm/memory: simplify error handling in insert_pages() mm/memory: return -ENOMEM for page-table allocation failure in
insert_pages()
mm/memory.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-)
base-commit: d2aad7fdcda7ae8a726926f2d6de7fe9e8ee7563
Signature · 1 line
2.43.0