Please use "PATCH hwc" for drm_hwcomposer patches. On Mon, Apr 23, 2018 at 7:06 PM, John Stultz <john.stu...@linaro.org> wrote: > The drm_hwcomposer has its own GL pre-compositor which is used > to squish layers when there are more layers then planes on the > display hardware. In many ways this duplicates the client-side > GL compositing that is done in SurfaceFlinger, but in theory can > be more highly optimized for the hardware. > > Unfortunately, due to these optimizations, the drm_hwcomposer's > pre-compositor becomes somewhat hardware specific (originally > targeting nvidia hardware, I believe). > > So on some hardware, the gl precompositor may not actually > initialize due to hardware missing features, or the hardware > supporting different shader APIs. > > Rather then try to rework the drm_hwcomposers precompositor > to be more generic, I instead suggest that when the > precompositor fails to initialize, we simply fall back to the > already more widely compatible client compositor in > SurfaceFlinger. > > Thus, this patch cleans up some of the precompositor > initialization, which didn't handle failures well. > > Feedback or alternative ideas would be greatly appreciated! > > Cc: Marissa Wall <maris...@google.com> > Cc: Sean Paul <seanp...@google.com> > Cc: Dmitry Shmidt <dimitr...@google.com> > Cc: Robert Foss <robert.f...@collabora.com> > Cc: Matt Szczesiak <matt.szczes...@arm.com> > Cc: Liviu Dudau <liviu.du...@arm.com> > Cc: David Hanna <david.hann...@gmail.com> > Cc: Rob Herring <rob.herr...@linaro.org> > Cc: Alexandru-Cosmin Gheorghe <alexandru-cosmin.gheor...@arm.com> > Cc: Alistair Strachan <astrac...@google.com> > Signed-off-by: John Stultz <john.stu...@linaro.org> > --- > drmdisplaycompositor.cpp | 40 +++++++++++++++++++++------------------- > drmdisplaycompositor.h | 3 +++ > 2 files changed, 24 insertions(+), 19 deletions(-)
Both patches look good to me. Reviewed-by: Rob Herring <r...@kernel.org> _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
