public inbox for libabigail@sourceware.org
 help / color / mirror / Atom feed
From: Dodji Seketeli <dodji@redhat.com>
To: libabigail@sourceware.org
Subject: [PATCH 1/4, applied] Make fe_iface::initialize independent from the kind of interface
Date: Fri, 07 Jul 2023 13:41:10 +0200	[thread overview]
Message-ID: <87a5w80x0p.fsf@redhat.com> (raw)
In-Reply-To: <87edlk0x3p.fsf@redhat.com> (Dodji Seketeli's message of "Fri, 07 Jul 2023 13:39:22 +0200")

Hello,

In the use case laid out by the "Library Splitting support" at
https://sourceware.org/bugzilla/show_bug.cgi?id=30034, tools might
want to be able to re-initialize a given front-end to load several
corpora in a row, independently from the kind of front-end (elf-based,
ABIXML, etc).  So we need a fe_iface::initialize() interface that is
similar to the elf_based_reader::initialize() interface but, unlike
that one, it should not depend on parameters that only make sense in a
elf-based context.  For instance, the fe_iface::initialize() interface
should not contain paths to debug-info as that information does not
make sense in an ABIXML context.

This patch thus renames fe_iface::reset() into fe_iface::initialize()
and makes it virtual.  It adjusts the plumbing accordingly, basically
making the other ::initialize() methods of classes that inherit
fe_iface use fe_iface::initialize() when appropriate.

The patch also provides an elf_based_reader::initialize()
implementation of the fe_iface::initialize() interface, so that
invoking ::initialize on elf_based_reader through the fe_iface
interface does the right thing so that the initial use case presented
at the beginning of this comment is supported.

	* include/abg-elf-based-reader.h (elf_based_reader::initialize):
	Rename elf_based_reader::reset into this.  Add a new virtual
	overload that implements fe_iface::initialize.
	* include/abg-elf-reader.h (reader::initialize): Rename
	elf_reader::reset into this.  Add a new virtual overload that
	implements fe_iface::initialize.
	* include/abg-fe-iface.h (fe_iface::initialize): Rename
	fe_iface::reset into this and make it virtual.
	* src/abg-btf-reader.cc (btf::reader::initialize): Adjust call to
	elf_based_reader::reset to elf_based_reader::initialize.
	* src/abg-ctf-reader.cc (ctf::reader::initialize): Likewise.
	* src/abg-dwarf-reader.cc (dwarf::reader::initialize): Likewise.
	* src/abg-elf-based-reader.cc (elf_based_reader::initialize):
	Rename elf_based_reader::reset into this.  Adjust call to
	elf::reader::reset into elf::reader::initialize.  Add a new
	virtual overload that implements fe_iface::initialize.
	* src/abg-elf-reader.cc (reader::initialize): Rename
	elf::reader::reset into this. Adjust call to fe_iface::reset into
	fe_iface::initialize.  Add a new virtual overload that implements
	fe_iface::initialize.
	* src/abg-fe-iface.cc (fe_iface::priv::initialize): Reset the
	corpus too.
	(fe_iface::initialize): Rename fe_iface::reset into this.  Invoke
	priv::initialize and set the new corpus_path.

Signed-off-by: Dodji Seketeli <dodji@redhat.com>
Applied to master.
---
 include/abg-elf-based-reader.h |  8 ++++++--
 include/abg-elf-reader.h       |  9 ++++++---
 include/abg-fe-iface.h         |  4 ++--
 src/abg-btf-reader.cc          |  2 +-
 src/abg-ctf-reader.cc          |  2 +-
 src/abg-dwarf-reader.cc        |  2 +-
 src/abg-elf-based-reader.cc    | 23 +++++++++++++++++++----
 src/abg-elf-reader.cc          | 25 +++++++++++++++++++------
 src/abg-fe-iface.cc            | 11 ++++-------
 9 files changed, 59 insertions(+), 27 deletions(-)

diff --git a/include/abg-elf-based-reader.h b/include/abg-elf-based-reader.h
index 597eec90..f51cb2cb 100644
--- a/include/abg-elf-based-reader.h
+++ b/include/abg-elf-based-reader.h
@@ -50,17 +50,21 @@ public:
   ~elf_based_reader();
 
   virtual void
-  reset(const std::string& elf_path,
-	const vector<char**>& debug_info_root_paths);
+  initialize(const std::string& elf_path,
+	     const vector<char**>& debug_info_root_paths);
 
   virtual ir::corpus_sptr
   read_and_add_corpus_to_group(ir::corpus_group& group,
 			       fe_iface::status& status);
+
   virtual void
   initialize(const string&		elf_path,
 	     const vector<char**>&	debug_info_root_paths,
 	     bool			load_all_types,
 	     bool			linux_kernel_mode) = 0;
+
+  virtual void
+  initialize(const std::string& corpus_path);
 };//end class elf_based_reader
 
 typedef std::shared_ptr<elf_based_reader> elf_based_reader_sptr;
