Enable/Disable Flow table lookup for incoming packets

[twood02]

This issue describes a simple bug/improvement for the OpenNetVM code, and is a great way for YOU to get involved with improving this open source project! If you want to fix it, just create a pull request with the fix as detailed in the Contributing document. If you have questions, post here!

---

By default, ONVM does a flow table lookup for all newly arrived packets (using our Flow Director API). However, many users don't use the built-in flow table, so this is wasteful since packets just get sent to the default (service ID 1).

Instead, this should be configurable either as a compile time variable or a command line argument to control whether a lookup is performed or not.

The code to change is here:

openNetVM/onvm/onvm_mgr/onvm_pkt.c

Lines 73 to 81 in fcac247

	ret = onvm_flow_dir_get_pkt(pkts[i], &flow_entry);
	if (ret >= 0) {
	sc = flow_entry->sc;
	meta->action = onvm_sc_next_action(sc, pkts[i]);
	meta->destination = onvm_sc_next_destination(sc, pkts[i]);
	} else {
	meta->action = onvm_sc_next_action(default_chain, pkts[i]);
	meta->destination = onvm_sc_next_destination(default_chain, pkts[i]);
	}

[harsh503]

For default case, should it perform a flow table lookup or not?

[twood02]

The default should be the same as now - a flow lookup should happen.

[kevindweb]

This is done, we should close the issue.

[koolzz]

Good point, use your powers @kevindweb