Changeset 39668 in vbox for trunk/src/VBox/Main

Timestamp:

Dec 20, 2011 9:46:30 PM (13 years ago)

Author:

vboxsync

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

75489

Message:

Implemented the log setting properties (read-only).

Location:

trunk/src/VBox/Main

Files:

• idl/VirtualBox.xidl (modified) (2 diffs)
• include/MachineDebuggerImpl.h (modified) (3 diffs)
• src-client/MachineDebuggerImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/idl/VirtualBox.xidl

@@ -13185 +13185 @@
   <interface
     name="IMachineDebugger" extends="$unknown"
-    uuid="1bfd2fa9-0d91-44d3-9515-368dcbb3eb4d"
+    uuid="a9abbb7c-d678-43b2-bed2-19ec0e32303d"
     wsmap="suppress"
     >
@@ -13541 +13541 @@
     </attribute>
 
-    <attribute name="logFlags" type="wstring" readonly="yes">
+    <attribute name="logDbgFlags" type="wstring" readonly="yes">
       <desc>The debug logger flags.</desc>
     </attribute>
 
-    <attribute name="logGroups" type="wstring" readonly="yes">
+    <attribute name="logDbgGroups" type="wstring" readonly="yes">
       <desc>The debug logger's group settings.</desc>
     </attribute>
 
-    <attribute name="logDestinations" type="wstring" readonly="yes">
+    <attribute name="logDbgDestinations" type="wstring" readonly="yes">
       <desc>The debug logger's destination settings.</desc>
+    </attribute>
+
+    <attribute name="logRelFlags" type="wstring" readonly="yes">
+      <desc>The release logger flags.</desc>
+    </attribute>
+
+    <attribute name="logRelGroups" type="wstring" readonly="yes">
+      <desc>The release logger's group settings.</desc>
+    </attribute>
+
+    <attribute name="logRelDestinations" type="wstring" readonly="yes">
+      <desc>The relase logger's destination settings.</desc>
     </attribute>
 

trunk/src/VBox/Main/include/MachineDebuggerImpl.h

@@ -22 +22 @@
 
 #include "VirtualBoxBase.h"
+#include <iprt/log.h>
 
 class Console;
@@ -63 +64 @@
     STDMETHOD(COMGETTER(LogEnabled)) (BOOL *aEnabled);
     STDMETHOD(COMSETTER(LogEnabled)) (BOOL aEnable);
-    STDMETHOD(COMGETTER(LogFlags)) (BSTR *a_pbstrSettings);
-    STDMETHOD(COMGETTER(LogGroups)) (BSTR *a_pbstrSettings);
-    STDMETHOD(COMGETTER(LogDestinations)) (BSTR *a_pbstrSettings);
+    STDMETHOD(COMGETTER(LogDbgFlags)) (BSTR *a_pbstrSettings);
+    STDMETHOD(COMGETTER(LogDbgGroups)) (BSTR *a_pbstrSettings);
+    STDMETHOD(COMGETTER(LogDbgDestinations)) (BSTR *a_pbstrSettings);
+    STDMETHOD(COMGETTER(LogRelFlags)) (BSTR *a_pbstrSettings);
+    STDMETHOD(COMGETTER(LogRelGroups)) (BSTR *a_pbstrSettings);
+    STDMETHOD(COMGETTER(LogRelDestinations)) (BSTR *a_pbstrSettings);
     STDMETHOD(COMGETTER(HWVirtExEnabled)) (BOOL *aEnabled);
     STDMETHOD(COMGETTER(HWVirtExNestedPagingEnabled)) (BOOL *aEnabled);
@@ -106 +110 @@
     bool queueSettings() const;
 
+    /** RTLogGetFlags, RTLogGetGroupSettings and RTLogGetDestinations function. */
+    typedef DECLCALLBACK(int) FNLOGGETSTR(PRTLOGGER, char *, size_t);
+    /** Function pointer.  */
+    typedef FNLOGGETSTR *PFNLOGGETSTR;
+    HRESULT logStringProps(PRTLOGGER pLogger, PFNLOGGETSTR pfnLogGetStr, const char *pszLogGetStr, BSTR *a_bstrSettings);
+
     Console * const mParent;
     // flags whether settings have been queued because

trunk/src/VBox/Main/src-client/MachineDebuggerImpl.cpp

@@ -447 +447 @@
 }
 
