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: property: use kzalloc_flex Date: Fri, 13 Mar 2026 13:43:38 +1000 Message-ID: In-Reply-To: <20260313000620.117065-1-rosenp@gmail.com> References: <20260313000620.117065-1-rosenp@gmail.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: drm: property: use kzalloc_flex Author: Rosen Penev Patches: 1 Reviewed: 2026-03-13T13:43:38.093496 --- This is a single-patch series that converts `struct drm_property` to use a = flexible array member for `values[]` instead of a separately allocated poin= ter, using the `kzalloc_flex` helper. The conversion is straightforward and= correct. The `values` pointer was never reassigned after initial allocatio= n, `struct drm_property` is always heap-allocated via pointers, and all exi= sting accesses to `property->values[i]` work identically with a flexible ar= ray member. The patch is **clean and correct**. It simplifies allocation (removing one = `kcalloc` and its error handling), simplifies deallocation (removing the se= parate `kfree(property->values)`), and adds `__counted_by` for runtime boun= ds checking =E2=80=94 all good improvements. **Recommendation: Accept / Reviewed-by worthy** --- Generated by Claude Code Patch Reviewer