On Tue, Aug 10, 2021 at 08:53:14PM +0800, Yang Yingliang wrote: > A spin lock is taken in __nat25_db_network_insert() and > update_BCNTIM() is called under spin lock so we should > use GFP_ATOMIC in both place. > > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Yang Yingliang <yangyingliang@xxxxxxxxxx> Good spot. This goes back to the original add a driver commit. It used to call rtw_malloc() but that was buggy. Fixes: 15865124feed ("staging: r8188eu: introduce new core dir for RTL8188eu driver") Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> regards, dan carpenter