> -----Original Message----- > From: Chris Wilson [mailto:chris at chris-wilson.co.uk] > Sent: Wednesday, January 04, 2012 5:47 AM > To: zhigang.gong at linux.intel.com > Cc: intel-gfx at lists.freedesktop.org; zhigang.gong at gmail.com; > zhigang.gong at linux.intel.com > Subject: Re: [PATCH 0/1] uxa/glamor: Route some drawing function to > glamor. > > 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? Exactly. That's the hardest part now. There are two DRI loaders, one is AIGLX and the other is EGL. Both loader have their own glapi.c. Currently, the AIGLX always get loaded before the DDX driver, and thus the glapi in glx will be used. One possible solution is as below, please help to review and comment: let the glamor egl module get loaded prior to the AIGLX And load the proper libGL.so thus the egl will create a complete and correct context. And then latter, we can call _glapi_create_table_from_handle to initialize the glx's dispatch structure. We can treat the context created by egl for glamor is the context dedicated for server client, and each time when call into glamor's rendering path, force current to glamor's context and restore to previous context at returning. Copy to KRH, I know you are the master of DRI2/AIGLX. Any suggestion here? Thanks. > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre
