net/core: ensure features get disabled on new lower devs
net/core: ensure features get disabled on new lower devs

With moving netdev_sync_lower_features() after the .ndo_set_features
calls, I neglected to verify that devices added *after* a flag had been
disabled on an upper device were properly added with that flag disabled as
well. This currently happens, because we exit __netdev_update_features()
when we see dev->features == features for the upper dev. We can retain the
optimization of leaving without calling .ndo_set_features with a bit of
tweaking and a goto here.

Fixes: fd867d51f889 ("net/core: generic support for disabling netdev features down stack")
CC: "David S. Miller" <fe08d3c717adf2ae63592e4c9aec6e3e404d8e3e@davemloft.net>
CC: Eric Dumazet <312c7d941f03386814be517ffdb3e0eb6624e275@google.com>
CC: Jay Vosburgh <91230355d07759b3db8fc654d247958e4c2cec2b@gmail.com>
CC: Veaceslav Falico <13e4c2c2c15eff98ad47d02f3ccee2d83dc7015d@gmail.com>
CC: Andy Gospodarek <235a7d7680e417ef53409d6a24f3311a70688752@cumulusnetworks.com>
CC: Jiri Pirko <1b548d10035ae428195c41f5d55eedb1437ccd8a@resnulli.us>
CC: Nikolay Aleksandrov <966b74740c67c4df944e25f0a65bfdba363503f7@blackwall.org>
CC: Michal Kubecek <48ad0bdf47651f7a71905c605c8960f443623cd5@suse.cz>
CC: Alexander Duyck <14b16e1046020048454db6cd357f0c60699c86fc@gmail.com>
CC: 1099b3bee480025a15c1b622ecdea77a1d6d7166@vger.kernel.org
Reported-by: Nikolay Aleksandrov <262b42b771725e3ebec40395f2b058e593df7462@cumulusnetworks.com>
Signed-off-by: Jarod Wilson <14059c7158912dcae849c377925c87d46da48cdf@redhat.com>
Signed-off-by: David S. Miller <fe08d3c717adf2ae63592e4c9aec6e3e404d8e3e@davemloft.net>

KristFoundation/Programs,KristFoundation/Programs,KristFoundation/Programs,KristFoundation/Programs,KristFoundation/Programs,KristFoundation/Programs