[PATCH] DRI2: handle drawable destruction properly at DRI2SwapComplete time

Jesse Barnes jbarnes at virtuousgeek.org
Mon Jan 25 12:27:42 PST 2010


On Mon, 25 Jan 2010 20:21:12 +0100
Simon Thum <simon.thum at gmx.de> wrote:

> Jesse Barnes wrote:
> > Simon reported an issue with kwin that turned out to be a general
> > problem.  If a drawable goes away before its swap completes, we'll
> > try to free it up. However, we free it improperly, which causes a
> > server crash in DRI2DestroyDrawable.  Fix that up by splitting the
> > free code out and calling it from DRI2SwapComplete.
> > 
> > Tested-by: Simon Thum <simon.thum at gmx.de>
> Note I didn't test it, maybe you're missing out someone? But feel free
> to add my
>   Reviewed-by: Simon Thum <simon.thum at gmx.de>
> if there's no concurrency problem with
>   xfree(pPriv);
> being executed before
>   dixSetPrivate(&pWin->devPrivates, dri2WindowPrivateKey, NULL);
> 
> (In case the swap stuff is parallelized somehow)

Oh I must have misread the bug?  I thought you tested and then ran into
the kernel busy buffer bug...

Anyway thanks for looking at it.

-- 
Jesse Barnes, Intel Open Source Technology Center


More information about the xorg-devel mailing list