From mboxrd@z Thu Jan 1 00:00:00 1970 From: Casey Leedom Subject: Re: [PATCH][RFC] cxgb4: Not need to hold the adap_rcu_lock lock when read adap_rcu_list Date: Mon, 23 Jun 2014 15:35:25 -0700 Message-ID: <53A8ABAD.9090904@chelsio.com> References: <1403256756-6557-1-git-send-email-rongqing.li@windriver.com> <20140623.145035.915335524137481573.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, eric.dumazet@gmail.com, hariprasad@chelsio.com, greearb@candelatech.com To: David Miller , rongqing.li@windriver.com Return-path: Received: from 99-65-72-227.uvs.sntcca.sbcglobal.net ([99.65.72.227]:41475 "EHLO stargate3.asicdesigners.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750931AbaFWWfc (ORCPT ); Mon, 23 Jun 2014 18:35:32 -0400 In-Reply-To: <20140623.145035.915335524137481573.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On 06/23/14 14:50, David Miller wrote: > From: > Date: Fri, 20 Jun 2014 17:32:36 +0800 > >> cxgb4_netdev maybe lead to dead lock, since it uses a spin lock, and be called >> in both thread and softirq context, but not disable BH, the lockdep report is >> below; In fact, cxgb4_netdev only reads adap_rcu_list with RCU protection, so >> not need to hold spin lock again. > I think this change is fine, and correct, but I would like to see some > reviews from the cxgb4 maintainers. Thanks David. Hari is gone on PTO so I think I'm the next logical person ... :-) I've gone back and reviewed the original patch, Eric Dumazet6's reply and revised patch and compared that against this proposed patch. Li RongQing is submitting the same patch that Eric suggested with the addition of a call to synchronize_rcu() the in driver remove() function. I'm not super familiar with the RCU system but that addition certainly seems innocuous enough. Other than that, everything looks fine.