[PATCH weston 7/8] compositor-drm: Implement clone mode, refactor output into logical ones

[Daniel Stone]

Hi,

On 22 November 2016 at 09:41, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> On Tue, 22 Nov 2016 09:06:00 +0000 Daniel Stone <daniel at fooishbar.org> wrote:
>> Thanks for this - it's a good point. It's strengthened my resolve a
>> little bit though, especially in the context of
>> https://phabricator.freedesktop.org/T7621, where our problem is that
>> repaint is inseparable from, and damaging[0] to, core state. If we
>> pulled primary_plane out of core, and started calculating repaint data
>> per-call instead, we could kill two birds with one stone. And also
>> promote surfaces to planes even if they do span multiple outputs.
>
> Indeed, and maybe even allow targeting renderers to arbitrary planes
> rather than the hardcoded primary plane. Then we'd "just" need the
> planner from hwc2 if it was a good fit...

I'd probably say these should be the other way around. Allocating new
GPU surfaces and populating them with buffers for every plane we want
to test, on the off chance that it might work, is a lot of work to do
in our repaint loop; I'd rather not be doing it for every view of
every frame ...

That being said, I have no real data to back this up, so hey, maybe it
is actually fine.

Cheers,
Daniel