On Sat, 31 Dec 2011 21:18:14 +0800, zhigang.gong at linux.intel.com wrote: > From: Zhigang Gong <zhigang.gong at linux.intel.com> > > I agree to pending the last patch which is to create glamor pixmap by default. > This patch is based on the other 3 patch. After apply this patch, and use > the latest glamor. I've pushed those 4 patches (all bar to use intel_glamor_create_textured_pixmap by default). I've a new segfault for you in glamor_core.c::glamor_validate_gc(), line 425 as pixmap_priv is NULL, which I only hit just as I thought I had completed testing the code. Afaics, the remaining big topics for the ddx are how to enable glx and integration between glamor/uxa render paths, right? -Chris -- Chris Wilson, Intel Open Source Technology Centre