Sedat, On Wed, 2010-04-21 at 00:06 -0700, Sedat Dilek wrote: > >> According to Johannes (on IRC) iwl3945 doesn't use internal scans yet > >> (see my patch below). > >> Thus, I am not sure if this needs to be fixed separately for iwl3945 > >> and if I ran into a scan race condition here. > > > > Does this patch make any difference for you? > > > > Speaking of my patch "iwl3945-fix-scan-races.patch" from [3]? > I was guessing iwl3945 has internal scans support - is that the fact? > Johannes told me someone is working on it. It has some support ... but nothing that currently requests an internal scan. > To answer your question: Not sure, but it seems not to help. Weird. > > > > It does. Thank you. As a sanity check, could you please try this patch? > > > > In general, it would be very helpful to give some comments on what > this offered patch is doing. > Not sure, if I can test it today. > >From your logs I see that it is right after disconnecting from AP (after loosing probe responses) that the problem occurs. I see that we send QOS commands to the device, to which it responds, but after that you start to get errors. I looked at when those QOS commands are sent and they are sent in ieee80211_set_disassoc _after_ the queues have been disabled. I am not sure if that is the right thing to do and wanted to check if it is indeed those commands that are causing the issue. The patch I asked you to test modified the code to not send QOS commands after queues have been disabled. > By the way, might help to set one of the module-options for iwl3945? > > parm: antenna:select antenna (1=Main, 2=Aux, default 0 [both]) (int) > parm: swcrypto:using software crypto (default 1 [software]) (int) > parm: debug:debug output mask (uint) > parm: disable_hw_scan:disable hardware scanning (default 0) (int) > parm: fw_restart3945:restart firmware in case of error (int) > > Especially "fw_restart3945" in my case? It should be set by default. > On which errors it is > restarting firmware? Firmware errors. Reinette -- 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
