Claude review: Re: [PATCH v2] cgroup/dmem: implement dmem.high soft limit via prioritized eviction

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: Re: [PATCH v2] cgroup/dmem: implement dmem.high soft limit via prioritized eviction
Author: Natalie Vock <natalie.vock@gmx.de>
Patches: 2
Reviewed: 2026-05-25T19:12:32.597474

---

This is a single-patch series (v2) adding a `dmem.high` soft limit to the dmem cgroup v2 controller. When device memory eviction is triggered (e.g., by hitting the hard `max` limit), buffer objects from cgroups exceeding their `high` limit are evicted first via a new prioritized pass in `ttm_bo_evict_alloc()`.

**v2 improves significantly over v1** by replacing sleep-on-allocation throttling (which would be catastrophic for GPU submission) with prioritized eviction. The approach reuses existing `page_counter` infrastructure (`page_counter.high`, `page_counter_set_high()`) and follows the cgroup v2 naming convention (min < low < high < max).

However, there are design concerns around protection bypass, missing documentation/selftests, and a semantic inconsistency with how the memory cgroup controller handles `high` vs protection guarantees.

---

---
Generated by Claude Code Patch Reviewer