On 4/23/24 02:29, Linux regression tracking (Thorsten Leemhuis) wrote:
On 23.04.24 11:06, Johannes Berg wrote:
On Tue, 2024-04-23 at 11:00 +0200, Linux regression tracking (Thorsten
Leemhuis) wrote:
On 16.04.24 08:17, Johannes Berg wrote:
On Mon, 2024-04-15 at 13:37 -0700, Ben Greear wrote:
Johannes, you had another suggestion: changing iwlwifi's request_module() to request_module_nowait() in
iwl_req_fw_callback()
Is that still best thing to try in your opinion?
I guess so, I don't have any better ideas so far anyway ...
[adding the iwlwifi maintainer; thread starts here:
https://lore.kernel.org/lkml/30f757e3-73c5-5473-c1f8-328bab98fd7d@xxxxxxxxxxxxxxx/
]
Johannes, Miri, what's the status wrt to this regression? From here
things look somewhat stalled -- but maybe there was progress and I just
missed it.
What do you want? It got bisected to an LED merge, but you ping _us_?
Way to go ...
Sorry, to me it sounded a bit like you had an idea for a fix and were
going to give it a try -- similar to how the maintainers for a r8169
driver and the igc driver provided fixes for bugs recent LED changes
exposed.
But sure, you are right, in the end some LED change seems to have cause
this, so the duty to fix it lies in that field. Therefore:
Lee, what's the status here to get this fixed before the final?
Ciao, Thorsten
Hello Johannes,
This patch makes the problem go away in my testbed with 24 Intel
iwlwifi radios. My guess is that it is just papering over the problem, but
maybe good enough? Would you like me to submit as official
patch to linux-wireless?
$ git diff
diff --git a/drivers/net/wireless/intel/iwlwifi/iwl-drv.c b/drivers/net/wireless/intel/iwlwifi/iwl-drv.c
index 4696d73c8971..993177e1de27 100644
--- a/drivers/net/wireless/intel/iwlwifi/iwl-drv.c
+++ b/drivers/net/wireless/intel/iwlwifi/iwl-drv.c
@@ -1744,7 +1744,7 @@ static void iwl_req_fw_callback(const struct firmware *ucode_raw, void *context)
* or hangs loading.
*/
if (load_module)
- request_module("%s", op->name);
+ request_module_nowait("%s", op->name);
failure = false;
goto free;
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
