diff options
author | cinap_lenrek <cinap_lenrek@felloff.net> | 2020-04-10 23:11:25 +0200 |
---|---|---|
committer | cinap_lenrek <cinap_lenrek@felloff.net> | 2020-04-10 23:11:25 +0200 |
commit | 1d3644a1684bd191f3a36568d94c2587c2d9f24f (patch) | |
tree | d846fb9fe81f38078db378a7264a9f02321f477d /sys/src/cmd/6l | |
parent | e610ffaf1b7847e82c18a1b9c687cdd7676f86c2 (diff) |
cc, ?l: fix gethunk() to actually grow allocation
the compilers and linkers use ther own memory allocator.
free memory is between hunk and hunk+nhunk. allocation
works by checking if nhunk is bigger or equal to the
amount needed, and if not, repeatedly call gethunk()
until there is. after that, the allocated amount is added
from hunk and subtracted from nhunk by the user.
the problem was when the needed amount was bigger than
the default NHUNK size gethunk() allocates per call.
gethunk() would not actually grow nhunk, but instead
just set hunk and nhunk variables to the last allocated
block. this resulted in a infinite loop of calls to
gethunk() until sbrk() would hit the maximum size for
the BSS segment.
this change makes gethunk() actually grow the hunk space,
increasing nhunk, and only updating hunk when nhunk was
previously zero. we assume that mysbrk() retuns increasing
addresses and that the space between the previous hunk+nhunk
and the new block base returned by mysbrk() is usable.
Diffstat (limited to 'sys/src/cmd/6l')
-rw-r--r-- | sys/src/cmd/6l/obj.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/sys/src/cmd/6l/obj.c b/sys/src/cmd/6l/obj.c index 5483eebae..e4e8a44fb 100644 --- a/sys/src/cmd/6l/obj.c +++ b/sys/src/cmd/6l/obj.c @@ -1222,8 +1222,11 @@ gethunk(void) diag("out of memory"); errorexit(); } - hunk = h; - nhunk = nh; + if(nhunk == 0) + hunk = h; + else + nh += (h - hunk) - nhunk; + nhunk += nh; thunk += nh; } |