[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CALAqxLV7sB+gN1FNhhaiE9FXmaFv1TNsmKxD1Xctab6fTERA4w@mail.gmail.com>
Date: Mon, 4 Nov 2019 10:44:15 -0800
From: John Stultz <john.stultz@...aro.org>
To: Brian Starkey <Brian.Starkey@....com>
Cc: lkml <linux-kernel@...r.kernel.org>,
"Andrew F. Davis" <afd@...com>, Laura Abbott <labbott@...hat.com>,
Benjamin Gaignard <benjamin.gaignard@...aro.org>,
Sumit Semwal <sumit.semwal@...aro.org>,
Liam Mark <lmark@...eaurora.org>,
Pratik Patel <pratikp@...eaurora.org>,
Vincent Donnefort <Vincent.Donnefort@....com>,
Sudipto Paul <Sudipto.Paul@....com>,
Christoph Hellwig <hch@...radead.org>,
Chenbo Feng <fengc@...gle.com>,
Alistair Strachan <astrachan@...gle.com>,
Hridya Valsaraju <hridya@...gle.com>,
Sandeep Patil <sspatil@...gle.com>,
Hillf Danton <hdanton@...a.com>,
Dave Airlie <airlied@...il.com>,
"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
nd <nd@....com>
Subject: Re: [PATCH v14 1/5] dma-buf: Add dma-buf heaps framework
On Mon, Nov 4, 2019 at 2:24 AM Brian Starkey <Brian.Starkey@....com> wrote:
> On Fri, Nov 01, 2019 at 09:42:34PM +0000, John Stultz wrote:
> > From: "Andrew F. Davis" <afd@...com>
> >
> > Additionally should the interface grow in the future, we have a
> > DMA_HEAP_IOC_GET_FEATURES ioctl which can return future feature
> > flags.
>
> The userspace patch doesn't use this - and there's no indication of
> what it's intended to allow in the future. I missed the discussion
> about it, do you have a link?
Yea. Sorry, It was feedback I got via IRC. If your curious the irc log
can be found here:
https://people.freedesktop.org/~cbrill/dri-log/?channel=dri-devel&highlight_names=&date=2019-10-30&show_html=true
> The new zeroing stuff all looks good to me - so wrt. that you can add
> back my r-b; would still like to get some clarification on the
> get_features, though.
>From Daniel's feedback it sounds like I'll drop the empty get_features
ioctl then.
> Couple of typos below.
Apologies for my bad spelling! Thanks for the continued attention and review!
-john
Powered by blists - more mailing lists