Messages in this thread | | | Date | Thu, 28 Apr 2016 10:45:51 -0300 | From | Gustavo Padovan <> | Subject | Re: [PATCH 00/12] De-stage Sync File Framework |
| |
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
| |