Page MenuHomeFreeBSD

hid: u2f: stop interrupts on last-close
AcceptedPublic

Authored by kevans on Sun, Jul 12, 7:48 PM.
Tags
None
Referenced Files
F163076373: D58199.diff
Sun, Jul 19, 9:08 PM
F162991846: D58199.id181853.diff
Sat, Jul 18, 11:40 PM
F162991389: D58199.id181853.diff
Sat, Jul 18, 11:33 PM
Unknown Object (File)
Sat, Jul 18, 10:17 PM
Unknown Object (File)
Sat, Jul 18, 5:59 PM
Unknown Object (File)
Fri, Jul 17, 4:27 PM
Unknown Object (File)
Fri, Jul 17, 6:48 AM
Unknown Object (File)
Fri, Jul 17, 6:14 AM
Subscribers

Details

Reviewers
wulf
aokblast
Summary

This fixes an issue with the Solo2 (and likely some of the Nitrokey
family) where hangs would occur with OpenSSH- it issues a CANCEL prior
to closing the device unconditionally, and without draining the read
endpoint we end up seeing the response to that CANCEL the next time
OpenSSH tries to connect. This throws the entire command/response
sequence out of whack.

This call used to break Yubikeys in some situations, but the fix that
landed in 28d85db46b48 ("xhci: Do not drop and add bits in xhci") seems
to have addressed that- presumably we sometimes end up stopping the
command and desyncing at the controller level. This probably implies
that we need a SYNCWRITE HID quirk, but that requires a little more work
in usbhid_sync_xfer() and this doesn't seem to cause any problems in
normal usage.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 74753
Build 71636: arc lint + arc unit