ext Kalle Valo wrote:
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.
Yeps, I can do the same for wl1271.
--
Cheers,
Luca.
--
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
