Re: [PATCH v5 2/2] rust: uaccess: refactor to use `overflow_assert!`

[Miguel Ojeda]

On Tue, Aug 19, 2025 at 2:39 AM Antonio Hickey
<contact@xxxxxxxxxxxxxxxxx> wrote:
>
> Using the `overflow_assert!` macro here adds documentation to
> the intent of the assertion, and avoids local `#ifdefs`s by
> encapsulating the conditional behavior to the macro itself.
>
> Co-developed-by: Daniel Cote <danielstonecote@xxxxxxxxx>
> Signed-off-by: Daniel Cote <danielstonecote@xxxxxxxxx>
> Signed-off-by: Antonio Hickey <contact@xxxxxxxxxxxxxxxxx>
> Link: https://github.com/Rust-for-Linux/linux/issues/1159
> Suggested-by: Miguel Ojeda <ojeda@xxxxxxxxxx>

You didn't pick the review tags you were given in v4 -- was there a
reason for that? (e.g. one usual reason is that the patch changed a
lot, but I don't think that is the case here?)

Cheers,
Miguel