* [Patch net-next] net: remove netif_set_real_num_rx_queues()
@ 2014-11-10 19:35 Cong Wang
2014-11-11 13:10 ` Edward Cree
0 siblings, 1 reply; 4+ messages in thread
From: Cong Wang @ 2014-11-10 19:35 UTC (permalink / raw)
To: netdev; +Cc: Eric Dumazet, David S. Miller, Cong Wang
vlan was the only user of netif_set_real_num_rx_queues(),
but it no longer calls it after
commit 4af429d29b341bb1735f04c2fb960178 ("vlan: lockless transmit path").
So we can just remove it.
Cc: Eric Dumazet <eric.dumazet@gmail.com>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
---
diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
index 888d551..4a6f770 100644
--- a/include/linux/netdevice.h
+++ b/include/linux/netdevice.h
@@ -2762,23 +2762,6 @@ static inline int netif_set_real_num_rx_queues(struct net_device *dev,
}
#endif
-static inline int netif_copy_real_num_queues(struct net_device *to_dev,
- const struct net_device *from_dev)
-{
- int err;
-
- err = netif_set_real_num_tx_queues(to_dev,
- from_dev->real_num_tx_queues);
- if (err)
- return err;
-#ifdef CONFIG_SYSFS
- return netif_set_real_num_rx_queues(to_dev,
- from_dev->real_num_rx_queues);
-#else
- return 0;
-#endif
-}
-
#ifdef CONFIG_SYSFS
static inline unsigned int get_netdev_rx_queue_index(
struct netdev_rx_queue *queue)
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Patch net-next] net: remove netif_set_real_num_rx_queues()
2014-11-10 19:35 [Patch net-next] net: remove netif_set_real_num_rx_queues() Cong Wang
@ 2014-11-11 13:10 ` Edward Cree
2014-11-11 17:07 ` Cong Wang
0 siblings, 1 reply; 4+ messages in thread
From: Edward Cree @ 2014-11-11 13:10 UTC (permalink / raw)
To: Cong Wang; +Cc: netdev, Eric Dumazet, David S. Miller
On 10/11/14 19:35, Cong Wang wrote:
> vlan was the only user of netif_set_real_num_rx_queues(),
> but it no longer calls it after
> commit 4af429d29b341bb1735f04c2fb960178 ("vlan: lockless transmit path").
> So we can just remove it.
>
> Cc: Eric Dumazet <eric.dumazet@gmail.com>
> Cc: David S. Miller <davem@davemloft.net>
> Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
>
> ---
> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
> index 888d551..4a6f770 100644
> --- a/include/linux/netdevice.h
> +++ b/include/linux/netdevice.h
> @@ -2762,23 +2762,6 @@ static inline int netif_set_real_num_rx_queues(struct net_device *dev,
> }
> #endif
>
> -static inline int netif_copy_real_num_queues(struct net_device *to_dev,
> - const struct net_device *from_dev)
Patch title says you're removing _set_ but body only removes _copy_.
Which one is right?
> -{
> - int err;
> -
> - err = netif_set_real_num_tx_queues(to_dev,
> - from_dev->real_num_tx_queues);
> - if (err)
> - return err;
> -#ifdef CONFIG_SYSFS
> - return netif_set_real_num_rx_queues(to_dev,
> - from_dev->real_num_rx_queues);
> -#else
> - return 0;
> -#endif
> -}
> -
> #ifdef CONFIG_SYSFS
> static inline unsigned int get_netdev_rx_queue_index(
> struct netdev_rx_queue *queue)
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Patch net-next] net: remove netif_set_real_num_rx_queues()
2014-11-11 13:10 ` Edward Cree
@ 2014-11-11 17:07 ` Cong Wang
2014-11-11 20:22 ` David Miller
0 siblings, 1 reply; 4+ messages in thread
From: Cong Wang @ 2014-11-11 17:07 UTC (permalink / raw)
To: Edward Cree
Cc: Linux Kernel Network Developers, Eric Dumazet, David S. Miller
On Tue, Nov 11, 2014 at 5:10 AM, Edward Cree <ecree@solarflare.com> wrote:
>> -static inline int netif_copy_real_num_queues(struct net_device *to_dev,
>> - const struct net_device *from_dev)
> Patch title says you're removing _set_ but body only removes _copy_.
> Which one is right?
Oops, my copy-n-paste error...
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Patch net-next] net: remove netif_set_real_num_rx_queues()
2014-11-11 17:07 ` Cong Wang
@ 2014-11-11 20:22 ` David Miller
0 siblings, 0 replies; 4+ messages in thread
From: David Miller @ 2014-11-11 20:22 UTC (permalink / raw)
To: xiyou.wangcong; +Cc: ecree, netdev, eric.dumazet
From: Cong Wang <xiyou.wangcong@gmail.com>
Date: Tue, 11 Nov 2014 09:07:29 -0800
> On Tue, Nov 11, 2014 at 5:10 AM, Edward Cree <ecree@solarflare.com> wrote:
>>> -static inline int netif_copy_real_num_queues(struct net_device *to_dev,
>>> - const struct net_device *from_dev)
>> Patch title says you're removing _set_ but body only removes _copy_.
>> Which one is right?
>
> Oops, my copy-n-paste error...
Please resubmit with fixed title, thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-11-11 20:22 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-10 19:35 [Patch net-next] net: remove netif_set_real_num_rx_queues() Cong Wang
2014-11-11 13:10 ` Edward Cree
2014-11-11 17:07 ` Cong Wang
2014-11-11 20:22 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).