[RFC][PATCH 1/5 v2] dma-buf: Add dma-buf heaps framework

John Stultz john.stultz at linaro.org
Wed Mar 6 16:57:42 UTC 2019


On Wed, Mar 6, 2019 at 8:12 AM Benjamin Gaignard
<benjamin.gaignard at linaro.org> wrote:
> Le mar. 5 mars 2019 à 21:54, John Stultz <john.stultz at linaro.org> a écrit :
> > +/**
> > + * DOC: DMABUF Heaps Userspace API
> > + *
> > + */
> > +
> > +/* Currently no flags */
> > +#define DMA_HEAP_VALID_FLAGS (0)
>
> I think here you need to allow flags like O_RDWR or O_CLOEXEC else
> mmap will fail.
>

Hm. So I meant for these to be just the bitmask of valid flags for the
allocate IOCTL (used to make sure no one is passing junk flags
accidentally), rather then valid flags for the heap open call.  But
this probably suggests I should call it something like
DMA_HEAP_ALLOC_VALID_FLAGS instead?

thanks
-john


More information about the dri-devel mailing list