Hi, On Fri, Sep 5, 2014 at 11:43 PM, Luis R. Rodriguez <mcgrof@xxxxxxxx> wrote: > Yeah this seems to be a corner case of the fact that we deal with > locking for the last request only through RCU and we only annotate > that the request was processed but don't add checks for when its > about to be processed. At least that's what I gather could be > the issue here given that if you see __reg_process_hint_driver() > we already check for two driver hints matching and and just > request the core to copy the existing regulatory domain onto > the new device. > > if (lr->initiator == NL80211_REGDOM_SET_BY_DRIVER && > !regdom_changes(driver_request->alpha2)) > return REG_REQ_ALREADY_SET; > > return REG_REQ_INTERSECT; > > The intersection would happen otherwise and I think this is caused > by a small race here. Can you try the following: Just retried with your patch applied but I can still get an "98" intersected regdomain :( [ 6.190000] ath: EEPROM regdomain: 0x0 [ 6.190000] ath: EEPROM indicates default country code should be used [ 6.190000] ath: doing EEPROM country->regdmn map search [ 6.190000] ath: country maps to regdmn code: 0x3a [ 6.190000] ath: Country alpha2 being used: US [ 6.190000] ath: Regpair used: 0x3a [ 6.200000] ieee80211 phy0: Selected rate control algorithm 'minstrel_ht' [ 6.210000] ieee80211 phy0: Atheros AR9340 Rev:0 mem=0xb8100000, irq=47 [ 6.210000] PCI: Enabling device 0000:00:00.0 (0000 -> 0002) [ 6.210000] ath: EEPROM regdomain: 0x0 [ 6.210000] ath: EEPROM indicates default country code should be used [ 6.210000] ath: doing EEPROM country->regdmn map search [ 6.210000] ath: country maps to regdmn code: 0x3a [ 6.210000] ath: Country alpha2 being used: US [ 6.210000] ath: Regpair used: 0x3a [ 6.220000] ieee80211 phy1: Selected rate control algorithm 'minstrel_ht' [ 6.240000] cfg80211: Calling CRDA for country: US [ 6.240000] cfg80211: Calling CRDA for country: US [ 6.240000] cfg80211: Current regulatory domain intersected: [ 6.240000] cfg80211: DFS Master region: unset [ 6.240000] cfg80211: (start_freq - end_freq @ bandwidth), (max_antenna_gain, max_eirp), (dfs_cac_time) [ 6.240000] cfg80211: (2402000 KHz - 2472000 KHz @ 40000 KHz), (N/A, 2000 mBm), (N/A) [ 6.240000] cfg80211: (2457000 KHz - 2472000 KHz @ 15000 KHz), (N/A, 2000 mBm), (N/A) [ 6.240000] cfg80211: (5170000 KHz - 5250000 KHz @ 80000 KHz), (N/A, 1700 mBm), (N/A) [ 6.240000] cfg80211: (5250000 KHz - 5330000 KHz @ 80000 KHz), (N/A, 2000 mBm), (0 s) [ 6.240000] cfg80211: (5735000 KHz - 5835000 KHz @ 80000 KHz), (N/A, 2000 mBm), (N/A) [ 6.240000] cfg80211: (57240000 KHz - 63720000 KHz @ 2160000 KHz), (N/A, 0 mBm), (N/A) [ 6.240000] cfg80211: Calling CRDA for country: US [ 6.240000] ieee80211 phy1: Atheros AR9300 Rev:3 mem=0xb0000000, irq=40 [ 6.240000] cfg80211: Current regulatory domain intersected: [ 6.240000] cfg80211: DFS Master region: unset [ 6.240000] cfg80211: (start_freq - end_freq @ bandwidth), (max_antenna_gain, max_eirp), (dfs_cac_time) [ 6.240000] cfg80211: (2402000 KHz - 2472000 KHz @ 40000 KHz), (N/A, 2000 mBm), (N/A) [ 6.240000] cfg80211: (2457000 KHz - 2472000 KHz @ 15000 KHz), (N/A, 2000 mBm), (N/A) [ 6.240000] cfg80211: (5170000 KHz - 5250000 KHz @ 80000 KHz), (N/A, 1700 mBm), (N/A) [ 6.240000] cfg80211: (5250000 KHz - 5330000 KHz @ 80000 KHz), (N/A, 2000 mBm), (0 s) [ 6.240000] cfg80211: (5735000 KHz - 5835000 KHz @ 80000 KHz), (N/A, 2000 mBm), (N/A) [ 6.240000] cfg80211: (57240000 KHz - 63720000 KHz @ 2160000 KHz), (N/A, 0 mBm), (N/A) I'll try to dive deeper into the regulatory code ... Helmut -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
