On Mon, Nov 4, 2019 at 11:34 AM John Stultz <john.stu...@linaro.org> wrote:
>
> On Sun, Nov 3, 2019 at 8:13 AM <sspa...@google.com> wrote:
> > On Fri, Nov 01, 2019 at 09:42:35PM +0000, John Stultz wrote:
> > > Add generic helper dmabuf ops for dma heaps, so we can reduce
> > > the amount of duplicative code for the exported dmabufs.
> > >
> > > This code is an evolution of the Android ION implementation, so
> > > thanks to its original authors and maintainters:
> > > Rebecca Schultz Zavin, Colin Cross, Laura Abbott, and others!
> > >
> > > Cc: Laura Abbott <labb...@redhat.com>
> > > Cc: Benjamin Gaignard <benjamin.gaign...@linaro.org>
> > > Cc: Sumit Semwal <sumit.sem...@linaro.org>
> > > Cc: Liam Mark <lm...@codeaurora.org>
> > > Cc: Pratik Patel <prat...@codeaurora.org>
> > > Cc: Brian Starkey <brian.star...@arm.com>
> > > Cc: Vincent Donnefort <vincent.donnef...@arm.com>
> > > Cc: Sudipto Paul <sudipto.p...@arm.com>
> > > Cc: Andrew F. Davis <a...@ti.com>
> > > Cc: Christoph Hellwig <h...@infradead.org>
> > > Cc: Chenbo Feng <fe...@google.com>
> > > Cc: Alistair Strachan <astrac...@google.com>
> > > Cc: Hridya Valsaraju <hri...@google.com>
> > > Cc: Sandeep Patil <sspa...@google.com>
> > > Cc: Hillf Danton <hdan...@sina.com>
> > > Cc: Dave Airlie <airl...@gmail.com>
> > > Cc: dri-devel@lists.freedesktop.org
> > > Reviewed-by: Benjamin Gaignard <benjamin.gaign...@linaro.org>
> > > Reviewed-by: Brian Starkey <brian.star...@arm.com>
> > > Acked-by: Laura Abbott <labb...@redhat.com>
> > > Tested-by: Ayan Kumar Halder <ayan.hal...@arm.com>
> > > Signed-off-by: John Stultz <john.stu...@linaro.org>
> >
> > I have one question and a naming suggestin below (sorry).
> >
> > Acked-by: Sandeep Patil <sspa...@android.com>
>
> > > +
> > > +static void dma_heap_buffer_vmap_put(struct heap_helper_buffer *buffer)
> > > +{
> > > + if (!--buffer->vmap_cnt) {
> >
> > nit: just checking here cause I don't know the order in which vmap_get() and
> > vmap_put() is expected to be called from dmabuf, the manual refcounting
> > based map/unmap is ok?
> >
> > I know ion had this for a while and it worked fine over the years, but I
> > don't know if anybody saw any issues with it.
Sorry, I hit send before replying to this bit. I'll double check on this.
thanks again for the review!
-john
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
