Changeset 73e5b62 in mainline for uspace/drv/bus/usb/xhci/hc.c

Timestamp:

2017-07-31T19:58:08Z (7 years ago)

Author:

Jaroslav Jindrak <dzejrou@…>

Branches:

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

Children:

7428b92

Parents:

c9bec1c

Message:

DCBAA now correctly holds physical addresses of the device contexts, a secondary array called dcbaa_virt was added that is used for deallocation.

File:

• uspace/drv/bus/usb/xhci/hc.c (modified) (3 diffs)

uspace/drv/bus/usb/xhci/hc.c

@@ -198 +198 @@
         int err;
 
-        hc->dcbaa = malloc32((1 + hc->max_slots) * sizeof(xhci_device_ctx_t*));
+        hc->dcbaa = malloc32((1 + hc->max_slots) * sizeof(uint64_t));
         if (!hc->dcbaa)
                 return ENOMEM;
 
+        hc->dcbaa_virt = malloc32((1 + hc->max_slots) * sizeof(xhci_device_ctx_t*));
+        if (!hc->dcbaa_virt) {
+                err = ENOMEM;
+                goto err_dcbaa;
+        }
+
         if ((err = xhci_trb_ring_init(&hc->command_ring, hc)))
-                goto err_dcbaa;
+                goto err_dcbaa_virt;
 
         if ((err = xhci_event_ring_init(&hc->event_ring, hc)))
@@ -220 +226 @@
 err_cmd_ring:
         xhci_trb_ring_fini(&hc->command_ring);
+err_dcbaa_virt:
+        free32(hc->dcbaa_virt);
 err_dcbaa:
         free32(hc->dcbaa);
@@ -468 +476 @@
         /* Idx 0 already deallocated by xhci_scratchpad_free. */
         for (unsigned i = 1; i < hc->max_slots + 1; ++i) {
-                if (hc->dcbaa[i] != NULL) {
-                        free32(hc->dcbaa[i]);
-                        hc->dcbaa[i] = NULL;
+                if (hc->dcbaa_virt[i] != NULL) {
+                        free32(hc->dcbaa_virt[i]);
+                        hc->dcbaa_virt[i] = NULL;
                 }
         }
 
         free32(hc->dcbaa);
+        free32(hc->dcbaa_virt);
 }