On Monday 24 September 2007, Johannes Berg wrote: > On Mon, 2007-09-24 at 19:20 +0200, Johannes Berg wrote: > > On Mon, 2007-09-24 at 18:41 +0200, Michael Buesch wrote: > > > > > - if (local->ops->config) > > > + if (local->open_count) > > > ret = local->ops->config(local_to_hw(local), &local->hw.conf); > > > > Isn't this racy? > > Also, shouldn't we then call ->config when the first interface is > brought up or something? I don't know. You are the one who knows the code. ;) I'm just pointing at problems that I'm not able to fix, because I don't know the code. - 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