Cspeed: increase system responsiveness using cgroups

Question

Recently, there was a much-discussed kernel patch that grouped processes by the TTY/terminal they were started from so your system got extremely responsive under pressure. Then, a .bashrc implementation was made that effectively does the same thing in only a couple lines of code. I decided to take that implementation (as modified at WebUpd8) and put it in a bash daemon with options.
How it works:
Linux 2.4 added a feature called "cgroups" which let you limit the resource usage of processes by prioritization, isolation, etc. For more information, see the Wikipedia article and the ArchWiki article on it. This script uses the WebUpd8 version of a bashrc modification that uses cgroups to increase your system's responsiveness under lots of pressure (for example, compiling a kernel with 'make -j64'). Note that it WILL NOT work for processes launched from X, like in your desktop environment. You'll need to open up a new terminal window or switch to a different TTY to see a difference in responsiveness.
Instructions:
After installing, if you use Arch Linux, add "cspeed" to your daemons array in /etc/rc.conf. Then, add the following line per-user in /home/user/.bashrc or system-wide at /etc/bash.bashrc:
/bin/bash /etc/bash.cgroups
Download:
AUR: http://aur.archlinux.org/packages.php?ID=43873
Github: https://github.com/sloshy/cspeed
Any bugs, suggestions, anything would be greatly appreciated !
Last edited by Sloshy (2010-12-01 18:01:29)

Answer 1

hbekel wrote:It seems you wanted to provide an easy way of enabling cgroups. That's ok of course, but it's not really the arch way. I personally wouldn't use it. Setting up cgroups isn't that hard after all, it's just a few lines...
Yeah, it's not so hard. This is more for the lazy people who don't want to modify this on their own and get it up-and-running quickly without any PEBKAC errors.
hbekel wrote:
Here are some problems I see:
- Modifying system wide config files... ouch. A big no no, at least for me. This type of hand-holding just isn't arch. Especially when you're adding to /etc/rc.local when you should at least have used an rc-script for that.
I'm going to be honest here: I'm rather new to Linux. I've only used it for a few years now, and I've used Arch for less than one. So, I wasn't sure what those were until you mentioned them right now. I'll transition it to an rc-script over the weekend.
hbekel wrote:- Dynamically creating/deleting a script below /usr/local/sbin without obvious reasons: the file is static, so it could simply be distributed (with the benefit of being properly tracked by a package manager). Note that the use of /usr/local/* is discouraged. No sane arch package installs anything there, and no arch user expects anything there unless he put it there himself.
Yeah, I'll fix that. I wasn't thinking when I made my script and didn't just pre-make this file. Thanks for pointing this out!
hbekel wrote:- Testing for a verbatim line in a config file to determine whether you need to edit it isn't very reliable... what if the user accidently leaves a space somewhere on your "test line" while editing the file later on? (Interestingly, you could have tested for the existence of the cgroup_clean script, which could have justified creating it dynamically... but that would've been a hack as well)
Hmm, good point. Maybe I could append "#cspeed" before the lines that are supposed to be there; would that work?
hbekel wrote:- Using inplace editing with sed to remove those changes is just... creepy. I'd never sed -i any important file, let alone a system config file.
Yeah, this has the potential to screw up badly. As I just said, maybe "#cspeed" could help with that.
hbekel wrote:Some bugs/oversights: It seems you forgot a ">>" on line 80, at least that seems to be your intent there. The undo function also edits /etc/skel/.bashrc, which you never touch during -s.
Good call! I did forget that; my mistake. And As for the /etc/skel/.bashrc edit, that was also a mistake; an earlier version of the script, before I pushed it to github, modified each individual user's .bashrc (and set it as the default for all new users) which, as you can imagine, is super messy. I'll remove the lines you mention .
Thank you so much for your comments; I'll fix my script up right away!
Last edited by Sloshy (2010-11-26 18:05:56)