Re: [PATCH v11 2/3] rust: add dma coherent allocator abstraction.

Abdiel Janulgue
2025-02-18T09:58:37Z

Hi Robin,

On 17/02/2025 15:52, Robin Murphy wrote:
> FWIW we've been wanting to steer away from relying on the default mask
> in new code, so it would be quite neat to actually enforce that
> allocations fail if dma_coherent_mask hasn't been explicitly set
> (assuming it's sufficiently cheap to keep a flag in the Device handle or
> something like that - it's not the end of the world if it isn't practical).

I just had a quick look on how to possible approach this and indeed
would refactor the Device binding a bit. If it is okay with you this
could go in a follow-up patch? I was hoping to upstream the initial
support first - at least with the dma_set_mask/dma_set_coherent_mask put
in place already.

Thanks!
/Abdiel