[RFC weston] compositor: optimize/simplify shaders

[Rob Clark]

On Thu, Aug 30, 2012 at 6:11 AM, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> On Wed, 29 Aug 2012 19:18:17 -0500
> Rob Clark <rob.clark at linaro.org> wrote:
>
>> On Tue, Aug 28, 2012 at 8:27 AM, Pekka Paalanen <ppaalanen at gmail.com> wrote:
>> > On Mon, 27 Aug 2012 17:03:10 +0300
>> > Pekka Paalanen <ppaalanen at gmail.com> wrote:
>> >
>> >> Hi Rob,
>> >>
>> >> I've started reviewing your patch and fixing the remaining bugs. So far
>> >> I think I got most of the blend/opaque region stuff sorted out. I
>> >> haven't still gotten to the geometry, where I can trigger visual bugs
>> >> with and without getting your TODO printouts.
>> >>
>> >> I'm guessing the xwayland surfaces will come and haunt us, because I
>> >> think they contain both an opaque region with undefined alpha values,
>> >> and opaque and non-opaque regions with valid alpha values.I haven't
>> >> even tested them yet, but I guess a quick fix would be to paint opaque
>> >> regions always with a shader that forces texture alpha to 1.0. Would it
>> >> better as a different shader program or a uniform flag for a shader, I
>> >> don't know.
>> >>
>> >> I'm not sure we should have nested functions in Weston code base. Also
>> >> like you noted, compositor_wayland.c does not build anymore.
>> >>
>> >> Here's my WIP tree that may be rebased!
>> >
>> > The tree is still here and updated:
>> > http://cgit.collabora.com/git/user/pq/wayland-demos.git/log/?h=shaders
>> > I don't think I will be rebasing it anymore.
>> >
>> > I merged Rob's updates there.
>> >
>> > Rob, I will leave cleaning up the series for you, once we are done. You
>> > probably want to squash some things etc.
>>
>> Fyi, I've squashed/reordered a bit.  And pushed some nice updates to
>> your debug draw stuff, so now it is quite fun to look at:
>>
>>   git://github.com/robclark/weston.git master
>
> Hi Rob,
>
> that's definitely interesting. I'm working based on your 'pq' branch,
> though, because you missed these commits:
>
> 1e17e4a compositor: fix blending for full-surface alpha
> 87af9c8 compositor: specialised fragment shader for RGBX
> 0c37514 compositor: re-enable full-surface alpha
> 82ffb58 compositor: fix build against WL_bind_wayland_display extension

oh woops, I quickly pushed 'em to my master branch.. would be a good
idea to double check that there is nothing missing.

> and it seems the 'pq' branch is up to date still.
>
> I got these with your 'pq' branch:
>
> http://people.collabora.com/~pq/geometry-debug-6.png
> It appears the clean-up painting paints something wrong, but I haven't
> been able to reproduce that otherwise. The flower is rotated about 45
> degrees. Notice the three green pixels down from the word "Thu" in the
> panel clock. Those pixels flicker when I move the mouse cursor, even
> though there should not be damage anywhere near. The green pixels are
> where the surface's top-left corner is.

three green pixels sound like some garbage left on one of the
flipchain buffers..  I guess that itself is just from the debug lines,
although the flower itself looks a bit messed up.

> http://people.collabora.com/~pq/geometry-debug-7.png
> This shows a problem with screen-aligned surfaces. The center vertex is
> not in the center, implying that there are duplicate vertices. Good
> thing I didn't remove the center vertex yet.

hmm, yeah, this stuff is kinda relying on the duplicate vertex check
in append_vertex()..

		if (x1 == x2) {
			append_vertex(clip(x1, cx1, cx2), clip(y1, cy1, cy2));
			append_vertex(clip(x2, cx1, cx2), clip(y2, cy1, cy2));
		} else if (y1 == y2) {
			append_vertex(clip(x1, cx1, cx2), clip(y1, cy1, cy2));
			append_vertex(clip(x2, cx1, cx2), clip(y2, cy1, cy2));
		} else {

but I guess maybe we end up w/ a duplicate vertex which isn't the last
vertex..  so this part is probably still not right..

BR,
-R

> If I disable the fan debug line clean-up, it is hard to spot or trigger
> any problems, apart from the center point thing.
>
>
> Thanks,
> pq
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel