netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH stable] Revert "dev: set iflink to 0 for virtual interfaces"
@ 2015-08-25 21:09 Stephen Hemminger
  2015-08-25 21:28 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: Stephen Hemminger @ 2015-08-25 21:09 UTC (permalink / raw)
  To: David Miller, Greg KH, Nicolas Dichtel; +Cc: netdev

Dave could you include this in next update to 4.1 stable update?

>From a72f813bf1aab4dfe47e3d4ab5e2e07bbcdf2b58 Mon Sep 17 00:00:00 2001
From: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Date: Mon, 6 Jul 2015 17:25:10 +0200
Subject: [PATCH] Revert "dev: set iflink to 0 for virtual interfaces"

commit 95ec655bc465ccb2a3329d4aff9a45e3c8188db5 upstream.
It should be applied to 4.1.y stable tree.

This reverts commit e1622baf54df8cc958bf29d71de5ad545ea7d93c.

The side effect of this commit is to add a '@NONE' after each virtual
interface name with a 'ip link'. It may break existing scripts.

Reported-by: Olivier Hartkopp <socketcan@hartkopp.net>
Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Tested-by: Oliver Hartkopp <socketcan@hartkopp.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
---
 net/core/dev.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/net/core/dev.c b/net/core/dev.c
index aa82f9a..8c416c99 100644
--- a/net/core/dev.c
+++ b/net/core/dev.c
@@ -672,10 +672,6 @@ int dev_get_iflink(const struct net_device *dev)
 	if (dev->netdev_ops && dev->netdev_ops->ndo_get_iflink)
 		return dev->netdev_ops->ndo_get_iflink(dev);
 
-	/* If dev->rtnl_link_ops is set, it's a virtual interface. */
-	if (dev->rtnl_link_ops)
-		return 0;
-
 	return dev->ifindex;
 }
 EXPORT_SYMBOL(dev_get_iflink);
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH stable] Revert "dev: set iflink to 0 for virtual interfaces"
  2015-08-25 21:09 [PATCH stable] Revert "dev: set iflink to 0 for virtual interfaces" Stephen Hemminger
@ 2015-08-25 21:28 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2015-08-25 21:28 UTC (permalink / raw)
  To: stephen; +Cc: greg, nicolas.dichtel, netdev

From: Stephen Hemminger <stephen@networkplumber.org>
Date: Tue, 25 Aug 2015 14:09:58 -0700

> Dave could you include this in next update to 4.1 stable update?

Queued up, thanks Stephen.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-08-25 21:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-25 21:09 [PATCH stable] Revert "dev: set iflink to 0 for virtual interfaces" Stephen Hemminger
2015-08-25 21:28 ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).