From: Giuliano Procida <gprocida@google.com>
To: Matthias Maennich <maennich@google.com>
Cc: libabigail@sourceware.org, Dodji Seketeli <dodji@seketeli.org>,
kernel-team@android.com
Subject: Re: [PATCH 3/8] abg-elf-helpers: move some versioning helpers from abg-dwarf-reader
Date: Mon, 20 Apr 2020 16:33:12 +0100 [thread overview]
Message-ID: <CAGvU0HmOxNq+0FSZ5vNPykCnk44u6LvAkcsXKQrsfsn-1W7OTg@mail.gmail.com> (raw)
In-Reply-To: <20200420110846.218792-4-maennich@google.com>
Looks good.
On Mon, 20 Apr 2020 at 12:09, Matthias Maennich <maennich@google.com> wrote:
>
> Move some definitions from abg-dwarf-reader to abg-elf-helpers that are
> strictly only related to ELF.
>
> * abg-dwarf-reader.cc(get_symbol_versionning_sections): Move function out.
> (get_version_definition_for_versym): Likewise.
> (get_version_needed_for_versym): Likewise.
> (get_version_for_symbol): Likewise.
> * abg-elf-helpers.cc(get_symbol_versionning_sections): Move function in.
> (get_version_definition_for_versym): Likewise.
> (get_version_needed_for_versym): Likewise.
> (get_version_for_symbol): Likewise.
> * abg-elf-helpers.cc(get_symbol_versionning_sections): Add declaration.
> (get_version_definition_for_versym): Likewise.
> (get_version_needed_for_versym): Likewise.
> (get_version_for_symbol): Likewise.
Reviewed-by: Giuliano Procida <gprocida@google.com>
> Signed-off-by: Matthias Maennich <maennich@google.com>
> ---
> src/abg-dwarf-reader.cc | 256 ----------------------------------------
> src/abg-elf-helpers.cc | 251 +++++++++++++++++++++++++++++++++++++++
> src/abg-elf-helpers.h | 28 +++++
> 3 files changed, 279 insertions(+), 256 deletions(-)
>
> diff --git a/src/abg-dwarf-reader.cc b/src/abg-dwarf-reader.cc
> index 6ec7e4b6e968..303c1f8df4c2 100644
> --- a/src/abg-dwarf-reader.cc
> +++ b/src/abg-dwarf-reader.cc
> @@ -294,12 +294,6 @@ static void
> add_symbol_to_map(const elf_symbol_sptr& sym,
> string_elf_symbols_map_type& map);
>
> -static bool
> -get_symbol_versionning_sections(Elf* elf_handle,
> - Elf_Scn*& versym_section,
> - Elf_Scn*& verdef_section,
> - Elf_Scn*& verneed_section);
> -
> static bool
> get_parent_die(const read_context& ctxt,
> const Dwarf_Die* die,
> @@ -867,256 +861,6 @@ compare_symbol_name(const string& symbol_name,
> return symbol_name == name;
> }
>
> -/// Return the SHT_GNU_versym, SHT_GNU_verdef and SHT_GNU_verneed
> -/// sections that are involved in symbol versionning.
> -///
> -/// @param elf_handle the elf handle to use.
> -///
> -/// @param versym_section the SHT_GNU_versym section found. If the
> -/// section wasn't found, this is set to nil.
> -///
> -/// @param verdef_section the SHT_GNU_verdef section found. If the
> -/// section wasn't found, this is set to nil.
> -///
> -/// @param verneed_section the SHT_GNU_verneed section found. If the
> -/// section wasn't found, this is set to nil.
> -///
> -/// @return true iff at least one of the sections where found.
> -static bool
> -get_symbol_versionning_sections(Elf* elf_handle,
> - Elf_Scn*& versym_section,
> - Elf_Scn*& verdef_section,
> - Elf_Scn*& verneed_section)
> -{
> - Elf_Scn* section = NULL;
> - GElf_Shdr mem;
> - Elf_Scn* versym = NULL, *verdef = NULL, *verneed = NULL;
> -
> - while ((section = elf_nextscn(elf_handle, section)) != NULL)
> - {
> - GElf_Shdr* h = gelf_getshdr(section, &mem);
> - if (h->sh_type == SHT_GNU_versym)
> - versym = section;
> - else if (h->sh_type == SHT_GNU_verdef)
> - verdef = section;
> - else if (h->sh_type == SHT_GNU_verneed)
> - verneed = section;
> - }
> -
> - if (versym || verdef || verneed)
> - {
> - // At least one the versionning sections was found. Return it.
> - versym_section = versym;
> - verdef_section = verdef;
> - verneed_section = verneed;
> - return true;
> - }
> -
> - return false;
> -}
> -
> -/// Get the version definition (from the SHT_GNU_verdef section) of a
> -/// given symbol represented by a pointer to GElf_Versym.
> -///
> -/// @param elf_hande the elf handle to use.
> -///
> -/// @param versym the symbol to get the version definition for.
> -///
> -/// @param verdef_section the SHT_GNU_verdef section.
> -///
> -/// @param version the resulting version definition. This is set iff
> -/// the function returns true.
> -///
> -/// @return true upon successful completion, false otherwise.
> -static bool
> -get_version_definition_for_versym(Elf* elf_handle,
> - GElf_Versym* versym,
> - Elf_Scn* verdef_section,
> - elf_symbol::version& version)
> -{
> - Elf_Data* verdef_data = elf_getdata(verdef_section, NULL);
> - GElf_Verdef verdef_mem;
> - GElf_Verdef* verdef = gelf_getverdef(verdef_data, 0, &verdef_mem);
> - size_t vd_offset = 0;
> -
> - for (;; vd_offset += verdef->vd_next)
> - {
> - for (;verdef != 0;)
> - {
> - if (verdef->vd_ndx == (*versym & 0x7fff))
> - // Found the version of the symbol.
> - break;
> - vd_offset += verdef->vd_next;
> - verdef = (verdef->vd_next == 0
> - ? 0
> - : gelf_getverdef(verdef_data, vd_offset, &verdef_mem));
> - }
> -
> - if (verdef != 0)
> - {
> - GElf_Verdaux verdaux_mem;
> - GElf_Verdaux *verdaux = gelf_getverdaux(verdef_data,
> - vd_offset + verdef->vd_aux,
> - &verdaux_mem);
> - GElf_Shdr header_mem;
> - GElf_Shdr* verdef_section_header = gelf_getshdr(verdef_section,
> - &header_mem);
> - size_t verdef_stridx = verdef_section_header->sh_link;
> - version.str(elf_strptr(elf_handle, verdef_stridx, verdaux->vda_name));
> - if (*versym & 0x8000)
> - version.is_default(false);
> - else
> - version.is_default(true);
> - return true;
> - }
> - if (!verdef || verdef->vd_next == 0)
> - break;
> - }
> - return false;
> -}
> -
> -/// Get the version needed (from the SHT_GNU_verneed section) to
> -/// resolve an undefined symbol represented by a pointer to
> -/// GElf_Versym.
> -///
> -/// @param elf_hande the elf handle to use.
> -///
> -/// @param versym the symbol to get the version definition for.
> -///
> -/// @param verneed_section the SHT_GNU_verneed section.
> -///
> -/// @param version the resulting version definition. This is set iff
> -/// the function returns true.
> -///
> -/// @return true upon successful completion, false otherwise.
> -static bool
> -get_version_needed_for_versym(Elf* elf_handle,
> - GElf_Versym* versym,
> - Elf_Scn* verneed_section,
> - elf_symbol::version& version)
> -{
> - if (versym == 0 || elf_handle == 0 || verneed_section == 0)
> - return false;
> -
> - size_t vn_offset = 0;
> - Elf_Data* verneed_data = elf_getdata(verneed_section, NULL);
> - GElf_Verneed verneed_mem;
> - GElf_Verneed* verneed = gelf_getverneed(verneed_data, 0, &verneed_mem);
> -
> - for (;verneed; vn_offset += verneed->vn_next)
> - {
> - size_t vna_offset = vn_offset;
> - GElf_Vernaux vernaux_mem;
> - GElf_Vernaux *vernaux = gelf_getvernaux(verneed_data,
> - vn_offset + verneed->vn_aux,
> - &vernaux_mem);
> - for (;vernaux != 0 && verneed;)
> - {
> - if (vernaux->vna_other == *versym)
> - // Found the version of the symbol.
> - break;
> - vna_offset += verneed->vn_next;
> - verneed = (verneed->vn_next == 0
> - ? 0
> - : gelf_getverneed(verneed_data, vna_offset, &verneed_mem));
> - }
> -
> - if (verneed != 0 && vernaux != 0 && vernaux->vna_other == *versym)
> - {
> - GElf_Shdr header_mem;
> - GElf_Shdr* verneed_section_header = gelf_getshdr(verneed_section,
> - &header_mem);
> - size_t verneed_stridx = verneed_section_header->sh_link;
> - version.str(elf_strptr(elf_handle,
> - verneed_stridx,
> - vernaux->vna_name));
> - if (*versym & 0x8000)
> - version.is_default(false);
> - else
> - version.is_default(true);
> - return true;
> - }
> -
> - if (!verneed || verneed->vn_next == 0)
> - break;
> - }
> - return false;
> -}
> -
> -/// Return the version for a symbol that is at a given index in its
> -/// SHT_SYMTAB section.
> -///
> -/// @param elf_handle the elf handle to use.
> -///
> -/// @param symbol_index the index of the symbol to consider.
> -///
> -/// @param get_def_version if this is true, it means that that we want
> -/// the version for a defined symbol; in that case, the version is
> -/// looked for in a section of type SHT_GNU_verdef. Otherwise, if
> -/// this parameter is false, this means that we want the version for
> -/// an undefined symbol; in that case, the version is the needed one
> -/// for the symbol to be resolved; so the version is looked fo in a
> -/// section of type SHT_GNU_verneed.
> -///
> -/// @param version the version found for symbol at @p symbol_index.
> -///
> -/// @return true iff a version was found for symbol at index @p
> -/// symbol_index.
> -static bool
> -get_version_for_symbol(Elf* elf_handle,
> - size_t symbol_index,
> - bool get_def_version,
> - elf_symbol::version& version)
> -{
> - Elf_Scn *versym_section = NULL,
> - *verdef_section = NULL,
> - *verneed_section = NULL;
> -
> - if (!get_symbol_versionning_sections(elf_handle,
> - versym_section,
> - verdef_section,
> - verneed_section))
> - return false;
> -
> - GElf_Versym versym_mem;
> - Elf_Data* versym_data = (versym_section)
> - ? elf_getdata(versym_section, NULL)
> - : NULL;
> - GElf_Versym* versym = (versym_data)
> - ? gelf_getversym(versym_data, symbol_index, &versym_mem)
> - : NULL;
> -
> - if (versym == 0 || *versym <= 1)
> - // I got these value from the code of readelf.c in elfutils.
> - // Apparently, if the symbol version entry has these values, the
> - // symbol must be discarded. This is not documented in the
> - // official specification.
> - return false;
> -
> - if (get_def_version)
> - {
> - if (*versym == 0x8001)
> - // I got this value from the code of readelf.c in elfutils
> - // too. It's not really documented in the official
> - // specification.
> - return false;
> -
> - if (verdef_section
> - && get_version_definition_for_versym(elf_handle, versym,
> - verdef_section, version))
> - return true;
> - }
> - else
> - {
> - if (verneed_section
> - && get_version_needed_for_versym(elf_handle, versym,
> - verneed_section, version))
> - return true;
> - }
> -
> - return false;
> -}
> -
> /// Lookup a symbol using the SysV ELF hash table.
> ///
> /// Note that this function hasn't been tested. So it hasn't been
> diff --git a/src/abg-elf-helpers.cc b/src/abg-elf-helpers.cc
> index b0113a4efd2c..b7b868a332ec 100644
> --- a/src/abg-elf-helpers.cc
> +++ b/src/abg-elf-helpers.cc
> @@ -500,6 +500,257 @@ Elf_Scn*
> find_data1_section(Elf* elf_handle)
> {return find_section(elf_handle, ".data1", SHT_PROGBITS);}
>
> +/// Return the SHT_GNU_versym, SHT_GNU_verdef and SHT_GNU_verneed
> +/// sections that are involved in symbol versionning.
> +///
> +/// @param elf_handle the elf handle to use.
> +///
> +/// @param versym_section the SHT_GNU_versym section found. If the
> +/// section wasn't found, this is set to nil.
> +///
> +/// @param verdef_section the SHT_GNU_verdef section found. If the
> +/// section wasn't found, this is set to nil.
> +///
> +/// @param verneed_section the SHT_GNU_verneed section found. If the
> +/// section wasn't found, this is set to nil.
> +///
> +/// @return true iff at least one of the sections where found.
> +bool
> +get_symbol_versionning_sections(Elf* elf_handle,
> + Elf_Scn*& versym_section,
> + Elf_Scn*& verdef_section,
> + Elf_Scn*& verneed_section)
> +{
> + Elf_Scn* section = NULL;
> + GElf_Shdr mem;
> + Elf_Scn* versym = NULL, *verdef = NULL, *verneed = NULL;
> +
> + while ((section = elf_nextscn(elf_handle, section)) != NULL)
> + {
> + GElf_Shdr* h = gelf_getshdr(section, &mem);
> + if (h->sh_type == SHT_GNU_versym)
> + versym = section;
> + else if (h->sh_type == SHT_GNU_verdef)
> + verdef = section;
> + else if (h->sh_type == SHT_GNU_verneed)
> + verneed = section;
> + }
> +
> + if (versym || verdef || verneed)
> + {
> + // At least one the versionning sections was found. Return it.
> + versym_section = versym;
> + verdef_section = verdef;
> + verneed_section = verneed;
> + return true;
> + }
> +
> + return false;
> +}
> +
> +/// Get the version definition (from the SHT_GNU_verdef section) of a
> +/// given symbol represented by a pointer to GElf_Versym.
> +///
> +/// @param elf_hande the elf handle to use.
> +///
> +/// @param versym the symbol to get the version definition for.
> +///
> +/// @param verdef_section the SHT_GNU_verdef section.
> +///
> +/// @param version the resulting version definition. This is set iff
> +/// the function returns true.
> +///
> +/// @return true upon successful completion, false otherwise.
> +bool
> +get_version_definition_for_versym(Elf* elf_handle,
> + GElf_Versym* versym,
> + Elf_Scn* verdef_section,
> + elf_symbol::version& version)
> +{
> + Elf_Data* verdef_data = elf_getdata(verdef_section, NULL);
> + GElf_Verdef verdef_mem;
> + GElf_Verdef* verdef = gelf_getverdef(verdef_data, 0, &verdef_mem);
> + size_t vd_offset = 0;
> +
> + for (;; vd_offset += verdef->vd_next)
> + {
> + for (;verdef != 0;)
> + {
> + if (verdef->vd_ndx == (*versym & 0x7fff))
> + // Found the version of the symbol.
> + break;
> + vd_offset += verdef->vd_next;
> + verdef = (verdef->vd_next == 0
> + ? 0
> + : gelf_getverdef(verdef_data, vd_offset, &verdef_mem));
> + }
> +
> + if (verdef != 0)
> + {
> + GElf_Verdaux verdaux_mem;
> + GElf_Verdaux *verdaux = gelf_getverdaux(verdef_data,
> + vd_offset + verdef->vd_aux,
> + &verdaux_mem);
> + GElf_Shdr header_mem;
> + GElf_Shdr* verdef_section_header = gelf_getshdr(verdef_section,
> + &header_mem);
> + size_t verdef_stridx = verdef_section_header->sh_link;
> + version.str(elf_strptr(elf_handle, verdef_stridx, verdaux->vda_name));
> + if (*versym & 0x8000)
> + version.is_default(false);
> + else
> + version.is_default(true);
> + return true;
> + }
> + if (!verdef || verdef->vd_next == 0)
> + break;
> + }
> + return false;
> +}
> +
> +/// Get the version needed (from the SHT_GNU_verneed section) to
> +/// resolve an undefined symbol represented by a pointer to
> +/// GElf_Versym.
> +///
> +/// @param elf_hande the elf handle to use.
> +///
> +/// @param versym the symbol to get the version definition for.
> +///
> +/// @param verneed_section the SHT_GNU_verneed section.
> +///
> +/// @param version the resulting version definition. This is set iff
> +/// the function returns true.
> +///
> +/// @return true upon successful completion, false otherwise.
> +bool
> +get_version_needed_for_versym(Elf* elf_handle,
> + GElf_Versym* versym,
> + Elf_Scn* verneed_section,
> + elf_symbol::version& version)
> +{
> + if (versym == 0 || elf_handle == 0 || verneed_section == 0)
> + return false;
> +
> + size_t vn_offset = 0;
> + Elf_Data* verneed_data = elf_getdata(verneed_section, NULL);
> + GElf_Verneed verneed_mem;
> + GElf_Verneed* verneed = gelf_getverneed(verneed_data, 0, &verneed_mem);
> +
> + for (;verneed; vn_offset += verneed->vn_next)
> + {
> + size_t vna_offset = vn_offset;
> + GElf_Vernaux vernaux_mem;
> + GElf_Vernaux *vernaux = gelf_getvernaux(verneed_data,
> + vn_offset + verneed->vn_aux,
> + &vernaux_mem);
> + for (;vernaux != 0 && verneed;)
> + {
> + if (vernaux->vna_other == *versym)
> + // Found the version of the symbol.
> + break;
> + vna_offset += verneed->vn_next;
> + verneed = (verneed->vn_next == 0
> + ? 0
> + : gelf_getverneed(verneed_data, vna_offset, &verneed_mem));
> + }
> +
> + if (verneed != 0 && vernaux != 0 && vernaux->vna_other == *versym)
> + {
> + GElf_Shdr header_mem;
> + GElf_Shdr* verneed_section_header = gelf_getshdr(verneed_section,
> + &header_mem);
> + size_t verneed_stridx = verneed_section_header->sh_link;
> + version.str(elf_strptr(elf_handle,
> + verneed_stridx,
> + vernaux->vna_name));
> + if (*versym & 0x8000)
> + version.is_default(false);
> + else
> + version.is_default(true);
> + return true;
> + }
> +
> + if (!verneed || verneed->vn_next == 0)
> + break;
> + }
> + return false;
> +}
> +
> +/// Return the version for a symbol that is at a given index in its
> +/// SHT_SYMTAB section.
> +///
> +/// @param elf_handle the elf handle to use.
> +///
> +/// @param symbol_index the index of the symbol to consider.
> +///
> +/// @param get_def_version if this is true, it means that that we want
> +/// the version for a defined symbol; in that case, the version is
> +/// looked for in a section of type SHT_GNU_verdef. Otherwise, if
> +/// this parameter is false, this means that we want the version for
> +/// an undefined symbol; in that case, the version is the needed one
> +/// for the symbol to be resolved; so the version is looked fo in a
> +/// section of type SHT_GNU_verneed.
> +///
> +/// @param version the version found for symbol at @p symbol_index.
> +///
> +/// @return true iff a version was found for symbol at index @p
> +/// symbol_index.
> +bool
> +get_version_for_symbol(Elf* elf_handle,
> + size_t symbol_index,
> + bool get_def_version,
> + elf_symbol::version& version)
> +{
> + Elf_Scn *versym_section = NULL,
> + *verdef_section = NULL,
> + *verneed_section = NULL;
> +
> + if (!get_symbol_versionning_sections(elf_handle,
> + versym_section,
> + verdef_section,
> + verneed_section))
> + return false;
> +
> + GElf_Versym versym_mem;
> + Elf_Data* versym_data = (versym_section)
> + ? elf_getdata(versym_section, NULL)
> + : NULL;
> + GElf_Versym* versym = (versym_data)
> + ? gelf_getversym(versym_data, symbol_index, &versym_mem)
> + : NULL;
> +
> + if (versym == 0 || *versym <= 1)
> + // I got these value from the code of readelf.c in elfutils.
> + // Apparently, if the symbol version entry has these values, the
> + // symbol must be discarded. This is not documented in the
> + // official specification.
> + return false;
> +
> + if (get_def_version)
> + {
> + if (*versym == 0x8001)
> + // I got this value from the code of readelf.c in elfutils
> + // too. It's not really documented in the official
> + // specification.
> + return false;
> +
> + if (verdef_section
> + && get_version_definition_for_versym(elf_handle, versym,
> + verdef_section, version))
> + return true;
> + }
> + else
> + {
> + if (verneed_section
> + && get_version_needed_for_versym(elf_handle, versym,
> + verneed_section, version))
> + return true;
> + }
> +
> + return false;
> +}
> +
> +
>
> } // end namespace elf_helpers
> } // end namespace abigail
> diff --git a/src/abg-elf-helpers.h b/src/abg-elf-helpers.h
> index 58720da0fa9e..7e1c231ccc4e 100644
> --- a/src/abg-elf-helpers.h
> +++ b/src/abg-elf-helpers.h
> @@ -96,6 +96,34 @@ find_data_section(Elf* elf_handle);
> Elf_Scn*
> find_data1_section(Elf* elf_handle);
>
> +bool
> +get_symbol_versionning_sections(Elf* elf_handle,
> + Elf_Scn*& versym_section,
> + Elf_Scn*& verdef_section,
> + Elf_Scn*& verneed_section);
> +
> +//
> +// Helpers for symbol versioning
> +//
> +
> +bool
> +get_version_definition_for_versym(Elf* elf_handle,
> + GElf_Versym* versym,
> + Elf_Scn* verdef_section,
> + elf_symbol::version& version);
> +
> +bool
> +get_version_needed_for_versym(Elf* elf_handle,
> + GElf_Versym* versym,
> + Elf_Scn* verneed_section,
> + elf_symbol::version& version);
> +
> +bool
> +get_version_for_symbol(Elf* elf_handle,
> + size_t symbol_index,
> + bool get_def_version,
> + elf_symbol::version& version);
> +
> } // end namespace elf_helpers
> } // end namespace abigail
>
> --
> 2.26.1.301.g55bc3eb7cb9-goog
>
next prev parent reply other threads:[~2020-04-20 15:33 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-20 11:08 [PATCH 0/8] Refactor dwarf-reader: split out ELF helpers Matthias Maennich
2020-04-20 11:08 ` [PATCH 1/8] abg-dwarf-reader split: create abg-elf-helpers.{h, cc} and test case Matthias Maennich
2020-04-20 15:35 ` Giuliano Procida
2020-04-20 11:08 ` [PATCH 2/8] abg-elf-helpers: move some elf helpers from abg-dwarf-reader Matthias Maennich
2020-04-20 15:34 ` Giuliano Procida
2020-04-20 11:08 ` [PATCH 3/8] abg-elf-helpers: move some versioning " Matthias Maennich
2020-04-20 15:33 ` Giuliano Procida [this message]
2020-04-20 11:08 ` [PATCH 4/8] abg-elf-helpers: move some kernel " Matthias Maennich
2020-04-20 14:32 ` Giuliano Procida
2020-04-20 15:30 ` Giuliano Procida
2020-04-20 11:08 ` [PATCH 5/8] abg-elf-helpers: consolidate the is_linux_kernel* helpers Matthias Maennich
2020-04-20 15:29 ` Giuliano Procida
2020-04-20 11:08 ` [PATCH 6/8] abg-dwarf-reader: migrate more ELF helpers to elf-helpers Matthias Maennich
2020-04-20 15:24 ` Giuliano Procida
2020-04-21 6:14 ` Matthias Maennich
2020-04-21 11:02 ` Giuliano Procida
2020-04-20 11:08 ` [PATCH 7/8] abg-elf-helpers: migrate more elf helpers (architecture specific helpers) Matthias Maennich
2020-04-20 15:25 ` Giuliano Procida
2020-04-20 11:08 ` [PATCH 8/8] abg-elf-helpers: migrate maybe_adjust_et_rel_sym_addr_to_abs_addr Matthias Maennich
2020-04-21 6:35 ` [PATCH v2 0/8] Refactor dwarf-reader: split out ELF helpers Matthias Maennich
2020-04-21 6:35 ` [PATCH v2 1/8] abg-dwarf-reader split: create abg-elf-helpers.{h, cc} and test case Matthias Maennich
2020-04-22 9:42 ` Dodji Seketeli
2020-04-21 6:35 ` [PATCH v2 2/8] abg-elf-helpers: move some elf helpers from abg-dwarf-reader Matthias Maennich
2020-04-22 9:44 ` Dodji Seketeli
2020-04-21 6:35 ` [PATCH v2 3/8] abg-elf-helpers: move some versioning " Matthias Maennich
2020-04-22 9:46 ` Dodji Seketeli
2020-04-21 6:35 ` [PATCH v2 4/8] abg-elf-helpers: move some kernel " Matthias Maennich
2020-04-22 9:47 ` Dodji Seketeli
2020-04-21 6:35 ` [PATCH v2 5/8] abg-elf-helpers: consolidate the is_linux_kernel* helpers Matthias Maennich
2020-04-22 9:48 ` Dodji Seketeli
2020-04-21 6:35 ` [PATCH v2 6/8] abg-dwarf-reader: migrate more ELF helpers to elf-helpers Matthias Maennich
2020-04-22 9:53 ` Dodji Seketeli
2020-04-21 6:35 ` [PATCH v2 7/8] abg-elf-helpers: migrate more elf helpers (architecture specific helpers) Matthias Maennich
2020-04-22 9:55 ` Dodji Seketeli
2020-04-21 6:35 ` [PATCH v2 8/8] abg-elf-helpers: migrate maybe_adjust_et_rel_sym_addr_to_abs_addr Matthias Maennich
2020-04-22 9:56 ` Dodji Seketeli
2020-04-22 9:57 ` [PATCH v2 0/8] Refactor dwarf-reader: split out ELF helpers Dodji Seketeli
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=CAGvU0HmOxNq+0FSZ5vNPykCnk44u6LvAkcsXKQrsfsn-1W7OTg@mail.gmail.com \
--to=gprocida@google.com \
--cc=dodji@seketeli.org \
--cc=kernel-team@android.com \
--cc=libabigail@sourceware.org \
--cc=maennich@google.com \
/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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).