svn commit: r357291 - in head/sys/dev: bwi bwn ipw iwi iwm iwn malo mwl ral rtwn/pci wi wpi wtap
Jeff Roberson
jroberson at jroberson.net
Thu Jan 30 20:51:27 UTC 2020
On Thu, 30 Jan 2020, Gleb Smirnoff wrote:
> On Thu, Jan 30, 2020 at 10:28:01AM +0000, Hans Petter Selasky wrote:
> H> Author: hselasky
> H> Date: Thu Jan 30 10:28:01 2020
> H> New Revision: 357291
> H> URL: https://svnweb.freebsd.org/changeset/base/357291
> H>
> H> Log:
> H> Widen EPOCH(9) usage in PCI WLAN drivers.
> H>
> H> Make sure all occurrences of ieee80211_input_xxx() in sys/dev are
> H> covered by a network epoch section. Do not depend on the interrupt
> H> handler nor any taskqueues being in a network epoch section.
> H>
> H> This patch should unbreak the PCI WLAN drivers after r357004.
> H>
> H> Pointy hat: glebius@
> H> Sponsored by: Mellanox Technologies
>
> Hey, I have reviewed all of them.
>
> The following drivers were not broken, and your change does 100%
> recursive epoch_enter:
>
> bwi, ipw, iwi, iwm, iwn, ral, rtwn, wi, wpi
>
> The following drivers use taskq and would be fixed by D23408:
>
> bwn, malo, mwl, wtap
>
> P.S. A funny note about wtap. You modified even a function that
> is a dead code - wtap_rx_deliver(). Gives some clue on quality
> of your sweep over all drivers.
I would strongly suggest that we not make more changes to this area
without a discussion on a review to make sure we're all in agreement.
There are some fine technical details which would benefit from multiple
eyes and failing to act together is creating more conflict than is
necessary. I volunteer to be on the reviews as an impartial third party.
Thanks,
Jeff
>
> --
> Gleb Smirnoff
>
More information about the svn-src-head
mailing list