On 28.07.2016 18:34, Michel Dänzer wrote: > Running XTS (the X test suite) on Xephyr using glamor running on Xorg, > I stumbled upon an issue: > > * XTS causes Xephyr to keep resetting and starting new server generations > * glamor creates a new GLX context for each server generation but re-uses > the same window > > This caused Xorg to accumulate stale Present event contexts from previous > GLX contexts, so it kept getting slower due to sending out an increasing > number of copies of the same Present events to Xephyr. > > The first patch fixes this problem, and the other two patches apply the > same principle to the other places using Present event contexts. > > Note that destroying a Present event context doesn't actually work > without this xserver fix: > > https://patchwork.freedesktop.org/patch/101430/
v2 of that fix has landed in xserver master. > But these changes shouldn't make things any worse than they already > are with X servers lacking that fix. > > [PATCH 1/3] loader/dri3: Destroy Present event context when > [PATCH 2/3] vl/dri3: Destroy Present event context when destroying > [PATCH 3/3] anv/x11: Destroy Present event context when destroying Any comments on the v2 patches? Unless there are any objections, I'm going to push v2 of patches 1 & 2 later this week. I'm not going to push patch 3 myself without getting at least a Tested-by: from somebody. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
