On 03/15/2011 07:04 PM, Eric Dumazet wrote:
> Le mardi 15 mars 2011 Ã 18:11 +0800, Lai Jiangshan a Ãcrit :
>>
>> There is no callback of this module maybe queued
>> since we use kfree_rcu(), we can safely remove the rcu_barrier().
>>
>> Signed-off-by: Lai Jiangshan <laijs@xxxxxxxxxxxxxx>
>> ---
>> net/sched/act_police.c | 1 -
>> 1 files changed, 0 insertions(+), 1 deletions(-)
>>
>> diff --git a/net/sched/act_police.c b/net/sched/act_police.c
>> index 083b091..22e3f9b 100644
>> --- a/net/sched/act_police.c
>> +++ b/net/sched/act_police.c
>> @@ -397,7 +397,6 @@ static void __exit
>> police_cleanup_module(void)
>> {
>> tcf_unregister_action(&act_police_ops);
>> - rcu_barrier(); /* Wait for completion of call_rcu()'s (tcf_police_free_rcu) */
>> }
>>
>> module_init(police_init_module);
>
>
> Why is it a separate patch, and not included in patch 5/17 ?
>
A simple_kfree_callback() a patch, if a module has multiple simple_kfree_callback()s,
"rcu_barrier()" in module_exit function should only be removed after all
callback()s are converted to kfree_rcu(). Separate patches makes things
clearer.
_______________________________________________
Containers mailing list
Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/containers
