<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Menlo;
        panose-1:2 11 6 9 3 8 4 2 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style>
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">I am sure it is before strip, it should be few kb only.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Regards<o:p></o:p></p>
<p class="MsoNormal">-Vijay<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">openbmc <openbmc-bounces+vijaykhemka=fb.com@lists.ozlabs.org> on behalf of Wilfred Smith <wilfredsmith@fb.com><br>
<b>Date: </b>Monday, September 9, 2019 at 9:10 AM<br>
<b>To: </b>Lei YU <mine260309@gmail.com><br>
<b>Cc: </b>"openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org><br>
<b>Subject: </b>[Potential Spoof] Re: Sdbusplus/tight memory conditions<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">Perhaps I’m doing it incorrectly… <o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">target_compile_options( sizing.bin PUBLIC -Os -s)<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">…gets me a binary of <span style="font-size:8.5pt;font-family:Menlo">2,092,428. Saves 200K, but still north of 2 MB<o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:8.5pt;font-family:Menlo">Wilfred</span><o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><br>
<br>
<o:p></o:p></p>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<div>
<p class="MsoNormal">On Sep 9, 2019, at 2:29 AM, Lei YU <<a href="mailto:mine260309@gmail.com">mine260309@gmail.com</a>> wrote:<o:p></o:p></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">Is the size before strip or after strip?<br>
I suspect it's not stripped so you get a quite large size.<br>
<br>
On Mon, Sep 9, 2019 at 4:34 PM Wilfred Smith <<a href="mailto:wilfredsmith@fb.com">wilfredsmith@fb.com</a>> wrote:<br>
<br>
<o:p></o:p></p>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<p class="MsoNormal"><br>
I’m having code size issues using the sdbusplus library. As I mentioned during the conference, I suspected it was a combinatorial expansion forced by the use of variant in a template. As a result, my “tiny” FRU dumping utility uses almost 3 MB on Tioga Pass
 and can only be stored on /tmp.. My sensor dumping utility, using the same structure by way of calling GetManagedObjects weighed in close to 9 MB, presumably because there are several loops that walk the structure, with even more templated function instantiation,
 which sux0rs when you only have a 32MB part. Even the best darn FRU utility in the world isn’t worth 25% of your available code space.<br>
<br>
I hope I’m doing something wrong. The data structure is as recommended by Ed Tanous and I don’t see an obvious way to simplify the variant out of the structure. My current thoughts for proceeding are to (a) write my own task specific D-Bus library that isn’t
 as template-happy or (b) to re-implement with low-level D-Bus calls. Neither is particularly desirable since I would think all components should use the same library to access the D-Bus<br>
<br>
My colleagues suspected I was statically linking the sdbusplus libraries, but as I’ll prove in the next section, the bloat doesn’t occur until I have code that touches the D-Bus response, and the code size scales directly with the number of variant types. That
 should be orthogonal to the penalty for linking statically, unless the template permutations are in the library itself, but libsdbusplus.so is a meager 18,140 bytes.Code size here is increasing by 80,345 bytes per additional variant type, which is 4x the size
 of the entire SO.<br>
<br>
My presumption is that I’m off in the weeds, and someone will kindly guide me back to the main road.<br>
<br>
Experiment 1: A minimal application<br>
#include <iostream><br>
<br>
int main( int argc, char *argv ) {<br>
std::cout << "Sizing app" << std::endl;<br>
}<br>
<br>
add_executable( sizing.bin source/sizing.cpp )<br>
target_link_libraries( sizing.bin sdbusplus.so )<br>
install( TARGETS sizing.bin )<br>
<br>
104,828 bytes with or without link to sdbusplus.so<br>
<br>
——————————————————————————————<br>
Experiment 2: Add code to perform a GetManagedObjects. This increases the code payload substantially more than expected, but is still reasonable.<br>
<br>
#include <iostream><br>
#include <sdbusplus/bus.hpp><br>
<br>
<br>
int main( int argc, char **argv ) {<br>
std::cout << "Sizing app" << std::endl;<br>
  auto bus = sdbusplus::bus::new_default_system();<br>
  auto method = bus.new_method_call("xyz.openbmc_project.FruDevice",<br>
                                       "/",<br>
                                       "org.freedesktop.DBus.ObjectManager",<br>
                                       "GetManagedObjects");<br>
  auto response = bus.call(method);<br>
<br>
}<br>
<br>
add_executable( sizing.bin source/sizing.cpp )<br>
target_link_libraries( sizing.bin sdbusplus.so<br>
                                 systemd.so )<br>
