Bug 254498

Summary: snow effect becomes very slow while present window effect is active
Product: [Plasma] kwin Reporter: g111
Component: compositingAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version: git master   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:

Description g111 2010-10-17 21:49:00 UTC
Version:           SVN (using Devel) 
OS:                Linux

When you activate the present windows effect (e.g. via Ctrl+F9) the snow effect becomes very slow. I am just reporting this because this might be a general performance issue with the present windows effect that only can be seen in this case?

Reproducible: Didn't try

Steps to Reproduce:
Activate the snow effect.
Activate the present windows effect.

Actual Results:  
The snow effect becomes very slow.

Expected Results:  
The snow should fall with normal speed.

4.5.72 (4.6 >= 20101012)

btw: The bugzilla severity entry "Minor: It's a bug a trivial bug to find and fix" should be renamed. a) IMHO there is a typo in this: "a bug" is too much. b) I cannot decide if it is trivial to fix. It should be named "Minor: It's a bug that is not that important"
Comment 1 Martin Flöser 2011-01-31 22:06:20 UTC
Git commit 3e8c9a65e2b165cd153ac15ccb59e30880de9258 by Martin Gr����lin.
Pushed by graesslin into branch 'graesslin/kwin-cleanup'.

Removing Snow Effect.

The usefulness of the snow effect is doubtable and it is rather
difficult to port the effect to the new rendering architecture
introduced with GLES. It is probably easier to rewrite the complete
effect from scratch, therefore it is removed for now.

I might consider readding it for 4.7.

This kind of "fixes" snow related bug reports.
BUG: 259362
BUG: 254498
FIXED-IN: 4.7.0

M  +0    -3    kwin/effects/CMakeLists.txt     
M  +0    -6    kwin/effects/configs_builtins.cpp     
D  +0    -33   kwin/effects/snow/CMakeLists.txt     
D  +0    -33   kwin/effects/snow/data/snow.frag     
D  +0    -47   kwin/effects/snow/data/snow.vert     
D  +0    -450  kwin/effects/snow/snow.cpp     
D  +0    -171  kwin/effects/snow/snow.desktop     
D  +0    -107  kwin/effects/snow/snow.h     
D  +0    -138  kwin/effects/snow/snow_config.cpp     
D  +0    -86   kwin/effects/snow/snow_config.desktop     
D  +0    -59   kwin/effects/snow/snow_config.h     
D  +0    -200  kwin/effects/snow/snow_config.ui     

http://commits.kde.org/a5d5b61a/3e8c9a65e2b165cd153ac15ccb59e30880de9258