OSDN Git Service

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 d27a43e..fa28328 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 2bea95b..9b2805e 100644 (file)
@@ -621,6 +621,14 @@ enum iwl_fw_dbg_trigger_mode {
 };
 
 /**
+ * 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
  * @IWL_FW_DBG_CONF_VIF_IBSS: IBSS mode
@@ -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;