Messages in this thread Patch in this message |  | | From | Melih Emik <> | | Subject | [PATCH] rust: dma: drop placeholder TODO comment | | Date | Wed, 6 May 2026 20:46:37 +0300 |
| |
The comment above Coherent already describes the outstanding design
considerations around device resource lifetime.
Drop the standalone placeholder TODO marker and keep the explanatory
text, including a minor wording fix.
No functional change intended.
Signed-off-by: Melih Emik <melihemik@noirlang.tr> --- rust/kernel/dma.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/rust/kernel/dma.rs b/rust/kernel/dma.rs index 4995ee5dc689..60c458adba0c 100644 --- a/rust/kernel/dma.rs +++ b/rust/kernel/dma.rs @@ -580,9 +580,7 @@ fn from(value: CoherentBox<T>) -> Self { /// to an allocated region of coherent memory and `dma_handle` is the DMA address base of the /// region. /// - The size in bytes of the allocation is equal to size information via pointer. -// TODO -// -// DMA allocations potentially carry device resources (e.g.IOMMU mappings), hence for soundness +// DMA allocations potentially carry device resources (e.g. IOMMU mappings), hence for soundness // reasons DMA allocation would need to be embedded in a `Devres` container, in order to ensure // that device resources can never survive device unbind. // -- 2.54.0
|  |