Merge lp:~rsalveti/ubuntu/maverick/ureadahead/fix-600359 into lp:ubuntu/maverick/ureadahead
Status: | Rejected |
---|---|
Rejected by: | Dave Walker |
Proposed branch: | lp:~rsalveti/ubuntu/maverick/ureadahead/fix-600359 |
Merge into: | lp:ubuntu/maverick/ureadahead |
Diff against target: |
46 lines (+19/-0) 3 files modified
conf/ureadahead-other.conf (+6/-0) conf/ureadahead.conf (+6/-0) debian/changelog (+7/-0) |
To merge this branch: | bzr merge lp:~rsalveti/ubuntu/maverick/ureadahead/fix-600359 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dave Walker (community) | Disapprove | ||
Evan Broder (community) | Needs Information | ||
Review via email: mp+32955@code.launchpad.net |
Commit message
* conf/ureadahead
Description of the change
Changing ureadahead initscript to avoid running ureadahead at systems with less than 512 MB of ram.
This is needed because ureadahead sets the tracing buffer to 128MB, and at a system with small amount of memory (Beagleboard) this can cause the boot to call the OOM, killing ureadahead and plymouthd.
Check bug 600359 for more details.
I decided to change only the initscript, instead of adding this restriction to the ureadahead code, because then the user can still run and test it if needed.
I also tried different memory thresholds, but didn't change the boot speed when comparing different bootcharts for BeagleBoard, so I just decided to avoid running it in this case.
Unmerged revisions
- 12. By Ricardo Salveti
-
conf/ureadahead
*.conf: only run ureadahead on systems with at least
512MB of ram. (LP: #600359)
This merge proposal references a bug which appears to have been fixed through an alternative approach. Is this merge still needed? If it is needed, please rebase the patch on the current version of lp:ubuntu/ureadahead. If it is no longer needed, please let us know so that we can remove this proposal, or feel free to delete the proposal yourself.