From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: Let userspace know about swapped out panthor GEM objects Date: Mon, 25 May 2026 20:03:23 +1000 Message-ID: In-Reply-To: <20260521-panthor-bo-reclaim-observability-v5-0-49313994da55@collabora.com> References: <20260521-panthor-bo-reclaim-observability-v5-0-49313994da55@collabora.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: Let userspace know about swapped out panthor GEM objects Author: Nicolas Frattaroli Patches: 4 Reviewed: 2026-05-25T20:03:23.030343 --- This is a clean, small 2-patch series (v5) that adds eviction observability= to panthor's GEM debugfs. The changes are well-scoped: patch 1 adds a `rec= laimed_count` atomic to track evictions with proper saturation, exposes it = via debugfs, and adds an "evicted" state flag; patch 2 tightens column widt= hs. Both patches carry Reviewed-by tags from Steven Price and Boris Brezill= on. The implementation is straightforward and correct. The use of `atomic_add_u= nless` for saturation at INT_MAX is appropriate. The ordering of the patche= s is slightly unusual =E2=80=94 the column width reduction in patch 2 is ap= plied to widths that were already set in patch 1 =E2=80=94 but this is a mi= nor aesthetic issue in sequencing, not a correctness problem. The cover let= ter mentions dropping an fdinfo patch from v4 to unblock the series, and th= e commit message for patch 1 still references fdinfo and `DRM_GEM_OBJECT_EV= ICTED` which no longer appears in the diff; this is a minor commit message = hygiene issue. **No blocking issues found.** This series looks ready to merge. --- Generated by Claude Code Patch Reviewer