True. My mistake. I was looking at Simon Horman’s private branch. Thanks for the quick turnaround.
Regards, Hexin On 4/24/17, 10:11 AM, "Ben Pfaff" <[email protected]> wrote: >On Mon, Apr 24, 2017 at 04:30:51PM +0000, Hexin Wang wrote: >> In ovs 2.6.90, we are seeing NULL reference to in_port if >> lookup_input_bundle() sets in_port to NULL. >> >> /* Learn source MAC. */ >> if (ctx->xin->may_learn && !in_port->is_layer3) { >> update_learning_table(ctx->xbridge, flow, wc, vlan, in_xbundle); >> } >> >> This happens when a flow being processed in xlate_normal is OFPP_NONE. >> (gdb) print ctx->xin->flow.in_port >> >> $5 = {odp_port = 65535, ofp_port = 65535} >> >> We don’t see this issue on 2.5.0. Is ofp_port 65535 expected in >> xlate_normal()? > >As far as I can tell, the code you cite never existed in the Open >vSwitch repository: only a single commit against master ever added the >string "is_layer3" to the code base, and not in the same file as any >reference to update_learning_table(). I guess that you are using some >fork of OVS that added it. In that case, probably you need to ask >whoever supplied the fork. _______________________________________________ discuss mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-discuss
