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

Groups > linux.kernel > #1460314

Re: [PATCH v2 0/6] de-stage SW_SYNC validation frawework

Show all headers | View raw

On Mon, Aug 08, 2016 at 06:24:16PM -0300, Gustavo Padovan wrote:
> From: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
> 
> Hi Greg,
> 
> This is the last step in the Sync Framwork de-stage task. It de-stage
> the SW_SYNC validation framework and the sync_debug info debugfs file.
> 
> The first 2 patches are clean up and improvements and the rest is preparation
> to de-stage and then finally the actual de-stage.
> 
> v2: 
>  - add documentation about the SW_SYNC ioctl API (comments from Pavel Machek)
>  - remove for now patch to add sync_pt name to debugfs
> 
> Please review,
> 
> Gustavo
> 
> ---
> Gustavo Padovan (6):
>   staging/android: remove doc from sw_sync
>   staging/android: do not let userspace trigger WARN_ON
>   staging/android: move trace/sync.h to sync_trace.h
>   staging/android: prepare sw_sync files for de-staging
>   staging/android: add Doc for SW_SYNC ioctl interface
>   dma-buf/sw_sync: de-stage SW_SYNC
> 
>  drivers/dma-buf/Kconfig              |  13 ++
>  drivers/dma-buf/Makefile             |   1 +
>  drivers/dma-buf/sw_sync.c            | 349 +++++++++++++++++++++++++++++++++++
>  drivers/dma-buf/sync_debug.c         | 230 +++++++++++++++++++++++
>  drivers/dma-buf/sync_debug.h         |  69 +++++++
>  drivers/dma-buf/sync_trace.h         |  32 ++++
>  drivers/staging/android/Kconfig      |  13 --
>  drivers/staging/android/Makefile     |   1 -
>  drivers/staging/android/sw_sync.c    | 344 ----------------------------------
>  drivers/staging/android/sync_debug.c | 230 -----------------------
>  drivers/staging/android/sync_debug.h |  84 ---------
>  drivers/staging/android/trace/sync.h |  32 ----
>  12 files changed, 694 insertions(+), 704 deletions(-)

>  create mode 100644 drivers/dma-buf/sw_sync.c
>  create mode 100644 drivers/dma-buf/sync_debug.c
>  create mode 100644 drivers/dma-buf/sync_debug.h
>  create mode 100644 drivers/dma-buf/sync_trace.h
>  delete mode 100644 drivers/staging/android/sw_sync.c
>  delete mode 100644 drivers/staging/android/sync_debug.c
>  delete mode 100644 drivers/staging/android/sync_debug.h
>  delete mode 100644 drivers/staging/android/trace/sync.h

When you send your next revision, could you use `git format-patch -M`?
A good 95% of the lines in these patches aren't actually modified, just
moved around, which makes it much harder to spot the actual changes :)

Cheers,
  Eric

> 
> -- 
> 2.5.5

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

Thread

[PATCH v2 0/6] de-stage SW_SYNC validation frawework Gustavo Padovan <gustavo@padovan.org> - 2016-08-08 23:30 +0200
  [PATCH v2 3/6] staging/android: move trace/sync.h to sync_trace.h Gustavo Padovan <gustavo@padovan.org> - 2016-08-08 23:30 +0200
    Re: [PATCH v2 3/6] staging/android: move trace/sync.h to sync_trace.h Eric Engestrom <eric.engestrom@imgtec.com> - 2016-08-11 11:50 +0200
  [PATCH v2 2/6] staging/android: do not let userspace trigger WARN_ON Gustavo Padovan <gustavo@padovan.org> - 2016-08-08 23:30 +0200
  [PATCH v2 1/6] staging/android: remove doc from sw_sync Gustavo Padovan <gustavo@padovan.org> - 2016-08-08 23:30 +0200
    Re: [PATCH v2 1/6] staging/android: remove doc from sw_sync Pavel Machek <pavel@ucw.cz> - 2016-08-10 20:30 +0200
      Re: [PATCH v2 1/6] staging/android: remove doc from sw_sync Gustavo Padovan <gustavo@padovan.org> - 2016-08-11 17:10 +0200
  Re: [PATCH v2 0/6] de-stage SW_SYNC validation frawework Eric Engestrom <eric.engestrom@imgtec.com> - 2016-08-11 11:30 +0200
    Re: [PATCH v2 0/6] de-stage SW_SYNC validation frawework Gustavo Padovan <gustavo@padovan.org> - 2016-08-11 17:20 +0200
  Re: [PATCH v2 6/6] dma-buf/sw_sync: de-stage SW_SYNC Eric Engestrom <eric.engestrom@imgtec.com> - 2016-08-11 12:00 +0200

csiph-web