diff options
author | Enrico Weigelt <info@metux.net> | 2019-06-05 17:06:40 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-06-05 19:57:23 -0400 |
commit | b90f5aa4d6268e81dd1fd51e5ef89d2892bf040d (patch) | |
tree | 81b4bee2f99ce243e855c4f45d1253eca07c4ea6 | |
parent | 4546e44ca2ec6fa3ca971ab6de3ef382f17ed544 (diff) |
net: openvswitch: drop unneeded likely() call around IS_ERR()
IS_ERR() already calls unlikely(), so this extra likely() call
around the !IS_ERR() is not needed.
Signed-off-by: Enrico Weigelt <info@metux.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/openvswitch/datapath.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/openvswitch/datapath.c b/net/openvswitch/datapath.c index dc9ff9367221..4076e080914a 100644 --- a/net/openvswitch/datapath.c +++ b/net/openvswitch/datapath.c | |||
@@ -1347,7 +1347,7 @@ static int ovs_flow_cmd_del(struct sk_buff *skb, struct genl_info *info) | |||
1347 | reply = ovs_flow_cmd_alloc_info((const struct sw_flow_actions __force *) flow->sf_acts, | 1347 | reply = ovs_flow_cmd_alloc_info((const struct sw_flow_actions __force *) flow->sf_acts, |
1348 | &flow->id, info, false, ufid_flags); | 1348 | &flow->id, info, false, ufid_flags); |
1349 | if (likely(reply)) { | 1349 | if (likely(reply)) { |
1350 | if (likely(!IS_ERR(reply))) { | 1350 | if (!IS_ERR(reply)) { |
1351 | rcu_read_lock(); /*To keep RCU checker happy. */ | 1351 | rcu_read_lock(); /*To keep RCU checker happy. */ |
1352 | err = ovs_flow_cmd_fill_info(flow, ovs_header->dp_ifindex, | 1352 | err = ovs_flow_cmd_fill_info(flow, ovs_header->dp_ifindex, |
1353 | reply, info->snd_portid, | 1353 | reply, info->snd_portid, |