[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    [W:0.020 / U:15.008 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site