[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] mkimage: avoid copying relocations for sections that won
From: |
Peter Jones |
Subject: |
Re: [PATCH 1/2] mkimage: avoid copying relocations for sections that won't be copied. |
Date: |
Tue, 20 Feb 2018 18:26:01 -0500 |
User-agent: |
NeoMutt/20171215 |
On Tue, Feb 20, 2018 at 03:48:44PM +0100, Daniel Kiper wrote:
> On Wed, Jan 31, 2018 at 11:26:59AM -0500, Peter Jones wrote:
> > +static int
> > +SUFFIX (is_kept_section) (Elf_Shdr *s, const struct
> > grub_install_image_target_desc *image_target);
> > +static int
> > +SUFFIX (is_kept_reloc_section) (Elf_Shdr *s, const struct
> > grub_install_image_target_desc *image_target,
> > + Elf_Shdr *sections, Elf_Half section_entsize,
> > Elf_Half num_sections,
> > + const char *strtab);
>
> Ugh... Could not you create a struct and pass the pointer to it here?
Fair enough - I did it that way because the whole file seems averse to
that sort of thing, and once you start doing it, it gets a bit messy.
Nevertheless, I'll send you a patchset that includes that in just a few
minutes.
> PS Please CC me on the patches next time.
Will do.
--
Peter
- [PATCH v3 2/7] mkimage: make it easier to run syntax checkers on grub-mkimagexx.c, (continued)
- [PATCH v3 4/7] mkimage: make locate_sections() set up vaddresses as well., Peter Jones, 2018/02/21
- Re: [PATCH v3 4/7] mkimage: make locate_sections() set up vaddresses as well., Daniel Kiper, 2018/02/23
- [PATCH v3 5/7] mkimage: refactor a bunch of section data into a struct., Peter Jones, 2018/02/21
- Re: [PATCH v3 5/7] mkimage: refactor a bunch of section data into a struct., Daniel Kiper, 2018/02/23
- Re: [PATCH v3 5/7] mkimage: refactor a bunch of section data into a struct., Peter Jones, 2018/02/23
- [PATCH v3 6/7] mkimage: avoid copying relocations for sections that won't be copied., Peter Jones, 2018/02/21
- Re: [PATCH v3 6/7] mkimage: avoid copying relocations for sections that won't be copied., Daniel Kiper, 2018/02/23
- Re: [PATCH v3 1/7] aout.h: Fix missing include., Daniel Kiper, 2018/02/23
Re: [PATCH 1/2] mkimage: avoid copying relocations for sections that won't be copied.,
Peter Jones <=