Commit 3ed34fbf authored by Shahar S Matityahu's avatar Shahar S Matityahu Committed by Luca Coelho

iwlwifi: dbg_ini: support FW response/notification region type

Allow the driver to collect FW response/notification region type
during dump and allow triggering dump collection for a given FW
response/notification.
Signed-off-by: default avatarShahar S Matityahu <shahar.s.matityahu@intel.com>
Signed-off-by: default avatarLuca Coelho <luciano.coelho@intel.com>
parent 068893b7
...@@ -1057,9 +1057,11 @@ iwl_fw_error_dump_file(struct iwl_fw_runtime *fwrt, ...@@ -1057,9 +1057,11 @@ iwl_fw_error_dump_file(struct iwl_fw_runtime *fwrt,
/** /**
* struct iwl_dump_ini_region_data - region data * struct iwl_dump_ini_region_data - region data
* @reg_tlv: region TLV * @reg_tlv: region TLV
* @dump_data: dump data
*/ */
struct iwl_dump_ini_region_data { struct iwl_dump_ini_region_data {
struct iwl_ucode_tlv *reg_tlv; struct iwl_ucode_tlv *reg_tlv;
struct iwl_fwrt_dump_data *dump_data;
}; };
static int iwl_dump_ini_prph_iter(struct iwl_fw_runtime *fwrt, static int iwl_dump_ini_prph_iter(struct iwl_fw_runtime *fwrt,
...@@ -1449,6 +1451,27 @@ iwl_dump_ini_err_table_iter(struct iwl_fw_runtime *fwrt, ...@@ -1449,6 +1451,27 @@ iwl_dump_ini_err_table_iter(struct iwl_fw_runtime *fwrt,
return sizeof(*range) + le32_to_cpu(range->range_data_size); return sizeof(*range) + le32_to_cpu(range->range_data_size);
} }
static int iwl_dump_ini_fw_pkt_iter(struct iwl_fw_runtime *fwrt,
struct iwl_dump_ini_region_data *reg_data,
void *range_ptr, int idx)
{
struct iwl_fw_ini_error_dump_range *range = range_ptr;
struct iwl_rx_packet *pkt = reg_data->dump_data->fw_pkt;
u32 pkt_len;
if (!pkt)
return -EIO;
pkt_len = iwl_rx_packet_payload_len(pkt);
memcpy(&range->fw_pkt_hdr, &pkt->hdr, sizeof(range->fw_pkt_hdr));
range->range_data_size = cpu_to_le32(pkt_len);
memcpy(range->data, pkt->data, pkt_len);
return sizeof(*range) + le32_to_cpu(range->range_data_size);
}
static void * static void *
iwl_dump_ini_mem_fill_header(struct iwl_fw_runtime *fwrt, iwl_dump_ini_mem_fill_header(struct iwl_fw_runtime *fwrt,
struct iwl_dump_ini_region_data *reg_data, struct iwl_dump_ini_region_data *reg_data,
...@@ -1753,6 +1776,23 @@ iwl_dump_ini_err_table_get_size(struct iwl_fw_runtime *fwrt, ...@@ -1753,6 +1776,23 @@ iwl_dump_ini_err_table_get_size(struct iwl_fw_runtime *fwrt,
return size; return size;
} }
static u32
iwl_dump_ini_fw_pkt_get_size(struct iwl_fw_runtime *fwrt,
struct iwl_dump_ini_region_data *reg_data)
{
u32 size = 0;
if (!reg_data->dump_data->fw_pkt)
return 0;
size += iwl_rx_packet_payload_len(reg_data->dump_data->fw_pkt);
if (size)
size += sizeof(struct iwl_fw_ini_error_dump) +
sizeof(struct iwl_fw_ini_error_dump_range);
return size;
}
/** /**
* struct iwl_dump_ini_mem_ops - ini memory dump operations * struct iwl_dump_ini_mem_ops - ini memory dump operations
* @get_num_of_ranges: returns the number of memory ranges in the region. * @get_num_of_ranges: returns the number of memory ranges in the region.
...@@ -1976,7 +2016,12 @@ static const struct iwl_dump_ini_mem_ops iwl_dump_ini_region_ops[] = { ...@@ -1976,7 +2016,12 @@ static const struct iwl_dump_ini_mem_ops iwl_dump_ini_region_ops[] = {
.fill_mem_hdr = iwl_dump_ini_err_table_fill_header, .fill_mem_hdr = iwl_dump_ini_err_table_fill_header,
.fill_range = iwl_dump_ini_err_table_iter, .fill_range = iwl_dump_ini_err_table_iter,
}, },
[IWL_FW_INI_REGION_RSP_OR_NOTIF] = {}, [IWL_FW_INI_REGION_RSP_OR_NOTIF] = {
.get_num_of_ranges = iwl_dump_ini_single_range,
.get_size = iwl_dump_ini_fw_pkt_get_size,
.fill_mem_hdr = iwl_dump_ini_mem_fill_header,
.fill_range = iwl_dump_ini_fw_pkt_iter,
},
[IWL_FW_INI_REGION_DEVICE_MEMORY] = { [IWL_FW_INI_REGION_DEVICE_MEMORY] = {
.get_num_of_ranges = iwl_dump_ini_mem_ranges, .get_num_of_ranges = iwl_dump_ini_mem_ranges,
.get_size = iwl_dump_ini_mem_get_size, .get_size = iwl_dump_ini_mem_get_size,
...@@ -2012,7 +2057,9 @@ static u32 iwl_dump_ini_trigger(struct iwl_fw_runtime *fwrt, ...@@ -2012,7 +2057,9 @@ static u32 iwl_dump_ini_trigger(struct iwl_fw_runtime *fwrt,
struct list_head *list) struct list_head *list)
{ {
struct iwl_fw_ini_trigger_tlv *trigger = dump_data->trig; struct iwl_fw_ini_trigger_tlv *trigger = dump_data->trig;
struct iwl_dump_ini_region_data reg_data = {}; struct iwl_dump_ini_region_data reg_data = {
.dump_data = dump_data,
};
int i; int i;
u32 size = 0; u32 size = 0;
u64 regions_mask = le64_to_cpu(trigger->regions_mask); u64 regions_mask = le64_to_cpu(trigger->regions_mask);
...@@ -2155,6 +2202,8 @@ static void iwl_dump_ini_list_free(struct list_head *list) ...@@ -2155,6 +2202,8 @@ static void iwl_dump_ini_list_free(struct list_head *list)
static void iwl_fw_error_dump_data_free(struct iwl_fwrt_dump_data *dump_data) static void iwl_fw_error_dump_data_free(struct iwl_fwrt_dump_data *dump_data)
{ {
dump_data->trig = NULL; dump_data->trig = NULL;
kfree(dump_data->fw_pkt);
dump_data->fw_pkt = NULL;
} }
static void iwl_fw_error_ini_dump(struct iwl_fw_runtime *fwrt, static void iwl_fw_error_ini_dump(struct iwl_fw_runtime *fwrt,
......
...@@ -65,6 +65,7 @@ ...@@ -65,6 +65,7 @@
#define __fw_error_dump_h__ #define __fw_error_dump_h__
#include <linux/types.h> #include <linux/types.h>
#include "fw/api/cmdhdr.h"
#define IWL_FW_ERROR_DUMP_BARKER 0x14789632 #define IWL_FW_ERROR_DUMP_BARKER 0x14789632
#define IWL_FW_INI_ERROR_DUMP_BARKER 0x14789633 #define IWL_FW_INI_ERROR_DUMP_BARKER 0x14789633
...@@ -327,6 +328,7 @@ struct iwl_fw_ini_fifo_hdr { ...@@ -327,6 +328,7 @@ struct iwl_fw_ini_fifo_hdr {
* @dram_base_addr: base address of dram monitor range * @dram_base_addr: base address of dram monitor range
* @page_num: page number of memory range * @page_num: page number of memory range
* @fifo_hdr: fifo header of memory range * @fifo_hdr: fifo header of memory range
* @fw_pkt: FW packet header of memory range
* @data: the actual memory * @data: the actual memory
*/ */
struct iwl_fw_ini_error_dump_range { struct iwl_fw_ini_error_dump_range {
...@@ -336,6 +338,7 @@ struct iwl_fw_ini_error_dump_range { ...@@ -336,6 +338,7 @@ struct iwl_fw_ini_error_dump_range {
__le64 dram_base_addr; __le64 dram_base_addr;
__le32 page_num; __le32 page_num;
struct iwl_fw_ini_fifo_hdr fifo_hdr; struct iwl_fw_ini_fifo_hdr fifo_hdr;
struct iwl_cmd_header fw_pkt_hdr;
}; };
__le32 data[]; __le32 data[];
} __packed; } __packed;
......
...@@ -940,6 +940,33 @@ int iwl_dbg_tlv_gen_active_trigs(struct iwl_fw_runtime *fwrt, u32 new_domain) ...@@ -940,6 +940,33 @@ int iwl_dbg_tlv_gen_active_trigs(struct iwl_fw_runtime *fwrt, u32 new_domain)
return 0; return 0;
} }
static bool iwl_dbg_tlv_check_fw_pkt(struct iwl_fw_runtime *fwrt,
struct iwl_fwrt_dump_data *dump_data,
union iwl_dbg_tlv_tp_data *tp_data,
u32 trig_data)
{
struct iwl_rx_packet *pkt = tp_data->fw_pkt;
struct iwl_cmd_header *wanted_hdr = (void *)&trig_data;
if (pkt && ((wanted_hdr->cmd == 0 && wanted_hdr->group_id == 0) ||
(pkt->hdr.cmd == wanted_hdr->cmd &&
pkt->hdr.group_id == wanted_hdr->group_id))) {
struct iwl_rx_packet *fw_pkt =
kmemdup(pkt,
sizeof(*pkt) + iwl_rx_packet_payload_len(pkt),
GFP_ATOMIC);
if (!fw_pkt)
return false;
dump_data->fw_pkt = fw_pkt;
return true;
}
return false;
}
static int static int
iwl_dbg_tlv_tp_trigger(struct iwl_fw_runtime *fwrt, iwl_dbg_tlv_tp_trigger(struct iwl_fw_runtime *fwrt,
struct list_head *active_trig_list, struct list_head *active_trig_list,
...@@ -1039,6 +1066,11 @@ void iwl_dbg_tlv_time_point(struct iwl_fw_runtime *fwrt, ...@@ -1039,6 +1066,11 @@ void iwl_dbg_tlv_time_point(struct iwl_fw_runtime *fwrt,
iwl_dbg_tlv_set_periodic_trigs(fwrt); iwl_dbg_tlv_set_periodic_trigs(fwrt);
iwl_dbg_tlv_send_hcmds(fwrt, hcmd_list); iwl_dbg_tlv_send_hcmds(fwrt, hcmd_list);
break; break;
case IWL_FW_INI_TIME_POINT_FW_RSP_OR_NOTIF:
iwl_dbg_tlv_send_hcmds(fwrt, hcmd_list);
iwl_dbg_tlv_tp_trigger(fwrt, trig_list, tp_data,
iwl_dbg_tlv_check_fw_pkt);
break;
default: default:
iwl_dbg_tlv_send_hcmds(fwrt, hcmd_list); iwl_dbg_tlv_send_hcmds(fwrt, hcmd_list);
iwl_dbg_tlv_tp_trigger(fwrt, trig_list, tp_data, NULL); iwl_dbg_tlv_tp_trigger(fwrt, trig_list, tp_data, NULL);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment