diff options
author | Sven Eckelmann <sven.eckelmann@gmx.de> | 2010-09-18 21:01:18 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-09-20 16:29:49 -0700 |
commit | 16f9530e99f5eb35dd1adafbae854dd94f8d3751 (patch) | |
tree | ff66e8898d8475721357ea7287418b4220c2cb08 /drivers/staging/batman-adv/types.h | |
parent | 952c699c06f562b07dba4b1f91dfc02de928545d (diff) | |
download | op-kernel-dev-16f9530e99f5eb35dd1adafbae854dd94f8d3751.zip op-kernel-dev-16f9530e99f5eb35dd1adafbae854dd94f8d3751.tar.gz |
Staging: batman-adv: Use synchronize_rcu instead of call_rcu
It is recommended [1] to use synchronize_rcu to simplify the code -
especially when otherwise extra locking is needed to protect other code
from picking stale elements. It also protects us for emitting to many
callbacks which may results in OOM conditions.
The only reason not to use it, would be in performance critical sections
or when we are not allowed to block.
[1] Documentation/RCU/checklist.txt
Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/batman-adv/types.h')
-rw-r--r-- | drivers/staging/batman-adv/types.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/staging/batman-adv/types.h b/drivers/staging/batman-adv/types.h index 478277f..b162644 100644 --- a/drivers/staging/batman-adv/types.h +++ b/drivers/staging/batman-adv/types.h @@ -44,7 +44,6 @@ struct batman_if { unsigned char *packet_buff; int packet_len; struct kobject *hardif_obj; - struct rcu_head rcu; struct packet_type batman_adv_ptype; struct net_device *soft_iface; }; |