[lkml]   [2012]   [Jul]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 2/7] netpoll: make __netpoll_cleanup non-block
    On Fri, 2012-07-27 at 14:40 -0400, Neil Horman wrote:
    > Here, and above I see you using an rcu_head to defer cleanup, until after all
    > pointer uses are dropped, but I don't see any modification of code points that
    > dereference any struct netpoll pointers to include
    > rcu_read_lock()/rcu_read_unlock(). Without those using rcu to defer cleanup is
    > pointless, as the rcu code won't know when its safe to run. You're no better
    > off that you would be just calling __netpoll_cleanup directly.

    Hi, Neil,

    Actually they are protected by rcu_read_lock_bh() which is implied by
    local_irq_disable(), see:

    commit f0f9deae9e7c421fa0c1c627beb8e174325e1ba7
    Author: Herbert Xu <>
    Date: Fri Sep 17 16:55:03 2010 -0700

    netpoll: Disable IRQ around RCU dereference in netpoll_rx

    We cannot use rcu_dereference_bh safely in netpoll_rx as we may
    be called with IRQs disabled. We could however simply disable
    IRQs as that too causes BH to be disabled and is safe in either

    Thanks to John Linville for discovering this bug and providing
    a patch.

    Signed-off-by: Herbert Xu <>
    Signed-off-by: David S. Miller <>


     \ /
      Last update: 2012-07-30 04:21    [W:0.021 / U:40.368 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site