Changeset 708d8fcd in mainline for uspace/drv/bus/usb/xhci/endpoint.h

Timestamp:

2018-01-10T00:46:29Z (7 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

f92f6b1

Parents:

4793023

git-author:

Ondřej Hlavatý <aearsis@…> (2018-01-09 19:26:55)

git-committer:

Ondřej Hlavatý <aearsis@…> (2018-01-10 00:46:29)

Message:

xhci: rewritten isochronous transfers

There was a fundamental problem with relying on hardware to send
RING_OVERRUN/UNDERRUN events, which QEMU (and possibly others) do not
send. That resulted in not knowing if the transfer is still on schedule,
and having to ring the doorbell every time. That is not feasible,
because then the transfer can be more frequent than it should be.
Furthermore, it ignored the fact that isochronous TRBs are to be
scheduled not too late, but also not too soon (see 4.11.2.5 of the xHCI
spec).

Now, scheduling the TRBs to hardware is called feeding, and can be
delayed by setting a timer. Ring overruns/underruns are detected also at
the end of handling an event.

File:

• uspace/drv/bus/usb/xhci/endpoint.h (modified) (2 diffs)

uspace/drv/bus/usb/xhci/endpoint.h

@@ -45 +45 @@
 #include <ddf/driver.h>
 
+#include "isoch.h"
+#include "transfers.h"
 #include "trb_ring.h"
-
-#include "transfers.h"
 
 typedef struct xhci_device xhci_device_t;
@@ -94 +94 @@
 
         /** This field is a valid pointer for (and only for) isochronous transfers. */
-        struct {
-                /** The maximum size of an isochronous transfer and therefore the size of buffers */
-                size_t max_size;
-
-                /** Isochronous scheduled transfers with respective buffers */
-                #define XHCI_ISOCH_BUFFER_COUNT 4
-                xhci_isoch_transfer_t transfers[XHCI_ISOCH_BUFFER_COUNT];
-
-                /** Indices to transfers */
-                size_t dequeue, enqueue;
-
-                /** Are isochronous transfers started? */
-                bool started;
-
-                /** Protects common buffers. */
-                fibril_mutex_t guard;
-
-                /** Signals filled buffer. */
-                fibril_condvar_t avail;
-        } isoch [0];
+        xhci_isoch_t isoch [0];
 } xhci_endpoint_t;