Return-path: Received: from s3.sipsolutions.net ([5.9.151.49]:34250 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966237AbdEWHWg (ORCPT ); Tue, 23 May 2017 03:22:36 -0400 Message-ID: <1495524153.2464.2.camel@sipsolutions.net> (sfid-20170523_092254_882502_A847A7B6) Subject: Re: 4.12-RC2 BUG: scheduling while atomic: irq/47-iwlwifi From: Johannes Berg To: Arend Van Spriel , Sander Eikelenboom Cc: linux-wireless , netdev@vger.kernel.org Date: Tue, 23 May 2017 09:22:33 +0200 In-Reply-To: <09ddb018-7093-2e2a-c84b-148889f7f06d@broadcom.com> (sfid-20170523_091928_491554_FD588B32) References: <1495450628.2653.14.camel@sipsolutions.net> <764a929c-ce8a-c859-a49e-2f20cb05ae44@broadcom.com> <532c257e-52a0-18c1-1afe-04d37c28e072@broadcom.com> <1495487095.26008.7.camel@sipsolutions.net> <09ddb018-7093-2e2a-c84b-148889f7f06d@broadcom.com> (sfid-20170523_091928_491554_FD588B32) Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, 2017-05-23 at 09:19 +0200, Arend Van Spriel wrote: > > > WARN_ON_ONCE(!rcu_read_lock_held() && !lockdep_rtnl_is_held()); > > Thought about something like this after sending the email. So there > are two call sites. One for scheduled scan results notification and > one in scheduled scan stop scenario. So for the latter it is not > needed to use the rcu_read_lock() as it should have RTNL lock hence > the two checks above? Right. The latter can't even really use rcu_read_lock() since it also wants to modify the list, and that's not sufficient protection for modifying. Thanks! johannes