Re: [PATCH v2 3/4] dma-buf: add export symbol for dma-heap

From: Christoph Hellwig
Date: Wed Dec 02 2020 - 08:52:18 EST


On Tue, Dec 01, 2020 at 09:51:43AM -0800, Minchan Kim wrote:
> From: Hyesoo Yu <hyesoo.yu@xxxxxxxxxxx>
>
> The heaps could be added as module, so some functions should
> be exported to register dma-heaps. And dma-heap of module can use
> cma area to allocate and free. However the function related cma
> is not exported now. Let's export them for next patches.
>
> Signed-off-by: Hyesoo Yu <hyesoo.yu@xxxxxxxxxxx>
> Signed-off-by: Minchan Kim <minchan@xxxxxxxxxx>

NAK. I really don't think any module has business gaining that
kind of low-level access to the various symbols. I think in general
your heap can just be built-in and we should be fine.