Re: [PATCH v2] semihosting/arm-compat: remove heuristic softmmu SYS_HEAPINFO

2021-06-13 Thread Peter Maydell
On Fri, 11 Jun 2021 at 18:03, Alex Bennée wrote: > > > Peter Maydell writes: > > > On Thu, 10 Jun 2021 at 15:16, Alex Bennée wrote: > >> > >> > >> Peter Maydell writes: > >> > (2) find the largest contiguous extent of that RAM which > >> > is not covered by a ROM blob, by iterating throug

Re: [PATCH v2] semihosting/arm-compat: remove heuristic softmmu SYS_HEAPINFO

2021-06-11 Thread Alex Bennée
Peter Maydell writes: > On Thu, 10 Jun 2021 at 15:16, Alex Bennée wrote: >> >> >> Peter Maydell writes: >> > I'm told that the Arm C compiler C library always assumes that >> > the "stack base" value is what it should set SP to, so reporting 0 >> > for that will break binaries that were built

Re: [PATCH v2] semihosting/arm-compat: remove heuristic softmmu SYS_HEAPINFO

2021-06-10 Thread Peter Maydell
On Thu, 10 Jun 2021 at 15:16, Alex Bennée wrote: > > > Peter Maydell writes: > > I'm told that the Arm C compiler C library always assumes that > > the "stack base" value is what it should set SP to, so reporting 0 > > for that will break binaries that were built with it. > > > > As the TODO comm

Re: [PATCH v2] semihosting/arm-compat: remove heuristic softmmu SYS_HEAPINFO

2021-06-10 Thread Alex Bennée
Peter Maydell writes: > On Thu, 10 Jun 2021 at 11:26, Alex Bennée wrote: >> >> The previous numbers were a guess at best. While we could extract the >> information from a loaded ELF file via -kernel we could still get >> tripped up by self decompressing or relocating code. Besides sane >> libr

Re: [PATCH v2] semihosting/arm-compat: remove heuristic softmmu SYS_HEAPINFO

2021-06-10 Thread Alex Bennée
Peter Maydell writes: > On Thu, 10 Jun 2021 at 11:26, Alex Bennée wrote: >> >> The previous numbers were a guess at best. While we could extract the >> information from a loaded ELF file via -kernel we could still get >> tripped up by self decompressing or relocating code. Besides sane >> libr

Re: [PATCH v2] semihosting/arm-compat: remove heuristic softmmu SYS_HEAPINFO

2021-06-10 Thread Peter Maydell
On Thu, 10 Jun 2021 at 11:26, Alex Bennée wrote: > > The previous numbers were a guess at best. While we could extract the > information from a loaded ELF file via -kernel we could still get > tripped up by self decompressing or relocating code. Besides sane > library code like newlib will fall ba

Re: [PATCH v2] semihosting/arm-compat: remove heuristic softmmu SYS_HEAPINFO

2021-06-10 Thread Peter Maydell
On Thu, 10 Jun 2021 at 11:26, Alex Bennée wrote: > > The previous numbers were a guess at best. While we could extract the > information from a loaded ELF file via -kernel we could still get > tripped up by self decompressing or relocating code. Besides sane > library code like newlib will fall ba

[PATCH v2] semihosting/arm-compat: remove heuristic softmmu SYS_HEAPINFO

2021-06-10 Thread Alex Bennée
The previous numbers were a guess at best. While we could extract the information from a loaded ELF file via -kernel we could still get tripped up by self decompressing or relocating code. Besides sane library code like newlib will fall back to known symbols to determine of the location of the heap