| From: |
| Abdiel Janulgue <abdiel.janulgue-AT-gmail.com> |
| To: |
| rust-for-linux-AT-vger.kernel.org, daniel.almeida-AT-collabora.com, dakr-AT-kernel.org, robin.murphy-AT-arm.com, aliceryhl-AT-google.com |
| Subject: |
| [PATCH v10 0/3] Add dma coherent allocator abstraction |
| Date: |
| Tue, 21 Jan 2025 21:14:01 +0200 |
| Message-ID: |
| <20250121191432.1178734-1-abdiel.janulgue@gmail.com> |
| Cc: |
| Miguel Ojeda <ojeda-AT-kernel.org>, Alex Gaynor <alex.gaynor-AT-gmail.com>, Boqun Feng <boqun.feng-AT-gmail.com>, Gary Guo <gary-AT-garyguo.net>, Björn Roy Baron <bjorn3_gh-AT-protonmail.com>, Benno Lossin <benno.lossin-AT-proton.me>, Andreas Hindborg <a.hindborg-AT-kernel.org>, Trevor Gross <tmgross-AT-umich.edu>, Valentin Obst <kernel-AT-valentinobst.de>, linux-kernel-AT-vger.kernel.org (open list), Christoph Hellwig <hch-AT-lst.de>, Marek Szyprowski <m.szyprowski-AT-samsung.com>, airlied-AT-redhat.com, iommu-AT-lists.linux.dev (open list:DMA MAPPING HELPERS), Abdiel Janulgue <abdiel.janulgue-AT-gmail.com> |
| Archive-link: |
| Article |
Changes since v9:
- Use ARef<Device> in the constructor arguments, docs clarification avoid
manually dropping the refcount for the device in into_parts(), use
add() instead of wrapping_add() in the pointer arithmetic for performance
(Alice Ryhl).
- Link to v9: https://lore.kernel.org/all/20250121084756.1051758-1-abdi...
Changes since v8:
- Add MAINTAINERS entry
- Fix build issues due to switch from core::ffi to crate:ffi in bindgen.
- Ensure the wrapped attribute is non-pub in struct Attrs, declare it
#[repr(transparent)] as well (Daniel Sedlak)
Changes since v7:
- Remove cpu_buf() and cpu_buf_mut() as exporting a r/w interface via
a slice is undefined behaviour due to slice's requirement that the
underlying pointer should not be modified (Alice Ryhl, Robin Murphy).
- Reintroduce r/w helpers instead which includes proper safety
invariants (Daniel Almeida).
Changes since v6:
- Include the dma_attrs in the constructor, use alloc::Flags as inpiration
Changes since v5:
- Remove unnecessary lifetime annotation when returning the CPU buffer.
Changes since v4:
- Documentation and example fixes, use Markdown formatting (Miguel Ojeda).
- Discard read()/write() helpers to remove bound on Copy and fix overhead
(Daniel Almeida).
- Improve error-handling in the constructor block (Andreas Hindborg).
Changes since v3:
- Reject ZST types by checking the type size in the constructor in
addition to requiring FromBytes/AsBytes traits for the type (Alice Ryhl).
Changes since v2:
- Fixed missing header for generating the bindings.
Changes since v1:
- Fix missing info in commit log where EOVERFLOW is used.
- Restrict the dma coherent allocator to numeric types for now for valid
behaviour (Daniel Almeida).
- Build slice dynamically.
Abdiel Janulgue (3):
rust: error: Add EOVERFLOW
rust: add dma coherent allocator abstraction.
MAINTAINERS: add entry for Rust dma mapping helpers device driver API
MAINTAINERS | 10 ++
rust/bindings/bindings_helper.h | 1 +
rust/kernel/dma.rs | 281 ++++++++++++++++++++++++++++++++
rust/kernel/error.rs | 1 +
rust/kernel/lib.rs | 1 +
5 files changed, 294 insertions(+)
create mode 100644 rust/kernel/dma.rs
base-commit: ceff0757f5dafb5be5205988171809c877b1d3e3
--
2.43.0
