On Fri, Jun 12, 2020 at 08:55:21AM +0200, Clément Leger wrote:
> Hi Oleksij,
>
> ----- On 12 Jun, 2020, at 07:22, Oleksij Rempel o.rempel@xxxxxxxxxxxxxx wrote:
>
> > Hi Clement,
> >
> > suddenly I was not able to apply you changes on current barebox/master
> > branch. There seems to be some missing changes?
>
> Sorry, I forgot the '^' on the base sha1 during my git format-patch.
> So patch 1 adding elf boundary computation is missing in this serie.
> Should I resend a V5 [RESEND] or a V6 ?
Better V6, to avoid confusion.
> >
> > For example, you patch is changing this struct:
> > @@ -405,7 +405,8 @@ struct elf_image {
> > u64 entry;
> > void *low_addr;
> > void *high_addr;
> > - void *buf;
> > + void *hdr_buf;
> > + char *filename;
> > };
> >
> > But, current version of it is:
> > struct elf_image {
> > struct list_head list;
> > u8 class;
> > u64 entry;
> > void *buf;
> > };
> >
> > On Thu, Jun 11, 2020 at 10:02:00AM +0200, Clement Leger wrote:
> >> Currently, when booting an elf file using "bootm /dev/mtdx", bootm will
> >> simply pass the file to the bootm handler and the read done on it will
> >> read the entire flash partition. This series starts by some cleanup and
> >> then modify the elf loader to load the elf file without copying the whole
> >> elf file to an intermediate buffer. Only the elf header is copied in
> >> order to parse the elf file into a list of segment and then, the segments
> >> are loaded directly from the file.
> >>
> >> A special handling for the elf file is also added in bootm data to allow
> >> using directly the elf file structure. Finally the mips bootm handler is
> >> modified to use bootm_load_os directly instead of manual elf loading.
> >>
> >> Compilation for both mips and arm has been tested but run on qemu-malta was not
> >> possible. Changes have been tested on kvx architecture for which bootm support
> >> has been added and will be submitted.
> >>
> >> Changes v4 -> v5
> >> - Rework elf loading to load from file rather than a buffer
> >> - Use calloc instead of xzalloc in elf file loading
> >> - Add check of program headers in elf check
> >>
> >> Changes v3 -> v4
> >> - Fix init of elf entry address to be used by bootm_load_elf
> >>
> >> Changes v2 -> v3
> >> - Integrate elf loading in bootm_load_os
> >> - Add patch to remove now unused elf_load_image/elf_release_image
> >> - Use malloc instead of xmalloc and check return value
> >>
> >> Changes v1 -> v2
> >> - Add BOOTM_ELF config to select elf support and add checks in code
> >> - Add an elf_get_mem_size function to avoid computing elf size in bootm.c
> >> - Use xmalloc and read_full in elf_open instead of xzalloc/read
> >> - Fix data->elf NULL reset
> >> - Remove elf struct entirely from mips bootm code
> >>
> >> Clement Leger (7):
> >> common: elf: fix warning on 32 bits architectures
> >> common: elf: use calloc instead of xzalloc
> >> common: elf: check number of elf program headers
> >> common: elf: load elf directly from file
> >> common: elf: add elf_open, elf_close and elf_load
> >> common: bootm: add support for elf file loading
> >> mips: lib: bootm: use bootm elf loading capabilities
> >>
> >> arch/mips/lib/bootm.c | 25 ++---
> >> common/Kconfig | 8 ++
> >> common/bootm.c | 33 ++++++
> >> common/elf.c | 226 +++++++++++++++++++++++++++++++++++-------
> >> include/bootm.h | 3 +
> >> include/elf.h | 9 +-
> >> 6 files changed, 248 insertions(+), 56 deletions(-)
> >>
> >> --
> >> 2.17.1
> >>
> >>
> >> _______________________________________________
> >> barebox mailing list
> >> barebox@xxxxxxxxxxxxxxxxxxx
> >> http://lists.infradead.org/mailman/listinfo/barebox
> >>
> >
> > --
> > Pengutronix e.K. | |
> > Steuerwalder Str. 21 | http://www.pengutronix.de/ |
> > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
> > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
>
> _______________________________________________
> barebox mailing list
> barebox@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/barebox
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
