In D47854#1090987, @jrtc27 wrote:

Honestly we should instead be removing the use of intptr_t for hwreset, it's not a sensible type to use for an ID.

I'll look at the screenshots and diagrams later. I don't want to defer this installment any further.

(adding doc mentors as reviewers)

While testing, I discovered the Nm's in SYNOPSIS were causing this page to get pulled into results for apropos sysctl. I rewrote it the way it's written in vt(4).

One more fixup.

Simplify error handling some more.

Jailhost before patch:

root@deblndw013x:/compat/linux/proc/self
# cat mounts
cat: mounts: Operation canceled

I wish we had any consensus on how we want to sort options. I'm partial to "-A -a -B -b --bee -C" sorting, but I don't want to bikeshed it. We have all different styles in the tree and any sorting is better!

Please add others you think see fit.

I tend to agree with kib@ for both of its inline comments. The second comment is the second alternative I listed. I'm fine either way.

For the record I have no stakes in SCTP and I'm not involved in the changes done here. This work with the user reporting it is a courtesy to the FreeBSD pf code and to triage panics which I only think is a good idea for production code.

What's the status now? From my point of you I've explained the previous behaviour, the change that caused this to exhibit the wrong behaviour and how it was fixed. I'd like to get to a more productive cooperation based on technical urgency. I don't think tests prevented this from happening and I don't think the issue gets any better discussing commit messages.