2020-09-28 09:25:02

by Luca Coelho

[permalink] [raw]
Subject: [PATCH 10/12] iwlwifi: remove wide_cmd_header field

From: Mordechay Goodstein <[email protected]>

Driver doesn't support fw without wide_cmd
so driver always sets it to true. instead of setting
it always to true just remove the field.

Signed-off-by: Mordechay Goodstein <[email protected]>
Signed-off-by: Luca Coelho <[email protected]>
---
drivers/net/wireless/intel/iwlwifi/iwl-trans.c | 2 +-
drivers/net/wireless/intel/iwlwifi/iwl-trans.h | 2 --
drivers/net/wireless/intel/iwlwifi/mvm/ops.c | 1 -
drivers/net/wireless/intel/iwlwifi/pcie/tx.c | 5 -----
4 files changed, 1 insertion(+), 9 deletions(-)

diff --git a/drivers/net/wireless/intel/iwlwifi/iwl-trans.c b/drivers/net/wireless/intel/iwlwifi/iwl-trans.c
index a99e0c5af512..073efce47e74 100644
--- a/drivers/net/wireless/intel/iwlwifi/iwl-trans.c
+++ b/drivers/net/wireless/intel/iwlwifi/iwl-trans.c
@@ -181,7 +181,7 @@ int iwl_trans_send_cmd(struct iwl_trans *trans, struct iwl_host_cmd *cmd)
if (!(cmd->flags & CMD_ASYNC))
lock_map_acquire_read(&trans->sync_cmd_lockdep_map);

- if (trans->wide_cmd_header && !iwl_cmd_groupid(cmd->id))
+ if (!iwl_cmd_groupid(cmd->id))
cmd->id = DEF_ID(cmd->id);

