[PULL] Reviewed glamor changes

Keith Packard keithp at keithp.com
Thu Apr 3 13:16:31 PDT 2014


Eric Anholt <eric at anholt.net> writes:

> keithp was saying he was holding the merge window open for a few last
> bits of glamor and wayland stuff for 1.16.  This is as far as I was able
> to review of keithp's code today.

I'm still waiting for Xwayland today. Note that x11perf -all crashes
because of the glamor putimage code, so I'd love to get the new version
of that merged for 1.16 as well as a bug fix.

> Eric Anholt (1):
>       Revert "glx: Make sure we get an FBConfig the root window's visual."
>
> Keith Packard (14):
>       glamor: glamor_poly_point_nf cannot fail for non-DDX pixmaps
>       glamor: SetWindowPixmap is not related to RENDER
>       glamor: Initialize XV shaders from glamor_xv_init instead of glamor_init
>       glamor: Public polyLines function is glamor_poly_lines_nf
>       glamor: Remove warning message when pixmap cannot be stored in a texture
>       glamor: Bail from composite when pixmap cannot be uploaded
>       fb: Publish fbGlyphs and fbUnrealizeGlyph
>       glamor: Compute supported GLSL version and save in screen private
>       glamor: Use plain GLSL 1.20 features for fill code.

I squashed in a patch for this; the vertex shader needed to see the
fill_size uniform now:

diff --git a/glamor/glamor_program.c b/glamor/glamor_program.c
index cdd95ea..0f4d0f0 100644
--- a/glamor/glamor_program.c
+++ b/glamor/glamor_program.c
@@ -107,6 +107,7 @@ static glamor_location_var location_vars[] = {
     {
         .location = glamor_program_location_fill,
         .vs_vars = ("uniform vec2 fill_offset;\n"
+                    "uniform vec2 fill_size;\n"
                     "varying vec2 fill_pos;\n"),
         .fs_vars = ("uniform sampler2D sampler;\n"
                     "uniform vec2 fill_size;\n"

>       glamor: Add simple upload/download functions in glamor_transfer
>       glamor: Add glamor_program based fill/set/get spans
>       glamor: Add glamor_program based poly_fill_rect
>       glamor: Use glamor_program for glamor_glyphblt
>       glamor: Add glamor_program based poly_text and image_text

Merged (with minor fix above):
   84f9774..3c34dd3  master -> master

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140403/b767c3cf/attachment.sig>


More information about the xorg-devel mailing list