[Make-wifi-fast] [PATCH v4] mac80211: Move reorder-sensitive TX handlers to after TXQ dequeue.
johannes at sipsolutions.net
Thu Sep 1 05:27:00 EDT 2016
> Yeah, was going to do that anyway. But since I'm touching the code
> anyway, this might be an opportunity to avoid constructs like this:
> if (!invoke_tx_handlers(tx))
> /* continue sending the packet */
> Most other succeed/fail functions seem to be of type bool, so it
> would help consistency as well. Unless there is some particular
> reason why this function happens to be using 0 to indicate success?
It's just convention in the kernel, really.
IMHO if a function has a bool return value it should be have a more
expressive name that indicates better what's going on, like e.g.
but of course that's not always done.
More information about the Make-wifi-fast