From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: rust: workqueue: add delayed work support for ARef<T>
Date: Wed, 25 Mar 2026 07:11:04 +1000
Message-ID: <review-patch3-20260323-aref-workitem-v3-3-f59729b812aa@collabora.com>
In-Reply-To: <20260323-aref-workitem-v3-3-f59729b812aa@collabora.com>
References: <20260323-aref-workitem-v3-0-f59729b812aa@collabora.com> <20260323-aref-workitem-v3-3-f59729b812aa@collabora.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Patch Review

Very small patch =E2=80=94 just adds the `RawDelayedWorkItem` marker trait =
for `ARef<T>`:

```rust
unsafe impl<T, const ID: u64> RawDelayedWorkItem<ID> for ARef<T>
where
    T: WorkItem<ID, Pointer =3D Self>,
    T: HasDelayedWork<T, ID>,
    T: AlwaysRefCounted,
{
}
```

This mirrors the `Arc<T>` impl at line 870-875 of the existing workqueue.rs=
 exactly. The safety justification references `HasDelayedWork` which ensure=
s the `work_struct` is actually embedded in a `delayed_work`. Correct.

---
Generated by Claude Code Patch Reviewer