On Fri, 2022-01-14 at 12:34 -0800, Aloka Dixit wrote: > > With that, what would be a better way: > (1) Making it work with pseudo code, still using GFP_KERNEL or > (2) Changing to GFP_ATOMIC but otherwise keep the code fairly similar to > v13 (preferably allocating an array instead of separate pointers as you > suggested)? > It's a good question. I'm not really into the code right now, but I'd say GFP_ATOMIC should be OK. I'm not even sure the functions you're modifying are always guaranteed to be called in a context where you can sleep? E.g. I believe the driver can call ieee80211_beacon_get_tim() in atomic context itself, e.g. even from timer/interrupt to get the next beacon? johannes
