i40e: change flags to use 64 bits
authorAlice Michael <alice.michael@intel.com>
Wed, 27 Dec 2017 13:17:50 +0000 (08:17 -0500)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Fri, 26 Jan 2018 21:23:15 +0000 (13:23 -0800)
As we have added more flags, we need to now use more
bits and have over flooded the 32 bit size.  So
make it 64.

Also change all the existing bits to unsigned long long
bits.

Signed-off-by: Alice Michael <alice.michael@intel.com>
Tested-by: Andrew Bowers <andrewx.bowers@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/i40e/i40e.h
drivers/net/ethernet/intel/i40e/i40e_ethtool.c

index e019baa905c569de929c5b67bb984d8a600f75dc..46e9f4e0a02cdabab19ba06138ea75a9aee52c53 100644 (file)
@@ -508,39 +508,40 @@ struct i40e_pf {
 #define I40E_HW_PORT_ID_VALID                  BIT(17)
 #define I40E_HW_RESTART_AUTONEG                        BIT(18)
 
-       u32 flags;
-#define I40E_FLAG_RX_CSUM_ENABLED              BIT(0)
-#define I40E_FLAG_MSI_ENABLED                  BIT(1)
-#define I40E_FLAG_MSIX_ENABLED                 BIT(2)
-#define I40E_FLAG_RSS_ENABLED                  BIT(3)
-#define I40E_FLAG_VMDQ_ENABLED                 BIT(4)
-#define I40E_FLAG_FILTER_SYNC                  BIT(5)
-#define I40E_FLAG_SRIOV_ENABLED                        BIT(6)
-#define I40E_FLAG_DCB_CAPABLE                  BIT(7)
-#define I40E_FLAG_DCB_ENABLED                  BIT(8)
-#define I40E_FLAG_FD_SB_ENABLED                        BIT(9)
-#define I40E_FLAG_FD_ATR_ENABLED               BIT(10)
-#define I40E_FLAG_FD_SB_AUTO_DISABLED          BIT(11)
-#define I40E_FLAG_FD_ATR_AUTO_DISABLED         BIT(12)
-#define I40E_FLAG_MFP_ENABLED                  BIT(13)
-#define I40E_FLAG_UDP_FILTER_SYNC              BIT(14)
-#define I40E_FLAG_HW_ATR_EVICT_ENABLED         BIT(15)
-#define I40E_FLAG_VEB_MODE_ENABLED             BIT(16)
-#define I40E_FLAG_VEB_STATS_ENABLED            BIT(17)
-#define I40E_FLAG_LINK_POLLING_ENABLED         BIT(18)
-#define I40E_FLAG_TRUE_PROMISC_SUPPORT         BIT(19)
-#define I40E_FLAG_TEMP_LINK_POLLING            BIT(20)
-#define I40E_FLAG_LEGACY_RX                    BIT(21)
-#define I40E_FLAG_PTP                          BIT(22)
-#define I40E_FLAG_IWARP_ENABLED                        BIT(23)
-#define I40E_FLAG_SERVICE_CLIENT_REQUESTED     BIT(24)
-#define I40E_FLAG_CLIENT_L2_CHANGE             BIT(25)
-#define I40E_FLAG_CLIENT_RESET                 BIT(26)
-#define I40E_FLAG_LINK_DOWN_ON_CLOSE_ENABLED   BIT(27)
-#define I40E_FLAG_SOURCE_PRUNING_DISABLED      BIT(28)
-#define I40E_FLAG_TC_MQPRIO                    BIT(29)
-#define I40E_FLAG_FD_SB_INACTIVE               BIT(30)
-#define I40E_FLAG_FD_SB_TO_CLOUD_FILTER                BIT(31)
+       u64 flags;
+#define I40E_FLAG_RX_CSUM_ENABLED              BIT_ULL(0)
+#define I40E_FLAG_MSI_ENABLED                  BIT_ULL(1)
+#define I40E_FLAG_MSIX_ENABLED                 BIT_ULL(2)
+#define I40E_FLAG_RSS_ENABLED                  BIT_ULL(3)
+#define I40E_FLAG_VMDQ_ENABLED                 BIT_ULL(4)
+#define I40E_FLAG_FILTER_SYNC                  BIT_ULL(5)
+#define I40E_FLAG_SRIOV_ENABLED                        BIT_ULL(6)
+#define I40E_FLAG_DCB_CAPABLE                  BIT_ULL(7)
+#define I40E_FLAG_DCB_ENABLED                  BIT_ULL(8)
+#define I40E_FLAG_FD_SB_ENABLED                        BIT_ULL(9)
+#define I40E_FLAG_FD_ATR_ENABLED               BIT_ULL(10)
+#define I40E_FLAG_FD_SB_AUTO_DISABLED          BIT_ULL(11)
+#define I40E_FLAG_FD_ATR_AUTO_DISABLED         BIT_ULL(12)
+#define I40E_FLAG_MFP_ENABLED                  BIT_ULL(13)
+#define I40E_FLAG_UDP_FILTER_SYNC              BIT_ULL(14)
+#define I40E_FLAG_HW_ATR_EVICT_ENABLED         BIT_ULL(15)
+#define I40E_FLAG_VEB_MODE_ENABLED             BIT_ULL(16)
+#define I40E_FLAG_VEB_STATS_ENABLED            BIT_ULL(17)
+#define I40E_FLAG_LINK_POLLING_ENABLED         BIT_ULL(18)
+#define I40E_FLAG_TRUE_PROMISC_SUPPORT         BIT_ULL(19)
+#define I40E_FLAG_TEMP_LINK_POLLING            BIT_ULL(20)
+#define I40E_FLAG_LEGACY_RX                    BIT_ULL(21)
+#define I40E_FLAG_PTP                          BIT_ULL(22)
+#define I40E_FLAG_IWARP_ENABLED                        BIT_ULL(23)
+#define I40E_FLAG_SERVICE_CLIENT_REQUESTED     BIT_ULL(24)
+#define I40E_FLAG_CLIENT_L2_CHANGE             BIT_ULL(25)
+#define I40E_FLAG_CLIENT_RESET                 BIT_ULL(26)
+#define I40E_FLAG_LINK_DOWN_ON_CLOSE_ENABLED   BIT_ULL(27)
+#define I40E_FLAG_SOURCE_PRUNING_DISABLED      BIT_ULL(28)
+#define I40E_FLAG_TC_MQPRIO                    BIT_ULL(29)
+#define I40E_FLAG_FD_SB_INACTIVE               BIT_ULL(30)
+#define I40E_FLAG_FD_SB_TO_CLOUD_FILTER                BIT_ULL(31)
+#define I40E_FLAG_DISABLE_FW_LLDP              BIT_ULL(32)
 
        struct i40e_client_instance *cinst;
        bool stat_offsets_loaded;
index dd6996e6539645074646d7deb71e4d25f1275016..8b0062ec8edbd7a3efd792b19825ac09799ce052 100644 (file)
@@ -4266,7 +4266,7 @@ static int i40e_set_priv_flags(struct net_device *dev, u32 flags)
        struct i40e_netdev_priv *np = netdev_priv(dev);
        struct i40e_vsi *vsi = np->vsi;
        struct i40e_pf *pf = vsi->back;
-       u32 orig_flags, new_flags, changed_flags;
+       u64 orig_flags, new_flags, changed_flags;
        u32 i, j;
 
        orig_flags = READ_ONCE(pf->flags);
@@ -4323,7 +4323,7 @@ flags_complete:
         * originally. We'll just punt with an error and log something in the
         * message buffer.
         */
-       if (cmpxchg(&pf->flags, orig_flags, new_flags) != orig_flags) {
+       if (cmpxchg64(&pf->flags, orig_flags, new_flags) != orig_flags) {
                dev_warn(&pf->pdev->dev,
                         "Unable to update pf->flags as it was modified by another thread...\n");
                return -EAGAIN;