On Wed, 1 Feb 2012 19:47:28 +0800, zhigang.gong at linux.intel.com wrote: > From: Zhigang Gong <zhigang.gong at linux.intel.com> > > The previous version calls glamor_egl_close_screen and > glamor_egl_free_screen manually which is not align with > standard process. Now glamor change the way to follow > standard method: > > glamor layer and glamor egl layer both have their internal > CloseScreens. The correct sequence is after the I830CloseScreen > is registered, then register glamor_egl_close_screen and > the last one is glamor_close_screen. So we move out the > intel_glamor_init from the intel_uxa_init to I830ScreenInit > and just after the registration of I830CloseScreen. > > As the glamor interfaces changed, we need to check the > glamor version when load the glamor egl module to make > sure we are loading the right glamor module. If > failed, it will switch back to UXA path. > > Signed-off-by: Zhigang Gong <zhigang.gong at linux.intel.com> I added the note about the glamor commit to this and the following patch and pushed. Thanks, -Chris -- Chris Wilson, Intel Open Source Technology Centre