On 1/12/21 10:31 PM, Ye Bin wrote: > sdev->handler_data = NULL; > + synchronize_rcu(); > kfree(h); What is the purpose of the new synchronize_rcu() call? If its purpose is to wait until *h is no longer in use, please use kfree_rcu() instead. Thanks, Bart.