On 08/21/2017 02:42 AM, Alexei Starovoitov wrote:
On 8/20/17 4:48 PM, Daniel Borkmann wrote:
[...]
I wonder why it was done the other way around in the first place then? dev_map_list is there only for notifier and since the map is freed with all the devices totally makes sense to isolate it from notifier as a first step.
Yep, agree. Initially this was done by the mutex, but that was not correct due to RCU for map helpers, of course.