sflow: Always add poller and sampler together.
authorNeil McKee <neil.mckee@inmon.com>
Wed, 5 May 2010 20:26:23 +0000 (13:26 -0700)
committerBen Pfaff <blp@nicira.com>
Wed, 5 May 2010 20:27:11 +0000 (13:27 -0700)
commit5586445e9945574da85f96976ca650e19b8216c1
tree1f03cdbddd968955686065edfec2b7f2e8b7f5ea
parentf9727d74050e1ff1c25ad0308a46e84c4d4870b5
sflow: Always add poller and sampler together.

he ofproto_sflow_add_poller() and ofproto_sflow_add_sampler() calls
should always be made together, either when a port is added
dynamically with ofproto_sflow_add_port() or when the sflow_agent is
first created in ofproto_sflow_set_options().  I was seeing odd
behavior where either the pollers or the samplers would never be
instantiated depending on the order that things happened.  (It's OK to
add the same sampler or poller again, because the library routines
sfl_agent_addPoller() and sfl_agent_addSampler() will just return the
existing one if it is there.  Perhaps we should add a comment to make
that clear?).

I changed the parameters to ofproto_sflow_add_sampler to make it work
the same way as ofproto_sflow_add_poller, where the options are
extracted from os->options within the function itself.
ofproto/ofproto-sflow.c