Bug 28712

Summary:	Patches to reduce 3ddeskd's memory-footprint dramatically and fix a minor bug
Product:	Gentoo Linux	Reporter:	Christoph Richter <toph>
Component:	Current packages	Assignee:	Desktop Misc. Team <desktop-misc>
Status:	RESOLVED FIXED
Severity:	minor	CC:	magnet, moixa, pau
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	The patch to fix the bug, cut down mem-consumption and make the use of compressed textures possible patch for the 0.2.5 ebuild, implements the patch from first attachment

Description Christoph Richter 2003-09-14 10:05:25 UTC

I've hacked around a bit with this piece of software, to reduce the whopping
>40MB memory-consumption of 3ddeskd down to about 17MB. This is done by 2 things:
 - free some imlib-objects that are only needed once, after the image
transformation is done. Which saves ~10MB at no cost.
 - introduce texture-compression, which saves another 17MB at the default
texture-size. Gets only activated if you put "compression 1" into your
3ddesktop.conf

It also fixes a minor bug regarding the --acquire=... option. The timing should
work more reliable now. 3ddeskd hung for me, when started with --acquire=X and X>99

Reproducible: Always
Steps to Reproduce:
1. Invoke 3ddeskd like this:
> 3ddeskd --acquire=100, or something >100


Actual Results:  
3ddeskd starts, acquires the first desktop and then hangs

Expected Results:  
3ddeskd should acquire all my desktops with the given delay.

(and shouldn't use more resident memory than my X-Server) :)

The patch is against 3ddesktop-0.2.5

Comment 1 Christoph Richter 2003-09-14 10:06:37 UTC

Created attachment 17704 [details, diff]
The patch to fix the bug, cut down mem-consumption and make the use of compressed textures possible

Comment 2 Tobias Sager 2004-01-09 09:40:27 UTC

Just as a sidenote: make sure you sent the patch to the main developer too..
(would be bard at systemtoolbox.com according to README)

Comment 3 Tobias Sager 2004-01-09 09:59:24 UTC

Created attachment 23494 [details, diff]
patch for the 0.2.5 ebuild, implements the patch from first attachment

Works for me.
Copy patch into files dir, named 3ddeskMEM+Compression+Fix-0.2.5.patch

Comment 4 Christoph Richter 2004-01-10 04:06:13 UTC

Of course I've sent this patch to the main developer, too. I think it's in his CVS-tree. But Brad seems to be really busy and a new release is not in sight.

Comment 5 Matt Keadle 2004-04-01 07:24:22 UTC

fwiw, i've applied the patch and rebuilt locally and it builds clean and does seem to run a bit smoother than before.

cheers

Comment 6 Brandon Hale (RETIRED) gentoo-dev

2004-04-13 08:44:33 UTC

In cvs, thanks