diff --git a/include/abg-elf-reader.h b/include/abg-elf-reader.h
index 9e370c0b..ab350607 100644
--- a/include/abg-elf-reader.h
+++ b/include/abg-elf-reader.h
@@ -72,9 +72,12 @@ class reader : public fe_iface
 
   ~reader();
 
-  void
-  reset(const std::string&	elf_path,
-	 const vector<char**>&	debug_info_roots);
+  virtual void
+  initialize(const std::string&	elf_path,
+	     const vector<char**>&	debug_info_roots);
+
+  virtual void
+  initialize(const std::string& elf_path);
 
   const vector<char**>&
   debug_info_root_paths() const;
diff --git a/include/abg-fe-iface.h b/include/abg-fe-iface.h
index 12b8b2bd..64feafb2 100644
--- a/include/abg-fe-iface.h
+++ b/include/abg-fe-iface.h
@@ -74,8 +74,8 @@ protected:
 
   virtual ~fe_iface();
 
-  void
-  reset(const std::string& corpus_path, environment& e);
+  virtual void
+  initialize(const std::string& corpus_path);
 
   const options_type&
   options() const;
diff --git a/src/abg-btf-reader.cc b/src/abg-btf-reader.cc
index 1ab2b4c8..995ff5b4 100644
--- a/src/abg-btf-reader.cc
+++ b/src/abg-btf-reader.cc
@@ -237,7 +237,7 @@ protected:
 	     bool			load_all_types,
 	     bool			linux_kernel_mode)
   {
-    reset(elf_path, debug_info_root_paths);
+    elf_based_reader::initialize(elf_path, debug_info_root_paths);
     btf__free(btf_handle_);
     options().load_all_types = load_all_types;
     options().load_in_linux_kernel_mode = linux_kernel_mode;
diff --git a/src/abg-ctf-reader.cc b/src/abg-ctf-reader.cc
index 8e9c204f..2de11c0f 100644
--- a/src/abg-ctf-reader.cc
+++ b/src/abg-ctf-reader.cc
@@ -288,7 +288,7 @@ public:
   {
     load_all_types = load_all_types;
     linux_kernel_mode = linux_kernel_mode;
-    reset(elf_path, debug_info_root_paths);
+    elf_based_reader::initialize(elf_path, debug_info_root_paths);
   }
 
   /// Setter of the current translation unit.
diff --git a/src/abg-dwarf-reader.cc b/src/abg-dwarf-reader.cc
index 80b9d752..c20cf16b 100644
--- a/src/abg-dwarf-reader.cc
+++ b/src/abg-dwarf-reader.cc
@@ -2018,7 +2018,7 @@ public:
 	     bool			load_all_types,
 	     bool			linux_kernel_mode)
   {
-    reset(elf_path, debug_info_root_paths);
+    elf_based_reader::initialize(elf_path, debug_info_root_paths);
     initialize(load_all_types, linux_kernel_mode);
   }
 
