Changeset 44502 in vbox for trunk/include/VBox

Timestamp:

Jan 31, 2013 8:11:08 PM (12 years ago)

Author:

vboxsync

Message:

PDM queue callback lock remarks.

File:

• trunk/include/VBox/vmm/pdmqueue.h (modified) (5 diffs)

trunk/include/VBox/vmm/pdmqueue.h

@@ -66 +66 @@
  * @param   pDevIns     The device instance.
  * @param   pItem       The item to consume. Upon return this item will be freed.
+ * @remarks The device critical section will NOT be entered before calling the
+ *          back.  No locks will be held, but for now it's safe to assume that
+ *          only one EMT will do queue callbacks at any one time.
  */
 typedef DECLCALLBACK(bool) FNPDMQUEUEDEV(PPDMDEVINS pDevIns, PPDMQUEUEITEMCORE pItem);
@@ -78 +81 @@
  * @param   pDevIns     The USB device instance.
  * @param   pItem       The item to consume. Upon return this item will be freed.
+ * @remarks No locks will be held, but for now it's safe to assume that only one
+ *          EMT will do queue callbacks at any one time.
  */
 typedef DECLCALLBACK(bool) FNPDMQUEUEUSB(PPDMUSBINS pUsbIns, PPDMQUEUEITEMCORE pItem);
@@ -90 +95 @@
  * @param   pDrvIns     The driver instance.
  * @param   pItem       The item to consume. Upon return this item will be freed.
+ * @remarks No locks will be held, but for now it's safe to assume that only one
+ *          EMT will do queue callbacks at any one time.
  */
 typedef DECLCALLBACK(bool) FNPDMQUEUEDRV(PPDMDRVINS pDrvIns, PPDMQUEUEITEMCORE pItem);
@@ -102 +109 @@
  * @param   pVM         The VM handle.
  * @param   pItem       The item to consume. Upon return this item will be freed.
+ * @remarks No locks will be held, but for now it's safe to assume that only one
+ *          EMT will do queue callbacks at any one time.
  */
 typedef DECLCALLBACK(bool) FNPDMQUEUEINT(PVM pVM, PPDMQUEUEITEMCORE pItem);
@@ -114 +123 @@
  * @param   pvUser      User argument.
  * @param   pItem       The item to consume. Upon return this item will be freed.
+ * @remarks No locks will be held, but for now it's safe to assume that only one
+ *          EMT will do queue callbacks at any one time.
  */
 typedef DECLCALLBACK(bool) FNPDMQUEUEEXT(void *pvUser, PPDMQUEUEITEMCORE pItem);