Changeset 36630 in vbox for trunk/include/VBox

Timestamp:

Apr 8, 2011 6:41:51 PM (14 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

71093

Message:

PCI: Main and VBoxManage work

File:

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

trunk/include/VBox/pci.h

@@ -1018 +1018 @@
 
 #if defined(__cplusplus) && defined(IN_RING3)
-/**
- * Document me.
+/* For RTStrPrintf(). */
+#include <iprt/string.h>
+
+/**
+ * Class representing PCI address. PCI device consist of
+ * bus, device and function numbers. Generally device PCI
+ * address could be changed during runtime, but only by 
+ * an OS PCI driver.
  *
  * @remarks C++ classes (structs included) are not generally accepted in
@@ -1028 +1034 @@
 struct PciBusAddress
 {
-    /** @todo r=bird: Add 'm', or 'm_' to data members. This is general
-     *        practice in most of the source tree. */
-    int  iBus;
-    int  iDevice;
-    int  iFn;
+    /** @todo: think if we'll need domain, which is higher
+     *  word of the address. */
+    int  miBus;
+    int  miDevice;
+    int  miFn;
 
     PciBusAddress()
@@ -1039 +1045 @@
     }
 
-    PciBusAddress(int bus, int device, int fn)
-    {
-        init(bus, device, fn);
+    PciBusAddress(int iBus, int iDevice, int iFn)
+    {
+        init(iBus, iDevice, iFn);
     }
 
     PciBusAddress& clear()
     {
-        iBus = iDevice = iFn = -1;
+        miBus = miDevice = miFn = -1;
         return *this;
     }
 
-    void init(int bus, int device, int fn)
-    {
-        iBus = bus;
-        iDevice = device;
-        iFn = fn;
+    void init(int iBus, int iDevice, int iFn)
+    {
+        miBus    = iBus;
+        miDevice = iDevice;
+        miFn     = iFn;
     }
 
     void init(const PciBusAddress &a)
     {
-        iBus = a.iBus;
-        iDevice = a.iDevice;
-        iFn = a.iFn;
+        miBus    = a.miBus;
+        miDevice = a.miDevice;
+        miFn     = a.miFn;
     }
 
     bool operator<(const PciBusAddress &a) const
     {
-        if (iBus < a.iBus)
+        if (miBus < a.miBus)
             return true;
 
-        if (iBus > a.iBus)
+        if (miBus > a.miBus)
             return false;
 
-        if (iDevice < a.iDevice)
+        if (miDevice < a.miDevice)
             return true;
 
-        if (iDevice > a.iDevice)
+        if (miDevice > a.miDevice)
             return false;
 
-        if (iFn < a.iFn)
+        if (miFn < a.miFn)
             return true;
 
-        if (iFn > a.iFn)
+        if (miFn > a.miFn)
             return false;
 
@@ -1089 +1095 @@
     bool operator==(const PciBusAddress &a) const
     {
-        return     (iBus == a.iBus)
-                && (iDevice == a.iDevice)
-                && (iFn == a.iFn);
+        return     (miBus    == a.miBus)
+                && (miDevice == a.miDevice)
+                && (miFn     == a.miFn);
     }
 
     bool operator!=(const PciBusAddress &a) const
     {
-        return     (iBus != a.iBus)
-                || (iDevice != a.iDevice)
-                || (iFn  != a.iFn);
+        return     (miBus    != a.miBus)
+                || (miDevice != a.miDevice)
+                || (miFn     != a.miFn);
     }
 
     bool valid() const
     {
-        return (iBus != -1) && (iDevice != -1) && (iFn != -1);
+        return (miBus    != -1) 
+            && (miDevice != -1) 
+            && (miFn     != -1);
     }
 
@@ -1109 +1117 @@
     {
         Assert(valid());
-        return (iBus << 8) | (iDevice << 3) | iFn;
+        return (miBus << 8) | (miDevice << 3) | miFn;
     }
 
     PciBusAddress& fromLong(int32_t value)
     {
-        iBus = (value >> 8) & 0xff;
-        iDevice = (value & 0xff) >> 3;
-        iFn = (value & 7);
+        miBus = (value >> 8) & 0xff;
+        miDevice = (value & 0xff) >> 3;
+        miFn = (value & 7);
         return *this;
+    }
+
+    /** Create string representation of this PCI address. */
+    bool format(char* szBuf, int32_t cBufSize)
+    {
+        if (cBufSize < (/* bus */ 2 + /* : */ 1 + /* device */ 2 + /* . */ 1 + /* function*/ 1 + /* \0 */1))
+            return false;
+        
+        if (valid())
+            RTStrPrintf(szBuf, cBufSize, "%02x:%02x.%01x", miBus, miDevice, miFn);
+        else
+            RTStrPrintf(szBuf, cBufSize, "%s", "<bad>");
+        
+        return true;
     }
 };