Thread (6 messages) 6 messages, 3 authors, 2013-01-29

Re: Can we replace the msleep(250) in dev.c netdev_wait_allrefs?

From: David Miller <davem@davemloft.net>
Date: 2013-01-29 20:31:17

From: Ben Greear <redacted>
Date: Tue, 29 Jan 2013 12:24:19 -0800
On 01/29/2013 12:21 PM, Eric Dumazet wrote:
quoted
On Tue, 2013-01-29 at 12:00 -0800, Ben Greear wrote:
quoted
I think there was a proposal to change it to
schedule_timeout_uninterruptible(1);
some time ago, but that change never made it upstream (at least not in
3.7.5)?

Thanks,
Ben
Honestly we should fix the bug at this point, not hide it.

We could use msleep(2500) to make sure people will complain ;)
Well, few will notice that either, or at least are unlikely
to know what to blame.

If we should just never reach that code, maybe a WARN_ON_ONCE() when
we do?
If you only notice some time in the future that something has hung in
this state, the original message could be lost.  Therefore, I
definitely prefer that this thing keeps barking into the logs
periodically.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help