在 2019/8/7 0:10, Pablo Neira Ayuso 写道: > >> >> +void flow_indr_add_block_ing_cb(struct flow_indr_block_ing_entry *entry) >> +{ > ... but registration does not protect the list with a mutex. > >> + list_add_tail_rcu(&entry->list, &block_ing_cb_list); >> +} >> +EXPORT_SYMBOL_GPL(flow_indr_add_block_ing_cb); yes, I think the flow_indr_add_block_ing_cb and flow_indr_del_block_ing_cb maybe used for more subsystem in the future. Both of them should add a mutex lock