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: alloc: remove `'static` bound on `ForeignOwnable`
Date: Tue, 26 May 2026 06:47:02 +1000
Message-ID: <review-patch2-20260525202921.124698-3-dakr@kernel.org>
In-Reply-To: <20260525202921.124698-3-dakr@kernel.org>
References: <20260525202921.124698-1-dakr@kernel.org> <20260525202921.124698-3-dakr@kernel.org>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

Removes `T: 'static` from `Box<T, A>` and `Pin<Box<T, A>>` `ForeignOwnable` impls, adding `Self: 'a` bounds on the GATs instead:

```rust
-unsafe impl<T: 'static, A> ForeignOwnable for Box<T, A>
+unsafe impl<T, A> ForeignOwnable for Box<T, A>
...
+    type Borrowed<'a> = &'a T where Self: 'a;
+    type BorrowedMut<'a> = &'a mut T where Self: 'a;
```

This is a well-known pattern for relaxing GAT bounds. The `Self: 'a` bound ensures that the borrowed types don't outlive the owning type, which replaces the `'static` requirement. Correct and necessary for the rest of the series.

---
Generated by Claude Code Patch Reviewer