iwlwifi: wrt: add fw force restart via triggers
authorShahar S Matityahu <shahar.s.matityahu@intel.com>
Thu, 28 Dec 2017 08:19:43 +0000 (10:19 +0200)
committerLuca Coelho <luciano.coelho@intel.com>
Wed, 28 Mar 2018 19:43:40 +0000 (22:43 +0300)
We can set triggers that cause a debug data collection when something
of interest happens (e.g. when too many probes are lost conscutively).
Normally, this triggers don't cause the FW to be restarted, but in
some cases that may be desired, so we recover from the problem.  To
support this, add a flag that indicates that the FW should be
restarted when the trigger fires.

Signed-off-by: Shahar S Matityahu <shahar.s.matityahu@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
drivers/net/wireless/intel/iwlwifi/fw/dbg.c
drivers/net/wireless/intel/iwlwifi/fw/file.h

index d27a43ea0d7cef12d5e7cefb90a4bd5e122e07b7..fa283285fcbed08e7df15ba3d1d5c513b8083b58 100644 (file)
@@ -1008,6 +1008,12 @@ int iwl_fw_dbg_collect(struct iwl_fw_runtime *fwrt,
 {
        struct iwl_fw_dump_desc *desc;
 
+       if (trigger && trigger->flags & IWL_FW_DBG_FORCE_RESTART) {
+               IWL_WARN(fwrt, "Force restart: trigger %d fired.\n", trig);
+               iwl_force_nmi(fwrt->trans);
+               return 0;
+       }
+
        desc = kzalloc(sizeof(*desc) + len, GFP_ATOMIC);
        if (!desc)
                return -ENOMEM;
index 2bea95bf4fc9f56f8b2dbe6beb718f5124c4b6d5..9b2805e1e3b14d16761390a17a45f0fbac072bf5 100644 (file)
@@ -620,6 +620,14 @@ enum iwl_fw_dbg_trigger_mode {
        IWL_FW_DBG_TRIGGER_MONITOR_ONLY = BIT(2),
 };
 
+/**
+ * enum iwl_fw_dbg_trigger_flags - the flags supported by wrt triggers
+ * @IWL_FW_DBG_FORCE_RESTART: force a firmware restart
+ */
+enum iwl_fw_dbg_trigger_flags {
+       IWL_FW_DBG_FORCE_RESTART = BIT(0),
+};
+
 /**
  * enum iwl_fw_dbg_trigger_vif_type - define the VIF type for a trigger
  * @IWL_FW_DBG_CONF_VIF_ANY: any vif type
@@ -656,6 +664,7 @@ enum iwl_fw_dbg_trigger_vif_type {
  * @occurrences: number of occurrences. 0 means the trigger will never fire.
  * @trig_dis_ms: the time, in milliseconds, after an occurrence of this
  *     trigger in which another occurrence should be ignored.
+ * @flags: &enum iwl_fw_dbg_trigger_flags
  */
 struct iwl_fw_dbg_trigger_tlv {
        __le32 id;
@@ -666,7 +675,8 @@ struct iwl_fw_dbg_trigger_tlv {
        u8 start_conf_id;
        __le16 occurrences;
        __le16 trig_dis_ms;
-       __le16 reserved[3];
+       u8 flags;
+       u8 reserved[5];
 
        u8 data[0];
 } __packed;