From: Shannon Nelson <shannon.nel...@oracle.com> Date: Tue, 6 Mar 2018 14:57:08 -0800
> This isn't broken for vlans because they use a separate features > connection (vlan_features) for inheriting features. This is > fine, but I don't think trying to add something like this to > every driver for every new upperdev is a good idea - I think > the upperdev should try to protect itself. I think this fix is correct. But for how many upperdevs are we going to duplicate code like this, and how many subtle differences and in fact bugs will result from all of that duplication? I think we really need something common for these upperdev drivers to use. Maybe just a macro defining feature bits to trim in this situation. Thanks.
