David Miller <davem@xxxxxxxxxxxxx> writes: >>> I really don't see the point, since it's just three lines of code, but I >>> wouldn't mind all that much either. >> >> My worry are the developers who even don't know what is a bottom half >> and might get it all wrong. (Yes, there really are such people.) > > And the difference between this and knowing you need to call the > ieee80211_rx_ni() thing is? > > You have to know what the heck a bottom half is to even know that you > would need to call the ieee80211_rx_ni() thing. > > And that's the same amount of knowledge necessary to simply wrap the > thing in a BH disable/enable sequence. I was thinking that it's possible to document it something like this: o in irq context use ieee80211_rx_irqsafe() o in a tasklet use ieee80211_rx() o in process context use ieee80211_rx_ni() Also in the future it might be easier to optimise something based on these functions. Maybe. But as Johannes didn't like the idea, and neither do you, I'm going to drop the idea. I'll add the BH disable/enable to wl1251 instead and hopefully Luciano does the same to wl1271. -- Kalle Valo -- 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
