[Merge] lp:~rsalveti/ubuntu/maverick/ureadahead/fix-600359 into lp:ubuntu/maverick/ureadahead
Evan Broder
evan at ebroder.net
Thu Nov 25 08:09:15 GMT 2010
You have been requested to review the proposed merge of lp:~rsalveti/ubuntu/maverick/ureadahead/fix-600359 into lp:ubuntu/maverick/ureadahead.
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.
--
https://code.launchpad.net/~rsalveti/ubuntu/maverick/ureadahead/fix-600359/+merge/32955
Your team Ubuntu Development Team is requested to review the proposed merge of lp:~rsalveti/ubuntu/maverick/ureadahead/fix-600359 into lp:ubuntu/maverick/ureadahead.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 1567 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20101125/1d17f7af/attachment.diff
More information about the Ubuntu-reviews
mailing list