net: correctly interlink lower/upper devices
authorVeaceslav Falico <vfalico@redhat.com>
Mon, 2 Sep 2013 14:26:51 +0000 (16:26 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 4 Sep 2013 04:27:26 +0000 (00:27 -0400)
Currently we're linking upper devices to lower ones, which results in
upside-down relationship: upper devices seeing lower devices via its upper
lists.

Fix this by correctly linking lower devices to the upper ones.

CC: "David S. Miller" <davem@davemloft.net>
CC: Eric Dumazet <edumazet@google.com>
CC: Jiri Pirko <jiri@resnulli.us>
CC: Alexander Duyck <alexander.h.duyck@intel.com>
CC: Cong Wang <amwang@redhat.com>
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c

index 07684e8..5c713f2 100644 (file)
@@ -4679,8 +4679,8 @@ static int __netdev_upper_dev_link(struct net_device *dev,
         * versa, and don't forget the devices itself. All of these
         * links are non-neighbours.
         */
-       list_for_each_entry(i, &upper_dev->upper_dev_list, list) {
-               list_for_each_entry(j, &dev->lower_dev_list, list) {
+       list_for_each_entry(i, &dev->lower_dev_list, list) {
+               list_for_each_entry(j, &upper_dev->upper_dev_list, list) {
                        ret = __netdev_adjacent_dev_link(i->dev, j->dev);
                        if (ret)
                                goto rollback_mesh;