From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bart Van Assche Subject: Re: [PATCH-v2 0/4] target: Eliminate se_port + t10_alua_tg_pt_gp_member Date: Tue, 26 May 2015 14:44:14 +0200 Message-ID: <55646A9E.5050401@sandisk.com> References: <1432623445-25776-1-git-send-email-nab@daterainc.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1432623445-25776-1-git-send-email-nab@daterainc.com> Sender: target-devel-owner@vger.kernel.org To: "Nicholas A. Bellinger" , target-devel Cc: linux-scsi , linux-kernel , Christoph Hellwig , Hannes Reinecke , Sagi Grimberg , "Paul E. McKenney" , Nicholas Bellinger List-Id: linux-scsi@vger.kernel.org On 05/26/15 08:57, Nicholas A. Bellinger wrote: > - Add various rcu_dereference and lockless_dereference RCU notation Hello Nic, Feedback from an RCU expert (which I'm not) would be appreciated here. But my understanding is that lockless_dereference(p) should be used for a pointer p that has *not* been annotated as an RCU pointer. I think in the for-next branch of the target repository that this macro is used to access RCU-annotated pointers. Is that why sparse complains about how lockless_dereference() is used in the target tree ? Bart.