Wen Gong <quic_wgong@xxxxxxxxxxx> wrote: > When wlan load, firmware report the reg code with 0x6C for QCA6174, > it is world reg which checked by ath_is_world_regd(), then the reg > will be save into reg_world_copy of ath_common in ath_regd_init(). > Later the regulatory of ath_common is updated to another country > code such as "US" in ath_reg_notifier_apply() by below call stack. > After that, regulatory_hint() is called in ath10k_mac_register() > and it lead "iw reg get" show two regdomain info as below. > > global > country US: DFS-FCC > (2400 - 2472 @ 40), (N/A, 30), (N/A) > (5150 - 5250 @ 80), (N/A, 23), (N/A), AUTO-BW > (5250 - 5350 @ 80), (N/A, 23), (0 ms), DFS, AUTO-BW > (5470 - 5730 @ 160), (N/A, 23), (0 ms), DFS > (5730 - 5850 @ 80), (N/A, 30), (N/A) > (57240 - 71000 @ 2160), (N/A, 40), (N/A) > > phy#0 > country US: DFS-FCC > (2400 - 2472 @ 40), (N/A, 30), (N/A) > (5150 - 5250 @ 80), (N/A, 23), (N/A), AUTO-BW > (5250 - 5350 @ 80), (N/A, 23), (0 ms), DFS, AUTO-BW > (5470 - 5730 @ 160), (N/A, 23), (0 ms), DFS > (5730 - 5850 @ 80), (N/A, 30), (N/A) > (57240 - 71000 @ 2160), (N/A, 40), (N/A) > > [ 4255.704975] Call Trace: > [ 4255.704983] ath_reg_notifier_apply+0xa6/0xc5 [ath] > [ 4255.704991] ath10k_reg_notifier+0x2f/0xd2 [ath10k_core] > [ 4255.705010] wiphy_regulatory_register+0x5f/0x69 [cfg80211] > [ 4255.705020] wiphy_register+0x459/0x8f0 [cfg80211] > [ 4255.705042] ? ieee80211_register_hw+0x3a6/0x7d1 [mac80211] > [ 4255.705049] ? __kmalloc+0xf4/0x218 > [ 4255.705058] ? ieee80211_register_hw+0x3a6/0x7d1 [mac80211] > [ 4255.705066] ? ath10k_mac_register+0x70/0xaab [ath10k_core] > [ 4255.705075] ieee80211_register_hw+0x51a/0x7d1 [mac80211] > [ 4255.705084] ath10k_mac_register+0x8b4/0xaab [ath10k_core] > [ 4255.705094] ath10k_core_register_work+0xa5e/0xb45 [ath10k_core] > [ 4255.705100] ? __schedule+0x61f/0x7d3 > [ 4255.705105] process_one_work+0x1b7/0x392 > [ 4255.705109] worker_thread+0x271/0x35d > [ 4255.705112] ? pr_cont_work+0x58/0x58 > [ 4255.705116] kthread+0x13f/0x147 > [ 4255.705119] ? pr_cont_work+0x58/0x58 > [ 4255.705123] ? kthread_destroy_worker+0x62/0x62 > [ 4255.705126] ret_from_fork+0x22/0x40 > > At this moment, the two regdomain info is same, when run "iw reg set KR", > the global regdomain info changed to KR, but the regdomain of phy#0 > does not change again. It leads inconsistent values between global and > phy#0 as below. > > global > country KR: DFS-JP > (2402 - 2482 @ 40), (N/A, 13), (N/A) > (5170 - 5250 @ 80), (N/A, 20), (N/A), AUTO-BW > (5250 - 5330 @ 80), (N/A, 20), (0 ms), DFS, AUTO-BW > (5490 - 5710 @ 160), (N/A, 30), (0 ms), DFS > (5735 - 5835 @ 80), (N/A, 30), (N/A) > (57000 - 66000 @ 2160), (N/A, 43), (N/A) > > phy#0 > country US: DFS-FCC > (2400 - 2472 @ 40), (N/A, 30), (N/A) > (5150 - 5250 @ 80), (N/A, 23), (N/A), AUTO-BW > (5250 - 5350 @ 80), (N/A, 23), (0 ms), DFS, AUTO-BW > (5470 - 5730 @ 160), (N/A, 23), (0 ms), DFS > (5730 - 5850 @ 80), (N/A, 30), (N/A) > (57240 - 71000 @ 2160), (N/A, 40), (N/A) > > The initial reg code is 0x6C which saved in reg_world_copy of ath_common, > and the code US is updated from cfg80211 later, so ath10k should also > check the initial reg code before regulatory_hint(). > > After this fix, regdomain info is same between "iw reg get" and "iw reg > set xx", it does not have the regdomain info of phy#0 again. > > global > country KR: DFS-JP > (2402 - 2482 @ 40), (N/A, 13), (N/A) > (5170 - 5250 @ 80), (N/A, 20), (N/A), AUTO-BW > (5250 - 5330 @ 80), (N/A, 20), (0 ms), DFS, AUTO-BW > (5490 - 5710 @ 160), (N/A, 30), (0 ms), DFS > (5735 - 5835 @ 80), (N/A, 30), (N/A) > (57000 - 66000 @ 2160), (N/A, 43), (N/A) > > This does not effect the channel list and power which ath10k used. > When the country code for regulatory_hint() in ath10k_mac_register() > is same with the global country code, then reg_set_rd_driver() of > cfg80211 called from crda which return -EALREADY to set_regdom() and > then update_all_wiphy_regulatory() will not be called while wlan load. > When run "iw reg set xx", reg_get_regdomain() which used by function > handle_channel() in net/wirelss/reg.c always use the regdomain > returned by get_cfg80211_regdom() because the initiator of last > regulatory_request is NL80211_REGDOM_SET_BY_USER, get_cfg80211_regdom() > is the global regdomain, then all the ieee80211_channel info is updated > in handle_channel() with the global regdomain. > > Tested-on: QCA6174 hw3.2 SDIO WLAN.RMH.4.4.1-00049 > Tested-on: QCA9984 hw1.0 PCI 10.4-3.6-00104 > > Signed-off-by: Wen Gong <quic_wgong@xxxxxxxxxxx> > Signed-off-by: Kalle Valo <quic_kvalo@xxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. 8723750e2753 ath10k: fix regdomain info of iw reg set/get -- https://patchwork.kernel.org/project/linux-wireless/patch/20220525132247.23459-1-quic_wgong@xxxxxxxxxxx/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
