diff options
| author |   Vlad Buslov <vladbu@mellanox.com> | 2019-09-13 18:28:41 +0300 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-09-16 09:18:03 +0200 |
| commit | 470d5060e6b3b8fae47d944601855e9ece7a2470 (patch) | |
| tree | b91a0dfb567b5ed2878d5543014a65196f7fe7ae /include/net/act_api.h | |
| parent | net: sched: take reference to psample group in flow_action infra (diff) | |
| download | wireguard-linux-470d5060e6b3b8fae47d944601855e9ece7a2470.tar.xz wireguard-linux-470d5060e6b3b8fae47d944601855e9ece7a2470.zip | |
net: sched: use get_dev() action API in flow_action infra
When filling in hardware intermediate representation tc_setup_flow_action()
directly obtains, checks and takes reference to dev used by mirred action,
instead of using act->ops->get_dev() API created specifically for this
purpose. In order to remove code duplication, refactor flow_action infra to
use action API when obtaining mirred action target dev. Extend get_dev()
with additional argument that is used to provide dev destructor to the
user.
Fixes: 5a6ff4b13d59 ("net: sched: take reference to action dev before calling offloads")
Signed-off-by: Vlad Buslov <vladbu@mellanox.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/act_api.h')
| -rw-r--r-- | include/net/act_api.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/net/act_api.h b/include/net/act_api.h index 4be8b0daedf0..b18c699681ca 100644 --- a/include/net/act_api.h +++ b/include/net/act_api.h @@ -101,8 +101,8 @@ struct tc_action_ops { struct netlink_ext_ack *); void (*stats_update)(struct tc_action *, u64, u32, u64, bool); size_t (*get_fill_size)(const struct tc_action *act); - struct net_device *(*get_dev)(const struct tc_action *a); - void (*put_dev)(struct net_device *dev); + struct net_device *(*get_dev)(const struct tc_action *a, + tc_action_priv_destructor *destructor); struct psample_group * (*get_psample_group)(const struct tc_action *a, tc_action_priv_destructor *destructor); |