ret = trans->ops->send_cmd(trans, cmd);
diff --git a/drivers/net/wireless/intel/iwlwifi/iwl-trans.h b/drivers/net/wireless/intel/iwlwifi/iwl-trans.h
index 868e137c51f0..8fe720ac1c74 100644
--- a/drivers/net/wireless/intel/iwlwifi/iwl-trans.h
+++ b/drivers/net/wireless/intel/iwlwifi/iwl-trans.h
@@ -960,7 +960,6 @@ struct iwl_trans_txqs {
* @hw_id_str: a string with info about HW ID. Set during transport allocation.
* @pm_support: set to true in start_hw if link pm is supported
* @ltr_enabled: set to true if the LTR is enabled
- * @wide_cmd_header: true when ucode supports wide command header format
* @num_rx_queues: number of RX queues allocated by the transport;
* the transport must set this before calling iwl_drv_start()
* @iml_len: the length of the image loader
@@ -1000,7 +999,6 @@ struct iwl_trans {

const struct iwl_hcmd_arr *command_groups;
int command_groups_size;
- bool wide_cmd_header;

u8 num_rx_queues;

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/ops.c b/drivers/net/wireless/intel/iwlwifi/mvm/ops.c
index e184a163946a..5a6f0fe2fb90 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/ops.c
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/ops.c
@@ -757,7 +757,6 @@ iwl_op_mode_mvm_start(struct iwl_trans *trans, const struct iwl_cfg *cfg,
trans_cfg.rx_buf_size = rb_size_default;
}

- trans->wide_cmd_header = true;
trans_cfg.bc_table_dword =
mvm->trans->trans_cfg->device_family < IWL_DEVICE_FAMILY_AX210;

diff --git a/drivers/net/wireless/intel/iwlwifi/pcie/tx.c b/drivers/net/wireless/intel/iwlwifi/pcie/tx.c
index e15939849a23..027b4e787ee6 100644
--- a/drivers/net/wireless/intel/iwlwifi/pcie/tx.c
+++ b/drivers/net/wireless/intel/iwlwifi/pcie/tx.c
@@ -1538,11 +1538,6 @@ static int iwl_pcie_enqueue_hcmd(struct iwl_trans *trans,
const u8 *cmddata[IWL_MAX_CMD_TBS_PER_TFD];
u16 cmdlen[IWL_MAX_CMD_TBS_PER_TFD];

- if (WARN(!trans->wide_cmd_header &&
- group_id > IWL_ALWAYS_LONG_GROUP,
- "unsupported wide command %#x\n", cmd->id))
- return -EINVAL;
-
if (group_id != 0) {
copy_size = sizeof(struct iwl_cmd_header_wide);
cmd_size = sizeof(struct iwl_cmd_header_wide);
--
2.28.0


2020-10-08 01:56:12

by Alexander Wetzel

[permalink] [raw]
Subject: [REGRESSION] Re: [PATCH 10/12] iwlwifi: remove wide_cmd_header field

Am 28.09.20 um 11:23 schrieb Luca Coelho:
> From: Mordechay Goodstein <[email protected]>
>
> Driver doesn't support fw without wide_cmd
> so driver always sets it to true. instead of setting
> it always to true just remove the field.
>

I guess that's only true for mvm but not dvm...

At least this patch kills my wireless using a Ultimate-N 6300.
Reverting to commit on top of wt-2020-10-06 works.

Here the last error I got during git bisect:


[ 375.671958] iwlwifi 0000:03:00.0: Radio type=0x0-0x3-0x1
[ 375.739218] iwlwifi 0000:03:00.0: Microcode SW error detected.
Restarting 0x82000000.
[ 375.739226] iwlwifi 0000:03:00.0: Loaded firmware version: 9.221.4.1
build 25532 6000-4.ucode
[ 375.739349] iwlwifi 0000:03:00.0: Start IWL Error Log Dump:
[ 375.739353] iwlwifi 0000:03:00.0: Status: 0x00000000, count: 5
[ 375.739357] iwlwifi 0000:03:00.0: 0x00000007 | BAD_COMMAND
[ 375.739361] iwlwifi 0000:03:00.0: 0x00000D10 | uPc
[ 375.739365] iwlwifi 0000:03:00.0: 0x00000D0E | branchlink1
[ 375.739368] iwlwifi 0000:03:00.0: 0x00000D0E | branchlink2
[ 375.739372] iwlwifi 0000:03:00.0: 0x000014DE | interruptlink1
[ 375.739376] iwlwifi 0000:03:00.0: 0x00000000 | interruptlink2
[ 375.739380] iwlwifi 0000:03:00.0: 0x00000000 | data1
[ 375.739383] iwlwifi 0000:03:00.0: 0x00000001 | data2
[ 375.739387] iwlwifi 0000:03:00.0: 0x00000463 | line
[ 375.739391] iwlwifi 0000:03:00.0: 0x0000B337 | beacon time
[ 375.739394] iwlwifi 0000:03:00.0: 0x0000DCC9 | tsf low
[ 375.739398] iwlwifi 0000:03:00.0: 0x00000000 | tsf hi
[ 375.739402] iwlwifi 0000:03:00.0: 0x00000000 | time gp1
[ 375.739406] iwlwifi 0000:03:00.0: 0x0000DCCD | time gp2
[ 375.739409] iwlwifi 0000:03:00.0: 0x00000000 | time gp3
[ 375.739413] iwlwifi 0000:03:00.0: 0x000109DD | uCode version
[ 375.739417] iwlwifi 0000:03:00.0: 0x00000074 | hw version
[ 375.739420] iwlwifi 0000:03:00.0: 0x0048D304 | board version
[ 375.739424] iwlwifi 0000:03:00.0: 0x00704001 | hcmd
[ 375.739427] iwlwifi 0000:03:00.0: 0x00122080 | isr0
[ 375.739431] iwlwifi 0000:03:00.0: 0x00000000 | isr1
[ 375.739435] iwlwifi 0000:03:00.0: 0x00000002 | isr2
[ 375.739438] iwlwifi 0000:03:00.0: 0x014400C0 | isr3
[ 375.739442] iwlwifi 0000:03:00.0: 0x00000000 | isr4
[ 375.739446] iwlwifi 0000:03:00.0: 0x01000112 | isr_pref
[ 375.739449] iwlwifi 0000:03:00.0: 0x0001BB08 | wait_event
[ 375.739453] iwlwifi 0000:03:00.0: 0x00000000 | l2p_control
[ 375.739457] iwlwifi 0000:03:00.0: 0x00000000 | l2p_duration
[ 375.739460] iwlwifi 0000:03:00.0: 0x00000000 | l2p_mhvalid
[ 375.739464] iwlwifi 0000:03:00.0: 0x00000000 | l2p_addr_match
[ 375.739468] iwlwifi 0000:03:00.0: 0x00000047 | lmpm_pmg_sel
[ 375.739471] iwlwifi 0000:03:00.0: 0x02061040 | timestamp
[ 375.739475] iwlwifi 0000:03:00.0: 0x00000008 | flow_handler
[ 375.739555] iwlwifi 0000:03:00.0: Log capacity 1024 is bogus, limit
to 512 entries
[ 375.739559] iwlwifi 0000:03:00.0: Start IWL Event Log Dump: display
last 20 entries
[ 375.739608] iwlwifi 0000:03:00.0: EVT_LOGT:0000056468:0x00000054:1334
[ 375.739646] iwlwifi 0000:03:00.0: EVT_LOGT:0000056469:0x01508c46:1334
[ 375.739680] iwlwifi 0000:03:00.0: EVT_LOGT:0000056470:0x00000017:0484
[ 375.739714] iwlwifi 0000:03:00.0: EVT_LOGT:0000056471:0x00000000:1334
[ 375.739749] iwlwifi 0000:03:00.0: EVT_LOGT:0000056475:0x00000000:1334
[ 375.739782] iwlwifi 0000:03:00.0: EVT_LOGT:0000056479:0x00000046:1334
[ 375.739816] iwlwifi 0000:03:00.0: EVT_LOGT:0000056479:0x00000046:1334
[ 375.739850] iwlwifi 0000:03:00.0: EVT_LOGT:0000056481:0x00000000:1334
[ 375.739883] iwlwifi 0000:03:00.0: EVT_LOGT:0000056485:0x00000000:1334
[ 375.739917] iwlwifi 0000:03:00.0: EVT_LOGT:0000056488:0x00000046:1334
[ 375.739950] iwlwifi 0000:03:00.0: EVT_LOGT:0000056489:0x00008c46:1334
[ 375.739984] iwlwifi 0000:03:00.0: EVT_LOGT:0000056490:0x00000000:1334
[ 375.740017] iwlwifi 0000:03:00.0: EVT_LOGT:0000056494:0x00000000:1334
[ 375.740051] iwlwifi 0000:03:00.0: EVT_LOGT:0000056497:0x00000054:1334
[ 375.740085] iwlwifi 0000:03:00.0: EVT_LOGT:0000056498:0x01508c46:1334
[ 375.740119] iwlwifi 0000:03:00.0: EVT_LOGT:0000056499:0x00000018:0484
[ 375.740152] iwlwifi 0000:03:00.0: EVT_LOGT:0000056506:0x00000019:0484
[ 375.740186] iwlwifi 0000:03:00.0: EVT_LOGT:0000056515:0x00000000:0661
[ 375.740226] iwlwifi 0000:03:00.0: EVT_LOGT:0000056521:0x00704001:0401
[ 375.740259] iwlwifi 0000:03:00.0: EVT_LOGT:0000056542:0x00000100:0125
[ 375.740315] iwlwifi 0000:03:00.0: Error Reply type 0x00000000 cmd
REPLY_ERROR (0x01) seq 0x0070 ser 0x00000001
[ 375.740339] iwlwifi 0000:03:00.0: FW error in SYNC CMD UNKNOWN
[ 375.740347] CPU: 6 PID: 354 Comm: wpa_supplicant Tainted: G T
5.9.0-rc6+ #33
[ 375.740351] Hardware name: LENOVO 2438CTO/2438CTO, BIOS G5ETB6WW
(2.76 ) 09/09/2019
[ 375.740356] Call Trace:
[ 375.740366] dump_stack+0x77/0xa0
[ 375.740380] iwl_trans_pcie_send_hcmd+0x43e/0x450 [iwlwifi]
[ 375.740387] ? wait_woken+0xb0/0xb0
[ 375.740397] iwl_trans_send_cmd+0x73/0x160 [iwlwifi]
[ 375.740406] iwl_send_calib_results+0x79/0xd0 [iwldvm]
[ 375.740414] iwl_load_ucode_wait_alive+0x184/0x1d0 [iwldvm]
[ 375.740421] ? iwl_send_calib_cfg+0xc0/0xc0 [iwldvm]
[ 375.740426] ? __local_bh_enable_ip+0x74/0xb0
[ 375.740433] iwl_run_init_ucode+0x90/0x130 [iwldvm]
[ 375.740439] ? iwl_alive_fn+0x80/0x80 [iwldvm]
[ 375.740446] iwlagn_mac_start+0xd2/0x2a0 [iwldvm]
[ 375.740466] drv_start+0x79/0x230 [mac80211]
[ 375.740483] ieee80211_do_open+0x1c4/0xb30 [mac80211]
[ 375.740499] ? ieee80211_check_concurrent_iface+0x151/0x1c0 [mac80211]
[ 375.740505] __dev_open+0xed/0x1a0
[ 375.740511] __dev_change_flags+0x1a0/0x200
[ 375.740515] ? dev_load+0x69/0x140
[ 375.740520] dev_change_flags+0x21/0x60
[ 375.740525] devinet_ioctl+0x701/0x8f0
[ 375.740530] inet_ioctl+0x194/0x1d0
[ 375.740536] sock_do_ioctl+0x3e/0x130
[ 375.740542] sock_ioctl+0x259/0x3e0
[ 375.740548] __x64_sys_ioctl+0x8d/0xc0
[ 375.740554] do_syscall_64+0x33/0x80
[ 375.740560] entry_SYSCALL_64_after_hwframe+0x44/0xa9
[ 375.740565] RIP: 0033:0x7fc781e68fd7
[ 375.740570] Code: 89 d8 48 f7 d8 49 39 c4 72 b9 e8 24 ff ff ff 85 c0
78 be 5b 4c 89 e0 5d 41 5c c3 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f
05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 61 5e 0c 00 f7 d8 64 89 01 48
[ 375.740575] RSP: 002b:00007fffb94c1828 EFLAGS: 00000246 ORIG_RAX:
0000000000000010
[ 375.740580] RAX: ffffffffffffffda RBX: 0000000000000001 RCX:
00007fc781e68fd7
[ 375.740584] RDX: 00007fffb94c1830 RSI: 0000000000008914 RDI:
0000000000000007
[ 375.740588] RBP: 0000000000000007 R08: 0000000000000007 R09:
0000560db64d3010
[ 375.740592] R10: 00007fc781f2fba0 R11: 0000000000000246 R12:
0000000000000000
[ 375.740596] R13: 0000560db64e93a0 R14: 00007fffb94c1830 R15:
0000000000000000
[ 375.740602] iwlwifi 0000:03:00.0: Error -5 on calib cmd 15
[ 375.740608] iwlwifi 0000:03:00.0: Could not complete ALIVE transition: -5
[ 375.752211] iwlwifi 0000:03:00.0: Failed to run INIT ucode: -5
[ 375.752251] iwlwifi 0000:03:00.0: Unable to initialize device.
[ 375.752305] ieee80211 phy4: Hardware restart was requested
[ 375.766178] iwlwifi 0000:03:00.0: Radio type=0x0-0x3-0x1
[ 375.829450] iwlwifi 0000:03:00.0: Microcode SW error detected.
Restarting 0x82000000.
[ 375.829482] iwlwifi 0000:03:00.0: Loaded firmware version: 9.221.4.1
build 25532 6000-4.ucode
[ 375.829630] iwlwifi 0000:03:00.0: Start IWL Error Log Dump:
[ 375.829634] iwlwifi 0000:03:00.0: Status: 0x00000000, count: 5
[ 375.829638] iwlwifi 0000:03:00.0: 0x00000007 | BAD_COMMAND
[ 375.829641] iwlwifi 0000:03:00.0: 0x00000D10 | uPc
[ 375.829645] iwlwifi 0000:03:00.0: 0x00000D0E | branchlink1
[ 375.829648] iwlwifi 0000:03:00.0: 0x00000D0E | branchlink2
[ 375.829652] iwlwifi 0000:03:00.0: 0x000014DE | interruptlink1
[ 375.829655] iwlwifi 0000:03:00.0: 0x00000000 | interruptlink2
[ 375.829659] iwlwifi 0000:03:00.0: 0x00000000 | data1
[ 375.829663] iwlwifi 0000:03:00.0: 0x00000001 | data2
[ 375.829666] iwlwifi 0000:03:00.0: 0x00000463 | line
[ 375.829670] iwlwifi 0000:03:00.0: 0x0000C31E | beacon time
[ 375.829673] iwlwifi 0000:03:00.0: 0x0000CCE2 | tsf low
[ 375.829677] iwlwifi 0000:03:00.0: 0x00000000 | tsf hi
[ 375.829680] iwlwifi 0000:03:00.0: 0x00000000 | time gp1
[ 375.829684] iwlwifi 0000:03:00.0: 0x0000CCE7 | time gp2
[ 375.829687] iwlwifi 0000:03:00.0: 0x00000000 | time gp3
[ 375.829691] iwlwifi 0000:03:00.0: 0x000109DD | uCode version
[ 375.829695] iwlwifi 0000:03:00.0: 0x00000074 | hw version
[ 375.829698] iwlwifi 0000:03:00.0: 0x0048D304 | board version
[ 375.829702] iwlwifi 0000:03:00.0: 0x00704001 | hcmd
[ 375.829705] iwlwifi 0000:03:00.0: 0x00122080 | isr0
[ 375.829709] iwlwifi 0000:03:00.0: 0x00000000 | isr1
[ 375.829712] iwlwifi 0000:03:00.0: 0x00000002 | isr2
[ 375.829716] iwlwifi 0000:03:00.0: 0x014000C0 | isr3
[ 375.829719] iwlwifi 0000:03:00.0: 0x00000000 | isr4
[ 375.829723] iwlwifi 0000:03:00.0: 0x01000112 | isr_pref
[ 375.829726] iwlwifi 0000:03:00.0: 0x0001BB08 | wait_event
[ 375.829730] iwlwifi 0000:03:00.0: 0x00000000 | l2p_control
[ 375.829733] iwlwifi 0000:03:00.0: 0x00000000 | l2p_duration
[ 375.829737] iwlwifi 0000:03:00.0: 0x00000000 | l2p_mhvalid
[ 375.829740] iwlwifi 0000:03:00.0: 0x00000000 | l2p_addr_match
[ 375.829744] iwlwifi 0000:03:00.0: 0x00000047 | lmpm_pmg_sel
[ 375.829747] iwlwifi 0000:03:00.0: 0x02061040 | timestamp
[ 375.829751] iwlwifi 0000:03:00.0: 0x00000008 | flow_handler
[ 375.829832] iwlwifi 0000:03:00.0: Log capacity 1024 is bogus, limit
to 512 entries
[ 375.829836] iwlwifi 0000:03:00.0: Start IWL Event Log Dump: display
last 20 entries
[ 375.829880] iwlwifi 0000:03:00.0: EVT_LOGT:0000052398:0x00000054:1334
[ 375.829918] iwlwifi 0000:03:00.0: EVT_LOGT:0000052399:0x01508c46:1334
[ 375.829951] iwlwifi 0000:03:00.0: EVT_LOGT:0000052400:0x00000017:0484
[ 375.829985] iwlwifi 0000:03:00.0: EVT_LOGT:0000052402:0x00000000:1334
[ 375.830018] iwlwifi 0000:03:00.0: EVT_LOGT:0000052405:0x00000000:1334
[ 375.830052] iwlwifi 0000:03:00.0: EVT_LOGT:0000052409:0x00000046:1334
[ 375.830086] iwlwifi 0000:03:00.0: EVT_LOGT:0000052409:0x00000046:1334
[ 375.830120] iwlwifi 0000:03:00.0: EVT_LOGT:0000052411:0x00000000:1334
[ 375.830153] iwlwifi 0000:03:00.0: EVT_LOGT:0000052415:0x00000000:1334
[ 375.830187] iwlwifi 0000:03:00.0: EVT_LOGT:0000052418:0x00000046:1334
[ 375.830221] iwlwifi 0000:03:00.0: EVT_LOGT:0000052419:0x00008c46:1334
[ 375.830254] iwlwifi 0000:03:00.0: EVT_LOGT:0000052420:0x00000000:1334
[ 375.830288] iwlwifi 0000:03:00.0: EVT_LOGT:0000052424:0x00000000:1334
[ 375.830321] iwlwifi 0000:03:00.0: EVT_LOGT:0000052427:0x00000054:1334
[ 375.830355] iwlwifi 0000:03:00.0: EVT_LOGT:0000052428:0x01508c46:1334
[ 375.830388] iwlwifi 0000:03:00.0: EVT_LOGT:0000052429:0x00000018:0484
[ 375.830422] iwlwifi 0000:03:00.0: EVT_LOGT:0000052436:0x00000019:0484
[ 375.830455] iwlwifi 0000:03:00.0: EVT_LOGT:0000052445:0x00000000:0661
[ 375.830489] iwlwifi 0000:03:00.0: EVT_LOGT:0000052451:0x00704001:0401
[ 375.830522] iwlwifi 0000:03:00.0: EVT_LOGT:0000052471:0x00000100:0125
[ 375.830604] iwlwifi 0000:03:00.0: Error Reply type 0x00000000 cmd
REPLY_ERROR (0x01) seq 0x0070 ser 0x00000001
[ 375.830615] iwlwifi 0000:03:00.0: FW error in SYNC CMD UNKNOWN
[ 375.830624] CPU: 6 PID: 354 Comm: wpa_supplicant Tainted: G T
5.9.0-rc6+ #33
[ 375.830629] Hardware name: LENOVO 2438CTO/2438CTO, BIOS G5ETB6WW
(2.76 ) 09/09/2019
[ 375.830633] Call Trace:
[ 375.830644] dump_stack+0x77/0xa0
[ 375.830668] iwl_trans_pcie_send_hcmd+0x43e/0x450 [iwlwifi]
[ 375.830675] ? wait_woken+0xb0/0xb0
[ 375.830685] iwl_trans_send_cmd+0x73/0x160 [iwlwifi]
[ 375.830694] iwl_send_calib_results+0x79/0xd0 [iwldvm]
[ 375.830701] iwl_load_ucode_wait_alive+0x184/0x1d0 [iwldvm]
[ 375.830708] ? iwl_send_calib_cfg+0xc0/0xc0 [iwldvm]
[ 375.830713] ? __local_bh_enable_ip+0x74/0xb0
[ 375.830720] iwl_run_init_ucode+0x90/0x130 [iwldvm]
[ 375.830726] ? iwl_alive_fn+0x80/0x80 [iwldvm]
[ 375.830732] iwlagn_mac_start+0xd2/0x2a0 [iwldvm]
[ 375.830752] drv_start+0x79/0x230 [mac80211]
[ 375.830769] ieee80211_do_open+0x1c4/0xb30 [mac80211]
[ 375.830784] ? ieee80211_check_concurrent_iface+0x151/0x1c0 [mac80211]
[ 375.830791] __dev_open+0xed/0x1a0
[ 375.830796] __dev_change_flags+0x1a0/0x200
[ 375.830801] ? dev_load+0x69/0x140
[ 375.830806] dev_change_flags+0x21/0x60
[ 375.830811] devinet_ioctl+0x701/0x8f0
[ 375.830816] inet_ioctl+0x194/0x1d0
[ 375.830822] sock_do_ioctl+0x3e/0x130
[ 375.830827] sock_ioctl+0x259/0x3e0
[ 375.830833] __x64_sys_ioctl+0x8d/0xc0
[ 375.830838] do_syscall_64+0x33/0x80
[ 375.830845] entry_SYSCALL_64_after_hwframe+0x44/0xa9
[ 375.830850] RIP: 0033:0x7fc781e68fd7
[ 375.830856] Code: 89 d8 48 f7 d8 49 39 c4 72 b9 e8 24 ff ff ff 85 c0
78 be 5b 4c 89 e0 5d 41 5c c3 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f
05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d 61 5e 0c 00 f7 d8 64 89 01 48
[ 375.830860] RSP: 002b:00007fffb94c16c8 EFLAGS: 00000246 ORIG_RAX:
0000000000000010
[ 375.830865] RAX: ffffffffffffffda RBX: 0000000000000001 RCX:
00007fc781e68fd7
[ 375.830869] RDX: 00007fffb94c16d0 RSI: 0000000000008914 RDI:
0000000000000008
[ 375.830873] RBP: 0000000000000008 R08: 0000000000000000 R09:
0000000000000007
[ 375.830877] R10: ffffffffffffffc8 R11: 0000000000000246 R12:
0000000000000000
[ 375.830881] R13: 0000560db6511e98 R14: 00007fffb94c16d0 R15:
00007fffb94c1830
[ 375.830887] iwlwifi 0000:03:00.0: Error -5 on calib cmd 15
[ 375.830893] iwlwifi 0000:03:00.0: Could not complete ALIVE transition: -5
[ 375.842315] iwlwifi 0000:03:00.0: Failed to run INIT ucode: -5
[ 375.842360] iwlwifi 0000:03:00.0: Unable to initialize device.
[ 375.842419] ieee80211 phy4: Hardware restart was requested

2020-10-09 08:23:18

by Luca Coelho

[permalink] [raw]
Subject: Re: [REGRESSION] Re: [PATCH 10/12] iwlwifi: remove wide_cmd_header field

On Wed, 2020-10-07 at 22:23 +0200, Alexander Wetzel wrote:
> Am 28.09.20 um 11:23 schrieb Luca Coelho:
> > From: Mordechay Goodstein <[email protected]>
> >
> > Driver doesn't support fw without wide_cmd
> > so driver always sets it to true. instead of setting
> > it always to true just remove the field.
> >
>
> I guess that's only true for mvm but not dvm...
>
> At least this patch kills my wireless using a Ultimate-N 6300.
> Reverting to commit on top of wt-2020-10-06 works.
>
> Here the last error I got during git bisect:
>
>
> [ 375.671958] iwlwifi 0000:03:00.0: Radio type=0x0-0x3-0x1
> [ 375.739218] iwlwifi 0000:03:00.0: Microcode SW error detected.
> Restarting 0x82000000.
> [ 375.739226] iwlwifi 0000:03:00.0: Loaded firmware version: 9.221.4.1
> build 25532 6000-4.ucode
> [ 375.739349] iwlwifi 0000:03:00.0: Start IWL Error Log Dump:
> [ 375.739353] iwlwifi 0000:03:00.0: Status: 0x00000000, count: 5
> [ 375.739357] iwlwifi 0000:03:00.0: 0x00000007 | BAD_COMMAND

Hmmm, this indeed looks wrong. I'll try to fix it or, in the worst
case, revert this patch.

--
Cheers,
Luca.