A debug message was supposed to print the node name, old state, new state, and reason for the change, in that order. However, the parameters that were printed were out of order. This resulted in nonsense output.
Signed-off-by: Mark Michelson <mmich...@redhat.com> --- lib/inc-proc-eng.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/inc-proc-eng.c b/lib/inc-proc-eng.c index 3bcdf95ad..78187c8fd 100644 --- a/lib/inc-proc-eng.c +++ b/lib/inc-proc-eng.c @@ -325,9 +325,8 @@ engine_set_node_state(struct engine_node *node, va_start(args, reason_fmt); char *reason = xvasprintf(reason_fmt, args); VLOG_DBG("node: %s, old_state %s, new_state %s, reason: %s.", - reason, node->name, - engine_node_state_name[node->state], - engine_node_state_name[state]); + node->name, engine_node_state_name[node->state], + engine_node_state_name[state], reason); va_end(args); free(reason); } -- 2.49.0 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev