Re: [PATCH v3 00/11] `Zeroable` improvements & bindings integration

["Benno Lossin" <lossin@kernel.org>]

On Wed Aug 27, 2025 at 7:19 PM CEST, Alice Ryhl wrote:
> On Thu, Aug 14, 2025 at 11:31 AM Benno Lossin <lossin@kernel.org> wrote:
>>
>> This came from a discussion at [1]. And I added some more useful parts
>> to the original idea.
>>
>> I'm not sure on the impact of build times and rust-analyzer. We're
>> adding a derive macro to every struct and union emitted by bindgen.
>> Building using my test-config took 27.7s before and 28.2s after this
>> change, but those are only two runs on my machine with a very reduced
>> config (that enables all Rust code that we have at the moment).
>>
>> Maybe we have to reevaluate this when more C code is scanned by bindgen.
>
> Imo we should re-export Zeroable from the ffi crate and use it via
> kernel::ffi in most cases. I don't really want to see pin_init here:
>
> let mut ctx: bindings::lsm_context = pin_init::zeroed();

Yeah sounds sensible.

---
Cheers,
Benno