From: Daniel Vetter <daniel@ffwll.ch> To: "Gustavo Padovan" <gustavo@padovan.org>, greg@kroah.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, daniels@collabora.com, "Gustavo Padovan" <gustavo.padovan@collabora.co.uk>, "Arve Hjønnevåg" <arve@android.com>, "Riley Andrews" <riandrews@android.com>, "Daniel Vetter" <daniel.vetter@ffwll.ch>, "Rob Clark" <robdclark@gmail.com>, "Greg Hackmann" <ghackmann@google.com>, "John Harrison" <John.C.Harrison@Intel.com> Subject: Re: [PATCH v3] staging/android: add TODO to de-stage android sync framework Date: Tue, 8 Dec 2015 09:20:32 +0100 [thread overview] Message-ID: <20151208082032.GB20822@phenom.ffwll.local> (raw) In-Reply-To: <20151207182845.GB2171@joana> On Mon, Dec 07, 2015 at 04:28:45PM -0200, Gustavo Padovan wrote: > Hi, > > any comments/update on this? Thanks My ack from the previous version still stands. -Daniel > > Gustavo > > 2015-11-26 Gustavo Padovan <gustavo@padovan.org>: > > > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > > > - remove CONFIG_SW_SYNC_USER, it is used only for testing/debugging and > > should not be upstreamed. > > - port CONFIG_SW_SYNC_USER tests interfaces to use debugfs somehow > > - port libsync tests to kselftest > > - clean up and ABI check for security issues > > - move the sync framework to drivers/base/dma-buf > > > > Cc: Arve Hjønnevåg <arve@android.com> > > Cc: Riley Andrews <riandrews@android.com> > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch> > > Cc: Rob Clark <robdclark@gmail.com> > > Cc: Greg Hackmann <ghackmann@google.com> > > Cc: John Harrison <John.C.Harrison@Intel.com> > > Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > --- > > drivers/staging/android/TODO | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO > > index 8f3ac37..64d8c87 100644 > > --- a/drivers/staging/android/TODO > > +++ b/drivers/staging/android/TODO > > @@ -25,5 +25,13 @@ ion/ > > exposes existing cma regions and doesn't reserve unecessarily memory when > > booting a system which doesn't use ion. > > > > +sync framework: > > + - remove CONFIG_SW_SYNC_USER, it is used only for testing/debugging and > > + should not be upstreamed. > > + - port CONFIG_SW_SYNC_USER tests interfaces to use debugfs somehow > > + - port libsync tests to kselftest > > + - clean up and ABI check for security issues > > + - move it to drivers/base/dma-buf > > + > > Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc: > > Arve Hjønnevåg <arve@android.com> and Riley Andrews <riandrews@android.com> > > -- > > 2.1.0 > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Vetter <daniel@ffwll.ch> To: "Gustavo Padovan" <gustavo@padovan.org>, greg@kroah.com, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, daniels@collabora.com, "Gustavo Padovan" <gustavo.padovan@collabora.co.uk>, "Arve Hjønnevåg" <arve@android.com>, "Riley Andrews" <riandrews@android.com>, "Daniel Vetter" <daniel.vetter@ffwll.ch>, "Rob Clark" <robdclark@gmail.com>, "Greg Hackmann" <ghackmann@google.com>, "John Harrison" <John.C.Harrison@Intel.com> Subject: Re: [PATCH v3] staging/android: add TODO to de-stage android sync framework Date: Tue, 8 Dec 2015 09:20:32 +0100 [thread overview] Message-ID: <20151208082032.GB20822@phenom.ffwll.local> (raw) In-Reply-To: <20151207182845.GB2171@joana> On Mon, Dec 07, 2015 at 04:28:45PM -0200, Gustavo Padovan wrote: > Hi, > > any comments/update on this? Thanks My ack from the previous version still stands. -Daniel > > Gustavo > > 2015-11-26 Gustavo Padovan <gustavo@padovan.org>: > > > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > > > - remove CONFIG_SW_SYNC_USER, it is used only for testing/debugging and > > should not be upstreamed. > > - port CONFIG_SW_SYNC_USER tests interfaces to use debugfs somehow > > - port libsync tests to kselftest > > - clean up and ABI check for security issues > > - move the sync framework to drivers/base/dma-buf > > > > Cc: Arve Hjønnevåg <arve@android.com> > > Cc: Riley Andrews <riandrews@android.com> > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch> > > Cc: Rob Clark <robdclark@gmail.com> > > Cc: Greg Hackmann <ghackmann@google.com> > > Cc: John Harrison <John.C.Harrison@Intel.com> > > Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk> > > --- > > drivers/staging/android/TODO | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO > > index 8f3ac37..64d8c87 100644 > > --- a/drivers/staging/android/TODO > > +++ b/drivers/staging/android/TODO > > @@ -25,5 +25,13 @@ ion/ > > exposes existing cma regions and doesn't reserve unecessarily memory when > > booting a system which doesn't use ion. > > > > +sync framework: > > + - remove CONFIG_SW_SYNC_USER, it is used only for testing/debugging and > > + should not be upstreamed. > > + - port CONFIG_SW_SYNC_USER tests interfaces to use debugfs somehow > > + - port libsync tests to kselftest > > + - clean up and ABI check for security issues > > + - move it to drivers/base/dma-buf > > + > > Please send patches to Greg Kroah-Hartman <greg@kroah.com> and Cc: > > Arve Hjønnevåg <arve@android.com> and Riley Andrews <riandrews@android.com> > > -- > > 2.1.0 > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2015-12-08 8:20 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-26 13:03 [PATCH v3] staging/android: add TODO to de-stage android sync framework Gustavo Padovan 2015-11-26 13:03 ` Gustavo Padovan 2015-12-07 18:28 ` Gustavo Padovan 2015-12-07 18:28 ` Gustavo Padovan 2015-12-08 8:20 ` Daniel Vetter [this message] 2015-12-08 8:20 ` Daniel Vetter
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20151208082032.GB20822@phenom.ffwll.local \ --to=daniel@ffwll.ch \ --cc=John.C.Harrison@Intel.com \ --cc=arve@android.com \ --cc=daniel.vetter@ffwll.ch \ --cc=daniels@collabora.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=ghackmann@google.com \ --cc=greg@kroah.com \ --cc=gustavo.padovan@collabora.co.uk \ --cc=gustavo@padovan.org \ --cc=linux-kernel@vger.kernel.org \ --cc=riandrews@android.com \ --cc=robdclark@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.