Re: [PATCH 2/2] ixgbe: add missing rtnl_lock in PM resume path

From: Jeff Kirsher
Date: Thu Apr 05 2012 - 19:03:10 EST


On Thu, 2012-04-05 at 11:11 -0400, Benjamin Poirier wrote:
> Upon resume from standby, ixgbe may trigger the ASSERT_RTNL() in
> netif_set_real_num_tx_queues(). The call stack is:
> netif_set_real_num_tx_queues
> ixgbe_set_num_queues
> ixgbe_init_interrupt_scheme
> ixgbe_resume
>
> Signed-off-by: Benjamin Poirier <bpoirier@xxxxxxx>
>
> ---
> Unlike patch 1 for igb, we cannot lock rtnl closer to the point where
> netif_set_real_num_tx_queues() is called because
> ixgbe_init_interrupt_scheme()
> is also called by the dcb code, already under rtnl.
> ---
> drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)

Thanks Benjamin, I will add the patch to my igb queue of patches.

Attachment: signature.asc
Description: This is a digitally signed message part