infiniband: remove dev_base_lock use

Le vendredi 29 octobre 2010 à 13:03 +0200, Eric Dumazet a écrit :
> dev_base_lock is the legacy way to lock the device list, and is planned
> to disappear. (writers hold RTNL, readers hold RCU lock)
> 
> Convert rdma_translate_ip() to RCU locking.
> 
> Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
> ---
>  drivers/infiniband/core/addr.c |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/infiniband/core/addr.c b/drivers/infiniband/core/addr.c
> index a5ea1bc..309b477 100644
> --- a/drivers/infiniband/core/addr.c
> +++ b/drivers/infiniband/core/addr.c
> @@ -130,8 +130,8 @@ int rdma_translate_ip(struct sockaddr *addr, struct rdma_dev_addr *dev_addr)
>  
>  #if defined(CONFIG_IPV6) || defined(CONFIG_IPV6_MODULE)
>  	case AF_INET6:
> -		read_lock(&dev_base_lock);
> -		for_each_netdev(&init_net, dev) {
> +		rcu_read_lock();
> +		for_each_netdev_rcu(&init_net, dev) {
>  			if (ipv6_chk_addr(&init_net,
>  					  &((struct sockaddr_in6 *) addr)->sin6_addr,
>  					  dev, 1)) {
> @@ -139,7 +139,7 @@ int rdma_translate_ip(struct sockaddr *addr, struct rdma_dev_addr *dev_addr)
>  				break;
>  			}
>  		}
> -		read_unlock(&dev_base_lock);
> +		rcu_read_unlock();
>  		break;
>  #endif
>  	}
> 

Here is a new version of patch.

David, it seems no infiniband guy commented this patch, could you take
it in your tree ?

Thanks

[PATCH] infiniband: remove dev_base_lock use

dev_base_lock is the legacy way to lock the device list, and is planned
to disappear. (writers hold RTNL, readers hold RCU lock)

Convert rdma_translate_ip() and update_ipv6_gids() to RCU locking.

Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
---
 drivers/infiniband/core/addr.c    |    6 +++---
 drivers/infiniband/hw/mlx4/main.c |    6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)



--
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

> David, it seems no infiniband guy commented this patch, could you take
 > it in your tree ?

Sorry... in any case this looks fine.  If all prerequisites are upstream
then I'm happy to take through my tree, but if Dave wants to take it:

Acked-by: Roland Dreier <rolandd@cisco.com>
--
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

From: Roland Dreier <rdreier@cisco.com>
Date: Wed, 24 Nov 2010 09:06:13 -0800

>  > David, it seems no infiniband guy commented this patch, could you take
>  > it in your tree ?
> 
> Sorry... in any case this looks fine.  If all prerequisites are upstream
> then I'm happy to take through my tree, but if Dave wants to take it:
> 
> Acked-by: Roland Dreier <rolandd@cisco.com>

I'd rather take it because it opens up some further simplifications
Eric wants to perform.

So I'll apply this to net-next-2.6, thanks for the review Roland.
--
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