1450af8d0SKP Singh // SPDX-License-Identifier: GPL-2.0 2450af8d0SKP Singh /* Copyright (c) 2019 Facebook */ 3450af8d0SKP Singh #include <linux/rculist.h> 4450af8d0SKP Singh #include <linux/list.h> 5450af8d0SKP Singh #include <linux/hash.h> 6450af8d0SKP Singh #include <linux/types.h> 7450af8d0SKP Singh #include <linux/spinlock.h> 8450af8d0SKP Singh #include <linux/bpf.h> 9450af8d0SKP Singh #include <linux/btf_ids.h> 10450af8d0SKP Singh #include <linux/bpf_local_storage.h> 11450af8d0SKP Singh #include <net/sock.h> 12450af8d0SKP Singh #include <uapi/linux/sock_diag.h> 13450af8d0SKP Singh #include <uapi/linux/btf.h> 140fe4b381SKP Singh #include <linux/rcupdate.h> 150fe4b381SKP Singh #include <linux/rcupdate_trace.h> 160fe4b381SKP Singh #include <linux/rcupdate_wait.h> 17450af8d0SKP Singh 18450af8d0SKP Singh #define BPF_LOCAL_STORAGE_CREATE_FLAG_MASK (BPF_F_NO_PREALLOC | BPF_F_CLONE) 19450af8d0SKP Singh 20450af8d0SKP Singh static struct bpf_local_storage_map_bucket * 21450af8d0SKP Singh select_bucket(struct bpf_local_storage_map *smap, 22450af8d0SKP Singh struct bpf_local_storage_elem *selem) 23450af8d0SKP Singh { 24450af8d0SKP Singh return &smap->buckets[hash_ptr(selem, smap->bucket_log)]; 25450af8d0SKP Singh } 26450af8d0SKP Singh 27450af8d0SKP Singh static int mem_charge(struct bpf_local_storage_map *smap, void *owner, u32 size) 28450af8d0SKP Singh { 29450af8d0SKP Singh struct bpf_map *map = &smap->map; 30450af8d0SKP Singh 31450af8d0SKP Singh if (!map->ops->map_local_storage_charge) 32450af8d0SKP Singh return 0; 33450af8d0SKP Singh 34450af8d0SKP Singh return map->ops->map_local_storage_charge(smap, owner, size); 35450af8d0SKP Singh } 36450af8d0SKP Singh 37450af8d0SKP Singh static void mem_uncharge(struct bpf_local_storage_map *smap, void *owner, 38450af8d0SKP Singh u32 size) 39450af8d0SKP Singh { 40450af8d0SKP Singh struct bpf_map *map = &smap->map; 41450af8d0SKP Singh 42450af8d0SKP Singh if (map->ops->map_local_storage_uncharge) 43450af8d0SKP Singh map->ops->map_local_storage_uncharge(smap, owner, size); 44450af8d0SKP Singh } 45450af8d0SKP Singh 46450af8d0SKP Singh static struct bpf_local_storage __rcu ** 47450af8d0SKP Singh owner_storage(struct bpf_local_storage_map *smap, void *owner) 48450af8d0SKP Singh { 49450af8d0SKP Singh struct bpf_map *map = &smap->map; 50450af8d0SKP Singh 51450af8d0SKP Singh return map->ops->map_owner_storage_ptr(owner); 52450af8d0SKP Singh } 53450af8d0SKP Singh 540a09a2f9SKumar Kartikeya Dwivedi static bool selem_linked_to_storage_lockless(const struct bpf_local_storage_elem *selem) 550a09a2f9SKumar Kartikeya Dwivedi { 560a09a2f9SKumar Kartikeya Dwivedi return !hlist_unhashed_lockless(&selem->snode); 570a09a2f9SKumar Kartikeya Dwivedi } 580a09a2f9SKumar Kartikeya Dwivedi 59450af8d0SKP Singh static bool selem_linked_to_storage(const struct bpf_local_storage_elem *selem) 60450af8d0SKP Singh { 61450af8d0SKP Singh return !hlist_unhashed(&selem->snode); 62450af8d0SKP Singh } 63450af8d0SKP Singh 640a09a2f9SKumar Kartikeya Dwivedi static bool selem_linked_to_map_lockless(const struct bpf_local_storage_elem *selem) 650a09a2f9SKumar Kartikeya Dwivedi { 660a09a2f9SKumar Kartikeya Dwivedi return !hlist_unhashed_lockless(&selem->map_node); 670a09a2f9SKumar Kartikeya Dwivedi } 680a09a2f9SKumar Kartikeya Dwivedi 69450af8d0SKP Singh static bool selem_linked_to_map(const struct bpf_local_storage_elem *selem) 70450af8d0SKP Singh { 71450af8d0SKP Singh return !hlist_unhashed(&selem->map_node); 72450af8d0SKP Singh } 73450af8d0SKP Singh 74450af8d0SKP Singh struct bpf_local_storage_elem * 75450af8d0SKP Singh bpf_selem_alloc(struct bpf_local_storage_map *smap, void *owner, 76b00fa38aSJoanne Koong void *value, bool charge_mem, gfp_t gfp_flags) 77450af8d0SKP Singh { 78450af8d0SKP Singh struct bpf_local_storage_elem *selem; 79450af8d0SKP Singh 80450af8d0SKP Singh if (charge_mem && mem_charge(smap, owner, smap->elem_size)) 81450af8d0SKP Singh return NULL; 82450af8d0SKP Singh 83e9aae8beSRoman Gushchin selem = bpf_map_kzalloc(&smap->map, smap->elem_size, 84b00fa38aSJoanne Koong gfp_flags | __GFP_NOWARN); 85450af8d0SKP Singh if (selem) { 86450af8d0SKP Singh if (value) 87836e49e1SXu Kuohai copy_map_value(&smap->map, SDATA(selem)->data, value); 88*9db44fddSKumar Kartikeya Dwivedi /* No need to call check_and_init_map_value as memory is zero init */ 89450af8d0SKP Singh return selem; 90450af8d0SKP Singh } 91450af8d0SKP Singh 92450af8d0SKP Singh if (charge_mem) 93450af8d0SKP Singh mem_uncharge(smap, owner, smap->elem_size); 94450af8d0SKP Singh 95450af8d0SKP Singh return NULL; 96450af8d0SKP Singh } 97450af8d0SKP Singh 980fe4b381SKP Singh void bpf_local_storage_free_rcu(struct rcu_head *rcu) 990fe4b381SKP Singh { 1000fe4b381SKP Singh struct bpf_local_storage *local_storage; 1010fe4b381SKP Singh 102d39d1445SHou Tao /* If RCU Tasks Trace grace period implies RCU grace period, do 103d39d1445SHou Tao * kfree(), else do kfree_rcu(). 104d39d1445SHou Tao */ 1050fe4b381SKP Singh local_storage = container_of(rcu, struct bpf_local_storage, rcu); 106d39d1445SHou Tao if (rcu_trace_implies_rcu_gp()) 107d39d1445SHou Tao kfree(local_storage); 108d39d1445SHou Tao else 1090fe4b381SKP Singh kfree_rcu(local_storage, rcu); 1100fe4b381SKP Singh } 1110fe4b381SKP Singh 1120fe4b381SKP Singh static void bpf_selem_free_rcu(struct rcu_head *rcu) 1130fe4b381SKP Singh { 1140fe4b381SKP Singh struct bpf_local_storage_elem *selem; 1150fe4b381SKP Singh 1160fe4b381SKP Singh selem = container_of(rcu, struct bpf_local_storage_elem, rcu); 117*9db44fddSKumar Kartikeya Dwivedi /* The can_use_smap bool is set whenever we need to free additional 118*9db44fddSKumar Kartikeya Dwivedi * fields in selem data before freeing selem. bpf_local_storage_map_free 119*9db44fddSKumar Kartikeya Dwivedi * only executes rcu_barrier to wait for RCU callbacks when it has 120*9db44fddSKumar Kartikeya Dwivedi * special fields, hence we can only conditionally dereference smap, as 121*9db44fddSKumar Kartikeya Dwivedi * by this time the map might have already been freed without waiting 122*9db44fddSKumar Kartikeya Dwivedi * for our call_rcu callback if it did not have any special fields. 123*9db44fddSKumar Kartikeya Dwivedi */ 124*9db44fddSKumar Kartikeya Dwivedi if (selem->can_use_smap) 125*9db44fddSKumar Kartikeya Dwivedi bpf_obj_free_fields(SDATA(selem)->smap->map.record, SDATA(selem)->data); 126d39d1445SHou Tao kfree(selem); 127*9db44fddSKumar Kartikeya Dwivedi } 128*9db44fddSKumar Kartikeya Dwivedi 129*9db44fddSKumar Kartikeya Dwivedi static void bpf_selem_free_tasks_trace_rcu(struct rcu_head *rcu) 130*9db44fddSKumar Kartikeya Dwivedi { 131*9db44fddSKumar Kartikeya Dwivedi /* Free directly if Tasks Trace RCU GP also implies RCU GP */ 132*9db44fddSKumar Kartikeya Dwivedi if (rcu_trace_implies_rcu_gp()) 133*9db44fddSKumar Kartikeya Dwivedi bpf_selem_free_rcu(rcu); 134d39d1445SHou Tao else 135*9db44fddSKumar Kartikeya Dwivedi call_rcu(rcu, bpf_selem_free_rcu); 1360fe4b381SKP Singh } 1370fe4b381SKP Singh 138450af8d0SKP Singh /* local_storage->lock must be held and selem->local_storage == local_storage. 139450af8d0SKP Singh * The caller must ensure selem->smap is still valid to be 140450af8d0SKP Singh * dereferenced for its smap->elem_size and smap->cache_idx. 141450af8d0SKP Singh */ 142c83597faSYonghong Song static bool bpf_selem_unlink_storage_nolock(struct bpf_local_storage *local_storage, 143450af8d0SKP Singh struct bpf_local_storage_elem *selem, 144dcf456c9SKP Singh bool uncharge_mem, bool use_trace_rcu) 145450af8d0SKP Singh { 146450af8d0SKP Singh struct bpf_local_storage_map *smap; 147450af8d0SKP Singh bool free_local_storage; 148450af8d0SKP Singh void *owner; 149450af8d0SKP Singh 1500fe4b381SKP Singh smap = rcu_dereference_check(SDATA(selem)->smap, bpf_rcu_lock_held()); 151450af8d0SKP Singh owner = local_storage->owner; 152450af8d0SKP Singh 153450af8d0SKP Singh /* All uncharging on the owner must be done first. 154450af8d0SKP Singh * The owner may be freed once the last selem is unlinked 155450af8d0SKP Singh * from local_storage. 156450af8d0SKP Singh */ 157450af8d0SKP Singh if (uncharge_mem) 158450af8d0SKP Singh mem_uncharge(smap, owner, smap->elem_size); 159450af8d0SKP Singh 160450af8d0SKP Singh free_local_storage = hlist_is_singular_node(&selem->snode, 161450af8d0SKP Singh &local_storage->list); 162450af8d0SKP Singh if (free_local_storage) { 163450af8d0SKP Singh mem_uncharge(smap, owner, sizeof(struct bpf_local_storage)); 164450af8d0SKP Singh local_storage->owner = NULL; 165450af8d0SKP Singh 166450af8d0SKP Singh /* After this RCU_INIT, owner may be freed and cannot be used */ 167450af8d0SKP Singh RCU_INIT_POINTER(*owner_storage(smap, owner), NULL); 168450af8d0SKP Singh 169450af8d0SKP Singh /* local_storage is not freed now. local_storage->lock is 170450af8d0SKP Singh * still held and raw_spin_unlock_bh(&local_storage->lock) 171450af8d0SKP Singh * will be done by the caller. 172450af8d0SKP Singh * 173450af8d0SKP Singh * Although the unlock will be done under 174c561d110STom Rix * rcu_read_lock(), it is more intuitive to 1750fe4b381SKP Singh * read if the freeing of the storage is done 176450af8d0SKP Singh * after the raw_spin_unlock_bh(&local_storage->lock). 177450af8d0SKP Singh * 178450af8d0SKP Singh * Hence, a "bool free_local_storage" is returned 1790fe4b381SKP Singh * to the caller which then calls then frees the storage after 1800fe4b381SKP Singh * all the RCU grace periods have expired. 181450af8d0SKP Singh */ 182450af8d0SKP Singh } 183450af8d0SKP Singh hlist_del_init_rcu(&selem->snode); 184450af8d0SKP Singh if (rcu_access_pointer(local_storage->cache[smap->cache_idx]) == 185450af8d0SKP Singh SDATA(selem)) 186450af8d0SKP Singh RCU_INIT_POINTER(local_storage->cache[smap->cache_idx], NULL); 187450af8d0SKP Singh 188dcf456c9SKP Singh if (use_trace_rcu) 189*9db44fddSKumar Kartikeya Dwivedi call_rcu_tasks_trace(&selem->rcu, bpf_selem_free_tasks_trace_rcu); 190dcf456c9SKP Singh else 191*9db44fddSKumar Kartikeya Dwivedi call_rcu(&selem->rcu, bpf_selem_free_rcu); 192dcf456c9SKP Singh 193450af8d0SKP Singh return free_local_storage; 194450af8d0SKP Singh } 195450af8d0SKP Singh 196dcf456c9SKP Singh static void __bpf_selem_unlink_storage(struct bpf_local_storage_elem *selem, 197dcf456c9SKP Singh bool use_trace_rcu) 198450af8d0SKP Singh { 199450af8d0SKP Singh struct bpf_local_storage *local_storage; 200450af8d0SKP Singh bool free_local_storage = false; 201a10787e6SSong Liu unsigned long flags; 202450af8d0SKP Singh 2030a09a2f9SKumar Kartikeya Dwivedi if (unlikely(!selem_linked_to_storage_lockless(selem))) 204450af8d0SKP Singh /* selem has already been unlinked from sk */ 205450af8d0SKP Singh return; 206450af8d0SKP Singh 2070fe4b381SKP Singh local_storage = rcu_dereference_check(selem->local_storage, 2080fe4b381SKP Singh bpf_rcu_lock_held()); 209a10787e6SSong Liu raw_spin_lock_irqsave(&local_storage->lock, flags); 210450af8d0SKP Singh if (likely(selem_linked_to_storage(selem))) 211450af8d0SKP Singh free_local_storage = bpf_selem_unlink_storage_nolock( 212dcf456c9SKP Singh local_storage, selem, true, use_trace_rcu); 213a10787e6SSong Liu raw_spin_unlock_irqrestore(&local_storage->lock, flags); 214450af8d0SKP Singh 215dcf456c9SKP Singh if (free_local_storage) { 216dcf456c9SKP Singh if (use_trace_rcu) 2170fe4b381SKP Singh call_rcu_tasks_trace(&local_storage->rcu, 2180fe4b381SKP Singh bpf_local_storage_free_rcu); 219dcf456c9SKP Singh else 220dcf456c9SKP Singh kfree_rcu(local_storage, rcu); 221dcf456c9SKP Singh } 222450af8d0SKP Singh } 223450af8d0SKP Singh 224450af8d0SKP Singh void bpf_selem_link_storage_nolock(struct bpf_local_storage *local_storage, 225450af8d0SKP Singh struct bpf_local_storage_elem *selem) 226450af8d0SKP Singh { 227450af8d0SKP Singh RCU_INIT_POINTER(selem->local_storage, local_storage); 22870b97111SMartin KaFai Lau hlist_add_head_rcu(&selem->snode, &local_storage->list); 229450af8d0SKP Singh } 230450af8d0SKP Singh 231450af8d0SKP Singh void bpf_selem_unlink_map(struct bpf_local_storage_elem *selem) 232450af8d0SKP Singh { 233450af8d0SKP Singh struct bpf_local_storage_map *smap; 234450af8d0SKP Singh struct bpf_local_storage_map_bucket *b; 235a10787e6SSong Liu unsigned long flags; 236450af8d0SKP Singh 2370a09a2f9SKumar Kartikeya Dwivedi if (unlikely(!selem_linked_to_map_lockless(selem))) 238450af8d0SKP Singh /* selem has already be unlinked from smap */ 239450af8d0SKP Singh return; 240450af8d0SKP Singh 2410fe4b381SKP Singh smap = rcu_dereference_check(SDATA(selem)->smap, bpf_rcu_lock_held()); 242450af8d0SKP Singh b = select_bucket(smap, selem); 243a10787e6SSong Liu raw_spin_lock_irqsave(&b->lock, flags); 244450af8d0SKP Singh if (likely(selem_linked_to_map(selem))) 245450af8d0SKP Singh hlist_del_init_rcu(&selem->map_node); 246a10787e6SSong Liu raw_spin_unlock_irqrestore(&b->lock, flags); 247450af8d0SKP Singh } 248450af8d0SKP Singh 249450af8d0SKP Singh void bpf_selem_link_map(struct bpf_local_storage_map *smap, 250450af8d0SKP Singh struct bpf_local_storage_elem *selem) 251450af8d0SKP Singh { 252450af8d0SKP Singh struct bpf_local_storage_map_bucket *b = select_bucket(smap, selem); 253a10787e6SSong Liu unsigned long flags; 254450af8d0SKP Singh 255a10787e6SSong Liu raw_spin_lock_irqsave(&b->lock, flags); 256450af8d0SKP Singh RCU_INIT_POINTER(SDATA(selem)->smap, smap); 257450af8d0SKP Singh hlist_add_head_rcu(&selem->map_node, &b->list); 258a10787e6SSong Liu raw_spin_unlock_irqrestore(&b->lock, flags); 259*9db44fddSKumar Kartikeya Dwivedi 260*9db44fddSKumar Kartikeya Dwivedi /* If our data will have special fields, smap will wait for us to use 261*9db44fddSKumar Kartikeya Dwivedi * its record in bpf_selem_free_* RCU callbacks before freeing itself. 262*9db44fddSKumar Kartikeya Dwivedi */ 263*9db44fddSKumar Kartikeya Dwivedi selem->can_use_smap = !IS_ERR_OR_NULL(smap->map.record); 264450af8d0SKP Singh } 265450af8d0SKP Singh 266dcf456c9SKP Singh void bpf_selem_unlink(struct bpf_local_storage_elem *selem, bool use_trace_rcu) 267450af8d0SKP Singh { 268450af8d0SKP Singh /* Always unlink from map before unlinking from local_storage 269450af8d0SKP Singh * because selem will be freed after successfully unlinked from 270450af8d0SKP Singh * the local_storage. 271450af8d0SKP Singh */ 272450af8d0SKP Singh bpf_selem_unlink_map(selem); 273dcf456c9SKP Singh __bpf_selem_unlink_storage(selem, use_trace_rcu); 274450af8d0SKP Singh } 275450af8d0SKP Singh 276e8b02296SMartin KaFai Lau /* If cacheit_lockit is false, this lookup function is lockless */ 277450af8d0SKP Singh struct bpf_local_storage_data * 278450af8d0SKP Singh bpf_local_storage_lookup(struct bpf_local_storage *local_storage, 279450af8d0SKP Singh struct bpf_local_storage_map *smap, 280450af8d0SKP Singh bool cacheit_lockit) 281450af8d0SKP Singh { 282450af8d0SKP Singh struct bpf_local_storage_data *sdata; 283450af8d0SKP Singh struct bpf_local_storage_elem *selem; 284450af8d0SKP Singh 285450af8d0SKP Singh /* Fast path (cache hit) */ 2860fe4b381SKP Singh sdata = rcu_dereference_check(local_storage->cache[smap->cache_idx], 2870fe4b381SKP Singh bpf_rcu_lock_held()); 288450af8d0SKP Singh if (sdata && rcu_access_pointer(sdata->smap) == smap) 289450af8d0SKP Singh return sdata; 290450af8d0SKP Singh 291450af8d0SKP Singh /* Slow path (cache miss) */ 2920fe4b381SKP Singh hlist_for_each_entry_rcu(selem, &local_storage->list, snode, 2930fe4b381SKP Singh rcu_read_lock_trace_held()) 294450af8d0SKP Singh if (rcu_access_pointer(SDATA(selem)->smap) == smap) 295450af8d0SKP Singh break; 296450af8d0SKP Singh 297450af8d0SKP Singh if (!selem) 298450af8d0SKP Singh return NULL; 299450af8d0SKP Singh 300450af8d0SKP Singh sdata = SDATA(selem); 301450af8d0SKP Singh if (cacheit_lockit) { 302a10787e6SSong Liu unsigned long flags; 303a10787e6SSong Liu 304450af8d0SKP Singh /* spinlock is needed to avoid racing with the 305450af8d0SKP Singh * parallel delete. Otherwise, publishing an already 306450af8d0SKP Singh * deleted sdata to the cache will become a use-after-free 307450af8d0SKP Singh * problem in the next bpf_local_storage_lookup(). 308450af8d0SKP Singh */ 309a10787e6SSong Liu raw_spin_lock_irqsave(&local_storage->lock, flags); 310450af8d0SKP Singh if (selem_linked_to_storage(selem)) 311450af8d0SKP Singh rcu_assign_pointer(local_storage->cache[smap->cache_idx], 312450af8d0SKP Singh sdata); 313a10787e6SSong Liu raw_spin_unlock_irqrestore(&local_storage->lock, flags); 314450af8d0SKP Singh } 315450af8d0SKP Singh 316450af8d0SKP Singh return sdata; 317450af8d0SKP Singh } 318450af8d0SKP Singh 319450af8d0SKP Singh static int check_flags(const struct bpf_local_storage_data *old_sdata, 320450af8d0SKP Singh u64 map_flags) 321450af8d0SKP Singh { 322450af8d0SKP Singh if (old_sdata && (map_flags & ~BPF_F_LOCK) == BPF_NOEXIST) 323450af8d0SKP Singh /* elem already exists */ 324450af8d0SKP Singh return -EEXIST; 325450af8d0SKP Singh 326450af8d0SKP Singh if (!old_sdata && (map_flags & ~BPF_F_LOCK) == BPF_EXIST) 327450af8d0SKP Singh /* elem doesn't exist, cannot update it */ 328450af8d0SKP Singh return -ENOENT; 329450af8d0SKP Singh 330450af8d0SKP Singh return 0; 331450af8d0SKP Singh } 332450af8d0SKP Singh 333450af8d0SKP Singh int bpf_local_storage_alloc(void *owner, 334450af8d0SKP Singh struct bpf_local_storage_map *smap, 335b00fa38aSJoanne Koong struct bpf_local_storage_elem *first_selem, 336b00fa38aSJoanne Koong gfp_t gfp_flags) 337450af8d0SKP Singh { 338450af8d0SKP Singh struct bpf_local_storage *prev_storage, *storage; 339450af8d0SKP Singh struct bpf_local_storage **owner_storage_ptr; 340450af8d0SKP Singh int err; 341450af8d0SKP Singh 342450af8d0SKP Singh err = mem_charge(smap, owner, sizeof(*storage)); 343450af8d0SKP Singh if (err) 344450af8d0SKP Singh return err; 345450af8d0SKP Singh 346e9aae8beSRoman Gushchin storage = bpf_map_kzalloc(&smap->map, sizeof(*storage), 347b00fa38aSJoanne Koong gfp_flags | __GFP_NOWARN); 348450af8d0SKP Singh if (!storage) { 349450af8d0SKP Singh err = -ENOMEM; 350450af8d0SKP Singh goto uncharge; 351450af8d0SKP Singh } 352450af8d0SKP Singh 353450af8d0SKP Singh INIT_HLIST_HEAD(&storage->list); 354450af8d0SKP Singh raw_spin_lock_init(&storage->lock); 355450af8d0SKP Singh storage->owner = owner; 356450af8d0SKP Singh 357450af8d0SKP Singh bpf_selem_link_storage_nolock(storage, first_selem); 358450af8d0SKP Singh bpf_selem_link_map(smap, first_selem); 359450af8d0SKP Singh 360450af8d0SKP Singh owner_storage_ptr = 361450af8d0SKP Singh (struct bpf_local_storage **)owner_storage(smap, owner); 362450af8d0SKP Singh /* Publish storage to the owner. 363450af8d0SKP Singh * Instead of using any lock of the kernel object (i.e. owner), 364450af8d0SKP Singh * cmpxchg will work with any kernel object regardless what 365450af8d0SKP Singh * the running context is, bh, irq...etc. 366450af8d0SKP Singh * 367450af8d0SKP Singh * From now on, the owner->storage pointer (e.g. sk->sk_bpf_storage) 368450af8d0SKP Singh * is protected by the storage->lock. Hence, when freeing 369450af8d0SKP Singh * the owner->storage, the storage->lock must be held before 370450af8d0SKP Singh * setting owner->storage ptr to NULL. 371450af8d0SKP Singh */ 372450af8d0SKP Singh prev_storage = cmpxchg(owner_storage_ptr, NULL, storage); 373450af8d0SKP Singh if (unlikely(prev_storage)) { 374450af8d0SKP Singh bpf_selem_unlink_map(first_selem); 375450af8d0SKP Singh err = -EAGAIN; 376450af8d0SKP Singh goto uncharge; 377450af8d0SKP Singh 378450af8d0SKP Singh /* Note that even first_selem was linked to smap's 379450af8d0SKP Singh * bucket->list, first_selem can be freed immediately 380450af8d0SKP Singh * (instead of kfree_rcu) because 381450af8d0SKP Singh * bpf_local_storage_map_free() does a 3820fe4b381SKP Singh * synchronize_rcu_mult (waiting for both sleepable and 3830fe4b381SKP Singh * normal programs) before walking the bucket->list. 384450af8d0SKP Singh * Hence, no one is accessing selem from the 385450af8d0SKP Singh * bucket->list under rcu_read_lock(). 386450af8d0SKP Singh */ 387450af8d0SKP Singh } 388450af8d0SKP Singh 389450af8d0SKP Singh return 0; 390450af8d0SKP Singh 391450af8d0SKP Singh uncharge: 392450af8d0SKP Singh kfree(storage); 393450af8d0SKP Singh mem_uncharge(smap, owner, sizeof(*storage)); 394450af8d0SKP Singh return err; 395450af8d0SKP Singh } 396450af8d0SKP Singh 397450af8d0SKP Singh /* sk cannot be going away because it is linking new elem 398450af8d0SKP Singh * to sk->sk_bpf_storage. (i.e. sk->sk_refcnt cannot be 0). 399450af8d0SKP Singh * Otherwise, it will become a leak (and other memory issues 400450af8d0SKP Singh * during map destruction). 401450af8d0SKP Singh */ 402450af8d0SKP Singh struct bpf_local_storage_data * 403450af8d0SKP Singh bpf_local_storage_update(void *owner, struct bpf_local_storage_map *smap, 404b00fa38aSJoanne Koong void *value, u64 map_flags, gfp_t gfp_flags) 405450af8d0SKP Singh { 406450af8d0SKP Singh struct bpf_local_storage_data *old_sdata = NULL; 407b00fa38aSJoanne Koong struct bpf_local_storage_elem *selem = NULL; 408450af8d0SKP Singh struct bpf_local_storage *local_storage; 409a10787e6SSong Liu unsigned long flags; 410450af8d0SKP Singh int err; 411450af8d0SKP Singh 412450af8d0SKP Singh /* BPF_EXIST and BPF_NOEXIST cannot be both set */ 413450af8d0SKP Singh if (unlikely((map_flags & ~BPF_F_LOCK) > BPF_EXIST) || 414450af8d0SKP Singh /* BPF_F_LOCK can only be used in a value with spin_lock */ 415450af8d0SKP Singh unlikely((map_flags & BPF_F_LOCK) && 416db559117SKumar Kartikeya Dwivedi !btf_record_has_field(smap->map.record, BPF_SPIN_LOCK))) 417450af8d0SKP Singh return ERR_PTR(-EINVAL); 418450af8d0SKP Singh 419b00fa38aSJoanne Koong if (gfp_flags == GFP_KERNEL && (map_flags & ~BPF_F_LOCK) != BPF_NOEXIST) 420b00fa38aSJoanne Koong return ERR_PTR(-EINVAL); 421b00fa38aSJoanne Koong 4220fe4b381SKP Singh local_storage = rcu_dereference_check(*owner_storage(smap, owner), 4230fe4b381SKP Singh bpf_rcu_lock_held()); 424450af8d0SKP Singh if (!local_storage || hlist_empty(&local_storage->list)) { 425450af8d0SKP Singh /* Very first elem for the owner */ 426450af8d0SKP Singh err = check_flags(NULL, map_flags); 427450af8d0SKP Singh if (err) 428450af8d0SKP Singh return ERR_PTR(err); 429450af8d0SKP Singh 430b00fa38aSJoanne Koong selem = bpf_selem_alloc(smap, owner, value, true, gfp_flags); 431450af8d0SKP Singh if (!selem) 432450af8d0SKP Singh return ERR_PTR(-ENOMEM); 433450af8d0SKP Singh 434b00fa38aSJoanne Koong err = bpf_local_storage_alloc(owner, smap, selem, gfp_flags); 435450af8d0SKP Singh if (err) { 436450af8d0SKP Singh kfree(selem); 437450af8d0SKP Singh mem_uncharge(smap, owner, smap->elem_size); 438450af8d0SKP Singh return ERR_PTR(err); 439450af8d0SKP Singh } 440450af8d0SKP Singh 441450af8d0SKP Singh return SDATA(selem); 442450af8d0SKP Singh } 443450af8d0SKP Singh 444450af8d0SKP Singh if ((map_flags & BPF_F_LOCK) && !(map_flags & BPF_NOEXIST)) { 445450af8d0SKP Singh /* Hoping to find an old_sdata to do inline update 446450af8d0SKP Singh * such that it can avoid taking the local_storage->lock 447450af8d0SKP Singh * and changing the lists. 448450af8d0SKP Singh */ 449450af8d0SKP Singh old_sdata = 450450af8d0SKP Singh bpf_local_storage_lookup(local_storage, smap, false); 451450af8d0SKP Singh err = check_flags(old_sdata, map_flags); 452450af8d0SKP Singh if (err) 453450af8d0SKP Singh return ERR_PTR(err); 4540a09a2f9SKumar Kartikeya Dwivedi if (old_sdata && selem_linked_to_storage_lockless(SELEM(old_sdata))) { 455450af8d0SKP Singh copy_map_value_locked(&smap->map, old_sdata->data, 456450af8d0SKP Singh value, false); 457450af8d0SKP Singh return old_sdata; 458450af8d0SKP Singh } 459450af8d0SKP Singh } 460450af8d0SKP Singh 461b00fa38aSJoanne Koong if (gfp_flags == GFP_KERNEL) { 462b00fa38aSJoanne Koong selem = bpf_selem_alloc(smap, owner, value, true, gfp_flags); 463b00fa38aSJoanne Koong if (!selem) 464b00fa38aSJoanne Koong return ERR_PTR(-ENOMEM); 465b00fa38aSJoanne Koong } 466b00fa38aSJoanne Koong 467a10787e6SSong Liu raw_spin_lock_irqsave(&local_storage->lock, flags); 468450af8d0SKP Singh 469450af8d0SKP Singh /* Recheck local_storage->list under local_storage->lock */ 470450af8d0SKP Singh if (unlikely(hlist_empty(&local_storage->list))) { 471450af8d0SKP Singh /* A parallel del is happening and local_storage is going 472450af8d0SKP Singh * away. It has just been checked before, so very 473450af8d0SKP Singh * unlikely. Return instead of retry to keep things 474450af8d0SKP Singh * simple. 475450af8d0SKP Singh */ 476450af8d0SKP Singh err = -EAGAIN; 477450af8d0SKP Singh goto unlock_err; 478450af8d0SKP Singh } 479450af8d0SKP Singh 480450af8d0SKP Singh old_sdata = bpf_local_storage_lookup(local_storage, smap, false); 481450af8d0SKP Singh err = check_flags(old_sdata, map_flags); 482450af8d0SKP Singh if (err) 483450af8d0SKP Singh goto unlock_err; 484450af8d0SKP Singh 485450af8d0SKP Singh if (old_sdata && (map_flags & BPF_F_LOCK)) { 486450af8d0SKP Singh copy_map_value_locked(&smap->map, old_sdata->data, value, 487450af8d0SKP Singh false); 488450af8d0SKP Singh selem = SELEM(old_sdata); 489450af8d0SKP Singh goto unlock; 490450af8d0SKP Singh } 491450af8d0SKP Singh 492b00fa38aSJoanne Koong if (gfp_flags != GFP_KERNEL) { 493450af8d0SKP Singh /* local_storage->lock is held. Hence, we are sure 494450af8d0SKP Singh * we can unlink and uncharge the old_sdata successfully 495450af8d0SKP Singh * later. Hence, instead of charging the new selem now 496450af8d0SKP Singh * and then uncharge the old selem later (which may cause 497450af8d0SKP Singh * a potential but unnecessary charge failure), avoid taking 498450af8d0SKP Singh * a charge at all here (the "!old_sdata" check) and the 499450af8d0SKP Singh * old_sdata will not be uncharged later during 500450af8d0SKP Singh * bpf_selem_unlink_storage_nolock(). 501450af8d0SKP Singh */ 502b00fa38aSJoanne Koong selem = bpf_selem_alloc(smap, owner, value, !old_sdata, gfp_flags); 503450af8d0SKP Singh if (!selem) { 504450af8d0SKP Singh err = -ENOMEM; 505450af8d0SKP Singh goto unlock_err; 506450af8d0SKP Singh } 507b00fa38aSJoanne Koong } 508450af8d0SKP Singh 509450af8d0SKP Singh /* First, link the new selem to the map */ 510450af8d0SKP Singh bpf_selem_link_map(smap, selem); 511450af8d0SKP Singh 512450af8d0SKP Singh /* Second, link (and publish) the new selem to local_storage */ 513450af8d0SKP Singh bpf_selem_link_storage_nolock(local_storage, selem); 514450af8d0SKP Singh 515450af8d0SKP Singh /* Third, remove old selem, SELEM(old_sdata) */ 516450af8d0SKP Singh if (old_sdata) { 517450af8d0SKP Singh bpf_selem_unlink_map(SELEM(old_sdata)); 518450af8d0SKP Singh bpf_selem_unlink_storage_nolock(local_storage, SELEM(old_sdata), 519dcf456c9SKP Singh false, true); 520450af8d0SKP Singh } 521450af8d0SKP Singh 522450af8d0SKP Singh unlock: 523a10787e6SSong Liu raw_spin_unlock_irqrestore(&local_storage->lock, flags); 524450af8d0SKP Singh return SDATA(selem); 525450af8d0SKP Singh 526450af8d0SKP Singh unlock_err: 527a10787e6SSong Liu raw_spin_unlock_irqrestore(&local_storage->lock, flags); 528b00fa38aSJoanne Koong if (selem) { 529b00fa38aSJoanne Koong mem_uncharge(smap, owner, smap->elem_size); 530b00fa38aSJoanne Koong kfree(selem); 531b00fa38aSJoanne Koong } 532450af8d0SKP Singh return ERR_PTR(err); 533450af8d0SKP Singh } 534450af8d0SKP Singh 535c83597faSYonghong Song static u16 bpf_local_storage_cache_idx_get(struct bpf_local_storage_cache *cache) 536450af8d0SKP Singh { 537450af8d0SKP Singh u64 min_usage = U64_MAX; 538450af8d0SKP Singh u16 i, res = 0; 539450af8d0SKP Singh 540450af8d0SKP Singh spin_lock(&cache->idx_lock); 541450af8d0SKP Singh 542450af8d0SKP Singh for (i = 0; i < BPF_LOCAL_STORAGE_CACHE_SIZE; i++) { 543450af8d0SKP Singh if (cache->idx_usage_counts[i] < min_usage) { 544450af8d0SKP Singh min_usage = cache->idx_usage_counts[i]; 545450af8d0SKP Singh res = i; 546450af8d0SKP Singh 547450af8d0SKP Singh /* Found a free cache_idx */ 548450af8d0SKP Singh if (!min_usage) 549450af8d0SKP Singh break; 550450af8d0SKP Singh } 551450af8d0SKP Singh } 552450af8d0SKP Singh cache->idx_usage_counts[res]++; 553450af8d0SKP Singh 554450af8d0SKP Singh spin_unlock(&cache->idx_lock); 555450af8d0SKP Singh 556450af8d0SKP Singh return res; 557450af8d0SKP Singh } 558450af8d0SKP Singh 559c83597faSYonghong Song static void bpf_local_storage_cache_idx_free(struct bpf_local_storage_cache *cache, 560450af8d0SKP Singh u16 idx) 561450af8d0SKP Singh { 562450af8d0SKP Singh spin_lock(&cache->idx_lock); 563450af8d0SKP Singh cache->idx_usage_counts[idx]--; 564450af8d0SKP Singh spin_unlock(&cache->idx_lock); 565450af8d0SKP Singh } 566450af8d0SKP Singh 567c83597faSYonghong Song int bpf_local_storage_map_alloc_check(union bpf_attr *attr) 568c83597faSYonghong Song { 569c83597faSYonghong Song if (attr->map_flags & ~BPF_LOCAL_STORAGE_CREATE_FLAG_MASK || 570c83597faSYonghong Song !(attr->map_flags & BPF_F_NO_PREALLOC) || 571c83597faSYonghong Song attr->max_entries || 572c83597faSYonghong Song attr->key_size != sizeof(int) || !attr->value_size || 573c83597faSYonghong Song /* Enforce BTF for userspace sk dumping */ 574c83597faSYonghong Song !attr->btf_key_type_id || !attr->btf_value_type_id) 575c83597faSYonghong Song return -EINVAL; 576c83597faSYonghong Song 577c83597faSYonghong Song if (!bpf_capable()) 578c83597faSYonghong Song return -EPERM; 579c83597faSYonghong Song 580c83597faSYonghong Song if (attr->value_size > BPF_LOCAL_STORAGE_MAX_VALUE_SIZE) 581c83597faSYonghong Song return -E2BIG; 582c83597faSYonghong Song 583c83597faSYonghong Song return 0; 584c83597faSYonghong Song } 585c83597faSYonghong Song 586c83597faSYonghong Song static struct bpf_local_storage_map *__bpf_local_storage_map_alloc(union bpf_attr *attr) 587c83597faSYonghong Song { 588c83597faSYonghong Song struct bpf_local_storage_map *smap; 589c83597faSYonghong Song unsigned int i; 590c83597faSYonghong Song u32 nbuckets; 591c83597faSYonghong Song 592c83597faSYonghong Song smap = bpf_map_area_alloc(sizeof(*smap), NUMA_NO_NODE); 593c83597faSYonghong Song if (!smap) 594c83597faSYonghong Song return ERR_PTR(-ENOMEM); 595c83597faSYonghong Song bpf_map_init_from_attr(&smap->map, attr); 596c83597faSYonghong Song 597c83597faSYonghong Song nbuckets = roundup_pow_of_two(num_possible_cpus()); 598c83597faSYonghong Song /* Use at least 2 buckets, select_bucket() is undefined behavior with 1 bucket */ 599c83597faSYonghong Song nbuckets = max_t(u32, 2, nbuckets); 600c83597faSYonghong Song smap->bucket_log = ilog2(nbuckets); 601c83597faSYonghong Song 602ddef81b5SYafang Shao smap->buckets = bpf_map_kvcalloc(&smap->map, sizeof(*smap->buckets), 603ddef81b5SYafang Shao nbuckets, GFP_USER | __GFP_NOWARN); 604c83597faSYonghong Song if (!smap->buckets) { 605c83597faSYonghong Song bpf_map_area_free(smap); 606c83597faSYonghong Song return ERR_PTR(-ENOMEM); 607c83597faSYonghong Song } 608c83597faSYonghong Song 609c83597faSYonghong Song for (i = 0; i < nbuckets; i++) { 610c83597faSYonghong Song INIT_HLIST_HEAD(&smap->buckets[i].list); 611c83597faSYonghong Song raw_spin_lock_init(&smap->buckets[i].lock); 612c83597faSYonghong Song } 613c83597faSYonghong Song 614552d42a3SMartin KaFai Lau smap->elem_size = offsetof(struct bpf_local_storage_elem, 615552d42a3SMartin KaFai Lau sdata.data[attr->value_size]); 616c83597faSYonghong Song 617c83597faSYonghong Song return smap; 618c83597faSYonghong Song } 619c83597faSYonghong Song 620c83597faSYonghong Song int bpf_local_storage_map_check_btf(const struct bpf_map *map, 621c83597faSYonghong Song const struct btf *btf, 622c83597faSYonghong Song const struct btf_type *key_type, 623c83597faSYonghong Song const struct btf_type *value_type) 624c83597faSYonghong Song { 625c83597faSYonghong Song u32 int_data; 626c83597faSYonghong Song 627c83597faSYonghong Song if (BTF_INFO_KIND(key_type->info) != BTF_KIND_INT) 628c83597faSYonghong Song return -EINVAL; 629c83597faSYonghong Song 630c83597faSYonghong Song int_data = *(u32 *)(key_type + 1); 631c83597faSYonghong Song if (BTF_INT_BITS(int_data) != 32 || BTF_INT_OFFSET(int_data)) 632c83597faSYonghong Song return -EINVAL; 633c83597faSYonghong Song 634c83597faSYonghong Song return 0; 635c83597faSYonghong Song } 636c83597faSYonghong Song 637c83597faSYonghong Song bool bpf_local_storage_unlink_nolock(struct bpf_local_storage *local_storage) 638450af8d0SKP Singh { 639450af8d0SKP Singh struct bpf_local_storage_elem *selem; 640c83597faSYonghong Song bool free_storage = false; 641c83597faSYonghong Song struct hlist_node *n; 642c83597faSYonghong Song 643c83597faSYonghong Song /* Neither the bpf_prog nor the bpf_map's syscall 644c83597faSYonghong Song * could be modifying the local_storage->list now. 645c83597faSYonghong Song * Thus, no elem can be added to or deleted from the 646c83597faSYonghong Song * local_storage->list by the bpf_prog or by the bpf_map's syscall. 647c83597faSYonghong Song * 648c83597faSYonghong Song * It is racing with bpf_local_storage_map_free() alone 649c83597faSYonghong Song * when unlinking elem from the local_storage->list and 650c83597faSYonghong Song * the map's bucket->list. 651c83597faSYonghong Song */ 652c83597faSYonghong Song hlist_for_each_entry_safe(selem, n, &local_storage->list, snode) { 653c83597faSYonghong Song /* Always unlink from map before unlinking from 654c83597faSYonghong Song * local_storage. 655c83597faSYonghong Song */ 656c83597faSYonghong Song bpf_selem_unlink_map(selem); 657c83597faSYonghong Song /* If local_storage list has only one element, the 658c83597faSYonghong Song * bpf_selem_unlink_storage_nolock() will return true. 659c83597faSYonghong Song * Otherwise, it will return false. The current loop iteration 660c83597faSYonghong Song * intends to remove all local storage. So the last iteration 661c83597faSYonghong Song * of the loop will set the free_cgroup_storage to true. 662c83597faSYonghong Song */ 663c83597faSYonghong Song free_storage = bpf_selem_unlink_storage_nolock( 664c83597faSYonghong Song local_storage, selem, false, false); 665c83597faSYonghong Song } 666c83597faSYonghong Song 667c83597faSYonghong Song return free_storage; 668c83597faSYonghong Song } 669c83597faSYonghong Song 670c83597faSYonghong Song struct bpf_map * 671c83597faSYonghong Song bpf_local_storage_map_alloc(union bpf_attr *attr, 672c83597faSYonghong Song struct bpf_local_storage_cache *cache) 673c83597faSYonghong Song { 674c83597faSYonghong Song struct bpf_local_storage_map *smap; 675c83597faSYonghong Song 676c83597faSYonghong Song smap = __bpf_local_storage_map_alloc(attr); 677c83597faSYonghong Song if (IS_ERR(smap)) 678c83597faSYonghong Song return ERR_CAST(smap); 679c83597faSYonghong Song 680c83597faSYonghong Song smap->cache_idx = bpf_local_storage_cache_idx_get(cache); 681c83597faSYonghong Song return &smap->map; 682c83597faSYonghong Song } 683c83597faSYonghong Song 684c83597faSYonghong Song void bpf_local_storage_map_free(struct bpf_map *map, 685c83597faSYonghong Song struct bpf_local_storage_cache *cache, 686c83597faSYonghong Song int __percpu *busy_counter) 687c83597faSYonghong Song { 688450af8d0SKP Singh struct bpf_local_storage_map_bucket *b; 689c83597faSYonghong Song struct bpf_local_storage_elem *selem; 690c83597faSYonghong Song struct bpf_local_storage_map *smap; 691450af8d0SKP Singh unsigned int i; 692450af8d0SKP Singh 693c83597faSYonghong Song smap = (struct bpf_local_storage_map *)map; 694c83597faSYonghong Song bpf_local_storage_cache_idx_free(cache, smap->cache_idx); 695c83597faSYonghong Song 696450af8d0SKP Singh /* Note that this map might be concurrently cloned from 697450af8d0SKP Singh * bpf_sk_storage_clone. Wait for any existing bpf_sk_storage_clone 698450af8d0SKP Singh * RCU read section to finish before proceeding. New RCU 699450af8d0SKP Singh * read sections should be prevented via bpf_map_inc_not_zero. 700450af8d0SKP Singh */ 701450af8d0SKP Singh synchronize_rcu(); 702450af8d0SKP Singh 703450af8d0SKP Singh /* bpf prog and the userspace can no longer access this map 704450af8d0SKP Singh * now. No new selem (of this map) can be added 705450af8d0SKP Singh * to the owner->storage or to the map bucket's list. 706450af8d0SKP Singh * 707450af8d0SKP Singh * The elem of this map can be cleaned up here 708450af8d0SKP Singh * or when the storage is freed e.g. 709450af8d0SKP Singh * by bpf_sk_storage_free() during __sk_destruct(). 710450af8d0SKP Singh */ 711450af8d0SKP Singh for (i = 0; i < (1U << smap->bucket_log); i++) { 712450af8d0SKP Singh b = &smap->buckets[i]; 713450af8d0SKP Singh 714450af8d0SKP Singh rcu_read_lock(); 715450af8d0SKP Singh /* No one is adding to b->list now */ 716450af8d0SKP Singh while ((selem = hlist_entry_safe( 717450af8d0SKP Singh rcu_dereference_raw(hlist_first_rcu(&b->list)), 718450af8d0SKP Singh struct bpf_local_storage_elem, map_node))) { 719bc235cdbSSong Liu if (busy_counter) { 720bc235cdbSSong Liu migrate_disable(); 721197827a0SHou Tao this_cpu_inc(*busy_counter); 722bc235cdbSSong Liu } 723dcf456c9SKP Singh bpf_selem_unlink(selem, false); 724bc235cdbSSong Liu if (busy_counter) { 725197827a0SHou Tao this_cpu_dec(*busy_counter); 726bc235cdbSSong Liu migrate_enable(); 727bc235cdbSSong Liu } 728450af8d0SKP Singh cond_resched_rcu(); 729450af8d0SKP Singh } 730450af8d0SKP Singh rcu_read_unlock(); 731450af8d0SKP Singh } 732450af8d0SKP Singh 733450af8d0SKP Singh /* While freeing the storage we may still need to access the map. 734450af8d0SKP Singh * 735450af8d0SKP Singh * e.g. when bpf_sk_storage_free() has unlinked selem from the map 736450af8d0SKP Singh * which then made the above while((selem = ...)) loop 737450af8d0SKP Singh * exit immediately. 738450af8d0SKP Singh * 739450af8d0SKP Singh * However, while freeing the storage one still needs to access the 740450af8d0SKP Singh * smap->elem_size to do the uncharging in 741450af8d0SKP Singh * bpf_selem_unlink_storage_nolock(). 742450af8d0SKP Singh * 743450af8d0SKP Singh * Hence, wait another rcu grace period for the storage to be freed. 744450af8d0SKP Singh */ 745450af8d0SKP Singh synchronize_rcu(); 746450af8d0SKP Singh 747*9db44fddSKumar Kartikeya Dwivedi /* Only delay freeing of smap, buckets are not needed anymore */ 748450af8d0SKP Singh kvfree(smap->buckets); 749*9db44fddSKumar Kartikeya Dwivedi 750*9db44fddSKumar Kartikeya Dwivedi /* When local storage has special fields, callbacks for 751*9db44fddSKumar Kartikeya Dwivedi * bpf_selem_free_rcu and bpf_selem_free_tasks_trace_rcu will keep using 752*9db44fddSKumar Kartikeya Dwivedi * the map BTF record, we need to execute an RCU barrier to wait for 753*9db44fddSKumar Kartikeya Dwivedi * them as the record will be freed right after our map_free callback. 754*9db44fddSKumar Kartikeya Dwivedi */ 755*9db44fddSKumar Kartikeya Dwivedi if (!IS_ERR_OR_NULL(smap->map.record)) { 756*9db44fddSKumar Kartikeya Dwivedi rcu_barrier_tasks_trace(); 757*9db44fddSKumar Kartikeya Dwivedi /* We cannot skip rcu_barrier() when rcu_trace_implies_rcu_gp() 758*9db44fddSKumar Kartikeya Dwivedi * is true, because while call_rcu invocation is skipped in that 759*9db44fddSKumar Kartikeya Dwivedi * case in bpf_selem_free_tasks_trace_rcu (and all local storage 760*9db44fddSKumar Kartikeya Dwivedi * maps pass use_trace_rcu = true), there can be call_rcu 761*9db44fddSKumar Kartikeya Dwivedi * callbacks based on use_trace_rcu = false in the earlier while 762*9db44fddSKumar Kartikeya Dwivedi * ((selem = ...)) loop or from bpf_local_storage_unlink_nolock 763*9db44fddSKumar Kartikeya Dwivedi * called from owner's free path. 764*9db44fddSKumar Kartikeya Dwivedi */ 765*9db44fddSKumar Kartikeya Dwivedi rcu_barrier(); 766*9db44fddSKumar Kartikeya Dwivedi } 76773cf09a3SYafang Shao bpf_map_area_free(smap); 768450af8d0SKP Singh } 769