All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.