[ovs-dev] [PATCH 1/3] datapath: Replace __force type cast with rcu_dereference_raw().
Pravin B Shelar
pshelar at nicira.com
Mon Oct 20 23:13:47 UTC 2014
rcu_dereference_raw() api is cleaner way of accessing RCU pointer
when no locking is required.
Signed-off-by: Pravin B Shelar <pshelar at nicira.com>
---
datapath/datapath.c | 2 +-
datapath/flow_table.c | 8 ++++----
datapath/vport.c | 2 +-
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/datapath/datapath.c b/datapath/datapath.c
index 9e2dcfa..5ff10e5 100644
--- a/datapath/datapath.c
+++ b/datapath/datapath.c
@@ -1202,7 +1202,7 @@ static int ovs_flow_cmd_del(struct sk_buff *skb, struct genl_info *info)
ovs_flow_tbl_remove(&dp->table, flow);
ovs_unlock();
- reply = ovs_flow_cmd_alloc_info((const struct sw_flow_actions __force *)flow->sf_acts,
+ reply = ovs_flow_cmd_alloc_info(rcu_dereference_raw(flow->sf_acts),
info, false);
if (likely(reply)) {
diff --git a/datapath/flow_table.c b/datapath/flow_table.c
index 4efef13..9656cfe 100644
--- a/datapath/flow_table.c
+++ b/datapath/flow_table.c
@@ -146,11 +146,11 @@ static void flow_free(struct sw_flow *flow)
{
int node;
- kfree((struct sw_flow_actions __force *)flow->sf_acts);
+ kfree(rcu_dereference_raw(flow->sf_acts));
for_each_node(node)
if (flow->stats[node])
kmem_cache_free(flow_stats_cache,
- (struct flow_stats __force *)flow->stats[node]);
+ rcu_dereference_raw(flow->stats[node]));
kmem_cache_free(flow_cache, flow);
}
@@ -334,10 +334,10 @@ skip_flows:
* error path. */
void ovs_flow_tbl_destroy(struct flow_table *table)
{
- struct table_instance *ti = (struct table_instance __force *)table->ti;
+ struct table_instance *ti = rcu_dereference_raw(table->ti);
free_percpu(table->mask_cache);
- kfree((struct mask_array __force *)table->mask_array);
+ kfree(rcu_dereference_raw(table->mask_array));
table_instance_destroy(ti, false);
}
diff --git a/datapath/vport.c b/datapath/vport.c
index fd52903..18f3956 100644
--- a/datapath/vport.c
+++ b/datapath/vport.c
@@ -166,7 +166,7 @@ struct vport *ovs_vport_alloc(int priv_size, const struct vport_ops *ops,
*/
void ovs_vport_free(struct vport *vport)
{
- kfree((struct vport_portids __force *)vport->upcall_portids);
+ kfree(rcu_dereference_raw(vport->upcall_portids));
free_percpu(vport->percpu_stats);
kfree(vport);
}
--
1.7.1
More information about the dev
mailing list