On Fri, Oct 25, 2019 at 11:48:32PM +0000, John Stultz wrote: > Now that the DMA BUF heaps core code has been queued, I wanted > to send out some of the pending changes that I've been working > on. > > For use with Android and their GKI effort, it is desired that > DMA BUF heaps are able to be loaded as modules. This is required > for migrating vendors off of ION which was also recently changed > to support modules. > > So this patch series simply provides the necessary exported > symbols and allows the system and CMA drivers to be built > as modules. > > Due to the fact that dmabuf's allocated from a heap may > be in use for quite some time, there isn't a way to safely > unload the driver once it has been loaded. Thus these > drivers do no implement module_exit() functions and will > show up in lsmod as "[permanent]" > > Feedback and thoughts on this would be greatly appreciated!
Do we actually want this? I figured if we just state that vendors should set up all the right dma-buf heaps in dt, is that not enough? Exporting symbols for no real in-tree users feels fishy. -Daniel > > thanks > -john > > Cc: Laura Abbott <[email protected]> > Cc: Benjamin Gaignard <[email protected]> > Cc: Sumit Semwal <[email protected]> > Cc: Liam Mark <[email protected]> > Cc: Pratik Patel <[email protected]> > Cc: Brian Starkey <[email protected]> > Cc: Andrew F. Davis <[email protected]> > Cc: Andrew Morton <[email protected]> > Cc: Yue Hu <[email protected]> > Cc: Mike Rapoport <[email protected]> > Cc: Chenbo Feng <[email protected]> > Cc: Alistair Strachan <[email protected]> > Cc: Sandeep Patil <[email protected]> > Cc: Hridya Valsaraju <[email protected]> > Cc: [email protected] > > John Stultz (1): > dma-buf: heaps: Allow system & cma heaps to be configured as a modules > > Sandeep Patil (1): > mm: cma: Export cma symbols for cma heap as a module > > drivers/dma-buf/dma-heap.c | 2 ++ > drivers/dma-buf/heaps/Kconfig | 4 ++-- > drivers/dma-buf/heaps/heap-helpers.c | 2 ++ > kernel/dma/contiguous.c | 1 + > mm/cma.c | 5 +++++ > 5 files changed, 12 insertions(+), 2 deletions(-) > > -- > 2.17.1 > > _______________________________________________ > dri-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