install( TARGETS sizing.bin )<br>
<br>
255,212 bytes<br>
<br>
——————————————————————————————————<br>
Experiment  3: Added the structure to read the result into, and the size grows too 10x. Note that I’m not even doing anything with the result. It’s the interaction between the “read” and the storage type.<br>
<br>
#include <iostream><br>
#include <sdbusplus/bus.hpp><br>
#include <boost/container/flat_map.hpp><br>
<br>
typedef sdbusplus::message::variant< std::string,<br>
                                    bool,<br>
                                    uint8_t,<br>
                                    int16_t,<br>
                                    uint16_t,<br>
                                    int32_t,<br>
                                    uint32_t,<br>
                                    int64_t,<br>
                                    uint64_t,<br>
                                    double,<br>
                                    std::vector<std::string>> AnyType;<br>
<br>
typedef boost::container::flat_map< std::string, AnyType> AnyTypeMap;<br>
typedef std::vector<std::pair<std::string, AnyTypeMap>> NamedArrayOfAnyTypeMaps;<br>
typedef std::vector<std::pair<sdbusplus::message::object_path, NamedArrayOfAnyTypeMaps>> ArrayOfObjectPathsAndTieredAnyTypeMaps;<br>
<br>
<br>
int main( int argc, char **argv ) {<br>
  ArrayOfObjectPathsAndTieredAnyTypeMaps result;<br>
<br>
std::cout << "Sizing app" << std::endl;<br>
  auto bus = sdbusplus::bus::new_default_system();<br>
  auto method = bus.new_method_call("xyz.openbmc_project.FruDevice",<br>
                                       "/",<br>
                                       "org.freedesktop.DBus.ObjectManager",<br>
                                       "GetManagedObjects");<br>
  auto response = bus.call(method);<br>
  response.read( result );<br>
}<br>
<br>
2,274,736 bytes<br>
<br>
—————————————————————————————————————<br>
Experiment 4: If I reduce the number of permutations to 2, the code payload remains huge, but shrinks.<br>
<br>
#include <iostream><br>
#include <sdbusplus/bus.hpp><br>
#include <boost/container/flat_map.hpp><br>
<br>
typedef sdbusplus::message::variant< std::string,<br>
//                                     bool,<br>
//                                     uint8_t,<br>
//                                     int16_t,<br>
//                                     uint16_t,<br>
//                                     int32_t,<br>
//                                     uint32_t,<br>
//                                     int64_t,<br>
//                                     uint64_t,<br>
//                                     double,<br>
                                    std::vector<std::string>> AnyType;<br>
<br>
typedef boost::container::flat_map< std::string, AnyType> AnyTypeMap;<br>
typedef std::vector<std::pair<std::string, AnyTypeMap>> NamedArrayOfAnyTypeMaps;<br>
typedef std::vector<std::pair<sdbusplus::message::object_path, NamedArrayOfAnyTypeMaps>> ArrayOfObjectPathsAndTieredAnyTypeMaps;<br>
<br>
<br>
int main( int argc, char **argv ) {<br>
  ArrayOfObjectPathsAndTieredAnyTypeMaps result;<br>
<br>
std::cout << "Sizing app" << std::endl;<br>
  auto bus = sdbusplus::bus::new_default_system();<br>
  auto method = bus.new_method_call("xyz.openbmc_project.FruDevice",<br>
                                       "/",<br>
                                       "org.freedesktop.DBus.ObjectManager",<br>
                                       "GetManagedObjects");<br>
  auto response = bus.call(method);<br>
  response.read( result );<br>
}<br>
<br>
1,551,624 bytes<o:p></o:p></p>
</blockquote>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</body>
</html>