[lkml]   [2007]   [Jun]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRE: [PATCH -rt] Preemption problem in kernel RT Patch

On Sat, 2007-06-23 at 10:08 -0400, Beauchemin, Mark wrote:
> Thanks for straightening me out. I was reading the function
> try_to_take_rt_mutex wrong... The problem makes more sense now. The tunnel
> code encapsulates the current packet in a new packet and calls ip_output
> to get it to the destination. If the routing table is changing(which
> I'm doing when this happens) it could be called recursively. The tunnel
> code tries to handle recursion at the top of ipip_tunnel_xmit:
> if (tunnel->recursion++) {
> tunnel->stat.collisions++;
> goto tx_error;
> }
> The problem is it tries to take dev->lock which it already owns in
> dev_queue_xmit before the check for recursion.

Hmm, this sounds scary. On a vanilla kernel (with debugging disabled),
this code will simply deadlock.

Do you have a test case? If you need more help, please contact the
netdev folks (

> Unfortunately, every time I put in debug to see the routing
> changes which cause the bug, it doesn't happen. I'll certainly try to
> reproduce it with CONFIG_PROVE_LOCKING on, but it won't be till end of next
> week as we have a release going out.

Well, you won't see much more than you already debugged. You see the
place where the lock was taken and the call trace of the function in the
same way you have seen it with the BUG_ON().


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2007-06-23 16:29    [from the cache]
©2003-2014 Jasper Spaans. hosted at Digital Ocean