From: Jiri Pirko <j...@mellanox.com>

Since the function name is misleading since it is not changing
anything, name it similarly to other cls.

Signed-off-by: Jiri Pirko <j...@mellanox.com>
---
 net/sched/cls_fw.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/net/sched/cls_fw.c b/net/sched/cls_fw.c
index 46a0e32..a30a098 100644
--- a/net/sched/cls_fw.c
+++ b/net/sched/cls_fw.c
@@ -190,10 +190,9 @@ static const struct nla_policy fw_policy[TCA_FW_MAX + 1] = 
{
        [TCA_FW_MASK]           = { .type = NLA_U32 },
 };
 
-static int
-fw_change_attrs(struct net *net, struct tcf_proto *tp, struct fw_filter *f,
-               struct nlattr **tb, struct nlattr **tca, unsigned long base,
-               bool ovr)
+static int fw_set_parms(struct net *net, struct tcf_proto *tp,
+                       struct fw_filter *f, struct nlattr **tb,
+                       struct nlattr **tca, unsigned long base, bool ovr)
 {
        struct fw_head *head = rtnl_dereference(tp->root);
        struct tcf_exts e;
@@ -276,7 +275,7 @@ static int fw_change(struct net *net, struct sk_buff 
*in_skb,
 
                tcf_exts_init(&fnew->exts, TCA_FW_ACT, TCA_FW_POLICE);
 
-               err = fw_change_attrs(net, tp, fnew, tb, tca, base, ovr);
+               err = fw_set_parms(net, tp, fnew, tb, tca, base, ovr);
                if (err < 0) {
                        tcf_exts_destroy(&fnew->exts);
                        kfree(fnew);
@@ -322,7 +321,7 @@ static int fw_change(struct net *net, struct sk_buff 
*in_skb,
        f->id = handle;
        f->tp = tp;
 
-       err = fw_change_attrs(net, tp, f, tb, tca, base, ovr);
+       err = fw_set_parms(net, tp, f, tb, tca, base, ovr);
        if (err < 0)
                goto errout;
 
-- 
2.9.3

Reply via email to