[Intel-gfx] [Glamor] [PATCH v3] uxa/glamor: Create glamor pixmap by default.

Chris Wilson chris at chris-wilson.co.uk
Mon Jan 16 11:54:33 CET 2012


On Thu, 12 Jan 2012 15:30:15 +0800, zhigang.gong at linux.intel.com wrote:
> From: Zhigang Gong <zhigang.gong at linux.intel.com>
> 
> Chris,
> 
> According to your previous comments. The X server will not die when
> fail to get a dri buffer for a glamor pixmap, instead it will return
> a BadAlloc. Your previous advice is to pass a BadMatch to the client,
> and I traced into X's dri2 module and found current code can only
> return a BadAlloc. IMHO, this should be ok. Right?

That looks to be the best solution in line with the existing failure
mechanisms along that path.
 
> Another change is to disable the reuse of bo for glamor textured
> pixmap.
> 
> The last change is reduce the performance gain. ;), I just found
> another 10-20% is come from my latest glamor patchset. 

Hmm, that does suggest that non-native glamor pixmaps were still in use
and that they were not being marked non-reusable i.e. they weren't DRI
pixmaps.

The patch looks good to me, so I've pushed it to master, thanks.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list