From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: ttm/pool: Use sentinels in debugfs Date: Tue, 28 Apr 2026 14:24:08 +1000 Message-ID: In-Reply-To: <20260427162253.682415-1-mario.limonciello@amd.com> References: <20260427162253.682415-1-mario.limonciello@amd.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Overall Series Review Subject: ttm/pool: Use sentinels in debugfs Author: Mario Limonciello Patches: 1 Reviewed: 2026-04-28T14:24:08.932872 --- This is a single-patch fix for the TTM pool debugfs output. The problem is straightforward: `ttm_pool_shrinker_count()` returns `SHRINK_EMPTY` (~0UL - 1) when there are no pages, and `ttm_pool_shrinker_scan()` returns `SHRINK_STOP` (~0UL) when nothing was freed. These are enormous unsigned long sentinel values that would be printed verbatim in the `page_pool_shrink` debugfs file, appearing as nonsensical huge numbers (e.g., 18446744073709551614/18446744073709551615) rather than meaningful zeros. The fix is correct and minimal. --- Generated by Claude Code Patch Reviewer