* [PATCH net-next] net: dev_addr_list: Introduce __dev_addr_add() and __dev_addr_del()
@ 2021-10-08 8:53 Yajun Deng
2021-10-08 9:02 ` yajun.deng
2021-10-08 18:25 ` Jakub Kicinski
0 siblings, 2 replies; 3+ messages in thread
From: Yajun Deng @ 2021-10-08 8:53 UTC (permalink / raw)
To: davem, kuba; +Cc: netdev, linux-kernel, Yajun Deng
Introduce helper functions __dev_addr_add() and __dev_addr_del() for
the same code, make the code more concise.
Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
---
net/core/dev_addr_lists.c | 117 ++++++++++++++++----------------------
1 file changed, 48 insertions(+), 69 deletions(-)
diff --git a/net/core/dev_addr_lists.c b/net/core/dev_addr_lists.c
index f0cb38344126..f6e33bdc0a30 100644
--- a/net/core/dev_addr_lists.c
+++ b/net/core/dev_addr_lists.c
@@ -614,6 +614,38 @@ int dev_addr_del(struct net_device *dev, const unsigned char *addr,
}
EXPORT_SYMBOL(dev_addr_del);
+static int __dev_addr_add(struct net_device *dev, const unsigned char *addr,
+ unsigned char addr_type, bool global, bool sync,
+ bool exclusive)
+{
+ int err;
+
+ netif_addr_lock_bh(dev);
+ err = __hw_addr_add_ex(&dev->uc, addr, dev->addr_len,
+ addr_type, global, sync,
+ 0, exclusive);
+ if (!err)
+ __dev_set_rx_mode(dev);
+ netif_addr_unlock_bh(dev);
+
+ return err;
+}
+
+static int __dev_addr_del(struct net_device *dev, const unsigned char *addr,
+ unsigned char addr_type, bool global, bool sync)
+{
+ int err;
+
+ netif_addr_lock_bh(dev);
+ err = __hw_addr_del_ex(&dev->uc, addr, dev->addr_len,
+ addr_type, global, sync);
+ if (!err)
+ __dev_set_rx_mode(dev);
+ netif_addr_unlock_bh(dev);
+
+ return err;
+}
+
/*
* Unicast list handling functions
*/
@@ -625,16 +657,9 @@ EXPORT_SYMBOL(dev_addr_del);
*/
int dev_uc_add_excl(struct net_device *dev, const unsigned char *addr)
{
- int err;
- netif_addr_lock_bh(dev);
- err = __hw_addr_add_ex(&dev->uc, addr, dev->addr_len,
- NETDEV_HW_ADDR_T_UNICAST, true, false,
- 0, true);
- if (!err)
- __dev_set_rx_mode(dev);
- netif_addr_unlock_bh(dev);
- return err;
+ return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_UNICAST,
+ true, false, true);
}
EXPORT_SYMBOL(dev_uc_add_excl);
@@ -648,15 +673,8 @@ EXPORT_SYMBOL(dev_uc_add_excl);
*/
int dev_uc_add(struct net_device *dev, const unsigned char *addr)
{
- int err;
-
- netif_addr_lock_bh(dev);
- err = __hw_addr_add(&dev->uc, addr, dev->addr_len,
- NETDEV_HW_ADDR_T_UNICAST);
- if (!err)
- __dev_set_rx_mode(dev);
- netif_addr_unlock_bh(dev);
- return err;
+ return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_UNICAST,
+ false, false, false);
}
EXPORT_SYMBOL(dev_uc_add);
@@ -670,15 +688,8 @@ EXPORT_SYMBOL(dev_uc_add);
*/
int dev_uc_del(struct net_device *dev, const unsigned char *addr)
{
- int err;
-
- netif_addr_lock_bh(dev);
- err = __hw_addr_del(&dev->uc, addr, dev->addr_len,
- NETDEV_HW_ADDR_T_UNICAST);
- if (!err)
- __dev_set_rx_mode(dev);
- netif_addr_unlock_bh(dev);
- return err;
+ return __dev_addr_del(dev, addr, NETDEV_HW_ADDR_T_UNICAST,
+ false, false);
}
EXPORT_SYMBOL(dev_uc_del);
@@ -810,33 +821,11 @@ EXPORT_SYMBOL(dev_uc_init);
*/
int dev_mc_add_excl(struct net_device *dev, const unsigned char *addr)
{
- int err;
-
- netif_addr_lock_bh(dev);
- err = __hw_addr_add_ex(&dev->mc, addr, dev->addr_len,
- NETDEV_HW_ADDR_T_MULTICAST, true, false,
- 0, true);
- if (!err)
- __dev_set_rx_mode(dev);
- netif_addr_unlock_bh(dev);
- return err;
+ return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
+ true, false, true);
}
EXPORT_SYMBOL(dev_mc_add_excl);
-static int __dev_mc_add(struct net_device *dev, const unsigned char *addr,
- bool global)
-{
- int err;
-
- netif_addr_lock_bh(dev);
- err = __hw_addr_add_ex(&dev->mc, addr, dev->addr_len,
- NETDEV_HW_ADDR_T_MULTICAST, global, false,
- 0, false);
- if (!err)
- __dev_set_rx_mode(dev);
- netif_addr_unlock_bh(dev);
- return err;
-}
/**
* dev_mc_add - Add a multicast address
* @dev: device
@@ -847,7 +836,8 @@ static int __dev_mc_add(struct net_device *dev, const unsigned char *addr,
*/
int dev_mc_add(struct net_device *dev, const unsigned char *addr)
{
- return __dev_mc_add(dev, addr, false);
+ return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
+ false, false, false);
}
EXPORT_SYMBOL(dev_mc_add);
@@ -860,24 +850,11 @@ EXPORT_SYMBOL(dev_mc_add);
*/
int dev_mc_add_global(struct net_device *dev, const unsigned char *addr)
{
- return __dev_mc_add(dev, addr, true);
+ return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
+ true, false, false);
}
EXPORT_SYMBOL(dev_mc_add_global);
-static int __dev_mc_del(struct net_device *dev, const unsigned char *addr,
- bool global)
-{
- int err;
-
- netif_addr_lock_bh(dev);
- err = __hw_addr_del_ex(&dev->mc, addr, dev->addr_len,
- NETDEV_HW_ADDR_T_MULTICAST, global, false);
- if (!err)
- __dev_set_rx_mode(dev);
- netif_addr_unlock_bh(dev);
- return err;
-}
-
/**
* dev_mc_del - Delete a multicast address.
* @dev: device
@@ -888,7 +865,8 @@ static int __dev_mc_del(struct net_device *dev, const unsigned char *addr,
*/
int dev_mc_del(struct net_device *dev, const unsigned char *addr)
{
- return __dev_mc_del(dev, addr, false);
+ return __dev_addr_del(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
+ false, false);
}
EXPORT_SYMBOL(dev_mc_del);
@@ -902,7 +880,8 @@ EXPORT_SYMBOL(dev_mc_del);
*/
int dev_mc_del_global(struct net_device *dev, const unsigned char *addr)
{
- return __dev_mc_del(dev, addr, true);
+ return __dev_addr_del(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
+ true, false);
}
EXPORT_SYMBOL(dev_mc_del_global);
--
2.32.0
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH net-next] net: dev_addr_list: Introduce __dev_addr_add() and __dev_addr_del()
2021-10-08 8:53 [PATCH net-next] net: dev_addr_list: Introduce __dev_addr_add() and __dev_addr_del() Yajun Deng
@ 2021-10-08 9:02 ` yajun.deng
2021-10-08 18:25 ` Jakub Kicinski
1 sibling, 0 replies; 3+ messages in thread
From: yajun.deng @ 2021-10-08 9:02 UTC (permalink / raw)
To: davem, kuba; +Cc: netdev, linux-kernel
Sorry, There are still some problems with this patch.
October 8, 2021 4:53 PM, "Yajun Deng" <yajun.deng@linux.dev> 写到:
> Introduce helper functions __dev_addr_add() and __dev_addr_del() for
> the same code, make the code more concise.
>
> Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
> ---
> net/core/dev_addr_lists.c | 117 ++++++++++++++++----------------------
> 1 file changed, 48 insertions(+), 69 deletions(-)
>
> diff --git a/net/core/dev_addr_lists.c b/net/core/dev_addr_lists.c
> index f0cb38344126..f6e33bdc0a30 100644
> --- a/net/core/dev_addr_lists.c
> +++ b/net/core/dev_addr_lists.c
> @@ -614,6 +614,38 @@ int dev_addr_del(struct net_device *dev, const unsigned char *addr,
> }
> EXPORT_SYMBOL(dev_addr_del);
>
> +static int __dev_addr_add(struct net_device *dev, const unsigned char *addr,
> + unsigned char addr_type, bool global, bool sync,
> + bool exclusive)
> +{
> + int err;
> +
> + netif_addr_lock_bh(dev);
> + err = __hw_addr_add_ex(&dev->uc, addr, dev->addr_len,
> + addr_type, global, sync,
> + 0, exclusive);
> + if (!err)
> + __dev_set_rx_mode(dev);
> + netif_addr_unlock_bh(dev);
> +
> + return err;
> +}
> +
> +static int __dev_addr_del(struct net_device *dev, const unsigned char *addr,
> + unsigned char addr_type, bool global, bool sync)
> +{
> + int err;
> +
> + netif_addr_lock_bh(dev);
> + err = __hw_addr_del_ex(&dev->uc, addr, dev->addr_len,
> + addr_type, global, sync);
> + if (!err)
> + __dev_set_rx_mode(dev);
> + netif_addr_unlock_bh(dev);
> +
> + return err;
> +}
> +
> /*
> * Unicast list handling functions
> */
> @@ -625,16 +657,9 @@ EXPORT_SYMBOL(dev_addr_del);
> */
> int dev_uc_add_excl(struct net_device *dev, const unsigned char *addr)
> {
> - int err;
>
> - netif_addr_lock_bh(dev);
> - err = __hw_addr_add_ex(&dev->uc, addr, dev->addr_len,
> - NETDEV_HW_ADDR_T_UNICAST, true, false,
> - 0, true);
> - if (!err)
> - __dev_set_rx_mode(dev);
> - netif_addr_unlock_bh(dev);
> - return err;
> + return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_UNICAST,
> + true, false, true);
> }
> EXPORT_SYMBOL(dev_uc_add_excl);
>
> @@ -648,15 +673,8 @@ EXPORT_SYMBOL(dev_uc_add_excl);
> */
> int dev_uc_add(struct net_device *dev, const unsigned char *addr)
> {
> - int err;
> -
> - netif_addr_lock_bh(dev);
> - err = __hw_addr_add(&dev->uc, addr, dev->addr_len,
> - NETDEV_HW_ADDR_T_UNICAST);
> - if (!err)
> - __dev_set_rx_mode(dev);
> - netif_addr_unlock_bh(dev);
> - return err;
> + return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_UNICAST,
> + false, false, false);
> }
> EXPORT_SYMBOL(dev_uc_add);
>
> @@ -670,15 +688,8 @@ EXPORT_SYMBOL(dev_uc_add);
> */
> int dev_uc_del(struct net_device *dev, const unsigned char *addr)
> {
> - int err;
> -
> - netif_addr_lock_bh(dev);
> - err = __hw_addr_del(&dev->uc, addr, dev->addr_len,
> - NETDEV_HW_ADDR_T_UNICAST);
> - if (!err)
> - __dev_set_rx_mode(dev);
> - netif_addr_unlock_bh(dev);
> - return err;
> + return __dev_addr_del(dev, addr, NETDEV_HW_ADDR_T_UNICAST,
> + false, false);
> }
> EXPORT_SYMBOL(dev_uc_del);
>
> @@ -810,33 +821,11 @@ EXPORT_SYMBOL(dev_uc_init);
> */
> int dev_mc_add_excl(struct net_device *dev, const unsigned char *addr)
> {
> - int err;
> -
> - netif_addr_lock_bh(dev);
> - err = __hw_addr_add_ex(&dev->mc, addr, dev->addr_len,
> - NETDEV_HW_ADDR_T_MULTICAST, true, false,
> - 0, true);
> - if (!err)
> - __dev_set_rx_mode(dev);
> - netif_addr_unlock_bh(dev);
> - return err;
> + return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
> + true, false, true);
> }
> EXPORT_SYMBOL(dev_mc_add_excl);
>
> -static int __dev_mc_add(struct net_device *dev, const unsigned char *addr,
> - bool global)
> -{
> - int err;
> -
> - netif_addr_lock_bh(dev);
> - err = __hw_addr_add_ex(&dev->mc, addr, dev->addr_len,
> - NETDEV_HW_ADDR_T_MULTICAST, global, false,
> - 0, false);
> - if (!err)
> - __dev_set_rx_mode(dev);
> - netif_addr_unlock_bh(dev);
> - return err;
> -}
> /**
> * dev_mc_add - Add a multicast address
> * @dev: device
> @@ -847,7 +836,8 @@ static int __dev_mc_add(struct net_device *dev, const unsigned char *addr,
> */
> int dev_mc_add(struct net_device *dev, const unsigned char *addr)
> {
> - return __dev_mc_add(dev, addr, false);
> + return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
> + false, false, false);
> }
> EXPORT_SYMBOL(dev_mc_add);
>
> @@ -860,24 +850,11 @@ EXPORT_SYMBOL(dev_mc_add);
> */
> int dev_mc_add_global(struct net_device *dev, const unsigned char *addr)
> {
> - return __dev_mc_add(dev, addr, true);
> + return __dev_addr_add(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
> + true, false, false);
> }
> EXPORT_SYMBOL(dev_mc_add_global);
>
> -static int __dev_mc_del(struct net_device *dev, const unsigned char *addr,
> - bool global)
> -{
> - int err;
> -
> - netif_addr_lock_bh(dev);
> - err = __hw_addr_del_ex(&dev->mc, addr, dev->addr_len,
> - NETDEV_HW_ADDR_T_MULTICAST, global, false);
> - if (!err)
> - __dev_set_rx_mode(dev);
> - netif_addr_unlock_bh(dev);
> - return err;
> -}
> -
> /**
> * dev_mc_del - Delete a multicast address.
> * @dev: device
> @@ -888,7 +865,8 @@ static int __dev_mc_del(struct net_device *dev, const unsigned char *addr,
> */
> int dev_mc_del(struct net_device *dev, const unsigned char *addr)
> {
> - return __dev_mc_del(dev, addr, false);
> + return __dev_addr_del(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
> + false, false);
> }
> EXPORT_SYMBOL(dev_mc_del);
>
> @@ -902,7 +880,8 @@ EXPORT_SYMBOL(dev_mc_del);
> */
> int dev_mc_del_global(struct net_device *dev, const unsigned char *addr)
> {
> - return __dev_mc_del(dev, addr, true);
> + return __dev_addr_del(dev, addr, NETDEV_HW_ADDR_T_MULTICAST,
> + true, false);
> }
> EXPORT_SYMBOL(dev_mc_del_global);
>
> --
> 2.32.0
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH net-next] net: dev_addr_list: Introduce __dev_addr_add() and __dev_addr_del()
2021-10-08 8:53 [PATCH net-next] net: dev_addr_list: Introduce __dev_addr_add() and __dev_addr_del() Yajun Deng
2021-10-08 9:02 ` yajun.deng
@ 2021-10-08 18:25 ` Jakub Kicinski
1 sibling, 0 replies; 3+ messages in thread
From: Jakub Kicinski @ 2021-10-08 18:25 UTC (permalink / raw)
To: Yajun Deng; +Cc: davem, netdev
On Fri, 8 Oct 2021 16:53:54 +0800 Yajun Deng wrote:
> Introduce helper functions __dev_addr_add() and __dev_addr_del() for
> the same code, make the code more concise.
>
> Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
Please leave net/core/dev_addr_lists.c alone for the time being.
I have a lot of changes queued to add the main address the the rbtree.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-10-08 18:25 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-08 8:53 [PATCH net-next] net: dev_addr_list: Introduce __dev_addr_add() and __dev_addr_del() Yajun Deng
2021-10-08 9:02 ` yajun.deng
2021-10-08 18:25 ` Jakub Kicinski
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).