Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1276177

Re: [PATCH] staging/android: add TODO to de-stage android sync framework

Show all headers | View raw

On Tue, Nov 24, 2015 at 09:51:12AM +0100, Daniel Vetter wrote:
> On Mon, Nov 23, 2015 at 05:41:53PM -0200, Gustavo Padovan wrote:
> > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> > 
> >  - remove sw_sync, it is used only for testing/debugging and should not
> > be upstreamed.
> >  - port sw_sync testcases to use debugfs somehow
> >  - 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>
> 
> This reflects my recollection of various discussions at conferences and on
> irc. Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch>

Coffee just kicked in ;-)

> > ---
> >  drivers/staging/android/TODO | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/drivers/staging/android/TODO b/drivers/staging/android/TODO
> > index 8f3ac37..2375dae 100644
> > --- a/drivers/staging/android/TODO
> > +++ b/drivers/staging/android/TODO
> > @@ -25,5 +25,12 @@ ion/
> >     exposes existing cma regions and doesn't reserve unecessarily memory when
> >     booting a system which doesn't use ion.
> >  
> > +sync framework:
> > + - remove sw_sync, it is used only for testing/debugging and should not be
> > +upstreamed.
> > + - port sw_sync testcases to use debugfs somehow

With all the effort going on around kselftest it'd be good to integrate
the existing testsuite google has into upstream too. Should probably be
listed here too.
-Daniel

> > + - 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

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH] staging/android: add TODO to de-stage android sync framework Gustavo Padovan <gustavo@padovan.org> - 2015-11-23 20:50 +0100
  Re: [PATCH] staging/android: add TODO to de-stage android sync  framework Daniel Vetter <daniel@ffwll.ch> - 2015-11-24 10:00 +0100
    Re: [PATCH] staging/android: add TODO to de-stage android sync  framework Daniel Vetter <daniel@ffwll.ch> - 2015-11-24 10:00 +0100
      Re: [PATCH] staging/android: add TODO to de-stage android sync  framework Greg Hackmann <ghackmann@google.com> - 2015-11-24 18:30 +0100
  Re: [PATCH] staging/android: add TODO to de-stage android sync  framework Greg Hackmann <ghackmann@google.com> - 2015-11-24 18:00 +0100
    Re: [PATCH] staging/android: add TODO to de-stage android sync  framework Gustavo Padovan <gustavo@padovan.org> - 2015-11-24 18:40 +0100

csiph-web