On Wednesday 04 October 2006 18:31, Jan Kiszka wrote: > Ivo van Doorn wrote: > > Hi, > > > >> Ivo told me about a patch for d80211 that moved certain timers to thread > >> context, effectively avoiding to call config from timer handlers, but I > >> didn't find any trace yet. Is there some modification in this direction > >> already scheduled? I'm not necessarily looking for work, at best I would > >> just enjoy to use it. ;) > > > > I have found the actual patch: > > [PATCH 1/5] d80211: make sleeping in hw->config possible > > And was send on august 1st by Jiri to the netdev list. > > It was based on a patch by Michael Buesch. > > Ah, looks like I didn't dug thoroughly enough. > > Anyway, this means my BUG proved the patch's claim wrong :o), at least > one atomic gremlin is left: > > ieee80211_sta_timer -> > ieee80211_sta_find_ibss -> > ieee80211_sta_join_ibss -> > ieee80211_ioctl_siwfreq -> > ieee80211_hw_config > > Anyone already an idea how to fix it?
You could replace the timer with a workqueue, the original patch also did that, so I think it would be good enough this time as well. :) Ivo - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
