Review Request 127072: WindowThumbnail: Discard glx pixmap in stopRedirecting()

[David Rosca]

> On Feb. 15, 2016, 7:11 a.m., Martin Gräßlin wrote:
> > do you know what exactly crashes if discard is not called? I just tried to read the code and also compare it to KWin's code to try to understand why it's needed. From all I can see I don't find a reason why it should crash. The pixmap should still be valid, etc. etc.
> > 
> > Comparing to KWin was tricky as it doesn't composite each window individually, but uses composite of root window. The only relatively related code is the fullscreen unredirection. And there interestingly on unredirection it doesn't call into discardPixmap().
> > 
> > If you don't know that fine and I'll give you shipit. I just wanted to understand ;-)

Sorry, no idea at all. It crashes only with swrast_dri, so probably bug there?


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127072/#review92376
-----------------------------------------------------------


On Feb. 13, 2016, 11:23 p.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127072/
> -----------------------------------------------------------
> 
> (Updated Feb. 13, 2016, 11:23 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Bugs: 357895
>     http://bugs.kde.org/show_bug.cgi?id=357895
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> BUG: 357895
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/windowthumbnail.cpp 21e655e 
> 
> Diff: https://git.reviewboard.kde.org/r/127072/diff/
> 
> 
> Testing
> -------
> 
> No more crash
> 
> 
> Thanks,
> 
> David Rosca
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160215/9cb57e32/attachment.html>