Messages in this thread | | | From | John Hubbard <> | Subject | [PATCH 0/2] mm: devmap: page-freeing related cleanups | Date | Thu, 14 Nov 2019 16:11:32 -0800 |
| |
Hi,
Here are the small devmap page-freeing cleanups as requested, so that they can be reviewed separately from the larger "track dma-pinned pages series" [1] that they were originally part of.
I did not add Jan Kara's and Jerome Glisse's reviewed-by tags, because those were for patch 2 but as part of [1], which is different enough from this series that I couldn't treat it as "close enough".
Testing: Dan notes that his patch (1) has passed nvdimm unit tests. However, I have not yet done much testing of the two patches together. What I've done so far is to boot up a system that has CONFIG_DEV_PAGEMAP_OPS=y and run LTP tests on it, but none of those tests are exercising free_devmap_managed_page().
Dan, if you could run this combined set through your unit tests, I'd feel a lot better about it. I'll also work on getting my HMM test setup revived.
[1] https://lore.kernel.org/r/20191113042710.3997854-1-jhubbard@nvidia.com
Dan Williams (1): mm: Cleanup __put_devmap_managed_page() vs ->page_free()
John Hubbard (1): mm: devmap: refactor 1-based refcounting for ZONE_DEVICE pages
drivers/nvdimm/pmem.c | 6 ---- include/linux/mm.h | 27 +++++++++++++-- mm/memremap.c | 76 ++++++++++++++++++++----------------------- 3 files changed, 60 insertions(+), 49 deletions(-)
-- 2.24.0
| |