Hi Denis, On 08/20/2018 12:40 AM, Denis Ryndine wrote: > Hello, > > The following may look like an error, could someone review. > > In rproc_elf_load_segments: > > /* grab the kernel address for this device address */ > ptr = rproc_da_to_va(rproc, da, memsz); > > The last parameter should be filesz. Otherwise this call may fail, as > the case when da is an address within a segment / memory. > (E.g. placing a RO data after the code within text segment /memory ). No, it's alright, memsz >= filesz usually. The actual loadable content will be filesz, the rest is zero initialized. Both these are from the program headers. Have you seen some issues around this? regards Suman
