<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"></blockquote><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"></blockquote>This capability already exists if you're using phosphor-pid-control<br><blockquote class="gmail_quote" style="margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>with entity manager<br><blockquote class="gmail_quote" style="margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>..<br><blockquote class="gmail_quote" style="margin:0px 0.8ex;border-left:1px solid rgb(204,204,204);border-right:1px solid rgb(204,204,204);padding-left:1ex;padding-right:1ex"></blockquote>Entity manager essentially will trigger a reconfigure of<br>phosphor-pid-control on every change, so new parameters can be tested. </blockquote><div> </div><div>I see. Let me make sure my mental model is correct.</div><div>You can modify PID configuration parameters published by entity-manager via dbus (is this a separate dbus path/interface than the pid-configuration interface?).</div><div>Entity-manager detects the property change/method call and then updates its pid-configuration on dbus.</div><div>phosphor-pid-control is listening on a signal/event and then essentially restarts itself and picks up the new configuration?</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I'm guessing you didn't know that existed, and clearly Patrick didn't<br>either.  So the next question is, where did you go to look for this<br>kind of thing (ie, where should we document this)?  There might not be<br>a perfect place, but hopefully we can make this more clear in the<br>future when people have these needs.</blockquote><div><br></div><div>I sure didn't, thanks for pointing this out.  As far as documentation improvements:</div><div><a href="https://github.com/openbmc/phosphor-pid-control/blob/master/configure.md">How to configure phosphor-pid-control</a> should probably mention something about the capability of configuring phosphor-pid-control via dbus.</div><div>As far as dbus pid-tuning goes, I suppose there should be some mention of it in <a href="https://github.com/openbmc/phosphor-pid-control/blob/master/tuning.md">https://github.com/openbmc/phosphor-pid-control/blob/master/tuning.md</a> .. I guess something along the lines of "...dbus configuration changes will be reflected in the operation of phosphor-pid-control.".</div><div>Intentionally left a bit vague because it doesn't HAVE to be entity-manager providing this configuration interface. It could be some other service that reads a config.json and publishes this information to dbus...</div><div>Which brings me to my next question</div><div>"Is the PID configuration interface formally defined as part of phosphor-dbus-interfaces?"</div><div><br></div><div><br></div><div>Thanks for the brain-dump Ed, it's making me rethink my approach. Adding yet another tuning interface has a clear potential to bloat and over complicate phosphor-pid-control. It seems like even if a system was using phosphor-hwmon for sensor telemetry monitoring, you could still use entity-manager just to publish and modify PID configurations.</div><div><br></div><div><br></div></div>