-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added NodePowerGovernorAgent #2927
base: dev
Are you sure you want to change the base?
Added NodePowerGovernorAgent #2927
Conversation
2ba6fd8
to
460c234
Compare
, m_do_write_batch(false) | ||
, m_last_power_budget(NAN) | ||
, m_power_budget_changed(false) | ||
, m_epoch_power_buf(geopm::make_unique<CircularBuffer<double> >(16)) // Magic number... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Magic taken straight from the existing power governor agent. If this is not needed for ongoing experiments we can remove it.
85fee63
to
1594055
Compare
1594055
to
537e6ff
Compare
Signed-off-by: lhlawson <lowren.h.lawson@intel.com>
537e6ff
to
e01d219
Compare
geopm::PowerGovernorAgent(3) -- agent that enforces a power cap | ||
=============================================================== |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
geopm::PowerGovernorAgent(3) -- agent that enforces a power cap | |
=============================================================== | |
geopm::NodePowerGovernorAgent(3) -- agent that enforces a node power cap | |
======================================================================== |
|
||
#include `<geopm/NodePowerGovernorAgent.hpp> <https://github.com/geopm/geopm/blob/dev/src/NodePowerGovernorAgent.hpp>`_ | ||
|
||
Link with ``-lgeopm`` **(MPI)** or ``-lgeopmpolicy`` **(non-MPI)** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think geopmpolicy is gone now
geopm_agent_power_governor(7) -- agent enforces a power cap | ||
=========================================================== |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
geopm_agent_power_governor(7) -- agent enforces a power cap | |
=========================================================== | |
geopm_agent_node_power_governor(7) -- agent enforces a node power cap | |
===================================================================== |
----------- | ||
|
||
The :doc:`geopm::NodePowerGovernorAgent(3) <GEOPM_CXX_MAN_NodePowerGovernorAgent.3>` enforces a per-compute-node power cap of the | ||
total platform power systems that support the Platform Power Limit feature. The platform power is determined by the platform vendor |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
total platform power systems that support the Platform Power Limit feature. The platform power is determined by the platform vendor | |
total platform power on systems that support the Platform Power Limit feature. The platform power is determined by the platform vendor |
The agent gates the Controller's control loop to a cadence of *5ms* | ||
(milliseconds). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This accurately describes the current revision of the agent. But maybe the agent should use the Waiter class, which has a configurable control loop gate.
2cd441d
to
0859ce3
Compare
Addition of a Node Power Governor Agent based upon the existing Power Governor Agent. Does not include a NodePowerGovernor
Additional Tasks (may be future PRs):