Re: [PATCH v7] nl80211: Stop scheduled scan if netlink client disappears

[Johannes Berg <johannes@sipsolutions.net>]

On Wed, 2014-11-26 at 11:07 +0200, Jukka Rissanen wrote:
> An attribute NL80211_ATTR_SOCKET_OWNER can be set by the scan initiator.
> If present, the attribute will cause the scan to be stopped if the client
> dies.
> 
> Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
> ---
> Hi,
> 
> v7:
> - convert the cfg80211_sched_scan_request to __rcu pointer in order
>   to avoid races when accessing it

You totally need to revisit how to use RCU.

Just some examples:
 * don't synchronize_rcu() after assignment - what did you do that for?
 * don't use rcu_read_lock() when you already have the write lock (rtnl)
 * don't use rcu_access_pointer(), use
    - local variables
    - rcu_dereference_protected() (or rather rtnl_dereference())
    - rcu_dereference()
 * actually fix it to assign the values before rcu_assign_pointer() so
they're
   all published together ...

Also, please make that a separate patch so it isn't mixed with the
functional changes.

johannes