public inbox for elfutils@sourceware.org
 help / color / mirror / Atom feed
From: Mark Wielaard <mark@klomp.org>
To: Omar Sandoval <osandov@osandov.com>
Cc: "Frank Ch. Eigler" <fche@redhat.com>, elfutils-devel@sourceware.org
Subject: Re: [RFC PATCH 0/2] elfutils: don't use dlopen() for libebl modules
Date: Tue, 09 Jul 2019 19:14:00 -0000	[thread overview]
Message-ID: <8617484f6d48734c1596c2cc2174fd84940ab3e0.camel@klomp.org> (raw)
In-Reply-To: <20190708210219.GA7174@vader>

Hi Omar,

On Mon, 2019-07-08 at 14:02 -0700, Omar Sandoval wrote:
> I imagine that little by little, most of the libebl functionality could
> be converted in this way, and that's how we could chip away at the size
> increase of the elfutils binaries.
> 
> Do you have any objections to patches 1-4 of my submission [2]?
> 
> Thanks for the timely response!
> 
> 1: https://sourceware.org/ml/elfutils-devel/2019-q3/msg00022.html
> 2: https://sourceware.org/ml/elfutils-devel/2019-q3/msg00020.html

I am still pondering this a little. But I do think (patches 1-4) is the
(first) step forward. But I think I would like to do a 0.177 release
with the last 4 months of fixes first. We normally do a release every
3-4 months, so it is about time, and it means your feature wouldn't
have to wait that long. Although it would be a couple of months before
it sees a release. But I think it would be good to have some time to
make sure the idea works out in practice.

So I would propose a release end of this week/start of next week. But
without the libebl build rewrite.

For the release there are then still 3 things pending:

- The eu-elfclassify tool
  https://sourceware.org/ml/elfutils-devel/2019-q2/msg00018.html
  I had hoped on some feedback from the rpm hackers, since they are
  one if the intended users. And there are some (small) missing
  features and tests. Lets see where we are end if the week to see
  whether we can include that, or also postpone it to the next release.
- The C-SKY backend:
  https://sourceware.org/ml/elfutils-devel/2019-q2/msg00007.html
  This is really just blocked on me not making enough time for a
  final review. It looks good, but I am confused about one aspect
  of the DWARF register numbering issue.
- The dwelf_elf_e_machine_string patch:
  https://sourceware.org/ml/elfutils-devel/2019-q2/msg00130.html
  I didn't see any objections, so I think this is good to go.

Then after the release, somewhere next week, we'll apply your patches
first and can then deal with any fallout and followups. I am thinking
of moving some of the functionality into libdw proper (as cleaned up,
exported api) to reduce the size increase a little. And add a mechanism
for only building some of the backends (or maybe just drop some old
ones that nobody uses anyway).

Cheers,

Mark

  parent reply	other threads:[~2019-07-09 19:14 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-03 20:04 Omar Sandoval
2019-07-03 20:04 ` [RFC PATCH 2/2] Don't " Omar Sandoval
2019-07-03 20:04 ` [RFC PATCH 1/2] libcpu: merge libcpu_{i386,x86_64,bpf} into one library Omar Sandoval
2019-07-03 21:33 ` [RFC PATCH 0/2] elfutils: don't use dlopen() for libebl modules Frank Ch. Eigler
2019-07-03 21:37   ` Omar Sandoval
2019-07-03 21:40     ` Frank Ch. Eigler
2019-07-03 21:46       ` Omar Sandoval
2019-07-04  0:56         ` Frank Ch. Eigler
2019-07-08 20:49           ` Mark Wielaard
2019-07-08 21:02             ` Omar Sandoval
2019-07-09  6:39               ` Florian Weimer
2019-07-09 19:14               ` Mark Wielaard [this message]
2019-07-09 19:25                 ` Omar Sandoval
2019-08-26 13:57                   ` Mark Wielaard
2019-07-08 20:33 ` Mark Wielaard

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=8617484f6d48734c1596c2cc2174fd84940ab3e0.camel@klomp.org \
    --to=mark@klomp.org \
    --cc=elfutils-devel@sourceware.org \
    --cc=fche@redhat.com \
    --cc=osandov@osandov.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).