Changeset 43428 in vbox for trunk/src/VBox/Devices/Parallel

Timestamp:

Sep 25, 2012 12:35:17 PM (13 years ago)

Author:

vboxsync

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

80927

Message:

Devices/Parallel: Extra Debug statements to debug issue with user. Will revert the changes soon.

File:

• trunk/src/VBox/Devices/Parallel/DrvHostParallel.cpp (modified) (12 diffs)

trunk/src/VBox/Devices/Parallel/DrvHostParallel.cpp

@@ -403 +403 @@
     hDevInfo = SetupDiGetClassDevs(NULL, 0, 0, DIGCF_PRESENT | DIGCF_ALLCLASSES);
     if (hDevInfo == INVALID_HANDLE_VALUE)
+    {
+        LogFlowFunc(("Invalid Handle \n"));
         return VERR_INVALID_HANDLE;
+    }
 
     /* Enumerate through all devices in Set. */
@@ -415 +418 @@
         while (!SetupDiGetDeviceRegistryProperty(hDevInfo, &DeviceInfoData, SPDRP_FRIENDLYNAME,
                                                  (PDWORD)&dwDataType, (uint8_t *)pBuf,
-                                                 dwBufSize, (PDWORD)&dwBufSize)
-               && GetLastError() == ERROR_INSUFFICIENT_BUFFER)
+                                                 dwBufSize, (PDWORD)&dwBufSize))
         {
-            if (pBuf)
-                 RTMemFree(pBuf);
-            /* Max size will never be more than 2048 bytes */
-            pBuf = (uint8_t *)RTMemAlloc(dwBufSize * 2);
+            if (GetLastError() == ERROR_INSUFFICIENT_BUFFER)
+            {
+                LogFlow(("ERROR_INSUFF_BUFF = %d. dwBufSz = %d\n", GetLastError(), dwBufSize));
+                if (pBuf)
+                    RTMemFree(pBuf);
+                 /* Max size will never be more than 2048 bytes */
+                if (dwBufSize > 1024 || dwBufSize < 0)
+                    dwBufSize = 1024;
+                pBuf = (uint8_t *)RTMemAlloc(dwBufSize * 2);
+            }
+            else
+            {
+                LogFlow(("GetDevProp Error = %d & dwBufSz = %d\n", GetLastError(), dwBufSize));
+                break;
+            }
         }
         if(!pBuf)
-            return VERR_NO_MEMORY;
-
+        {
+            LogFlowFunc(("No Memory to save ParportString\n"));
+            /* Trying once more  with fixed length, assuming friendly name
+             * will not be bigger than 2048
+             */
+            pBuf = (uint8_t *)RTMemAlloc(2048);
+            if (!pBuf)
+            {
+                LogFlowFunc(("Fixed allocation failed\n"));
+                return VERR_NO_MEMORY;
+            }
+            if (!SetupDiGetDeviceRegistryProperty(hDevInfo, &DeviceInfoData, SPDRP_FRIENDLYNAME,
+                                                 (PDWORD)&dwDataType, (uint8_t *)pBuf,
+                                                 2048, NULL))
+            {
+                LogFlowFunc(("GetDevProp failed with ERR = %d\n", GetLastError()));
+                return VERR_GENERAL_FAILURE;
+
+            }
+        }
         if (RTStrStr((char*)pBuf, "LPT"))
         {
@@ -472 +503 @@
                 pThis->u32LptAddrStatus  = pThis->u32LptAddr + STATUS_REG_OFFSET;
             }
+            else
+                LogFlowFunc(("u32Parport Addr No Available \n"));
             if (pThis->fParportAvail)
+            {
+                LogFlow(("Parport found . Break from inner loop \n"));
                 break;
+            }
+        }
+        else
+        {
+            LogFlow(("LPT: Parallel Port not available \n"));
         }
         if (pBuf)
