Re: [PATCH 00/12] De-stage Sync File Framework

From	Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Newsgroups	linux.kernel
Subject	Re: [PATCH 00/12] De-stage Sync File Framework
Date	2016-04-28 15:50 +0200
Message-ID	<rsUuo-3Xq-51@gated-at.bofh.it> (permalink)
References	<rsAvD-47z-3@gated-at.bofh.it> <rsD0u-6ts-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

2016-04-27 Daniel Vetter <daniel@ffwll.ch>:

> On Wed, Apr 27, 2016 at 01:27:07PM -0300, Gustavo Padovan wrote:
> > From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> > 
> > Hi,
> > 
> > This patchset sits on top of Sync ABI Rework v13:
> > 
> > https://www.spinics.net/lists/dri-devel/msg105667.html
> > 
> > The first eight clean up and prepare sync_file for de-staging. The last four
> > patches do the de-staging, moving files to drivers/dma-buf/ and include/linux/
> > plus adding Documentation.
> > 
> > As the de-stage depends upon many changes on the staging tree it would
> > be good to get all the patches merged through the staging tree if Sumit
> > agrees with that.
> > 
> > The next step on the Sync de-stage is clean up the remaining bits 
> > of the Sync Framework, mainly SW_SYNC, which is only used for testing.
> 
> Ok I looked once more at all this stuff, and there's some nitpicks I
> discussed with Gustavo on irc. But really their small, and I think
> perfectly ok to address them once sync_file is destaged. Especially since
> there's a lot more work pending on top of this, so we really want to get
> sync_file.[hc] destaged in 4.7. It'll take us a few iterations and a few
> drivers using this in anger in upstream to perfect the internal interfaces
> anyway, but let's get this started.
> 
> Had some real nitpicks on the docs patch, but that can also be merged
> later on imo. Except for that patch, on the series:
> 
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> 

Thanks for the review, I'm sending v2 fixing the nitpicks you pointed
out.

	Gustavo

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

Thread

[PATCH 00/12] De-stage Sync File Framework Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 12/12] Documentation: add Sync File doc Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
    Re: [PATCH 12/12] Documentation: add Sync File doc Daniel Vetter <daniel@ffwll.ch> - 2016-04-27 21:10 +0200
  [PATCH 04/12] staging/android: make sync_file_merge() static Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 06/12] staging/android: prepare sync_file for de-staging Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
    Re: [PATCH 06/12] staging/android: prepare sync_file for de-staging Pavel Machek <pavel@ucw.cz> - 2016-05-02 23:10 +0200
      Re: [PATCH 06/12] staging/android: prepare sync_file for de-staging Gustavo Padovan <gustavo@padovan.org> - 2016-05-02 23:20 +0200
  [PATCH 02/12] staging/android: drop sync_file_install() and sync_file_put() Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 07/12] staging/android: improve documentation for sync_file Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 03/12] staging/android: move sync_file functions comments to sync.c Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 08/12] staging/android: style fix: alignment to match the open parenthesis Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 05/12] staging/android: make sync_file_fdget() static Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 11/12] Documentation: include sync_file into DocBook Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 09/12] dma-buf/sync_file: de-stage sync_file headers Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 10/12] dma-buf/sync_file: de-stage sync_file Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:30 +0200
  [PATCH 01/12] staging/android: remove redundant comments on sync_merge_data Gustavo Padovan <gustavo@padovan.org> - 2016-04-27 18:40 +0200
    Re: [PATCH 01/12] staging/android: remove redundant comments on  sync_merge_data Pavel Machek <pavel@ucw.cz> - 2016-05-02 23:10 +0200
      Re: [PATCH 01/12] staging/android: remove redundant comments on  sync_merge_data Gustavo Padovan <gustavo@padovan.org> - 2016-05-02 23:20 +0200
  Re: [PATCH 00/12] De-stage Sync File Framework Daniel Vetter <daniel@ffwll.ch> - 2016-04-27 21:10 +0200
    Re: [PATCH 00/12] De-stage Sync File Framework Gustavo Padovan <gustavo.padovan@collabora.co.uk> - 2016-04-28 15:50 +0200

csiph-web