Page MenuHomeFreeBSD
Differential D50944

bsdinstall: Emit a warning if the system has too little memory
ClosedPublic
Actions

Authored by emaste on Thu, Jun 19, 7:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jul 7, 1:06 PM
Unknown Object (File)
Sun, Jul 6, 8:36 PM
Unknown Object (File)
Sat, Jul 5, 9:29 PM
Unknown Object (File)
Sat, Jul 5, 3:34 PM
Unknown Object (File)
Sat, Jul 5, 11:01 AM
Unknown Object (File)
Sat, Jul 5, 9:06 AM
Unknown Object (File)
Sat, Jul 5, 3:47 AM
Unknown Object (File)
Thu, Jul 3, 9:27 PM
View All 9 Files
Subscribers
adrian
imp
jrtc27

Details

Reviewers
adrian
Group Reviewers
Installer
Commits
rGeb5884c564ae: bsdinstall: Emit a warning if the system has too little memory
Summary

PR: 251993
Event: Kitchener-Waterloo Hackathon 202506
Sponsored by: The FreeBSD Foundation

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste created this revision.Thu, Jun 19, 7:49 PM
Herald added a subscriber: jrtc27. · View Herald TranscriptThu, Jun 19, 7:49 PM
emaste requested review of this revision.Thu, Jun 19, 7:49 PM
adrian added a subscriber: adrian.Thu, Jun 19, 7:51 PM

I'd at least put a comment into here as to why we've chosen this. Eg, what I just asked about in IRC and you said "pkgbase OOM's on 256mb RAM".

emaste updated this revision to Diff 157327.Thu, Jun 19, 7:54 PM

Add a comment

adrian accepted this revision.Thu, Jun 19, 8:19 PM

oh and heh, maybe add what the minimum should be in the dialog, so people don't have to guess?

This revision is now accepted and ready to land.Thu, Jun 19, 8:19 PM
emaste updated this revision to Diff 157335.Thu, Jun 19, 8:49 PM

Show recommended minimum too

This revision now requires review to proceed.Thu, Jun 19, 8:49 PM
emaste updated this revision to Diff 157371.Fri, Jun 20, 3:06 PM

Reduce installer memory requirement to 256 MiB (QEMU's default was 128 MiB where the pkg OOM was observed).

adrian accepted this revision.Fri, Jun 20, 6:54 PM
This revision is now accepted and ready to land.Fri, Jun 20, 6:54 PM
Closed by commit rGeb5884c564ae: bsdinstall: Emit a warning if the system has too little memory (authored by emaste). · Explain WhyFri, Jun 20, 7:23 PM
This revision was automatically updated to reflect the committed changes.
emaste added a commit: rGeb5884c564ae: bsdinstall: Emit a warning if the system has too little memory.
Herald added a subscriber: imp. · View Herald TranscriptFri, Jun 20, 7:23 PM

Revision Contents
Changeset List

PathSize
usr.sbin/
bsdinstall/
scripts/
8 lines

Diff 157371

View Options

usr.sbin/bsdinstall/scripts/auto

Loading...