Commit 41411e2f authored by wenxu's avatar wenxu Committed by David S. Miller

net/sched: act_tunnel_key: Add dst_cache support

The metadata_dst is not init the dst_cache which make the
ip_md_tunnel_xmit can't use the dst_cache. It will lookup
route table every packets.
Signed-off-by: default avatarwenxu <wenxu@ucloud.cn>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent caf337bd
...@@ -321,12 +321,18 @@ static int tunnel_key_init(struct net *net, struct nlattr *nla, ...@@ -321,12 +321,18 @@ static int tunnel_key_init(struct net *net, struct nlattr *nla,
goto err_out; goto err_out;
} }
#ifdef CONFIG_DST_CACHE
ret = dst_cache_init(&metadata->u.tun_info.dst_cache, GFP_KERNEL);
if (ret)
goto release_tun_meta;
#endif
if (opts_len) { if (opts_len) {
ret = tunnel_key_opts_set(tb[TCA_TUNNEL_KEY_ENC_OPTS], ret = tunnel_key_opts_set(tb[TCA_TUNNEL_KEY_ENC_OPTS],
&metadata->u.tun_info, &metadata->u.tun_info,
opts_len, extack); opts_len, extack);
if (ret < 0) if (ret < 0)
goto release_tun_meta; goto release_dst_cache;
} }
metadata->u.tun_info.mode |= IP_TUNNEL_INFO_TX; metadata->u.tun_info.mode |= IP_TUNNEL_INFO_TX;
...@@ -342,14 +348,14 @@ static int tunnel_key_init(struct net *net, struct nlattr *nla, ...@@ -342,14 +348,14 @@ static int tunnel_key_init(struct net *net, struct nlattr *nla,
&act_tunnel_key_ops, bind, true); &act_tunnel_key_ops, bind, true);
if (ret) { if (ret) {
NL_SET_ERR_MSG(extack, "Cannot create TC IDR"); NL_SET_ERR_MSG(extack, "Cannot create TC IDR");
goto release_tun_meta; goto release_dst_cache;
} }
ret = ACT_P_CREATED; ret = ACT_P_CREATED;
} else if (!ovr) { } else if (!ovr) {
NL_SET_ERR_MSG(extack, "TC IDR already exists"); NL_SET_ERR_MSG(extack, "TC IDR already exists");
ret = -EEXIST; ret = -EEXIST;
goto release_tun_meta; goto release_dst_cache;
} }
t = to_tunnel_key(*a); t = to_tunnel_key(*a);
...@@ -359,7 +365,7 @@ static int tunnel_key_init(struct net *net, struct nlattr *nla, ...@@ -359,7 +365,7 @@ static int tunnel_key_init(struct net *net, struct nlattr *nla,
NL_SET_ERR_MSG(extack, "Cannot allocate tunnel key parameters"); NL_SET_ERR_MSG(extack, "Cannot allocate tunnel key parameters");
ret = -ENOMEM; ret = -ENOMEM;
exists = true; exists = true;
goto release_tun_meta; goto release_dst_cache;
} }
params_new->tcft_action = parm->t_action; params_new->tcft_action = parm->t_action;
params_new->tcft_enc_metadata = metadata; params_new->tcft_enc_metadata = metadata;
...@@ -376,6 +382,10 @@ static int tunnel_key_init(struct net *net, struct nlattr *nla, ...@@ -376,6 +382,10 @@ static int tunnel_key_init(struct net *net, struct nlattr *nla,
return ret; return ret;
release_dst_cache:
#ifdef CONFIG_DST_CACHE
dst_cache_destroy(&metadata->u.tun_info.dst_cache);
#endif
release_tun_meta: release_tun_meta:
dst_release(&metadata->dst); dst_release(&metadata->dst);
...@@ -391,8 +401,15 @@ static void tunnel_key_release(struct tc_action *a) ...@@ -391,8 +401,15 @@ static void tunnel_key_release(struct tc_action *a)
{ {
struct tcf_tunnel_key *t = to_tunnel_key(a); struct tcf_tunnel_key *t = to_tunnel_key(a);
struct tcf_tunnel_key_params *params; struct tcf_tunnel_key_params *params;
#ifdef CONFIG_DST_CACHE
struct ip_tunnel_info *info;
#endif
params = rcu_dereference_protected(t->params, 1); params = rcu_dereference_protected(t->params, 1);
#ifdef CONFIG_DST_CACHE
info = &params->tcft_enc_metadata->u.tun_info;
dst_cache_destroy(&info->dst_cache);
#endif
tunnel_key_release_params(params); tunnel_key_release_params(params);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment