Hi Mikulas, This is V5 for dm-pcache, please consider merging. As you requested, I've squashed these patches into a single patch, based on v6.16. If you find anything that needs tweaking while merging, feel free to modify it directly. I can also send a rebased version on top of the linux-dm branch if you prefer. This patch has gone through quite a bit of testing on my side. I can share the highlights. Once you merge this patch into the linux-dm branch, I’ll keep running ongoing tests against dm-6.18. 1. dtg-tests for pcache[1]: - Includes `dmsetup` management operations, including failure/exception scenarios. - Failslab tests (covering the NOWAIT → NOIO fallback path you mentioned). - `fail_make_request` fault-injection tests. - Backing device delay tests. - xfstests (group `generic/rw`) under different parameters. 2. Striped pmem as the cache_dev[2]: ran the same dtg-tests suite with a striped pmem configuration. 3. KASAN and memleak modes[3]: all of the above tests were also executed with KASAN and memory leak detection enabled. 4. Coverage with gcov: I used gcov to gather code coverage during the test runs[4]. - The results show 100% function coverage and 90%+ line coverage. The remaining uncovered code is mostly `BUG()` lines and a few hard-to-simulate error paths. - In the coverage report you can see the code path for `mempool_alloc` with `GFP_NOIO` is exercised and covered.
[1]: https://datatravelguide.github.io/dtg-blog/pcache/pcache_v5_result_virtio/results.html [2]: https://datatravelguide.github.io/dtg-blog/pcache/pcache_v5_result_ram/results.html [3]: https://datatravelguide.github.io/dtg-blog/pcache/pcache_v5_result_kasan/results.html [4]: https://datatravelguide.github.io/dtg-blog/pcache/pcache_cov_v5/dm-pcache/index.html Changelogs V5 from V4: - Fix `get_n_vecs()` bug when the buffer isn’t page-aligned; use - `bio_add_max_vecs()` instead. - Use `kvcalloc()` for allocations larger than 32,768 bytes. - For mempool allocations, try `GFP_NOWAIT` first; if that fails, fall back to `GFP_NOIO`. V4 from V3: - go back to use mempool for allocation of cache_key and backing_dev_req. - introduce backing_bvec_cache and backing_dev->bvec_pool to allocate bvecs for backing_dev_req. Currently, the obj_size of bvec_cache is max size of bvecs. The bvecs are allocated only for write‑back backing_dev_req requests, and only a limited number of writeback requests are allocated at the same time, so the resulting memory overhead should not be too severe. If in the futer we found that this implementation results in unacceptable memory waste, we can optimize it by switching to a multi‑level slab design. - remove ret-value checking for bio_init_clone() as there should not be REQ_INTEGRITY in bio bi_opf, so bio_init_clone() never fail. - remove ret-value checking for backing_dev_req_alloc() and cache_key_alloc(). V3 from V2: - rebased against linux-dm dm-6.17 - add missing include file bitfiled.h (Mikulas) - move kmem_cache from per-device to per-module (Mikulas) - fix a memleak in error path found by failslab testing - retry pcache_request in defer_req() when memory allocation fail. V2 from V1: - introduce req_alloc() and req_init() in backing_dev.c, then we can do req_alloc() before holding spinlock and do req_init() in subtree_walk(). - introduce pre_alloc_key and pre_alloc_req in walk_ctx, that means we can pre-allocate cache_key or backing_dev_request before subtree walking. - use mempool_alloc() with NOIO for the allocation of cache_key and backing_dev_req. - some coding style changes from comments of Jonathan. V1 from RFC-V2: - use crc32c to replace crc32 - only retry pcache_req when cache full, add pcache_req into defer_list, and wait cache invalidation happen. - new format for pcache table, it is more easily extended with new parameters later. - remove __packed. - use spin_lock_irq in req_complete_fn to replace spin_lock_irqsave. - fix bug in backing_dev_bio_end with spin_lock_irqsave. - queue_work() inside spinlock. - introduce inline_bvecs in backing_dev_req. - use kmalloc_array for bvecs allocation. - calculate ->off with dm_target_offset() before use it. Dongsheng Yang (1): dm-pcache: add persistent cache target in device-mapper .../admin-guide/device-mapper/dm-pcache.rst | 201 ++++ MAINTAINERS | 8 + drivers/md/Kconfig | 2 + drivers/md/Makefile | 1 + drivers/md/dm-pcache/Kconfig | 17 + drivers/md/dm-pcache/Makefile | 3 + drivers/md/dm-pcache/backing_dev.c | 374 ++++++++ drivers/md/dm-pcache/backing_dev.h | 100 ++ drivers/md/dm-pcache/cache.c | 445 +++++++++ drivers/md/dm-pcache/cache.h | 632 +++++++++++++ drivers/md/dm-pcache/cache_dev.c | 299 ++++++ drivers/md/dm-pcache/cache_dev.h | 70 ++ drivers/md/dm-pcache/cache_gc.c | 170 ++++ drivers/md/dm-pcache/cache_key.c | 888 ++++++++++++++++++ drivers/md/dm-pcache/cache_req.c | 835 ++++++++++++++++ drivers/md/dm-pcache/cache_segment.c | 293 ++++++ drivers/md/dm-pcache/cache_writeback.c | 258 +++++ drivers/md/dm-pcache/dm_pcache.c | 497 ++++++++++ drivers/md/dm-pcache/dm_pcache.h | 67 ++ drivers/md/dm-pcache/pcache_internal.h | 117 +++ drivers/md/dm-pcache/segment.c | 61 ++ drivers/md/dm-pcache/segment.h | 74 ++ 22 files changed, 5412 insertions(+) create mode 100644 Documentation/admin-guide/device-mapper/dm-pcache.rst create mode 100644 drivers/md/dm-pcache/Kconfig create mode 100644 drivers/md/dm-pcache/Makefile create mode 100644 drivers/md/dm-pcache/backing_dev.c create mode 100644 drivers/md/dm-pcache/backing_dev.h create mode 100644 drivers/md/dm-pcache/cache.c create mode 100644 drivers/md/dm-pcache/cache.h create mode 100644 drivers/md/dm-pcache/cache_dev.c create mode 100644 drivers/md/dm-pcache/cache_dev.h create mode 100644 drivers/md/dm-pcache/cache_gc.c create mode 100644 drivers/md/dm-pcache/cache_key.c create mode 100644 drivers/md/dm-pcache/cache_req.c create mode 100644 drivers/md/dm-pcache/cache_segment.c create mode 100644 drivers/md/dm-pcache/cache_writeback.c create mode 100644 drivers/md/dm-pcache/dm_pcache.c create mode 100644 drivers/md/dm-pcache/dm_pcache.h create mode 100644 drivers/md/dm-pcache/pcache_internal.h create mode 100644 drivers/md/dm-pcache/segment.c create mode 100644 drivers/md/dm-pcache/segment.h -- 2.43.0