Author: hselasky
Date: Sun Mar 15 09:05:26 2015
New Revision: 280018
URL: https://svnweb.freebsd.org/changeset/base/280018

Log:
  MFC r279865:
  Ensure setting promiscious mode when a network interface is up, is
  always non-blocking by not locking a SX type of mutex.
  
  Sponsored by: Mellanox Technologies

Modified:
  stable/10/sys/ofed/drivers/net/mlx4/en_netdev.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/ofed/drivers/net/mlx4/en_netdev.c
==============================================================================
--- stable/10/sys/ofed/drivers/net/mlx4/en_netdev.c     Sun Mar 15 08:09:56 
2015        (r280017)
+++ stable/10/sys/ofed/drivers/net/mlx4/en_netdev.c     Sun Mar 15 09:05:26 
2015        (r280018)
@@ -1916,19 +1916,22 @@ static int mlx4_en_ioctl(struct ifnet *d
                error = -mlx4_en_change_mtu(dev, ifr->ifr_mtu);
                break;
        case SIOCSIFFLAGS:
-               mutex_lock(&mdev->state_lock);
                if (dev->if_flags & IFF_UP) {
-                       if ((dev->if_drv_flags & IFF_DRV_RUNNING) == 0)
+                       if ((dev->if_drv_flags & IFF_DRV_RUNNING) == 0) {
+                               mutex_lock(&mdev->state_lock);
                                mlx4_en_start_port(dev);
-                       else
+                               mutex_unlock(&mdev->state_lock);
+                       } else {
                                mlx4_en_set_rx_mode(dev);
+                       }
                } else {
+                       mutex_lock(&mdev->state_lock);
                        if (dev->if_drv_flags & IFF_DRV_RUNNING) {
                                mlx4_en_stop_port(dev);
-                                if_link_state_change(dev, LINK_STATE_DOWN);
+                               if_link_state_change(dev, LINK_STATE_DOWN);
                        }
+                       mutex_unlock(&mdev->state_lock);
                }
-               mutex_unlock(&mdev->state_lock);
                break;
        case SIOCADDMULTI:
        case SIOCDELMULTI:
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to