mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Oleksij Rempel <o.rempel@pengutronix.de>
To: "Clément Leger" <cleger@kalray.eu>
Cc: Barebox List <barebox@lists.infradead.org>,
	Oleksij Rempel <linux@rempel-privat.de>
Subject: Re: [PATCH v5 0/7] elf: add better bootm support
Date: Fri, 12 Jun 2020 08:57:18 +0200	[thread overview]
Message-ID: <20200612065718.ebc2tb74fwbrqhm3@pengutronix.de> (raw)
In-Reply-To: <962943155.6989814.1591944921269.JavaMail.zimbra@kalray.eu>


[-- Attachment #1.1: Type: text/plain, Size: 4645 bytes --]

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@pengutronix.de 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@lists.infradead.org
> >> 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@lists.infradead.org
> 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 #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 149 bytes --]

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

      reply	other threads:[~2020-06-12  6:57 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-11  8:02 Clement Leger
2020-06-11  8:02 ` [PATCH v5 1/7] common: elf: fix warning on 32 bits architectures Clement Leger
2020-06-11  8:02 ` [PATCH v5 2/7] common: elf: use calloc instead of xzalloc Clement Leger
2020-06-11  8:02 ` [PATCH v5 3/7] common: elf: check number of elf program headers Clement Leger
2020-06-11  8:02 ` [PATCH v5 4/7] common: elf: load elf directly from file Clement Leger
2020-06-11  8:02 ` [PATCH v5 5/7] common: elf: add elf_open, elf_close and elf_load Clement Leger
2020-06-11  8:02 ` [PATCH v5 6/7] common: bootm: add support for elf file loading Clement Leger
2020-06-11  8:02 ` [PATCH v5 7/7] mips: lib: bootm: use bootm elf loading capabilities Clement Leger
2020-06-12  5:22 ` [PATCH v5 0/7] elf: add better bootm support Oleksij Rempel
2020-06-12  6:55   ` Clément Leger
2020-06-12  6:57     ` Oleksij Rempel [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200612065718.ebc2tb74fwbrqhm3@pengutronix.de \
    --to=o.rempel@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=cleger@kalray.eu \
    --cc=linux@rempel-privat.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox