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

From: Pavel Machek
Date: Sun Jun 26 2016 - 17:45:33 EST


Hi!

> From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx>
>
> Hi Greg,
>
> This is the last step in the Sync Framwork de-stage task. It

Typo: "fram_e_work"

> de-stage
> the SW_SYNC validation framework and the sync_debug info debugfs file.
>
> The first 3 patches are clean up and improvements and the rest is preparation
> to de-stage and then finally the actual de-stage.

Could we get some kind of description what the sync framework does?
There are no useful comments in /sw_sync.c. There's no Documentation/
files. I don't know what this is supposed to do...

Pavel

> Please review,
>
> Gustavo
>
> ---
> Gustavo Padovan (7):
> staging/android: move trace/sync.h to sync_trace.h
> staging/android: remove doc from sw_sync
> staging/android: display sync_pt name on debugfs
> staging/android: do not let userspace trigger WARN_ON
> staging/android: prepare sw_sync files for de-staging
> dma-buf/sw_sync: de-stage SW_SYNC
> staging/android: remove sync framework TODO
>
> drivers/dma-buf/Kconfig | 14 +++++++
> drivers/dma-buf/Makefile | 1 +
> drivers/{staging/android => dma-buf}/sw_sync.c | 46 +++-------------------
> drivers/{staging/android => dma-buf}/sync_debug.c | 7 ++--
> drivers/{staging/android => dma-buf}/sync_debug.h | 26 +++++-------
> .../android/trace/sync.h => dma-buf/sync_trace.h} | 6 +--
> drivers/staging/android/Kconfig | 13 ------
> drivers/staging/android/Makefile | 1 -
> drivers/staging/android/TODO | 8 ----
> 9 files changed, 38 insertions(+), 84 deletions(-)
> rename drivers/{staging/android => dma-buf}/sw_sync.c (84%)
> rename drivers/{staging/android => dma-buf}/sync_debug.c (97%)
> rename drivers/{staging/android => dma-buf}/sync_debug.h (72%)
> rename drivers/{staging/android/trace/sync.h => dma-buf/sync_trace.h} (84%)
>

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html