Merge pull request #1184 from martinling/flush-on-...
by Michael Ossmann 2 years 10 months
Merge pull request #1184 from martinling/flush-on-setup

Flush queues on control endpoint when a new SETUP is received.
0c85e101
Flush queues on control endpoint when a new SETUP ...
by Martin Ling 2 years 10 months
Flush queues on control endpoint when a new SETUP is received.

On macOS 12.6 running libusb 1.0.26, the host was seen to sometimes
make an incomplete ClearFeature(ENDPOINT_HALT) request when
ClearPipeStallBothEnds was called from libusb_cancel_transfer.

The host would send the SETUP packet, and the firmware would
call usb_transfer_schedule_ack to acknowledge the upcoming IN
token. However, the host would then not send the IN token,
proceeding directly to the next SETUP.

Since an empty transfer was queued to schedule the ACK, the
firmware would leak one transfer from the free_transfers list.

After a few iterations of this, the firmware would run out of
free transfers and the next request would hang waiting for one.

Fix this by flushing the transfer queues for the control endpoint
when a new SETUP is received, since that token implicitly cancels
any previously ongoing request.
1db39c0e
Add documentation request issue template
by straithe 2 years 10 months
44244f52
remove retired content and move jawbreakercontent
by straithe 2 years 10 months
12b9d4cd
firmware: use python3 interpreter, not python
by Michael Ossmann 2 years 10 months
1f9f597e
firmware: improve platform detection
by Michael Ossmann 2 years 10 months
firmware: improve platform detection

Slow down the detection of pull-up and pull-down resistors to make
detection more reliable.
f3d10dcc
firmware: fix duration of delay_us_at_mhz()
by Michael Ossmann 2 years 10 months
b026c070
Merge all_finished_lock with transfer_lock.
by Martin Ling 2 years 10 months
f7c219e3
Hold transfer lock during initial submissions in p...
by Martin Ling 2 years 10 months
Hold transfer lock during initial submissions in prepare_transfers.
367ce3d7
Extend scope of transfer_lock in callback.
by Martin Ling 2 years 10 months
559a45bd
Report a bug