@@ -479 +519 @@
         if (pThis->fParportAvail)
         {
+            LogFlow(("Parport Available. Break from outer loop \n"));
             /* Parallel port address has been found. No need to iterate further. */
             break;
@@ -586 +627 @@
         rc = RTErrConvertFromErrno(errno);
 # else /* VBOX_WITH_WIN_PARPORT_SUP */
-    if (pThis->fParportAvail)
+    if (!pThis->fParportAvail)
+        LogFlowFunc(("Parport Not Available\n"));
+    //if (pThis->fParportAvail)
     {
         for (size_t i = 0; i < cbWrite; i++)
@@ -629 +672 @@
         rc = RTErrConvertFromErrno(errno);
 # else  /* VBOX_WITH_WIN_PARPORT_SUP */
-    if (pThis->fParportAvail)
+    if (!pThis->fParportAvail)
+        LogFlowFunc(("Parport Not Available\n"));
+    //if (pThis->fParportAvail)
     {
         *((uint8_t*)(pvBuf)) = 0; /* Initialize the buffer. */
@@ -659 +704 @@
     uint64_t u64Data;
     u64Data = (uint8_t)iMode;
-    LogFlowFunc(("calling R0 to write CTRL, data=%#x\n", u64Data));
-    if (pThis->fParportAvail)
-    {
+    if (!pThis->fParportAvail)
+        LogFlowFunc(("Parport Not available\n"));
+    //if (pThis->fParportAvail)
+    {
+        LogFlowFunc(("calling R0 to SetPortDirection, data=%#x\n", u64Data));
         rc = PDMDrvHlpCallR0(pThis->CTX_SUFF(pDrvIns), DRVHOSTPARALLELR0OP_SETPORTDIRECTION, u64Data);
         AssertRC(rc);
@@ -686 +733 @@
     uint64_t u64Data;
     u64Data = (uint8_t)fReg;
-    LogFlowFunc(("calling R0 to write CTRL, data=%#x\n", u64Data));
-    if (pThis->fParportAvail)
-    {
+    if (!pThis->fParportAvail)
+        LogFlowFunc(("Parport Not Available\n"));
+    //if (pThis->fParportAvail)
+    {
+        LogFlowFunc(("calling R0 to write CTRL, data=%#x\n", u64Data));
         rc = PDMDrvHlpCallR0(pThis->CTX_SUFF(pDrvIns), DRVHOSTPARALLELR0OP_WRITECONTROL, u64Data);
         AssertRC(rc);
@@ -718 +767 @@
 # else /* VBOX_WITH_WIN_PARPORT_SUP */
     *pfReg = 0; /* Initialize the buffer*/
-    if (pThis->fParportAvail)
+    if (!pThis->fParportAvail)
+        LogFlowFunc(("Parport Not Available\n"));
+    //if (pThis->fParportAvail)
     {
         LogFlowFunc(("calling R0 to read control from parallel port\n"));
@@ -738 +789 @@
     int rcLnx = 0;
     uint8_t fReg = 0;
+    LogFlowFunc(("%d Status  Reg\n", *pfReg));
 # ifndef  VBOX_WITH_WIN_PARPORT_SUP
     rcLnx = ioctl(RTFileToNative(pThis->hFileDevice), PPRSTATUS, &fReg);
@@ -749 +801 @@
 # else /* VBOX_WITH_WIN_PARPORT_SUP */
     *pfReg = 0; /* Intialize the buffer. */
-    if (pThis->fParportAvail)
-    {
-        LogFlowFunc(("calling R0 to read status from parallel port\n"));
+    if (!pThis->fParportAvail)
+        LogFlowFunc(("fParport Not Available.. Error!!!!!!!!!! \n"));
+    //if (pThis->fParportAvail)
+    {
+        LogFlowFunc(("calling R0 to read status from parallel port. fParport should be available\n"));
         rc = PDMDrvHlpCallR0(pThis->CTX_SUFF(pDrvIns), DRVHOSTPARALLELR0OP_READSTATUS, 0);
         AssertRC(rc);
+        LogFlow(("value read from status = %d\n", *pfReg));
         *pfReg = pThis->u8ReadInStatus;
     }
+
 # endif /* VBOX_WITH_WIN_PARPORT_SUP */
     return rc;
@@ -995 +1051 @@
         rc = RTFileOpen(&pThis->hWinFileDevice, (char *)pThis->szParportName,
                         RTFILE_O_READWRITE | RTFILE_O_OPEN | RTFILE_O_DENY_NONE);
+        LogFlowFunc(("RTFileOpen Return = %d\n", rc));
     }
 #endif