bisdn / basebox

A tiny OpenFlow controller for OF-DPA switches.
Mozilla Public License 2.0
45 stars 9 forks source link

properly fix race in port tap device creation on of connection #257

Closed KanjiMonster closed 4 years ago

KanjiMonster commented 4 years ago

The fix in #254 was not enough due to a misunderstanding of how rofl-common works. So instead of trying to figure out the place where to enable netlink processing without missing messages, just enable it always.

Description

Mostly see #254, but now enable when instantiating nbi_impl, instead of waiting for the signal that the switch is ready.

Motivation and Context

See #254

How Has This Been Tested?

Two rounds of running the testing pipeline (with the pending carrier changes) on the ag7648, one round without the changes, build pipeline 9455 (currently running).