openbmc / entity-manager

Run-time JSON driven system configuration manager
Other
26 stars 48 forks source link

Memory leak in FruDevice #33

Closed Max-918 closed 5 months ago

Max-918 commented 5 months ago

In BMC:

~#: ps -l | grep fru
S   UID   PID  PPID   VSZ   RSS TTY   STIME TIME     CMD
S     0   310     1 15716  **4756** 0:0   09:47 00:00:03 /usr/bin/fru-device

When continuously calling the ReScanBus method which under FruDevice:

~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5
~#: busctl call xyz.openbmc_project.FruDevice /xyz/openbmc_project/FruDevice xyz.openbmc_project.FruDeviceManager ReScanBus q 5

~#: ps -l | grep fru
S   UID   PID  PPID   VSZ   RSS TTY   STIME TIME     CMD
S     0   310     1 15716  **4764** 0:0   09:47 00:00:03 /usr/bin/fru-device

Long term continuous calling of ReScanBus method means a lot of memory leakage