#933 closed Enhancement (invalid)
[PATCH] client apps memory leaks cause OOM crashes (w/ patch)
| Reported by: | misterc | Owned by: | davea |
|---|---|---|---|
| Priority: | Minor | Milestone: | Undetermined |
| Component: | Client - Daemon | Version: | |
| Keywords: | patch oom oom_adj debian | Cc: |
Description
From my Debian bug http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=534418 (reposted here as they lack active maintainers):
Client apps can have crufty code leading to OOM conditions. The attached patch prevents this from taking down the system by configuring oom_adj to favor removal.
Yes, this is a patch for the Debian init script, but should be easily convertible for the universal init script.
Attachments (1)
Change History (7)
Changed 15 years ago by
| Attachment: | boinc-client.patch added |
|---|
comment:1 Changed 14 years ago by
| Summary: | client apps memory leaks cause OOM crashes (w/ patch) → [PATCH] client apps memory leaks cause OOM crashes (w/ patch) |
|---|
comment:2 Changed 14 years ago by
comment:3 follow-up: 5 Changed 14 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
This patch is for init scripts used by distribution packages. I first thought BOINC also uses an init script for starting the Client, but apparently it does not. So this is invalid here.
comment:4 Changed 14 years ago by
It would be good to collect some Linux init scripts, put them in the BOINC repository, and link to them from the wiki (I couldn't find any such links).
comment:5 Changed 14 years ago by
Replying to ChristianB:
This patch is for init scripts used by distribution packages. I first thought BOINC also uses an init script for starting the Client, but apparently it does not. So this is invalid here.
The patch was for the Debian init script, but the point of this ticket was to get something similar applied to the init script in the BOINC repository.
comment:6 Changed 14 years ago by
Links to init scripts are in the user wiki: http://boinc.berkeley.edu/wiki/Starting_BOINC_on_boot_%28Unix%29
I don't think we need additional init scripts in the repository (they have to be maintained, which is difficult). This should be kind of self regulated through the community and the page above.
what file does this patch apply to?