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/qxl: Convert qxl release idr to xarray Date: Thu, 23 Apr 2026 08:54:43 +1000 Message-ID: In-Reply-To: <20260421060023.96618-1-liuqiangneo@163.com> References: <20260421060023.96618-1-liuqiangneo@163.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/qxl: Convert qxl release idr to xarray Author: liuqiangneo@163.com Patches: 1 Reviewed: 2026-04-23T08:54:43.877264 --- This is a single-patch series converting the QXL release IDR to xarray. The= conversion is mechanically correct and the v2 changelog shows the author a= ddressed prior review feedback (xa_limit_31b, GFP_KERNEL, u32 cast). The pa= tch is straightforward and reasonable =E2=80=94 xarray provides built-in lo= cking that simplifies the code by removing the external `release_idr_lock` = spinlock. There are two issues worth raising: one correctness bug with the seqno assi= gnment ordering, and one minor concern about the `qxl_release_from_id_locke= d` function doing unnecessary explicit locking. --- Generated by Claude Code Patch Reviewer