[PATCH 17/17] net,act_police,rcu: remove rcu_barrier()
Lai Jiangshan
laijs at cn.fujitsu.com
Tue Mar 15 20:13:11 PDT 2011
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 at cn.fujitsu.com>
>> ---
>> 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.
More information about the Containers
mailing list