diff mbox series

net: tipc: add net device refcount tracker for bearer

Message ID 202308041653414100323@zte.com.cn
State New
Headers
Series net: tipc: add net device refcount tracker for bearer |

Commit Message

Yang Yang Aug. 4, 2023, 8:53 a.m. UTC 
  From: xu xin <xu.xin16@zte.com.cn>

Add net device refcount tracker to the struct tipc_bearer.

Signed-off-by: xu xin <xu.xin16@zte.com.cn>
Reviewed-by: Yang Yang <yang.yang.29@zte.com.cn>
Cc: Kuang Mingfu <kuang.mingfu@zte.com.cn>
---
 net/tipc/bearer.c | 10 +++++-----
 net/tipc/bearer.h |  1 +
 2 files changed, 6 insertions(+), 5 deletions(-)

Comments

Simon Horman Aug. 5, 2023, 11:26 a.m. UTC | #1 
On Fri, Aug 04, 2023 at 04:53:41PM +0800, yang.yang29@zte.com.cn wrote:
> From: xu xin <xu.xin16@zte.com.cn>
> 
> Add net device refcount tracker to the struct tipc_bearer.
> 
> Signed-off-by: xu xin <xu.xin16@zte.com.cn>
> Reviewed-by: Yang Yang <yang.yang.29@zte.com.cn>
> Cc: Kuang Mingfu <kuang.mingfu@zte.com.cn>

...

