mac802154: Rename kfree_rcu() to kvfree_rcu_mightsleep()

The k[v]free_rcu() macro's single-argument form is deprecated.
Therefore switch to the new k[v]free_rcu_mightsleep() variant. The goal
is to avoid accidental use of the single-argument forms, which can
introduce functionality bugs in atomic contexts and latency bugs in
non-atomic contexts.

The callers are holding a mutex so the context allows blocking. Hence
using the API with a single argument will be fine, but use its new name.

There is no functionality change with this patch.

Fixes: 57588c71177f ("mac802154: Handle passive scanning")
Acked-by: Stefan Schmidt <stefan@datenfreihafen.org>
Reviewed-by: Paul E. McKenney <paulmck@kernel.org>
Signed-off-by: Joel Fernandes (Google) <joel@joelfernandes.org>

diff --git a/net/mac802154/scan.c b/net/mac802154/scan.c
index 9b0933a185ebc9981ff8c64478742f7f6e9d8e9e..5c191bedd72c4440b4ddffdef8848c2011451e8b 100644 (file)
--- a/net/mac802154/scan.c
+++ b/net/mac802154/scan.c
@@ -52,7 +52,7 @@ static int mac802154_scan_cleanup_locked(struct ieee802154_local *local,
        request = rcu_replace_pointer(local->scan_req, NULL, 1);
        if (!request)
                return 0;
-       kfree_rcu(request);
+       kvfree_rcu_mightsleep(request);
 
        /* Advertize first, while we know the devices cannot be removed */
        if (aborted)
@@ -403,7 +403,7 @@ int mac802154_stop_beacons_locked(struct ieee802154_local *local,
        request = rcu_replace_pointer(local->beacon_req, NULL, 1);
        if (!request)
                return 0;
-       kfree_rcu(request);
+       kvfree_rcu_mightsleep(request);
 
        nl802154_beaconing_done(wpan_dev);