Changeset 51524 in vbox for trunk/src/VBox/HostServices/SharedOpenGL

Timestamp:

Jun 4, 2014 8:06:12 AM (11 years ago)

Author:

vboxsync

Message:

DevVGA/crOpenGL: bugfixes, logging

Location:

trunk/src/VBox/HostServices/SharedOpenGL

Files:

• crserver/crservice.cpp (modified) (3 diffs)
• crserverlib/server_main.c (modified) (8 diffs)
• crserverlib/server_misc.c (modified) (5 diffs)
• crserverlib/server_presenter.cpp (modified) (1 diff)

trunk/src/VBox/HostServices/SharedOpenGL/crserver/crservice.cpp

@@ -973 +973 @@
         default:
         {
+            WARN(("svcCall: unexpected u32Function %d", u32Function));
             rc = VERR_NOT_IMPLEMENTED;
         }
@@ -1517 +1518 @@
         }
         default:
+            WARN(("svcHostCallPerform: unexpected u32Function %d", u32Function));
             rc = VERR_NOT_IMPLEMENTED;
             break;
@@ -1592 +1594 @@
                 }
                 default:
-                    WARN(("invalid function"));
+                    WARN(("svcHostCall: invalid function %d", pCtl->enmType));
                     return VERR_INVALID_PARAMETER;
             }

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib/server_main.c

@@ -2825 +2825 @@
     {
         if (MAPPED(SCREEN(i)))
-            crWarning("Screen count is changing, but screen[%i] is still mapped", i);
+            WARN(("Screen count is changing, but screen[%i] is still mapped", i));
         return VERR_NOT_IMPLEMENTED;
     }
@@ -3406 +3406 @@
         case SHCRGL_GUEST_FN_SET_VERSION:
         {
-            crWarning("invalid function");
+            WARN(("SHCRGL_GUEST_FN_SET_VERSION: invalid function"));
             rc = VERR_NOT_IMPLEMENTED;
             break;
@@ -3413 +3413 @@
         case SHCRGL_GUEST_FN_SET_PID:
         {
-            crWarning("invalid function");
+            WARN(("SHCRGL_GUEST_FN_SET_PID: invalid function"));
             rc = VERR_NOT_IMPLEMENTED;
             break;
@@ -3420 +3420 @@
         default:
         {
-            crWarning("invalid function");
+            WARN(("invalid function, %d", u32Function));
             rc = VERR_NOT_IMPLEMENTED;
             break;
@@ -3586 +3586 @@
         }
         default:
-            WARN(("invalid function"));
+            WARN(("crVBoxCrCmdGuestCtl: invalid function %d", pCtl->u32Type));
             return VERR_INVALID_PARAMETER;
     }
@@ -4161 +4161 @@
         case SHCRGL_GUEST_FN_SET_VERSION:
         {
-            crWarning("invalid function");
+            WARN(("crVBoxServerCrHgsmiCmd, SHCRGL_GUEST_FN_SET_VERSION: invalid function"));
             rc = VERR_NOT_IMPLEMENTED;
             break;
@@ -4168 +4168 @@
         case SHCRGL_GUEST_FN_SET_PID:
         {
-            crWarning("invalid function");
+            WARN(("crVBoxServerCrHgsmiCmd, SHCRGL_GUEST_FN_SET_PID: invalid function"));
             rc = VERR_NOT_IMPLEMENTED;
             break;
@@ -4175 +4175 @@
         default:
         {
-            crWarning("invalid function");
+            WARN(("crVBoxServerCrHgsmiCmd: invalid functionm %d", u32Function));
             rc = VERR_NOT_IMPLEMENTED;
             break;

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib/server_misc.c

@@ -728 +728 @@
 
         if (!mural->fRedirected)
+        {
+            WARN(("mural not redirected!"));
             return VERR_NOT_IMPLEMENTED;
+        }
 
         enmBuf = fDraw ? ctx->buffer.drawBuffer : ctx->buffer.readBuffer;
@@ -744 +747 @@
                 break;
             default:
-                crWarning("unsupported enum buf");
+                WARN(("unsupported enum buf %d", enmBuf));
                 return VERR_NOT_IMPLEMENTED;
                 break;
@@ -779 +782 @@
     if (pAp->level)
     {
-        crWarning("non-zero level not implemented");
+        WARN(("non-zero level not implemented"));
         return VERR_NOT_IMPLEMENTED;
     }
@@ -792 +795 @@
     if (tobj->target != GL_TEXTURE_2D && tobj->target != GL_TEXTURE_RECTANGLE_NV)
     {
-        crWarning("non-texture[rect|2d] not implemented");
+        WARN(("non-texture[rect|2d] not implemented"));
         return VERR_NOT_IMPLEMENTED;
     }
@@ -823 +826 @@
     if (mask != GL_COLOR_BUFFER_BIT)
     {
-        crWarning("not supported blit mask %d", mask);
+        WARN(("not supported blit mask %d", mask));
         return VERR_NOT_IMPLEMENTED;
     }

trunk/src/VBox/HostServices/SharedOpenGL/crserverlib/server_presenter.cpp

@@ -5654 +5654 @@
         if (hSrcFb)
         {
-            WARN(("blit from one framebuffer, wow"));
+            LOG(("blit from one framebuffer, wow"));
 
             int rc = CrFbUpdateBegin(hSrcFb);