On Mon, Apr 21, 2025 at 1:40 PM T.J. Mercier tjmercier@google.com wrote:
new file mode 100644 index 000000000000..b4b8be1d6aa4 --- /dev/null +++ b/kernel/bpf/dmabuf_iter.c
Maybe we should add this file to drivers/dma-buf. I would like to hear other folks thoughts on this.
This is fine with me, and would save us the extra CONFIG_DMA_SHARED_BUFFER check that's currently needed in kernel/bpf/Makefile but would require checking CONFIG_BPF instead. Sumit / Christian any objections to moving the dmabuf bpf iterator implementation into drivers/dma-buf?
The driver directory would need to 'depends on BPF_SYSCALL'. Are you sure you want this? imo kernel/bpf/ is fine for this.
You also probably want .feature = BPF_ITER_RESCHED in bpf_dmabuf_reg_info.
Also have you considered open coded iterator for dmabufs? Would it help with the interface to user space?