venti: make error messages consistent, remove duplicate messages.

removes duplicate messaes about restrictions, and duplicate
checks, makes the error match the rest of the errors.
This commit is contained in:
Ori Bernstein 2021-06-21 12:29:52 +00:00
parent 068f403e71
commit 214dc1ec6c

View file

@ -55,20 +55,17 @@ freemem(void)
Bterm(bp); Bterm(bp);
if (pgsize > 0 && userpgs > 0 && userused > 0) if (pgsize > 0 && userpgs > 0 && userused > 0)
size = (userpgs - userused) * pgsize; size = (userpgs - userused) * pgsize;
} else { } else
fprint(2, "%s: failed to open /dev/swap\n", argv0); fprint(2, "%s: failed to open /dev/swap\n", argv0);
}
/* cap it to keep the size within 32 bits */ /* cap it to keep the size within 32 bits */
if (size >= 3840UL * 1024 * 1024){
size = 3840UL * 1024 * 1024;
fprint(2, "%s: Reduced free memory detected to 3840MiB because we don't support 64-bit addresses yet.\n", argv0);
}
/* FIXME: we use signed 32-bit integers in some places for some fucking reason. /* FIXME: we use signed 32-bit integers in some places for some fucking reason.
Limiting accordingly for now. * Limiting accordingly for now.
*/ * if (size >= 3840UL * 1024 * 1024)
if (size >= 2047UL * 1024 * 1024){ * size = 3840UL * 1024 * 1024;
*/
if (size >= 2047UL * 1024 * 1024) {
size = 2047UL * 1024 * 1024; size = 2047UL * 1024 * 1024;
fprint(2, "%s: Reduced free memory detected to 2047MiB because we have bugz.\n", argv0); fprint(2, "%s: mem pct overflows: restricting to 2047MiB\n", argv0);
} }
return size; return size;
} }
@ -110,7 +107,7 @@ allocbypcnt(u32int mempcnt, u32int stfree)
else { else {
if (avail >= 2047UL * 1024 * 1024){ if (avail >= 2047UL * 1024 * 1024){
avail = 2047UL * 1024 * 1024; /* sanity */ avail = 2047UL * 1024 * 1024; /* sanity */
fprint(2, "%s: restricting memory usage to 2047MiB\n", argv0); fprint(2, "%s: mem pct overflows: restricting to 2047MiB\n", argv0);
} }
avail /= 2; avail /= 2;
all.icmem = avail; all.icmem = avail;