[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