liquidio: allow override of firmware present in flash
authorRick Farrington <ricardo.farrington@cavium.com>
Sat, 23 Sep 2017 00:12:43 +0000 (17:12 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 26 Sep 2017 03:25:39 +0000 (20:25 -0700)
Signed-off-by: Rick Farrington <ricardo.farrington@cavium.com>
Signed-off-by: Felix Manlunas <felix.manlunas@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/cavium/liquidio/lio_main.c
drivers/net/ethernet/cavium/liquidio/liquidio_image.h
drivers/net/ethernet/cavium/liquidio/octeon_device.c
drivers/net/ethernet/cavium/liquidio/octeon_device.h

index e7f54948173f7e6764b57ba5b99636ba206ba323..ce08f710de0b5c125279a97e32181c7c6d320a05 100644 (file)
@@ -59,9 +59,9 @@ static int debug = -1;
 module_param(debug, int, 0644);
 MODULE_PARM_DESC(debug, "NETIF_MSG debug bits");
 
-static char fw_type[LIO_MAX_FW_TYPE_LEN] = LIO_FW_NAME_TYPE_NIC;
+static char fw_type[LIO_MAX_FW_TYPE_LEN] = LIO_FW_NAME_TYPE_AUTO;
 module_param_string(fw_type, fw_type, sizeof(fw_type), 0444);
-MODULE_PARM_DESC(fw_type, "Type of firmware to be loaded. Default \"nic\".  Use \"none\" to load firmware from flash.");
+MODULE_PARM_DESC(fw_type, "Type of firmware to be loaded (default is \"auto\"), which uses firmware in flash, if present, else loads \"nic\".");
 
 static u32 console_bitmask;
 module_param(console_bitmask, int, 0644);
@@ -1115,10 +1115,10 @@ liquidio_probe(struct pci_dev *pdev,
        return 0;
 }
 
-static bool fw_type_is_none(void)
+static bool fw_type_is_auto(void)
 {
-       return strncmp(fw_type, LIO_FW_NAME_TYPE_NONE,
-                      sizeof(LIO_FW_NAME_TYPE_NONE)) == 0;
+       return strncmp(fw_type, LIO_FW_NAME_TYPE_AUTO,
+                      sizeof(LIO_FW_NAME_TYPE_AUTO)) == 0;
 }
 
 /**
@@ -1302,7 +1302,7 @@ static void octeon_destroy_resources(struct octeon_device *oct)
                 * Implementation note: only soft-reset the device
                 * if it is a CN6XXX OR the LAST CN23XX device.
                 */
-               if (fw_type_is_none())
+               if (atomic_read(oct->adapter_fw_state) == FW_IS_PRELOADED)
                        octeon_pci_flr(oct);
                else if (OCTEON_CN6XXX(oct) || !refcount)
                        oct->fn_list.soft_reset(oct);
@@ -1934,7 +1934,7 @@ static int load_firmware(struct octeon_device *oct)
        char fw_name[LIO_MAX_FW_FILENAME_LEN];
        char *tmp_fw_type;
 
-       if (fw_type[0] == '\0')
+       if (fw_type_is_auto())
                tmp_fw_type = LIO_FW_NAME_TYPE_NIC;
        else
                tmp_fw_type = fw_type;
@@ -3882,9 +3882,9 @@ octeon_recv_vf_drv_notice(struct octeon_recv_info *recv_info, void *buf)
 static int octeon_device_init(struct octeon_device *octeon_dev)
 {
        int j, ret;
-       int fw_loaded = 0;
        char bootcmd[] = "\n";
        char *dbg_enb = NULL;
+       enum lio_fw_state fw_state;
        struct octeon_device_priv *oct_priv =
                (struct octeon_device_priv *)octeon_dev->priv;
        atomic_set(&octeon_dev->status, OCT_DEV_BEGIN_STATE);
@@ -3916,24 +3916,40 @@ static int octeon_device_init(struct octeon_device *octeon_dev)
 
        octeon_dev->app_mode = CVM_DRV_INVALID_APP;
 
-       if (OCTEON_CN23XX_PF(octeon_dev)) {
-               if (!cn23xx_fw_loaded(octeon_dev) && !fw_type_is_none()) {
-                       fw_loaded = 0;
-                       /* Do a soft reset of the Octeon device. */
-                       if (octeon_dev->fn_list.soft_reset(octeon_dev))
-                               return 1;
-                       /* things might have changed */
-                       if (!cn23xx_fw_loaded(octeon_dev))
-                               fw_loaded = 0;
-                       else
-                               fw_loaded = 1;
-               } else {
-                       fw_loaded = 1;
-               }
-       } else if (octeon_dev->fn_list.soft_reset(octeon_dev)) {
-               return 1;
+       /* CN23XX supports preloaded firmware if the following is true:
+        *
+        * The adapter indicates that firmware is currently running AND
+        * 'fw_type' is 'auto'.
+        *
+        * (default state is NEEDS_TO_BE_LOADED, override it if appropriate).
+        */
+       if (OCTEON_CN23XX_PF(octeon_dev) &&
+           cn23xx_fw_loaded(octeon_dev) && fw_type_is_auto()) {
+               atomic_cmpxchg(octeon_dev->adapter_fw_state,
+                              FW_NEEDS_TO_BE_LOADED, FW_IS_PRELOADED);
        }
 
+       /* If loading firmware, only first device of adapter needs to do so. */
+       fw_state = atomic_cmpxchg(octeon_dev->adapter_fw_state,
+                                 FW_NEEDS_TO_BE_LOADED,
+                                 FW_IS_BEING_LOADED);
+
+       /* Here, [local variable] 'fw_state' is set to one of:
+        *
+        *   FW_IS_PRELOADED:       No firmware is to be loaded (see above)
+        *   FW_NEEDS_TO_BE_LOADED: The driver's first instance will load
+        *                          firmware to the adapter.
+        *   FW_IS_BEING_LOADED:    The driver's second instance will not load
+        *                          firmware to the adapter.
+        */
+
+       /* Prior to f/w load, perform a soft reset of the Octeon device;
+        * if error resetting, return w/error.
+        */
+       if (fw_state == FW_NEEDS_TO_BE_LOADED)
+               if (octeon_dev->fn_list.soft_reset(octeon_dev))
+                       return 1;
+
        /* Initialize the dispatch mechanism used to push packets arriving on
         * Octeon Output queues.
         */
@@ -4063,7 +4079,7 @@ static int octeon_device_init(struct octeon_device *octeon_dev)
 
        atomic_set(&octeon_dev->status, OCT_DEV_IO_QUEUES_DONE);
 
-       if ((!OCTEON_CN23XX_PF(octeon_dev)) || !fw_loaded) {
+       if (fw_state == FW_NEEDS_TO_BE_LOADED) {
                dev_dbg(&octeon_dev->pci_dev->dev, "Waiting for DDR initialization...\n");
                if (!ddr_timeout) {
                        dev_info(&octeon_dev->pci_dev->dev,
@@ -4125,6 +4141,8 @@ static int octeon_device_init(struct octeon_device *octeon_dev)
                        dev_err(&octeon_dev->pci_dev->dev, "Could not load firmware to board\n");
                        return 1;
                }
+
+               atomic_set(octeon_dev->adapter_fw_state, FW_HAS_BEEN_LOADED);
        }
 
        handshake[octeon_dev->octeon_id].init_ok = 1;
index 78a3685f6fe0faac471fcdc2dabc09a8c69990f8..5bf5e8791dfb84db66d9a7a20bc0772937a18aa3 100644 (file)
@@ -24,6 +24,7 @@
 #define LIO_FW_BASE_NAME        "lio_"
 #define LIO_FW_NAME_SUFFIX      ".bin"
 #define LIO_FW_NAME_TYPE_NIC    "nic"
+#define LIO_FW_NAME_TYPE_AUTO   "auto"
 #define LIO_FW_NAME_TYPE_NONE   "none"
 #define LIO_MAX_FIRMWARE_VERSION_LEN 16
 
index 29d53b1763a72902a56f3c73ebafaaae3b38c654..e4aa3395a57811c35c81eb4933695bcfe94418a0 100644 (file)
@@ -541,6 +541,7 @@ static char oct_dev_app_str[CVM_DRV_APP_COUNT + 1][32] = {
 
 static struct octeon_device *octeon_device[MAX_OCTEON_DEVICES];
 static atomic_t adapter_refcounts[MAX_OCTEON_DEVICES];
+static atomic_t adapter_fw_states[MAX_OCTEON_DEVICES];
 
 static u32 octeon_device_count;
 /* locks device array (i.e. octeon_device[]) */
@@ -770,6 +771,10 @@ int octeon_register_device(struct octeon_device *oct,
        oct->adapter_refcount = &adapter_refcounts[oct->octeon_id];
        atomic_set(oct->adapter_refcount, 0);
 
+       /* Like the reference count, the f/w state is shared 'per-adapter' */
+       oct->adapter_fw_state = &adapter_fw_states[oct->octeon_id];
+       atomic_set(oct->adapter_fw_state, FW_NEEDS_TO_BE_LOADED);
+
        spin_lock(&octeon_devices_lock);
        for (idx = (int)oct->octeon_id - 1; idx >= 0; idx--) {
                if (!octeon_device[idx]) {
@@ -780,11 +785,15 @@ int octeon_register_device(struct octeon_device *oct,
                        atomic_inc(oct->adapter_refcount);
                        return 1; /* here, refcount is guaranteed to be 1 */
                }
-               /* if another device is at same bus/dev, use its refcounter */
+               /* If another device is at same bus/dev, use its refcounter
+                * (and f/w state variable).
+                */
                if ((octeon_device[idx]->loc.bus == bus) &&
                    (octeon_device[idx]->loc.dev == dev)) {
                        oct->adapter_refcount =
                                octeon_device[idx]->adapter_refcount;
+                       oct->adapter_fw_state =
+                               octeon_device[idx]->adapter_fw_state;
                        break;
                }
        }
index 894af199ddefdd1c7696f228e733552194dbc591..33d19c4509bc4a2ad7b3432a8b2f1b63c336b14c 100644 (file)
@@ -50,6 +50,13 @@ enum octeon_pci_swap_mode {
        OCTEON_PCI_32BIT_LW_SWAP = 3
 };
 
+enum lio_fw_state {
+       FW_IS_PRELOADED = 0,
+       FW_NEEDS_TO_BE_LOADED = 1,
+       FW_IS_BEING_LOADED = 2,
+       FW_HAS_BEEN_LOADED = 3,
+};
+
 enum {
        OCTEON_CONFIG_TYPE_DEFAULT = 0,
        NUM_OCTEON_CONFS,
@@ -557,6 +564,9 @@ struct octeon_device {
        } loc;
 
        atomic_t *adapter_refcount; /* reference count of adapter */
+
+       atomic_t *adapter_fw_state; /* per-adapter, lio_fw_state */
+
        bool ptp_enable;
 };