Bug 912457 - Kmix eats a lot of memory
Summary: Kmix eats a lot of memory
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: kmix
Version: 18
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Than Ngo
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-02-18 17:34 UTC by Jan Grulich
Modified: 2013-08-02 03:22 UTC (History)
7 users (show)

Fixed In Version: kmix-4.10.5-2.fc18
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-07-25 00:36:41 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
KDE Software Compilation 309464 0 None None None Never

Description Jan Grulich 2013-02-18 17:34:15 UTC
Description of problem:
Kmix eats a lot of memory after a long use. 

Version-Release number of selected component (if applicable):
kmix-4.10.0-1.fc18.x86_64

How reproducible:
Just use your computer for some time.
  
Actual results:
Kmix uses a lot of memory, even around 400MB.

Expected results:
Kmix uses normal amount of memory.

Comment 1 Rex Dieter 2013-02-18 17:59:43 UTC
For comparision, on my f18/x86_64 box, system activity reports for kmix:

Memory: 32,184k
Shared Mem: 10,604k

How are you measuring memory usage?

Comment 2 Jan Grulich 2013-02-18 21:59:09 UTC
I'm using ksysguard. I just found out that switching songs in Clementine increases memory usage of kmix. Now uses 70MB of memory.

Comment 3 Jan Grulich 2013-02-18 22:32:54 UTC
It seems it's a bug in PA, when I disabled it, kmix uses the same amount of memory.

Comment 4 Kevin Kofler 2013-03-10 18:29:32 UTC
This is not new in 4.10, is it? Can we take this off the blocker tracker?

Comment 5 Kevin Kofler 2013-03-20 23:05:57 UTC
Taking this off the 4.10 tracker.

Comment 6 Lukáš Tinkl 2013-07-01 14:02:45 UTC
Looks like http://commits.kde.org/kde-workspace/984b9c4ad6b35fa07019e4d4d083e07749482a29 has the right fix

Comment 7 Rex Dieter 2013-07-01 14:07:02 UTC
I pulled that in

* Wed Jun 26 2013 Rex Dieter <rdieter> 4.10.4-6
- kmix: media track change memory leaks with pulseaudio+oxygen widget style (kde#309464, #912457)


but, followup comment
https://bugs.kde.org/show_bug.cgi?id=309464#c100

based on a conversation I had with apachelogger, looks like that's not the correct fix afterall. :(

Comment 8 Jan Grulich 2013-07-01 14:15:56 UTC
It seems to be okay with Fedora 19. The reason is probably that F19 has newer version of PulseAudio. 

[root@fedora jgrulich]# rpm -q pulseaudio
pulseaudio-3.0-10.fc19.x86_64

Comment 9 Rex Dieter 2013-07-01 14:17:24 UTC
I still leaked for me, last I'd tested on f19 too.  Are you sure?

Comment 10 Jan Grulich 2013-07-01 14:24:37 UTC
I can switch songs arbitrarily and KMix still uses the same amount of memory (~ 19Mb). Before it grew what song to + ~0.5Mb.

Comment 11 Rex Dieter 2013-07-01 14:26:19 UTC
Just to be paranoid:
rpm -q kde-workspace kmix
please

and I'll retest myself (or anyone else, I'd like confirmation).

Comment 12 Jan Grulich 2013-07-01 14:28:19 UTC
[root@fedora jgrulich]# rpm -q kde-workspace kmix
kde-workspace-4.10.4-6.fc19.x86_64
kmix-4.10.4-1.fc19.x86_64

Comment 13 Rex Dieter 2013-07-01 14:31:55 UTC
right, see comment #7 where I pulled in (the apparently wrong) upstream fix.  apachelogger thinks that fix trades a memleak for an occasional crash. :(

Comment 14 Kevin Kofler 2013-07-01 22:13:23 UTC
I must say I agree with him, changing a deleteLater to a delete looks scary to me. It only makes sense when you really know what you're doing, which I had assumed the author of the patch did when I had seen the diff in the commit, but the patch has now been reverted upstream, so that assumption looks no longer valid. We should remove the patch ASAP.

The code was probably using deleteLater for a reason. In fact, the typical fix to use-after-free bugs is to change the delete to a deleteLater. I agree with apachelogger that we really need to find out why deleteLater is not working in KMix.

The offending patch is all the more scary as it's a global Oxygen change, not specific to KMix, so it can cause crashes all over KDE!

Comment 15 Fedora Update System 2013-07-16 14:26:02 UTC
kmix-4.10.5-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/kmix-4.10.5-2.fc19

Comment 16 Fedora Update System 2013-07-17 03:02:57 UTC
Package kmix-4.10.5-2.fc19:
* should fix your issue,
* was pushed to the Fedora 19 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing kmix-4.10.5-2.fc19'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-13061/kmix-4.10.5-2.fc19
then log in and leave karma (feedback).

Comment 17 Fedora Update System 2013-07-23 13:49:16 UTC
kmix-4.10.5-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/kmix-4.10.5-2.fc18

Comment 18 Fedora Update System 2013-07-25 00:36:41 UTC
kmix-4.10.5-2.fc19 has been pushed to the Fedora 19 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 19 Fedora Update System 2013-08-02 03:22:45 UTC
kmix-4.10.5-2.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.