<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=gb2312"><meta name=Generator content="Microsoft Word 14 (filtered medium)"><style></style></head><body lang=ZH-CN link=blue vlink=purple style='text-justify-trim:punctuation'><div class=WordSection1>Hi Keith,<o:p></o:p><o:p> </o:p>I just came back from a long vacation and I checked all the comments for<o:p></o:p>the glamor’s pull request. All the comments came from Alan. Thanks Alan.<o:p></o:p>Here are a summary of those comments and current status:<o:p></o:p><o:p> </o:p><![if !supportLists]>1.  <![endif]>Alan reported a build fail problem. <o:p></o:p>Fixed and tested by Alan.<o:p></o:p><![if !supportLists]>2.  <![endif]>Alan pointed out that the coding style is not consistent with X’s and<o:p></o:p>even have some conflicts with the code itself. <o:p></o:p>Fixed in the latest version. Alan took a quick glance, and said it’s <o:p></o:p>more consistent with itself. And after that, I made some change to use <o:p></o:p>camelCase naming rule according to X coding style.<o:p></o:p><o:p> </o:p>Please check it out and if any other thing need to be fixed, please let me <o:p></o:p>know.<o:p></o:p><o:p> </o:p>Thanks.<o:p></o:p><o:p> </o:p>--Zhigang<o:p></o:p><o:p> </o:p>The following changes since commit 7fb4bef0394a5d09680985d34bce8252b61493cb:<o:p></o:p><o:p> </o:p>  Merge remote-tracking branch 'mattst88/for-keith' (2011-09-21 14:34:27 -0700)<o:p></o:p><o:p> </o:p>are available in the git repository at:<o:p></o:p><o:p> </o:p>  git://people.freedesktop.org/~gongzg/xserver-glamor glamor-style-fixes<o:p></o:p><o:p> </o:p>Eric Anholt (100):<o:p></o:p>      Add build infrastructure for glamor.<o:p></o:p>      More ephyr/glamor glue.<o:p></o:p>      Add exa/glamor to _DEPENDENCIES so the bin gets rebuilt with the libs.<o:p></o:p>      glamor: Use -lgl for Xephyr if GLX is unset.<o:p></o:p>      glamor: Start trying to hook up the rendering bits<o:p></o:p>      glamor: Use a nicer struct initializer for gcops.<o:p></o:p>      glamor: glFlush from the blockhandler so rendering happens.<o:p></o:p>      glamor: first real attempt at rendering.<o:p></o:p>      ephyr: merge the host glamor bits into hostx.<o:p></o:p>      add fill files, merge with first real attempt at rendering.<o:p></o:p>      ephyr: Make sure a glamor-using window is created with a glx visual.<o:p></o:p>      ephyr: Use GLEW.<o:p></o:p>      glamor: Fix up block/wakeup handler.<o:p></o:p>      glamor: remove gratuitous flush.<o:p></o:p>      glamor: Fix up DrawPixels arguments to bear some relation to reality.<o:p></o:p>      glamor: Move to using shader objects.<o:p></o:p>      glamor: Add getspans implementation.<o:p></o:p>      glamor: Maybe fix up the format/type for setspans.<o:p></o:p>      glamor: Create FBOs for pixmaps.<o:p></o:p>      glamor: Fill out glamor_get_color_4f_from_pixel() a bit.<o:p></o:p>      glamor: Add untested support for tile filling.<o:p></o:p>      glamor: Free resources when destroying pixmaps.<o:p></o:p>      glamor: Add stub Composite support.<o:p></o:p>      glamor: Fill in 1 and 24-bit getspans.<o:p></o:p>      glamor: Add more solid_fail_regions for software fallbacks.<o:p></o:p>      glamor: Add trapezoids code.<o:p></o:p>      glamor: Add 8bpp to get/setspans.<o:p></o:p>      glamor: Add untested putimage support.<o:p></o:p>      glamor: Add ALU support to solid, tile, and putimage.<o:p></o:p>      glamor: Hook up miGetImage to fix some invalid accesses.<o:p></o:p>      glamor: Move setspans to a separate file.<o:p></o:p>      glamor: Fix memory leak in getspans.<o:p></o:p>      glamor: Fix bad fallthrough in getspans (accessing invalid memory).<o:p></o:p>      glamor: Give setspans the same format support as getspans.<o:p></o:p>      glamor: add alu support to setspans.<o:p></o:p>      glamor: Move planemask to a function in case I decide to fill it in.<o:p></o:p>      glamor: Fix some screen_xy offsets to be the right way around.<o:p></o:p>      glamor: Add clipping to setspans.<o:p></o:p>      glamor: Start adding render acceleration support.<o:p></o:p>      glamor: make the polylines complaint a little more useful.<o:p></o:p>      glamor: Add untested PutImage XYBitmap support.<o:p></o:p>      glamor: Add untested PutImage 1bpp XYPixmap support.<o:p></o:p>      glamor: Accelerate PolyFillRect using glamor_fill instead of spans.<o:p></o:p>      glamor: Add untested copyarea implementation<o:p></o:p>      glamor: Add support for solid source pictures.<o:p></o:p>      glamor: Fix pixmap private getter in copyarea.<o:p></o:p>      glamor: Add prepare/finishaccess code based on UXA.<o:p></o:p>      glamor: Fix the type for copyarea.<o:p></o:p>      glamor: Move glamor_poly_lines to a separate file.<o:p></o:p>      glamor: Add fallbacks for glamor_copy_area().<o:p></o:p>      glamor: Add fallbacks for Render.<o:p></o:p>      glamor: Use GL_MESA_pack_invert to avoid complexity in prepare_access.<o:p></o:p>      glamor: Add glamor_prepare_access_gc() from UXA.<o:p></o:p>      glamor: Add fallback support to glamor_set_spans().<o:p></o:p>      glamor: Add fallback support to glamor_put_image().<o:p></o:p>      glamor: Fix up the access flags for glamor_prepare_access().<o:p></o:p>      glamor: Fix the row length of 1bpp prepare_access.<o:p></o:p>      ephyr: Hook the glamor into damage and draw into the backbuffer.<o:p></o:p>      glamor: Add fallback support for glamor_get_spans().<o:p></o:p>      glamor: Add check for EXT_bgra which we rely on.<o:p></o:p>      glamor: All the fallbacks in the world.<o:p></o:p>      glamor: Replace the glDrawPixels in glamor_finish_access with GLSL.<o:p></o:p>      glamor: Use the pixmap stride in fallbacks instead of trying to guess it.<o:p></o:p>      glamor: Fix prepare_access on 1bpp data.<o:p></o:p>      glamor: Enable fill acceleration and flip opposite for pixmaps too.<o:p></o:p>      glamor: Add CopyWindow implementation so it doesn't crash.<o:p></o:p>      glamor: Add acceleration for copyarea not from the screen.<o:p></o:p>      glamor: Pull in UXA code for dumping some composites down to copy_n_to_n.<o:p></o:p>      glamor: Provide more information about the operands to fallback composites.<o:p></o:p>      glamor: Split the copy path out into its own function.<o:p></o:p>      glamor: Align stride of fallback pixmap data to 32 bits.<o:p></o:p>      glamor: Set active texture in finishaccess drawing.<o:p></o:p>      glamor: Restore planemask to all-on when finishing a fill.<o:p></o:p>      glamor: Fix up and enable accelerated composite.<o:p></o:p>      glamor: Add the glyph cache from UXA (de-camelCased).<o:p></o:p>      glamor: Rework the Render shader setup to be easily modified, like cairo-gl.<o:p></o:p>      glamor: Don't try to CopyArea from a Solid source picture.<o:p></o:p>      glamor: Set active texture on glamor_copy_n_to_n setup.<o:p></o:p>      glamor: Add support for accel of x8r8g8b8 source pictures.<o:p></o:p>      glamor: Implement glCopyPixels based src == dest CopyArea acceleration<o:p></o:p>      glamor: Fix screen_x/screen_y handling for compositing.<o:p></o:p>      glamor: Fix off-by-one in CopyPixels CopyArea path.<o:p></o:p>      glamor: Fix and enable tile fill acceleration (aka the root weave).<o:p></o:p>      glamor: Fix up the wide/non-solid lines fallback.<o:p></o:p>      glamor: Enable glamor_fill_spans().<o:p></o:p>      glamor: Fix and enable glamor_get_spans().<o:p></o:p>      glamor: Fix render source transforms.<o:p></o:p>      glamor: Fix and enable ZPixmap PutImage acceleration.<o:p></o:p>      glamor: Add support for a1 composite sources.<o:p></o:p>      glamor: Fix up the fallback message for no texture present on compositing.<o:p></o:p>      glamor: Add a little mechanism for only printing fallbacks when they happen.<o:p></o:p>      glamor: Rework shader setup to make extending the IN types easier.<o:p></o:p>      glamor: Add support for component alpha rendering.<o:p></o:p>      glamor: Convert the shaders path to handling glamor_composite_rect_t.<o:p></o:p>      glamor: Use glamor_composite_with_shader once per group of glyphs.<o:p></o:p>      glamor: Use a VBO to accumulate multiple glyph quads at once.<o:p></o:p>      glamor: Replace the immediate mode in glamor_fill() with glDrawArrays().<o:p></o:p>      glamor: Fix the segfault at screen fini.<o:p></o:p>      glamor: Reduce the noise from wide lines "fallback".<o:p></o:p>      glamor: Add support for using EXT_framebuffer_blit to do CopyArea.<o:p></o:p><o:p> </o:p>Kristian H酶gsberg (1):<o:p></o:p>      glamor ddx driver using EGL on KMS<o:p></o:p><o:p> </o:p>Zhenyu Wang (3):<o:p></o:p>      glamor: Use function pointer to get extenstion calls<o:p></o:p>      glamor: Require libdrm CFLAGS for building<o:p></o:p>      glamor: Fix direct call for glEGLImageTargetTexture2DOES<o:p></o:p><o:p> </o:p>Zhigang Gong (132):<o:p></o:p>      glamor: Fixed some compiling and linking errors.<o:p></o:p>      glamor: Fixed one conflict data type(XID) bug.<o:p></o:p>      glamor: Call glamor_composite_rects only when enable RENDER.<o:p></o:p>      glamor: Merged with latest xserver.<o:p></o:p>      glamor: Fixed one linking error.<o:p></o:p>      glamor: Fixed one segfault bug when close screen.<o:p></o:p>      glamor:  Resolved merge conflictions with Kristian's glamor-ddx patch.<o:p></o:p>      glamor: Add glamor-ddx driver to the build tree.<o:p></o:p>      glamor: Add new feature which is to flip output on y axis.<o:p></o:p>      glamor-ddx: Migrate the code to latest mesa library.<o:p></o:p>      glamor-ddx: Fix a compiling error. Need revisit.<o:p></o:p>      glamor-ddx: Move the cursor EGL image to crtc strcture.<o:p></o:p>      glamor: Silent compilation warnings due to some deprecated APIs.<o:p></o:p>      glamor-ddx: Fixed one bug when a client reset the connection.<o:p></o:p>      glamor: Switch to software fb for too large pixmap.<o:p></o:p>      glamor: glamor_set_alu should enable GL_COLOR_LOGIC_OP.<o:p></o:p>      glamor: Add fallback code path for glamor_fill.<o:p></o:p>      glamor-ddx: Remove debug message when moving cursor.<o:p></o:p>      glamor: Use buffer object as much as possible.<o:p></o:p>      glamor: Don't write back read only pixmap to fbo.<o:p></o:p>      glamor: improve glamor_finish_access.<o:p></o:p>      glamor: Don't use glamor_pixmap_type currently.<o:p></o:p>      glamor: Optimize glamor_finish_access.<o:p></o:p>      glamor: Don't print those fallback messages by default.<o:p></o:p>      glamor: Optimize composite when soure/mask is xrgb.<o:p></o:p>      glamor: Use software fb for 1bpp pixmap.<o:p></o:p>      glamor: Add render triangles support.<o:p></o:p>      glamor: Silence compilation warnings.<o:p></o:p>      glamor: For non-supported fill style fallback to fbFill.<o:p></o:p>      glamor: Fallback to fbPolylines for diagonal poly_line.<o:p></o:p>      glamor: Simplify fill acceleration for spans/polyfillrect by only clipping once.<o:p></o:p>      glamor : Add dynamic texture uploading feature.<o:p></o:p>      glamor: Concentrate and reduce some coords processing code.<o:p></o:p>      glamor: Don't always fallback everything when change window attr.<o:p></o:p>      glamor: Initialize pixmap private's container to correct value.<o:p></o:p>      glamor: Fixed one bug when enable dynamic pixmap uploading.<o:p></o:p>      glamor: Move the blend set up after the pixmap uploading.<o:p></o:p>      glamor: Don't need to read dest if op is SRC or CLEAR.<o:p></o:p>      glamor: Prepare/finish access once if src equal to dst.<o:p></o:p>      glamor: Change to use system memory for write only access.<o:p></o:p>      glamor: Reduce one extra copy in glamor_trapezoids.<o:p></o:p>      glamor: Don't map the vbo to system memory.<o:p></o:p>      glamor: Add one option to force create a cpu memory pixmap.<o:p></o:p>      glamor: Reduce source pixmap's size.<o:p></o:p>      glamor: Fix the coords calculation in glamor_fill.<o:p></o:p>      glamor: Convert picture to proper size and format.<o:p></o:p>      glamor: Remove one extra area copy in glamor_glyph.<o:p></o:p>      glamor: Fix one typo bug in glamor_tile.<o:p></o:p>      glamor: silence compilation warnings.<o:p></o:p>      glamor: convert if too large source or mask .<o:p></o:p>      glamor-ddx: Reinit crtc if we need to reopen the card.<o:p></o:p>      glamor-ddx: Add missed drmmode_crtc_destroy function.<o:p></o:p>      Revert "glamor: Don't need to read dest if op is SRC or CLEAR."<o:p></o:p>      glamor: Optimize fallback case for the polylines.<o:p></o:p>      glamor: Fix multiple crtc setup.<o:p></o:p>      glamor: Fixed two unintialized warnings.<o:p></o:p>      glamor: We don't need to check format in compositing.<o:p></o:p>      glamor: Don't need to pad the tile image if pixmap is ni texture.<o:p></o:p>      glamor: comment out the message when creating a system memory pixmap.<o:p></o:p>      glamor: Should return when done gl drawing.<o:p></o:p>      glamor: Fixed a bug when computing the bounds of boxes.<o:p></o:p>      glamor: Implement delayed solid filling.<o:p></o:p>      glamor: Avoid 2D bitblit if possible.<o:p></o:p>      glamor: Use small internal texture format if possible.<o:p></o:p>      glamor: Fix one bug for Xephyr.<o:p></o:p>      glamor: Change to use official APIs rather than ARB version function.<o:p></o:p>      glamor-es2: Add gles2 option.<o:p></o:p>      glamor-ddx: Use glamor ddx driver only if enable GLAMOR_DDX.<o:p></o:p>      glamor: Add glamor related macros to xorg config header file.<o:p></o:p>      glamor-es2: Fix some non-es2 functions.<o:p></o:p>      glamor-es: Use glVertexAttribPointer to replace glVertexPointer.<o:p></o:p>      glamor_gles2: Use gl_flavor to determine which version of GL.<o:p></o:p>      glamor: Check GLAMOR_GLES2 in configure process.<o:p></o:p>      glamor-ddx: Define macro GLAMOR_DDX in command line.<o:p></o:p>      glamor-gles2: Add initial support for GLES2.<o:p></o:p>      glamor: Define macro GLAMOR_GLES2 if enable glamor_gles2<o:p></o:p>      glamor: Concentrate FBO's creation to one function.<o:p></o:p>      glamor-ddx: Silence one warning.<o:p></o:p>      glamor: Disable ALPHA8 fbo.<o:p></o:p>      glamor:  Unify the variable name which used to indicate no alpha.<o:p></o:p>      glamor: Only fallbac glamor_setspan when we are using gles2.<o:p></o:p>      glamor: Remove useless glVertexPointer related code.<o:p></o:p>      glamor: Remove glu3 which is unnecessary.<o:p></o:p>      glamor-ddx: Hardware cursor's format should be GL_RGBA.<o:p></o:p>      glamor:  GLES2 doesn't support GL_CLAMP_TO_BORDER.<o:p></o:p>      glamor: GLES2 doesn't support glEnable/Disable Texture 2D.<o:p></o:p>      glamor: Add color conversion support by using new shader.<o:p></o:p>      glamor: Convert pixmap to supported format before getspans.<o:p></o:p>      glamor: Fix the copy of overlaped region in one pixmap.<o:p></o:p>      glamor: Add device independent glamor to the installation package.<o:p></o:p>      glamor-ddx: Add code to check required egl extensions.<o:p></o:p>      glamor:  egl version 7.11 should be ok for us.<o:p></o:p>      glamor: Don't need to check status when set the target.<o:p></o:p>      glamor-dix: Make a glamor dix module.<o:p></o:p>      glamor: Change to use the original drawable in glamor_fill.<o:p></o:p>      glamor: Fallback to software fb when repeat is needed.<o:p></o:p>      glamor-gles2: Add explicit precision qualifiers for gles2.<o:p></o:p>      glamor-egl: Move EGL surfaceless related code to dix module.<o:p></o:p>      glamor: Remove the version check for EGL/gles.<o:p></o:p>      glamor-egl: Fix direct call for glEGLImageTargetTexture2DOES.<o:p></o:p>      glamor: Change glamor_egl_init API.<o:p></o:p>      glamor: Move glamor_glyphs_init to create screen resources stage.<o:p></o:p>      glamor: Improve glyphs cache mechanism.<o:p></o:p>      glamor: Route UnrealizeGlyph to glamor_glyph_unrealize.<o:p></o:p>      glamor: Let GLAMOR_DDX implicit GLAMOR.<o:p></o:p>      glamor: Don't direct call to any gl functions.<o:p></o:p>      glamor: Re-arrange some macros/definitions in header files.<o:p></o:p>      glamor-egl: Added new function to create image for a pixmap.<o:p></o:p>      glamor-ddx: Change naming of glamor ddx module.<o:p></o:p>      glamor-ddx: Use gbm to allocate front buffer and cursor image.<o:p></o:p>      glamor-ddx: Fix the broken switching between VTs.<o:p></o:p>      glamor-egl: Add gbm device support.<o:p></o:p>      glamor-ddx: Reuse glamor_dix module to handle egl platform.<o:p></o:p>      glamor-ddx: Don't need to call preInit in screen_init.<o:p></o:p>      glamor-ddx: Destroy all the pixmap cursors when close current screen.<o:p></o:p>      glamor: Correct the version check for GLES2.<o:p></o:p>      ephyr-glamor: Set texture 0 to screen pixmap.<o:p></o:p>      ephyr-glamor:  Implment gl disptach function for host glx.<o:p></o:p>      glamor: Refine the screen pixmap initialization for Xephyr.<o:p></o:p>      glamor-ddx: Don't double free the cursor memory.<o:p></o:p>      glamor: Check libgbm if enable glamor ddx.<o:p></o:p>      glamor-ddx: Rename glamor.c to glamor_ddx.c.<o:p></o:p>      glamor: Don't call to glamor functions if it's dumb.<o:p></o:p>      glamor: Abort if fail to initialize glamor for xephyr.<o:p></o:p>      xephyr-glamor: Abort if use glamor over GLES2.<o:p></o:p>      glamor: Don't build glamor egl/gbm related code if GLAMOR_DDX disabled<o:p></o:p>      glamor: Fix style inconsistencies problem.<o:p></o:p>      xephyr-glamor: Change to use camelCase style.<o:p></o:p>      glamor-ddx: Change to use camelCase style.<o:p></o:p>      glamor: Change to use camelCase for glamor functions.<o:p></o:p>      glamor: Modify structure's name according to X coding style.<o:p></o:p>      glamor: Silence compilation warnings.<o:p></o:p><o:p> </o:p><o:p> </o:p></div></body></html>