On Fri, Apr 1, 2011 at 12:59 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2011-04-01 at 12:42 -0700, Luis R. Rodriguez wrote: >> When we restore regulatory settings its possible CRDA >> will not reply because of a bogus user entry. In this >> case the bogus entry will prevent any further processing >> on cfg80211 for regulatory domains even if we restore >> regulatory settings. >> >> To prevent this we suck out all pending requests when >> restoring regulatory settings and add them back into the >> queue after we have queued up the reset work. > > What if CRDA replies in order, i.e. replies to the user requested one > first instead of the disassoc requested one? Are you questioning the order of udev events and how CRDA processes them? If CRDA is present it should get the udev event for any valid request and process it accordingly. If the request is bogus it'll prevent any further processing on cfg80211 given that we simply bail out of processing requests until last_request->processed is true. The fix for that lies in the timeout on patch 2. This patch just ensures that we make sure to clear out any pending requests prior to doing a restore of regulatory settings. > Why do we even require crda to reply to the first in list, rather than > any one? The order should not matter except that we want the queue to be cleared before processing core hints when doing restoration, otherwise the next user hint in the queue can be bogus and it will prevent a restore. Luis -- 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
