FS#40966 - [volumeicon] 100% cpu if soundcard is disconnected (+patch)

Attached to Project: Community Packages
Opened by Steven Honeyman (stevenhoneyman) - Tuesday, 24 June 2014, 20:38 GMT
Last edited by Alexander F. Rødseth (xyproto) - Thursday, 03 July 2014, 23:39 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Alexander F. Rødseth (xyproto)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

If you unplug your soundcard, volumeicon goes into an infinite loop when ALSA can't see the card. This causes CPU usage to go to 100%
(bug is also in 'official release 0.5.0' - the code responsible for this hasn't changed at all even in Git release)

Steps to reproduce:

1. Start volumeicon (any version)
2. Unplug soundcard
3. Watch CPU go to 100%

Fix:

As upstream seems dead (no reply to my github pull request, or email(s) even from months ago), I've attached an incredibly simple patch which fixes this problem.
The author just forgot to pass G_IO_ERR status to his handle_events function, so without this patch, it never sees the error!

I know Arch policy isn't to patch code... but unfortunately, upstream is gone :(
   volumeicon-fix_100_cpu_bug.pa... (1.3 KiB)
This task depends upon

Closed by  Alexander F. Rødseth (xyproto)
Thursday, 03 July 2014, 23:39 GMT
Reason for closing:  Fixed
Comment by Alexander F. Rødseth (xyproto) - Saturday, 28 June 2014, 09:10 GMT
Thanks for reporting! What a pity that upstream is unresponsive. Volumeicon was specifically introduced to overcome the shortcomings of the previous volume icon tray application. Will apply the patch and fix this. Suggestions for better volume control applications that should rather be in community are welcome.
Comment by Steven Honeyman (stevenhoneyman) - Saturday, 28 June 2014, 12:40 GMT
I've spent half of the morning looking, and unfortunately the best I can find (apart from Volumeicon) is "PNMixer" https://github.com/nicklan/pnmixer
It's only Gtk2 supported, but some recent commits on Git. I've raised a few tickets for obvious bugs, so will see if they get a reply or not

It is a real shame. Everything else is either meant for a bloated DE, a dead project, or is just a messy Python script etc.

I've replied to your other comment in the related bugreport too (patches applied to my github fork)

Thanks,
Steven.
Comment by Steven Honeyman (stevenhoneyman) - Saturday, 28 June 2014, 12:56 GMT
There is this also, but it's a little *too* lightweight: https://github.com/Unia/gvolicon
Comment by Alexander F. Rødseth (xyproto) - Thursday, 03 July 2014, 23:32 GMT
Updated the package to use the stevenhoneyman/volumeicon repo. The updated package will appear in [community] shortly. Please re-open this bug if there are still problems with this.

Loading...