[Devel] [PATCH rh7] net: drop direct dst_gc_task() call on netdev shutdown

Konstantin Khorenko khorenko at virtuozzo.com
Mon Dec 18 17:28:57 MSK 2017


It was added on a rebase from 2.6.32-x to 3.10.0-x as a part
of big (cumulative) patch.

i suspect it was added as a part of work of getting rid of
leaking network devices, but we have not seen that problem
for a long time already and with this call added
a Container is stopped slower (waits for dst_gc_task() finish).

Let's drop this call now.
If we ever get new leaking network devices, we'll see
appropriate messages and dig the problem.

Signed-off-by: Konstantin Khorenko <khorenko at virtuozzo.com>
---
 net/core/dst.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/net/core/dst.c b/net/core/dst.c
index 8f3026c..95482da 100644
--- a/net/core/dst.c
+++ b/net/core/dst.c
@@ -435,7 +435,6 @@ static int dst_dev_event(struct notifier_block *this, unsigned long event,
 	switch (event) {
 	case NETDEV_UNREGISTER_FINAL:
 	case NETDEV_DOWN:
-		dst_gc_task(NULL);
 		mutex_lock(&dst_gc_mutex);
 		for (dst = dst_busy_list; dst; dst = dst->next) {
 			last = dst;
-- 
2.1.4



More information about the Devel mailing list