Re: [PATCH v5 15/20] rust: io: implement a view type for `Coherent`
"Gary Guo" <[email protected]>
| Newsgroups | org.kernel.vger.linux-pwm,dev.linux.lists.driver-core,dev.linux.lists.nova-gpu,org.freedesktop.lists.dri-devel,org.kernel.vger.linux-kernel,org.kernel.vger.linux-pci,org.kernel.vger.rust-for-linux |
|---|---|
| Message-ID | <[email protected]> |
On Fri Jul 3, 2026 at 2:14 PM BST, Alexandre Courbot wrote: > On Fri Jun 26, 2026 at 11:45 PM JST, Gary Guo wrote: >> Implement a `CoherentView` type which is a view of `Coherent`. To be able >> to give out DMA handles, the view type contains both CPU and DMA pointers, >> and the projection method projects both at once. >> >> Delegate most of the `Io` implementation to `SysMemBackend`. Provide a >> method to erase the DMA handle and give out a `SysMem` view, if the user >> does not need the `dma_handle`. >> >> Signed-off-by: Gary Guo <[email protected]> > > Of the Sashiko comments, the second one (adding exclusivity requirement > to the safety comment) looks actionable; with that: Most of the text is taken from existing `as_mut` method on the full `Coherent`. I think these can be addressed, but I'd prefer to leave it to a further patch (perhaps as a good first issue). Best, Gary > > Reviewed-by: Alexandre Courbot <[email protected]> > > (one nit below) >