> @@ -479,7 +479,7 @@ void tipc_disable_l2_media(struct tipc_bearer *b)
>  	dev_remove_pack(&b->pt);
>  	RCU_INIT_POINTER(dev->tipc_ptr, NULL);
>  	synchronize_net();
> -	dev_put(dev);
> +	netdev_put(dev, &b->devtracker);
>  }
> 
>  /**
> diff --git a/net/tipc/bearer.h b/net/tipc/bearer.h
> index 41eac1ee0c09..1adeaf94aa62 100644
> --- a/net/tipc/bearer.h
> +++ b/net/tipc/bearer.h
> @@ -174,6 +174,7 @@ struct tipc_bearer {
>  	u16 encap_hlen;
>  	unsigned long up;
>  	refcount_t refcnt;
> +	netdevice_tracker	devtracker;

Hi Xu Xin and Yang Yang,

Please add netdevice_tracker to the kernel doc for struct tipc_bearer,
which appears just above the definition of the structure.

>  };
> 
>  struct tipc_bearer_names {

With that fixed, feel free to add:

Reviewed-by: Simon Horman <horms@kernel.org>
Simon Horman Aug. 6, 2023, 7:51 p.m. UTC | #2 
On Sat, Aug 05, 2023 at 01:26:56PM +0200, Simon Horman wrote:
> On Fri, Aug 04, 2023 at 04:53:41PM +0800, yang.yang29@zte.com.cn wrote:
> > From: xu xin <xu.xin16@zte.com.cn>
> > 
> > Add net device refcount tracker to the struct tipc_bearer.
> > 
> > Signed-off-by: xu xin <xu.xin16@zte.com.cn>
> > Reviewed-by: Yang Yang <yang.yang.29@zte.com.cn>
> > Cc: Kuang Mingfu <kuang.mingfu@zte.com.cn>
> 
> ...
> 
> > @@ -479,7 +479,7 @@ void tipc_disable_l2_media(struct tipc_bearer *b)
> >  	dev_remove_pack(&b->pt);
> >  	RCU_INIT_POINTER(dev->tipc_ptr, NULL);
> >  	synchronize_net();
> > -	dev_put(dev);
> > +	netdev_put(dev, &b->devtracker);
> >  }
> > 
> >  /**
> > diff --git a/net/tipc/bearer.h b/net/tipc/bearer.h
> > index 41eac1ee0c09..1adeaf94aa62 100644
> > --- a/net/tipc/bearer.h
> > +++ b/net/tipc/bearer.h
> > @@ -174,6 +174,7 @@ struct tipc_bearer {
> >  	u16 encap_hlen;
> >  	unsigned long up;
> >  	refcount_t refcnt;
> > +	netdevice_tracker	devtracker;
> 
> Hi Xu Xin and Yang Yang,
> 
> Please add netdevice_tracker to the kernel doc for struct tipc_bearer,
> which appears just above the definition of the structure.
> 
> >  };
> > 
> >  struct tipc_bearer_names {
> 
> With that fixed, feel free to add:
> 
> Reviewed-by: Simon Horman <horms@kernel.org>

Given review of other, similar patches, by Eric.
I think it would be best to confirm that this patch
has been tested.
Tung Quang Nguyen Aug. 7, 2023, 12:25 a.m. UTC | #3 
>Add net device refcount tracker to the struct tipc_bearer.
>
>Signed-off-by: xu xin <xu.xin16@zte.com.cn>
>Reviewed-by: Yang Yang <yang.yang.29@zte.com.cn>
>Cc: Kuang Mingfu <kuang.mingfu@zte.com.cn>
>---
> net/tipc/bearer.c | 10 +++++-----
> net/tipc/bearer.h |  1 +
> 2 files changed, 6 insertions(+), 5 deletions(-)
>
>diff --git a/net/tipc/bearer.c b/net/tipc/bearer.c index 2cde375477e3..01ffd1f2337a 100644
>--- a/net/tipc/bearer.c
>+++ b/net/tipc/bearer.c
>@@ -426,15 +426,15 @@ int tipc_enable_l2_media(struct net *net, struct tipc_bearer *b,
> 	struct net_device *dev;
>
> 	/* Find device with specified name */
>-	dev = dev_get_by_name(net, dev_name);
>+	dev = netdev_get_by_name(net, dev_name, &b->devtracker, GFP_KERNEL);
> 	if (!dev)
> 		return -ENODEV;
> 	if (tipc_mtu_bad(dev)) {
>-		dev_put(dev);
>+		netdev_put(dev, &b->devtracker);
> 		return -EINVAL;
> 	}
> 	if (dev == net->loopback_dev) {
>-		dev_put(dev);
>+		netdev_put(dev, &b->devtracker);
Loopback device is using tn->loopback_pt.dev_tracker, not b->devtracker.
Did you try to configure bearer on a net namespace to test if your patch works ?
diff mbox series

Patch

diff --git a/net/tipc/bearer.c b/net/tipc/bearer.c
index 2cde375477e3..01ffd1f2337a 100644
--- a/net/tipc/bearer.c
+++ b/net/tipc/bearer.c
@@ -426,15 +426,15 @@  int tipc_enable_l2_media(struct net *net, struct tipc_bearer *b,
 	struct net_device *dev;

 	/* Find device with specified name */
-	dev = dev_get_by_name(net, dev_name);
+	dev = netdev_get_by_name(net, dev_name, &b->devtracker, GFP_KERNEL);
 	if (!dev)
 		return -ENODEV;
 	if (tipc_mtu_bad(dev)) {
-		dev_put(dev);
+		netdev_put(dev, &b->devtracker);
 		return -EINVAL;
 	}
 	if (dev == net->loopback_dev) {
-		dev_put(dev);
+		netdev_put(dev, &b->devtracker);
 		pr_info("Enabling <%s> not permitted\n", b->name);
 		return -EINVAL;
 	}
@@ -445,7 +445,7 @@  int tipc_enable_l2_media(struct net *net, struct tipc_bearer *b,
 		tipc_net_init(net, node_id, 0);
 	}
 	if (!tipc_own_id(net)) {
-		dev_put(dev);
+		netdev_put(dev, &b->devtracker);
 		pr_warn("Failed to obtain node identity\n");
 		return -EINVAL;
 	}
@@ -479,7 +479,7 @@  void tipc_disable_l2_media(struct tipc_bearer *b)
 	dev_remove_pack(&b->pt);
 	RCU_INIT_POINTER(dev->tipc_ptr, NULL);
 	synchronize_net();
-	dev_put(dev);
+	netdev_put(dev, &b->devtracker);
 }

 /**
diff --git a/net/tipc/bearer.h b/net/tipc/bearer.h
index 41eac1ee0c09..1adeaf94aa62 100644
--- a/net/tipc/bearer.h
+++ b/net/tipc/bearer.h
@@ -174,6 +174,7 @@  struct tipc_bearer {
 	u16 encap_hlen;
 	unsigned long up;
 	refcount_t refcnt;
+	netdevice_tracker	devtracker;
 };

 struct tipc_bearer_names {