openbmc.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: "Tanous, Ed" <ed.tanous@intel.com>
To: Patrick Venture <venture@google.com>,
	Ratan K Gupta <ratagupt@in.ibm.com>,
	 Brad Bishop <bradleyb@fuzziesquirrel.com>,
	OpenBMC Maillist <openbmc@lists.ozlabs.org>
Subject: RE: Sdbusplus-based Shared Library
Date: Mon, 19 Mar 2018 18:24:44 +0000	[thread overview]
Message-ID: <7E9441B1E5EFFD4681F54958E82169932F66E12E@ORSMSX114.amr.corp.intel.com> (raw)
In-Reply-To: <CAO=notwXV+EaWJqV61qkZaDSA3mxMhpXbq3iWUqRp2Qg52wMGA@mail.gmail.com>

Have you tried prototyping to see how much space you'll save?  I suspect it won't be a lot for a few reasons.

1. A lot of sdbusplus is template instantiations, and unless we forward declare a number of base template instantiations for use in the shared library, most of the application code will end up in the binary anyway.
2. Sdbus calls into libsystemd, which is already a shared library.
3. The filesystem is already compressed, so I suspect that any duplicated methods that aren't inlined will have the same binary code pattern and get duplicated by the squashfs filesystem.

Those are all the reasons why I haven't really looked into it for the dbus stuff;  I can't speak to the timer stuff, but I suspect the wins in size there are going to be small.

Normal disclaimer, on this specific library, I haven't prototyped anything, just done back of the napkin guesses, so I could very easily be missing something here.

-Ed

> -----Original Message-----
> From: openbmc [mailto:openbmc-
> bounces+ed.tanous=intel.com@lists.ozlabs.org] On Behalf Of Patrick
> Venture
> Sent: Monday, March 19, 2018 10:12 AM
> To: Ratan K Gupta <ratagupt@in.ibm.com>; Brad Bishop
> <bradleyb@fuzziesquirrel.com>; OpenBMC Maillist
> <openbmc@lists.ozlabs.org>
> Subject: Sdbusplus-based Shared Library
> 
> We have a lot of duplication across daemons using sdbusplus, and no new
> utility library.  To avoid every daemon having their own timer object, and
> their own this or that, I suggest we create this shared library.
> 
> Thoughts?  We can start simple, just have a timer object in there, and then
> the new timers being introduced to phosphor-hwmon and phosphor-host-
> ipmid can be the first customers.
> 
> Patrick

  reply	other threads:[~2018-03-19 18:24 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-19 17:11 Sdbusplus-based Shared Library Patrick Venture
2018-03-19 18:24 ` Tanous, Ed [this message]
2018-03-19 19:33   ` Patrick Venture
2018-03-20  2:32     ` Lei YU
2018-03-20  5:39     ` Tanous, Ed
2018-03-27  5:47     ` Andrew Jeffery
2018-03-27  6:04       ` Ratan Gupta
2018-03-27 15:43         ` Patrick Venture
2018-03-29  1:39           ` Brad Bishop
2018-03-29  3:25             ` Patrick Venture
2018-03-29  3:43               ` Brad Bishop
2018-07-18  8:36                 ` Lei YU
2018-03-20  6:36 ` Deepak Kodihalli
2018-07-18 11:34 ` vishwa
2018-07-18 13:53   ` Thomaiyar, Richard Marian

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=7E9441B1E5EFFD4681F54958E82169932F66E12E@ORSMSX114.amr.corp.intel.com \
    --to=ed.tanous@intel.com \
    --cc=bradleyb@fuzziesquirrel.com \
    --cc=openbmc@lists.ozlabs.org \
    --cc=ratagupt@in.ibm.com \
    --cc=venture@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 NNTP newsgroup(s).