From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/dumb-buffer: Drop buffer-size limits for now Date: Thu, 04 Jun 2026 12:51:35 +1000 Message-ID: In-Reply-To: <20260602112842.252279-1-tzimmermann@suse.de> References: <20260602112842.252279-1-tzimmermann@suse.de> 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: drm/dumb-buffer: Drop buffer-size limits for now Author: Thomas Zimmermann Patches: 3 Reviewed: 2026-06-04T12:51:35.907861 --- This is a single patch that removes overly restrictive hard-coded size limi= ts (8192x8192 @ 32bpp) from `drm_mode_create_dumb()`, added by the recent c= ommit 5ab62dd3687b. The limits broke CI tests and were too aggressive =E2= =80=94 they'd reject legitimate use cases like 8K displays (7680x4320) and = 64bpp formats (`XRGB16161616F`) that are explicitly supported elsewhere in = the same file (line 176 of the current tree). The removal is correct: the r= emaining arithmetic overflow checks in `drm_mode_create_dumb()` are mathema= tically sound and sufficient to prevent the integer overflows the original = commit was targeting. **Verdict: The patch is good and should be applied.** Minor nits on tag ord= ering below. --- --- Generated by Claude Code Patch Reviewer