On Thu, Jul 13, 2017 at 11:44:51PM -0700, Han Zhou wrote: > This reverts commit 67ac844b55d4c5f6bbfa01773c82b3d6d8b62131. > > The commit introduced a problem that "File exists" will be reported > when trying to open br0.
Earlier today I applied Eelco's additional fix (appended below). I hope that it resolves all the issues. If it doesn't, I think that we will have to revert both this new commit and the one that you propose to revert as well. It seems like this area should not be so tricky. I am a bit frustrated. commit 8c2c225e481d6560faf73bb49c33f026bc58a664 Author: Eelco Chaudron <[email protected]> Date: Fri Jul 14 14:33:27 2017 +0200 netdev: Fix netdev_open() to track and recreate classless interfaces Due to commit 67ac844 an existing issue with OVS persisten ports surfaced. If we revert the commit we no longer get the error, and basic traffic will flow. However the wrong netdev class is used, hence the wrong callbacks get called. The main issue is with netdev_open() being called with type = NULL before the interface is actually configured in the system. This patch tracks these "auto" generated interfaces, and once netdev_open() gets called with a valid type, re-configures (re-create) it. Signed-off-by: Eelco Chaudron <[email protected]> Signed-off-by: Ben Pfaff <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
