"Nathan J. Froyd" <froydnj@...> writes:
> The second implements one of the FIXMEs in src/runtime/gencgc.c by
> replacing all instances of the bare 4096 with the symbolic constant
Thanks for your work here. I'm reluctant to apply this in its present
form though, because it may mislead future maintainers into believing
that they can change the gencgc page size by tweaking this number. In
fact there are a number of other places that need fixing too - look
for e.g. logical masking operations here and there.
If you can come up with a patch such that rebuilding SBCL with
PAGE_BYTES != 4096 does not cause gencgc to fall over in a pile
(afaics it doesn't have to be the hardware page size[*], some multiple
thereof should be OK), I'll gladly take it ;-)
[*] where by "hardware page size" in this instance I mean "smallest
meaningful unit for mprotect"
http://www.cliki.net/ - Link farm for free CL-on-Unix resources