-STDMETHODIMP MachineDebugger::COMGETTER(LogFlags)(BSTR *a_pbstrSettings)
-{
-    ReturnComNotImplemented();
-}
-
-STDMETHODIMP MachineDebugger::COMGETTER(LogGroups)(BSTR *a_pbstrSettings)
-{
-    ReturnComNotImplemented();
-}
-
-STDMETHODIMP MachineDebugger::COMGETTER(LogDestinations)(BSTR *a_pbstrSettings)
-{
-    ReturnComNotImplemented();
+HRESULT MachineDebugger::logStringProps(PRTLOGGER pLogger, PFNLOGGETSTR pfnLogGetStr,
+                                        const char *pszLogGetStr, BSTR *a_pbstrSettings)
+{
+    /* Make sure the VM is powered up. */
+    AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
+    Console::SafeVMPtr ptrVM(mParent);
+    HRESULT hrc = ptrVM.rc();
+    if (FAILED(hrc))
+        return hrc;
+
+    /* Make sure we've got a logger. */
+    if (!pLogger)
+    {
+        Bstr bstrEmpty;
+        bstrEmpty.cloneTo(a_pbstrSettings);
+        return S_OK;
+    }
+
+    /* Do the job. */
+    size_t cbBuf = _1K;
+    for (;;)
+    {
+        char *pszBuf = (char *)RTMemTmpAlloc(cbBuf);
+        AssertReturn(pszBuf, E_OUTOFMEMORY);
+
+        int rc = pfnLogGetStr(pLogger, pszBuf, cbBuf);
+        if (RT_SUCCESS(rc))
+        {
+            try
+            {
+                Bstr bstrRet(pszBuf);
+                bstrRet.detachTo(a_pbstrSettings);
+                hrc = S_OK;
+            }
+            catch (std::bad_alloc)
+            {
+                hrc = E_OUTOFMEMORY;
+            }
+            RTMemTmpFree(pszBuf);
+            return hrc;
+        }
+        RTMemTmpFree(pszBuf);
+        AssertReturn(rc == VERR_BUFFER_OVERFLOW, setError(VBOX_E_IPRT_ERROR, tr("%s returned %Rrc"), pszLogGetStr, rc));
+
+        /* try again with a bigger buffer. */
+        cbBuf *= 2;
+        AssertReturn(cbBuf <= _256K, setError(E_FAIL, tr("%s returns too much data"), pszLogGetStr));
+    }
+}
+
+
+STDMETHODIMP MachineDebugger::COMGETTER(LogDbgFlags)(BSTR *a_pbstrSettings)
+{
+    CheckComArgOutPointerValid(a_pbstrSettings);
+
+    AutoCaller autoCaller(this);
+    HRESULT hrc = autoCaller.rc();
+    if (SUCCEEDED(hrc))
+        hrc = logStringProps(RTLogGetDefaultInstance(), RTLogGetFlags, "RTGetFlags", a_pbstrSettings);
+
+    return hrc;
+}
+
+STDMETHODIMP MachineDebugger::COMGETTER(LogDbgGroups)(BSTR *a_pbstrSettings)
+{
+    CheckComArgOutPointerValid(a_pbstrSettings);
+
+    AutoCaller autoCaller(this);
+    HRESULT hrc = autoCaller.rc();
+    if (SUCCEEDED(hrc))
+        hrc = logStringProps(RTLogGetDefaultInstance(), RTLogGetGroupSettings, "RTLogGetGroupSettings", a_pbstrSettings);
+
+    return hrc;
+}
+
+STDMETHODIMP MachineDebugger::COMGETTER(LogDbgDestinations)(BSTR *a_pbstrSettings)
+{
+    CheckComArgOutPointerValid(a_pbstrSettings);
+
+    AutoCaller autoCaller(this);
+    HRESULT hrc = autoCaller.rc();
+    if (SUCCEEDED(hrc))
+        hrc = logStringProps(RTLogRelDefaultInstance(), RTLogGetDestinations, "RTLogGetDestinations", a_pbstrSettings);
+
+    return hrc;
+}
+
+
+STDMETHODIMP MachineDebugger::COMGETTER(LogRelFlags)(BSTR *a_pbstrSettings)
+{
+    CheckComArgOutPointerValid(a_pbstrSettings);
+
+    AutoCaller autoCaller(this);
+    HRESULT hrc = autoCaller.rc();
+    if (SUCCEEDED(hrc))
+        hrc = logStringProps(RTLogRelDefaultInstance(), RTLogGetFlags, "RTGetFlags", a_pbstrSettings);
+
+    return hrc;
+}
+
+STDMETHODIMP MachineDebugger::COMGETTER(LogRelGroups)(BSTR *a_pbstrSettings)
+{
+    CheckComArgOutPointerValid(a_pbstrSettings);
+
+    AutoCaller autoCaller(this);
+    HRESULT hrc = autoCaller.rc();
+    if (SUCCEEDED(hrc))
+        hrc = logStringProps(RTLogRelDefaultInstance(), RTLogGetGroupSettings, "RTLogGetGroupSettings", a_pbstrSettings);
+
+    return hrc;
+}
+
+STDMETHODIMP MachineDebugger::COMGETTER(LogRelDestinations)(BSTR *a_pbstrSettings)
+{
+    CheckComArgOutPointerValid(a_pbstrSettings);
+
+    AutoCaller autoCaller(this);
+    HRESULT hrc = autoCaller.rc();
+    if (SUCCEEDED(hrc))
+        hrc = logStringProps(RTLogRelDefaultInstance(), RTLogGetDestinations, "RTLogGetDestinations", a_pbstrSettings);
+
+    return hrc;
 }