diff mbox

[net-next,1/1] net sched actions: rename act_get_notify() to tcf_get_notify()

Message ID 1499283238-2551-1-git-send-email-mrv@mojatatu.com
State Deferred, archived
Delegated to: David Miller
Headers show

Commit Message

Roman Mashak July 5, 2017, 7:33 p.m. UTC
Make name consistent with other TC event notification routines.

Signed-off-by: Roman Mashak <mrv@mojatatu.com>
---
 net/sched/act_api.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Jiri Pirko July 5, 2017, 8:31 p.m. UTC | #1
Wed, Jul 05, 2017 at 09:33:58PM CEST, mrv@mojatatu.com wrote:
>Make name consistent with other TC event notification routines.
>
>Signed-off-by: Roman Mashak <mrv@mojatatu.com>
>---
> net/sched/act_api.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/net/sched/act_api.c b/net/sched/act_api.c
>index aed6cf2..f2e9ed3 100644
>--- a/net/sched/act_api.c
>+++ b/net/sched/act_api.c
>@@ -835,7 +835,7 @@ static int tca_get_fill(struct sk_buff *skb, struct list_head *actions,
> }
> 
> static int
>-act_get_notify(struct net *net, u32 portid, struct nlmsghdr *n,
>+tcf_get_notify(struct net *net, u32 portid, struct nlmsghdr *n,

Rather call it "tcf_action_get_notify"

+ net-next is closed if I'm not mistaken
Roman Mashak July 5, 2017, 9:13 p.m. UTC | #2
Jiri Pirko <jiri@resnulli.us> writes:

> Wed, Jul 05, 2017 at 09:33:58PM CEST, mrv@mojatatu.com wrote:
>>Make name consistent with other TC event notification routines.
>>
>>Signed-off-by: Roman Mashak <mrv@mojatatu.com>
>>---
>> net/sched/act_api.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>>diff --git a/net/sched/act_api.c b/net/sched/act_api.c
>>index aed6cf2..f2e9ed3 100644
>>--- a/net/sched/act_api.c
>>+++ b/net/sched/act_api.c
>>@@ -835,7 +835,7 @@ static int tca_get_fill(struct sk_buff *skb, struct list_head *actions,
>> }
>> 
>> static int
>>-act_get_notify(struct net *net, u32 portid, struct nlmsghdr *n,
>>+tcf_get_notify(struct net *net, u32 portid, struct nlmsghdr *n,
>
> Rather call it "tcf_action_get_notify"

We already have tcf_add_notify() and tcf_del_notify() as a common style.

>
> + net-next is closed if I'm not mistaken
diff mbox

Patch

diff --git a/net/sched/act_api.c b/net/sched/act_api.c
index aed6cf2..f2e9ed3 100644
--- a/net/sched/act_api.c
+++ b/net/sched/act_api.c
@@ -835,7 +835,7 @@  static int tca_get_fill(struct sk_buff *skb, struct list_head *actions,
 }
 
 static int
-act_get_notify(struct net *net, u32 portid, struct nlmsghdr *n,
+tcf_get_notify(struct net *net, u32 portid, struct nlmsghdr *n,
 	       struct list_head *actions, int event)
 {
 	struct sk_buff *skb;
@@ -1018,7 +1018,7 @@  static int tca_action_flush(struct net *net, struct nlattr *nla,
 	}
 
 	if (event == RTM_GETACTION)
-		ret = act_get_notify(net, portid, n, &actions, event);
+		ret = tcf_get_notify(net, portid, n, &actions, event);
 	else { /* delete */
 		ret = tcf_del_notify(net, n, &actions, portid);
 		if (ret)