Changeset 101753 in vbox for trunk/src/VBox

Timestamp:

Nov 3, 2023 3:57:39 PM (13 months ago)

Author:

vboxsync

Message:

added more detailed logging, tstRTSg.cpp testcase

Location:

trunk/src/VBox/Runtime

Files:

• common/misc/sg.cpp (modified) (1 diff)
• testcase/tstRTSg.cpp (modified) (5 diffs)

trunk/src/VBox/Runtime/common/misc/sg.cpp

@@ -1 +1 @@
 /* $Id$ */
 /** @file
- * IPRT - S/G buffer handling.
+ * IPRT - Scatter / Gather buffer handling.
  */
 

trunk/src/VBox/Runtime/testcase/tstRTSg.cpp

@@ -1 +1 @@
 /* $Id$ */
 /** @file
- * IPRT Testcase - S/G Buffers.
+ * IPRT Testcase - Scatter / Gather Buffers.
  */
 
@@ -198 +198 @@
             {
                 size_t const cbInitial = (size_t)RTRandU64Ex(iRun, cbSgBuf1);
-                RTTESTI_CHECK(RTSgBufAdvance(pSgBuf1, cbInitial) == cbInitial);
+                size_t cbAdvanced = RTSgBufAdvance(pSgBuf1, cbInitial);
+                RTTESTI_CHECK_MSG(cbAdvanced == cbInitial, ("cbAdvanced=%zu, cbInitial=%zu\n", cbAdvanced, cbInitial));
+                /* should probably print part of pSgBuf1 values... */
                 cbLeft -= cbInitial;
             }
             for (;;)
             {
-                RTTESTI_CHECK(RTSgBufCalcLengthLeft(pSgBuf1) == cbLeft);
+                RTTESTI_CHECK_MSG(RTSgBufCalcLengthLeft(pSgBuf1) == cbLeft, ("pSgBuf1_calcLen=%zu, cbLeft=%zu\n",
+                                                                             RTSgBufCalcLengthLeft(pSgBuf1), cbLeft));
                 size_t cbThisSeg = 0;
                 void *pvThisSeg = RTSgBufGetNextSegment(pSgBuf1, &cbThisSeg);
@@ -212 +215 @@
                 cbLeft -= cbThisSeg;
             }
-            RTTESTI_CHECK(cbLeft == 0);
+            RTTESTI_CHECK_MSG(cbLeft == 0, ("cbLeft=%zu\n", cbLeft));
             RTTESTI_CHECK(RTSgBufIsAtEnd(pSgBuf1));
             RTTESTI_CHECK(RTSgBufCalcLengthLeft(pSgBuf1) == 0);
@@ -234 +237 @@
                 {
                     cbInitial = (size_t)RTRandU64Ex(iRun, cbSgBuf1);
-                    RTTESTI_CHECK(RTSgBufAdvance(pSgBuf1, cbInitial) == cbInitial);
+                    size_t cbAdvanced = RTSgBufAdvance(pSgBuf1, cbInitial);
+                    RTTESTI_CHECK_MSG(cbAdvanced == cbInitial, ("cbAdvanced=%zu, cbInitial=%zu\n",
+                                                                cbAdvanced, cbInitial));
                     cbLeft -= cbInitial;
                 }
@@ -249 +254 @@
                 RTTESTI_CHECK(cbCopied <= cbLeft);
                 RTTESTI_CHECK(cbCopied <= cbToCopy);
-                RTTESTI_CHECK(cbCopied == RT_MIN(cbToCopy, cbLeft));
+                RTTESTI_CHECK_MSG(cbCopied == RT_MIN(cbToCopy, cbLeft), ("cbCopied=%zu, cbToCopy=%zu, cbLeft=%zu\n",
+                                                                         cbCopied, cbToCopy, cbLeft));
                 RTTESTI_CHECK(memcmp(&pbSrc[cbInitial], pbDst, cbCopied) == 0);
 
-                RTTESTI_CHECK(RTSgBufCalcLengthLeft(pSgBuf1) == cbLeft - cbCopied);
+                RTTESTI_CHECK_MSG(RTSgBufCalcLengthLeft(pSgBuf1) == cbLeft - cbCopied,
+                                  ("pSgBuf1_calcLen=%zu cbLeft=%zu cbCopied=%zu\n",
+                                   RTSgBufCalcLengthLeft(pSgBuf1), cbLeft, cbCopied));
             }