Changeset 5f0b366 in mainline for uspace/lib/usbhost/src/endpoint.c

Timestamp:

2018-01-25T02:05:57Z (7 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

b357377

Parents:

e8277c0

git-author:

Ondřej Hlavatý <aearsis@…> (2018-01-24 19:34:07)

git-committer:

Ondřej Hlavatý <aearsis@…> (2018-01-25 02:05:57)

Message:

usbhost: prepare bandwidth accounting privatization to usb2_bus

The endpoint does not account the bandwidth by default. The next step is
moving the count_bw callback to the usb2 bus.

File:

• uspace/lib/usbhost/src/endpoint.c (modified) (3 diffs)

uspace/lib/usbhost/src/endpoint.c

@@ -74 +74 @@
 
         ep->max_transfer_size = ep->max_packet_size * ep->packets_per_uframe;
-
-        ep->bandwidth = endpoint_count_bw(ep, ep->max_transfer_size);
 }
 
@@ -201 +199 @@
         ep->active_batch = NULL;
         fibril_condvar_signal(&ep->avail);
-}
-
-/**
- * Call the bus operation to count bandwidth.
- *
- * @param ep Endpoint on which the transfer will take place.
- * @param size The payload size.
- */
-ssize_t endpoint_count_bw(endpoint_t *ep, size_t size)
-{
-        assert(ep);
-
-        const bus_ops_t *ops = BUS_OPS_LOOKUP(get_bus_ops(ep), endpoint_count_bw);
-        if (!ops)
-                return 0;
-
-        return ops->endpoint_count_bw(ep, size);
 }
 
@@ -262 +243 @@
         }
 
+        /** Limit transfers with reserved bandwidth to the amount reserved */
+        if ((ep->transfer_type == USB_TRANSFER_INTERRUPT
+            || ep->transfer_type == USB_TRANSFER_ISOCHRONOUS)
+            && size > ep->max_transfer_size)
+                return ENOSPC;
+
         /* Offline devices don't schedule transfers other than on EP0. */
-        if (!device->online && ep->endpoint > 0) {
+        if (!device->online && ep->endpoint > 0)
                 return EAGAIN;
-        }
-
-        const size_t bw = endpoint_count_bw(ep, size);
-        /* Check if we have enough bandwidth reserved */
-        if (ep->bandwidth < bw) {
-                usb_log_error("Endpoint(%d:%d) %s needs %zu bw "
-                    "but only %zu is reserved.\n",
-                    device->address, ep->endpoint, name, bw, ep->bandwidth);
-                return ENOSPC;
-        }
 
         usb_transfer_batch_t *batch = usb_transfer_batch_create(ep);