Changeset 86666 in vbox for trunk/src/VBox/VMM/include

Timestamp:

Oct 21, 2020 3:01:32 PM (5 years ago)

Author:

vboxsync

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

141042

Message:

include/VBox,VMM,DBGF: Some boilerplate for the new breakpoint manager which is disabled by default (can be built with VBOX_WITH_LOTS_OF_DBGF_BPS), bugref:9837

File:

• trunk/src/VBox/VMM/include/DBGFInternal.h (modified) (8 diffs)

trunk/src/VBox/VMM/include/DBGFInternal.h

@@ -747 +747 @@
 
 
+#ifndef VBOX_WITH_LOTS_OF_DBGF_BPS
 /**
  * Breakpoint search optimization.
@@ -761 +762 @@
 /** Pointer to a breakpoint search optimziation structure. */
 typedef DBGFBPSEARCHOPT *PDBGFBPSEARCHOPT;
+#else
+/**
+ * Hardware breakpoint state.
+ */
+typedef struct DBGFBPHW
+{
+    /** The flat GC address of the breakpoint. */
+    RTGCUINTPTR     GCPtr;
+    /** The breakpoint handle if active, NIL_DBGFBP if disabled. */
+    DBGFBP          hBp;
+    /** The access type (one of the X86_DR7_RW_* value). */
+    uint8_t         fType;
+    /** The access size. */
+    uint8_t         cb;
+    /** Flag whether the breakpoint is currently enabled. */
+    bool            fEnabled;
+    /** Padding. */
+    uint8_t         bPad;
+} DBGFBPHW;
+AssertCompileSize(DBGFBPHW, 16);
+/** Pointer to a hardware breakpoint state. */
+typedef DBGFBPHW *PDBGFBPHW;
+/** Pointer to a const hardware breakpoint state. */
+typedef const DBGFBPHW *PCDBGFBPHW;
+#endif
 
 
@@ -790 +816 @@
     /** The number of enabled hardware I/O breakpoints. */
     uint8_t                     cEnabledHwIoBreakpoints;
+#ifndef VBOX_WITH_LOTS_OF_DBGF_BPS
     /** The number of enabled INT3 breakpoints. */
     uint8_t                     cEnabledInt3Breakpoints;
     uint8_t                     abPadding; /**< Unused padding space up for grabs. */
+#else
+    uint16_t                    u16Pad;
+#endif
     uint32_t                    uPadding;
 
@@ -830 +860 @@
     uint32_t                    u32Padding[2]; /**< Alignment padding. */
 
+#ifndef VBOX_WITH_LOTS_OF_DBGF_BPS
     /** Array of hardware breakpoints. (0..3)
      * This is shared among all the CPUs because life is much simpler that way. */
@@ -843 +874 @@
     /** INT3 breakpoint search optimizations. */
     DBGFBPSEARCHOPT             Int3;
+#else
+    /** Array of hardware breakpoints (0..3).
+     * This is shared among all the CPUs because life is much simpler that way. */
+    DBGFBPHW                    aHwBreakpoints[4];
+#endif
 
     /**
@@ -907 +943 @@
     uint32_t                offVM;
 
+#ifndef VBOX_WITH_LOTS_OF_DBGF_BPS
     /** Current active breakpoint (id).
      * This is ~0U if not active. It is set when a execution engine
@@ -915 +952 @@
      * @todo drop this in favor of aEvents!  */
     uint32_t                iActiveBp;
+#else
+    /** Current active breakpoint handle.
+     * This is NIL_DBGFBP if not active. It is set when a execution engine
+     * encounters a breakpoint and returns VINF_EM_DBG_BREAKPOINT.
+     *
+     * @todo drop this in favor of aEvents!  */
+    DBGFBP                  hBpActive;
+#endif
     /** Set if we're singlestepping in raw mode.
      * This is checked and cleared in the \#DB handler. */
@@ -1092 +1137 @@
 void dbgfR3AsTerm(PUVM pUVM);
 void dbgfR3AsRelocate(PUVM pUVM, RTGCUINTPTR offDelta);
+#ifdef VBOX_WITH_LOTS_OF_DBGF_BPS
+DECLHIDDEN(int) dbgfR3BpInit(PVM pVM);
+DECLHIDDEN(int) dbgfR3BpTerm(PVM pVM);
+#else
 int  dbgfR3BpInit(PVM pVM);
+#endif
 int  dbgfR3InfoInit(PUVM pUVM);
 int  dbgfR3InfoTerm(PUVM pUVM);