From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Gospodarek Subject: Re: [PATCHv2] bond_alb: don't disable softirq under bond_alb_xmit Date: Mon, 9 Jan 2012 17:20:46 -0500 Message-ID: <20120109222046.GC32485@quad.redhat.com> References: <1326146497-27711-1-git-send-email-maxim.uvarov@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, fubar@us.ibm.com, andy@greyhouse.net, davem@davemloft.net, Cong Wang To: Maxim Uvarov Return-path: Received: from mx1.redhat.com ([209.132.183.28]:19893 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933246Ab2AIWUz (ORCPT ); Mon, 9 Jan 2012 17:20:55 -0500 Content-Disposition: inline In-Reply-To: <1326146497-27711-1-git-send-email-maxim.uvarov@oracle.com> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, Jan 09, 2012 at 02:01:37PM -0800, Maxim Uvarov wrote: > No need to lock soft irqs under bond_alb_xmit() > which already has softirq disabled. > > Changes: > 1. add non-bh/bh version to tlb_clear_slave() > > 2. represent BH and non BH hash table locks > _lock_rx_hashtbl_bh/_unlock_rx_hashtbl_bh > _lock_rx_hashtbl/_unlock_rx_hashtbl > _lock_tx_hashtbl_bh/_unlock_tx_hashtbl_bh > _lock_tx_hashtbl/_unlock_tx_hashtbl > > Signed-off-by: Maxim Uvarov > Signed-off-by: Cong Wang > --- > drivers/net/bonding/bond_alb.c | 112 +++++++++++++++++++++++++++------------- > 1 files changed, 76 insertions(+), 36 deletions(-) > Did you test this with CONFIG_DEBUG_LOCKDEP=y at all?