| To: | "'Paul Rolland'" <rol@xxxxxxxxxx>, "'Stephen Hemminger'" <shemminger@xxxxxxxx>, <cfriesen@xxxxxxxxxxxxxxxxxx>, <paulus@xxxxxxxxx> |
|---|---|
| Subject: | Re: [BUG]: problem when shutting down ppp connection since 2.5.70 |
| From: | "Paul Rolland" <rol@xxxxxxxxxx> |
| Date: | Mon, 14 Jul 2003 13:43:51 +0200 |
| Cc: | <linux-ppp@xxxxxxxxxxxxxxx>, <netdev@xxxxxxxxxxx> |
| Importance: | Normal |
| In-reply-to: | |
| Sender: | netdev-bounce@xxxxxxxxxxx |
Hello,
I'm sorry, the patch was not complete, it should have considered the
BUG_ON too...
Here is one that is fine on my system :
--- dev.c.orig 2003-07-14 13:41:33.000000000 +0200
+++ dev.c 2003-07-14 13:34:27.000000000 +0200
@@ -2742,7 +2742,7 @@
unsigned long rebroadcast_time, warning_time;
rebroadcast_time = warning_time = jiffies;
- while (atomic_read(&dev->refcnt) != 0) {
+ while (atomic_read(&dev->refcnt) > 0) {
if (time_after(jiffies, rebroadcast_time + 1 * HZ)) {
rtnl_shlock();
rtnl_exlock();
@@ -2836,7 +2836,7 @@
dev->reg_state = NETREG_UNREGISTERED;
netdev_wait_allrefs(dev);
- BUG_ON(atomic_read(&dev->refcnt));
+ BUG_ON(atomic_read(&dev->refcnt) > 0);
netdev_finish_unregister(dev);
break;
Still don't understand why refcnt is really bad (negative value),
but at least the machine is working...
Paul
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
I have a vitally important role serving as a bad example.
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [BUG]: problem when shutting down ppp connection since 2.5.70, Paul Rolland |
|---|---|
| Next by Date: | [PATCH] IPVS' Kconfig LBLC and LBLCR configuration typo, Alberto Bertogli |
| Previous by Thread: | Re: [BUG]: problem when shutting down ppp connection since 2.5.70, Paul Rolland |
| Next by Thread: | [PATCH] Don't call request_module() under spinlock in xfrm_get_type(), James Morris |
| Indexes: | [Date] [Thread] [Top] [All Lists] |