* [PATCH net-next] net: Fix ndo_setup_tc comment
@ 2017-01-26 22:44 Florian Fainelli
2017-01-26 22:59 ` John Fastabend
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Florian Fainelli @ 2017-01-26 22:44 UTC (permalink / raw)
To: netdev; +Cc: davem, john.r.fastabend, jiri, jhs, Florian Fainelli
Commit 16e5cc647173 ("net: rework setup_tc ndo op to consume
general tc operand") changed the ndo_setup_tc() signature, but did not
update the comments in netdevice.h, so do that now.
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
include/linux/netdevice.h | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
index 3868c32d98af..d63cacb67ea6 100644
--- a/include/linux/netdevice.h
+++ b/include/linux/netdevice.h
@@ -964,11 +964,12 @@ struct netdev_xdp {
* with PF and querying it may introduce a theoretical security risk.
* int (*ndo_set_vf_rss_query_en)(struct net_device *dev, int vf, bool setting);
* int (*ndo_get_vf_port)(struct net_device *dev, int vf, struct sk_buff *skb);
- * int (*ndo_setup_tc)(struct net_device *dev, u8 tc)
- * Called to setup 'tc' number of traffic classes in the net device. This
- * is always called from the stack with the rtnl lock held and netif tx
- * queues stopped. This allows the netdevice to perform queue management
- * safely.
+ * int (*ndo_setup_tc)(struct net_device *dev, u32 handle,
+ * __be16 protocol, struct tc_to_netdev *tc);
+ * Called to setup any 'tc' scheduler, classifier or action on @dev.
+ * This is always called from the stack with the rtnl lock held and netif
+ * tx queues stopped. This allows the netdevice to perform queue
+ * management safely.
*
* Fiber Channel over Ethernet (FCoE) offload functions.
* int (*ndo_fcoe_enable)(struct net_device *dev);
--
2.9.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] net: Fix ndo_setup_tc comment
2017-01-26 22:44 [PATCH net-next] net: Fix ndo_setup_tc comment Florian Fainelli
@ 2017-01-26 22:59 ` John Fastabend
2017-01-27 7:12 ` Jiri Pirko
2017-01-27 14:15 ` David Miller
2 siblings, 0 replies; 4+ messages in thread
From: John Fastabend @ 2017-01-26 22:59 UTC (permalink / raw)
To: Florian Fainelli, netdev; +Cc: davem, john.r.fastabend, jiri, jhs
On 17-01-26 02:44 PM, Florian Fainelli wrote:
> Commit 16e5cc647173 ("net: rework setup_tc ndo op to consume
> general tc operand") changed the ndo_setup_tc() signature, but did not
> update the comments in netdevice.h, so do that now.
>
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> ---
Thanks.
Acked-by: John Fastabend <john.r.fastabend@intel.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] net: Fix ndo_setup_tc comment
2017-01-26 22:44 [PATCH net-next] net: Fix ndo_setup_tc comment Florian Fainelli
2017-01-26 22:59 ` John Fastabend
@ 2017-01-27 7:12 ` Jiri Pirko
2017-01-27 14:15 ` David Miller
2 siblings, 0 replies; 4+ messages in thread
From: Jiri Pirko @ 2017-01-27 7:12 UTC (permalink / raw)
To: Florian Fainelli; +Cc: netdev, davem, john.r.fastabend, jiri, jhs
Thu, Jan 26, 2017 at 11:44:17PM CET, f.fainelli@gmail.com wrote:
>Commit 16e5cc647173 ("net: rework setup_tc ndo op to consume
>general tc operand") changed the ndo_setup_tc() signature, but did not
>update the comments in netdevice.h, so do that now.
>
>Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Reviewed-by: Jiri Pirko <jiri@mellanox.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] net: Fix ndo_setup_tc comment
2017-01-26 22:44 [PATCH net-next] net: Fix ndo_setup_tc comment Florian Fainelli
2017-01-26 22:59 ` John Fastabend
2017-01-27 7:12 ` Jiri Pirko
@ 2017-01-27 14:15 ` David Miller
2 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2017-01-27 14:15 UTC (permalink / raw)
To: f.fainelli; +Cc: netdev, john.r.fastabend, jiri, jhs
From: Florian Fainelli <f.fainelli@gmail.com>
Date: Thu, 26 Jan 2017 14:44:17 -0800
> Commit 16e5cc647173 ("net: rework setup_tc ndo op to consume
> general tc operand") changed the ndo_setup_tc() signature, but did not
> update the comments in netdevice.h, so do that now.
>
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Applied.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-01-27 14:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-26 22:44 [PATCH net-next] net: Fix ndo_setup_tc comment Florian Fainelli
2017-01-26 22:59 ` John Fastabend
2017-01-27 7:12 ` Jiri Pirko
2017-01-27 14:15 ` David Miller
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.