On 2019-02-28 11:17, Kalle Valo wrote:
Maya Erez <merez@xxxxxxxxxxxxxx> writes:
From: Ahmad Masri <amasri@xxxxxxxxxxxxxx>
+
+ max_assoc_sta = min_t(uint, max_assoc_sta, fw_max_assoc_sta);
You shouldn't modify max_assoc_sta like this. If you have two wil6210
devices on the same host they will share that variable and there's a
race condition. Sure, I guess currently they will use the same firmware
so the values will be the same, but still this is a bug. I think you
should have wil->max_assoc_sta to make this device specific value.
I'll apply this anyway as it breaks all other patches if I remove this
patch. But please fix this in a follow up patch.
we'll fix that.
--
Maya Erez
Qualcomm Israel, Inc. on behalf of Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a
Linux Foundation Collaborative Project
