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

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

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

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 >>

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 >>

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

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

[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