From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============5566131038627512497=="
MIME-Version: 1.0
From: Thomas Renninger <trenn at suse.de>
Subject: [Powertop] Too much memory consumption by powertop
Date: Tue, 28 Apr 2015 17:58:05 +0200
Message-ID: <1765755.qjI46OxpdT@skinner>
To: powertop@lists.01.org
List-ID: <powertop@lists.01.org>

--===============5566131038627512497==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

As powertop list is for subscribers only, here Takashi's
findings concerning a big memory waste fowarded:

----------  Forwarded Message  ----------

Subject: Too much memory consumption by powertop
Date: Tuesday, April 28, 2015, 05:32:41 PM
From: Takashi Iwai <tiwai(a)suse.de>
To: powertop(a)lists.01.org
CC: trenn(a)suse.de

Hi,

I noticed that powertop consumes way too much memory than needed.  It
constantly takes over 170MB RES, sometimes over 200MB.

Then I looked at profile, and it showed that most of memory is
allocated for newpad_sp().  Indeed, the current code allocates a pad
of the fixed size of 1000x1000 (!) for each tab, and we have five
tabs, so five times of that.  No wonder that such a small neat CUI
program is so hungry.

For example, a hack like below already reduces the memory footprint
to around 20MB, 90% cut.  Not bad.

--- a/src/display.cpp
+++ b/src/display.cpp
@@ -48,7 +48,7 @@ void create_tab(const string &name, cons
        if (!w)
                w =3D new(class tab_window);
 =

-       w->win =3D newpad(1000,1000);
+       w->win =3D newpad(80,160); // XXX FIXME XXX
        tab_names.push_back(name);
        tab_windows[name] =3D w;
        tab_translations[name] =3D translation;

Can anyone look into this and give a better / smarter solution?


thanks,

Takashi
-----------------------------------------

--===============5566131038627512497==--