Thanks for your review.
Our kernel driver has been upstreamed in the openeuler and anolis community, and the kernel community code is being prepared for upstreaming.
Then we will not submit this patch yet, and we will resubmit the v2 version of NBL PMD for Nebulamatrix NICs without this patch.
openeuler community:
https://gitee.com/openeuler/kernel/pulls/11667
anolis community:
https://gitee.com/anolis/cloud-kernel/pulls/5185
https://gitee.com/anolis/cloud-kernel/pulls/5059
------------------------------------------------------------------
> Hello, Stephen:
> Thanks for your review. This patch is particularly important for our NBL driver because our coexistence implementation depends on it.
> Unlike RDMA devices that support hardware page table translation, our hardware requires a software-managed approach to avoid IOVA conflicts since both user-space and kernel-space DMA operations share the same domain. This architectural constraint makes the patch essential for:
> 1. Maintaining correct memory isolation
> 2. Preventing IOVA collisions in shared DMA operations
> 3. Enabling safe driver coexistence
> We'd be happy to:
> 1. Provide additional technical details if needed
> 2. Make any necessary adjustments based on your feedback
> Anyway, we really hope this patch could be accepted, Thanks again
The approach is sound just want more feedback from the developers.
Other drivers may have similar things to worry about.
If the driver is upstream then it makes it easier to test and review.