Skip to content

Commit

Permalink
Don't set governors parameters when getting them
Browse files Browse the repository at this point in the history
Every time the governor parameter were fetched, they would be
deserialized from the attribute, then serialized again to the same
values. Stop doing that.
  • Loading branch information
lmoureaux committed Jun 30, 2024
1 parent 9254ee0 commit cccb608
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions client/governor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -701,10 +701,8 @@ void cmafec_get_fe_parameter(struct city *pcity, struct cm_parameter *dest)
{
struct cm_parameter parameter;

// our fe_parameter could be stale. our agents parameter is uptodate
if (cma_is_city_under_agent(pcity, &parameter)) {
cm_copy_parameter(dest, &parameter);
cmafec_set_fe_parameter(pcity, dest);
} else {
// Create a dummy parameter to return.
cm_init_parameter(dest);
Expand Down

0 comments on commit cccb608

Please sign in to comment.