diff --git a/src/abg-elf-based-reader.cc b/src/abg-elf-based-reader.cc
index d1d9a2df..f565c26a 100644
--- a/src/abg-elf-based-reader.cc
+++ b/src/abg-elf-based-reader.cc
@@ -61,7 +61,7 @@ elf_based_reader::elf_based_reader(const std::string& elf_path,
 elf_based_reader::~elf_based_reader()
 {delete priv_;}
 
-/// Reset (re-initialize) the resources used by the current reader.
+/// (re)Initialize) the resources used by the current reader.
 ///
 /// This frees the resources of the current reader and gets it ready
 /// to read data from another ELF file.
@@ -71,13 +71,28 @@ elf_based_reader::~elf_based_reader()
 /// @param debug_info_root_paths a vector of paths to look into for
 /// split debug info files.
 void
-elf_based_reader::reset(const std::string& elf_path,
-			const vector<char**>& debug_info_root_paths)
+elf_based_reader::initialize(const std::string& elf_path,
+			     const vector<char**>& debug_info_root_paths)
 {
-  elf::reader::reset(elf_path, debug_info_root_paths);
+  elf::reader::initialize(elf_path, debug_info_root_paths);
   priv_->initialize();
 }
 
+/// (re)Initialize the resources used by the current reader.
+///
+/// This invokes fe_iface::initialize as wel as the virtual pure
+/// elf_based_reader::initialize() interface.
+///
+/// @param corpus_path path to the corpus to be built.
+void
+elf_based_reader::initialize(const std::string& corpus_path)
+{
+  fe_iface::initialize(corpus_path);
+  vector<char**> v;
+  initialize(corpus_path, v, /*load_all_type=*/false,
+	     /*linux_kernel_mode=*/false);
+}
+
 /// Read an ABI corpus and add it to a given corpus group.
 ///
 /// @param group the corpus group to consider.  The new corpus is
diff --git a/src/abg-elf-reader.cc b/src/abg-elf-reader.cc
index af65cecf..7884c672 100644
--- a/src/abg-elf-reader.cc
+++ b/src/abg-elf-reader.cc
@@ -527,8 +527,8 @@ reader::reader(const string&		elf_path,
 reader::~reader()
 {delete priv_;}
 
-/// Resets (erase) the resources used by the current @ref
-/// elf::reader type.
+/// Re-initialize the resources used by the current @ref elf::reader
+/// type.
 ///
 /// This lets the reader in a state where it's ready to read from
 /// another ELF file.
@@ -538,11 +538,10 @@ reader::~reader()
 /// @param debug_info_roots a vector of directory paths to look into
 /// for split debug information files.
 void
-reader::reset(const std::string&	elf_path,
-	      const vector<char**>&	debug_info_roots)
+reader::initialize(const std::string&		elf_path,
+		   const vector<char**>&	debug_info_roots)
 {
-  fe_iface::options_type opts = options();
-  fe_iface::reset(elf_path, opts.env);
+  fe_iface::initialize(elf_path);
   corpus_path(elf_path);
   priv_->initialize(debug_info_roots);
   priv_->crack_open_elf_file();
@@ -550,6 +549,20 @@ reader::reset(const std::string&	elf_path,
   priv_->locate_ctf_debug_info();
 }
 
+/// Re-initialize the resources used by the current @ref elf::reader
+/// type.
+///
+/// This lets the reader in a state where it's ready to read from
+/// another ELF file.
+///
+/// @param elf_path the new ELF path to read from.
+void
+reader::initialize(const std::string&	elf_path)
+{
+  vector<char**> v;
+  initialize(elf_path, v);
+}
+
 /// Getter of the vector of directory paths to look into for split
 /// debug information files.
 ///
diff --git a/src/abg-fe-iface.cc b/src/abg-fe-iface.cc
index 6ec50bfd..7c62fe7c 100644
--- a/src/abg-fe-iface.cc
+++ b/src/abg-fe-iface.cc
@@ -43,11 +43,11 @@ struct fe_iface::priv
   void
   initialize()
   {
-    //TODO: initialize the options.
     corpus_path.clear();
     dt_soname.clear();
     suppressions.clear();
     corpus_group.reset();
+    corpus.reset();
   }
 }; //end struct fe_iface::priv
 
@@ -78,14 +78,11 @@ fe_iface::~fe_iface()
 ///
 /// @param corpus_path the path to the file for which a new corpus is
 /// to be created.
-///
-/// @param e the environment in which the Front End operates.
 void
-fe_iface::reset(const std::string& corpus_path,
-		environment& e)
+fe_iface::initialize(const std::string& corpus_path)
 {
-  delete priv_;
-  priv_ = new fe_iface::priv(corpus_path, e);
+  priv_->initialize();
+  priv_->corpus_path = corpus_path;
 }
 
 /// Getter of the the options of the current Front End Interface.
-- 
2.39.3



-- 
		Dodji


  reply	other threads:[~2023-07-07 11:41 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-07 11:39 [PATCH 0/4, applied] Binaries set comparison support Dodji Seketeli
2023-07-07 11:41 ` Dodji Seketeli [this message]
2023-07-07 11:42 ` [PATCH 2/4, applied] corpus,tools-utils: Support loading a corpus, its deps & other binaries Dodji Seketeli
2023-07-07 11:43 ` [PATCH 3/4, applied] abidw: Add --{follow,list}-dependencies & --add-binaries support Dodji Seketeli
2023-07-07 11:43 ` [PATCH 4/4, applied] abidiff: Add --{follow,list}-dependencies & add-binaries{1,2} support 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=87a5w80x0p.fsf@redhat.com \
    --to=dodji@redhat.com \
    --cc=libabigail@sourceware.org \
    /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).