Changes in pfvar.h and removal of rwlock.h includes broke the build (noted by @kristof). This updated patch unbreaks the build.

In D15502#327153, @kristof wrote:

Looks good at first glance.
I like the move of the sys/rmlock.h include into pfvar.h, so the header becomes more self-contained (i.e. you don't need extra includes to make your include of pfvar.h work). I'm running test builds for that, because I thought it broke when I tried. Perhaps I messed something else up though.

For others and for context: Olivier has tested this change and saw a 65% increase in packets per second through pf. We really, really want this change.

In D11072#237288, @emaste wrote:

Be sure to send these upstream as well

Improve the logic when detecting when whether or not we should reverse byte order for ip_len and ip_off.

Use a less intrusive method to find out if we should include "pcap-int.h"

Unbreak on 11.x+ by removing reference to unused header file.

Turns out there was a much simpler solution for this build failure. By simply removing reference to "pcap-int.h" (which was not used at all) now this port builds on 10.x, 11.x and 12x just fine.

Removed reference to unnecessary header files to unbreak on 11.x and 12.x

Addressed in D8897

In D9091#187890, @bapt wrote:

Why adding a new one instead of upgrading the old one?

Update net/radvd to 2.15

Fixed as per @bapt recommendations