[ovs-dev] [PATCH] netflow: Populate NetFlow output interface field.
Jesse Gross
jesse at nicira.com
Tue Oct 27 20:52:50 UTC 2009
Justin Pettit wrote:
>> case OFPP_CONTROLLER:
>> add_controller_action(ctx->out, oao);
>> + ctx->nf_output_iface = ODPP_LOCAL;
>
> Shouldn't this line be below for OFPP_LOCAL?
>
> I spoke with Pete about what should be done with packets that are only
> sent to the controller, and he suggested that we don't generate
> NetFlow records for them.
>
This was correct give what I was intending to do (report controller
traffic as local) since add_output_action already sets the output
interface. However, now I'm not generating NetFlow records for
controller traffic.
>> break;
>> case OFPP_LOCAL:
>> add_output_action(ctx, ODPP_LOCAL);
>
> Also, I think it would be good to document the special meaning of
> these ports. Perhaps in the ovs-vswitch.conf man page. While you're
> in there, it might be nice to describe that when the "add-id-to-iface"
> flag is enabled that the maximum number of supported ports is going to
> around 509.
>
Good idea, I updated the man page.
I went ahead and pushed it with these modifications. There may be
additional NetFlow modifications that we want but those are independent
of this.
More information about the dev
mailing list