Changeset 820

Timestamp:

02/29/08 20:37:26 (5 months ago)

Author:

astrange

Message:

Write frame-dropping stats to the log if logging is on.

Files:

• trunk/FFusionCodec.c (modified) (10 diffs)

trunk/FFusionCodec.c

@@ -91 +91 @@
-};
-
+struct per_frame_decode_stats
+{
+        unsigned                begin_calls;
+        unsigned                decode_calls;
+        unsigned                draw_calls;
+        unsigned                end_calls;
+};
+
+struct decode_stats
+{
+        struct per_frame_decode_stats type[4];
+        int             max_frames_begun;
+        int             max_frames_decoded;
+};
+
-typedef struct
-{
@@ -111 +126 @@
-        FFusionData             data;
-        struct decode_glob      decode;
+        struct decode_stats stats;
-} FFusionGlobalsRecord, *FFusionGlobals;
-
@@ -168 +184 @@
-#include <QuickTime/ComponentDispatchHelper.c>
-
-
+static 
-{
-        FSRef *ref = (FSRef *)args;
@@ -210 +226 @@
-        pthread_t thread;
-        pthread_create(&thread, NULL, launchUpdateChecker, updateCheckRef);
+}
+
+static void RecomputeMaxCounts(FFusionGlobals glob)
+{
+        int i;
+        unsigned begun = 0, decoded = 0, ended = 0, drawn = 0;
+        
+        for (i = 0; i < 4; i++) {
+                struct per_frame_decode_stats *f = &glob->stats.type[i];
+                
+                begun += f->begin_calls;
+                decoded += f->decode_calls;
+                drawn += f->draw_calls;
+                ended += f->end_calls;
+        }
+        
+        signed begin_diff = begun - ended, decode_diff = decoded - drawn;
+        
+        if (abs(begin_diff) > glob->stats.max_frames_begun) glob->stats.max_frames_begun = begin_diff;
+        if (abs(decode_diff) > glob->stats.max_frames_decoded) glob->stats.max_frames_decoded = decode_diff;
+}
+
+static void DumpFrameDropStats(FFusionGlobals glob)
+{
+        static const char types[4] = {'?', 'I', 'P', 'B'};
+        int i;
+        
+        if (!glob->fileLog || glob->decode.lastFrame == 0) return;
+        
+        Codecprintf(glob->fileLog, "Type\t| BeginBand\t| DecodeBand\t| DrawBand\t| dropped before decode\t| dropped before draw\n");
+        
+        for (i = 0; i < 4; i++) {
+                struct per_frame_decode_stats *f = &glob->stats.type[i];
+                                
+                Codecprintf(glob->fileLog, "%c\t| %d\t\t| %d\t\t| %d\t\t| %d/%f%%\t\t| %d/%f%%\n", types[i], f->begin_calls, f->decode_calls, f->draw_calls,
+                                        f->begin_calls - f->decode_calls,(f->begin_calls > f->decode_calls) ? ((float)(f->begin_calls - f->decode_calls)/(float)f->begin_calls) * 100. : 0.,
+                                        f->decode_calls - f->draw_calls,(f->decode_calls > f->draw_calls) ? ((float)(f->decode_calls - f->draw_calls)/(float)f->decode_calls) * 100. : 0.);
+        }
-}
-
@@ -316 +370 @@
-{
-    FFusionDebugPrint("%p closed.\n", glob);
+        DumpFrameDropStats(glob);
-
-    // Make sure to close the base component and deallocate our storage
@@ -436 +491 @@
-                cap->baseCodecShouldCallDecodeBandForAllFrames = true;
-                cap->subCodecSupportsScheduledBackwardsPlaybackWithDifferenceFrames = !doExperimentalFlags;
-                
-        //  XXX enabling this seems to cause rare visible artifacts in h.264?
-                cap->subCodecSupportsDrawInDecodeOrder = doExperimentalFlags; 
-                cap->subCodecSupportsDecodeSmoothing = true; 
@@ -1023 +1076 @@
-        myDrp->GOPStartFrameNumber = glob->begin.lastIFrame;
-        
+        glob->stats.type[drp->frameType].begin_calls++;
+        RecomputeMaxCounts(glob);
-        FFusionDebugPrint("%p BeginBand: frame #%d type %d. (%sskippable)\n", glob, myDrp->frameNumber, type, not(skippable));
+        
-    return noErr;
-}
@@ -1050 +1106 @@
-    FFusionDecompressRecord *myDrp = (FFusionDecompressRecord *)drp->userDecompressRecord;
-        
+        glob->stats.type[drp->frameType].decode_calls++;
+        RecomputeMaxCounts(glob);
-        FFusionDebugPrint("%p DecodeBand #%d qtType %d. (packed %d)\n", glob, myDrp->frameNumber, drp->frameType, glob->packedType);
-
@@ -1178 +1236 @@
-        int i, j;
-        
+        glob->stats.type[drp->frameType].draw_calls++;
+        RecomputeMaxCounts(glob);
-        FFusionDebugPrint("%p DrawBand #%d. (%sdecoded)\n", glob, myDrp->frameNumber, not(myDrp->decoded));
-        
@@ -1271 +1331 @@
-{
-        FFusionDecompressRecord *myDrp = (FFusionDecompressRecord *)drp->userDecompressRecord;
+        glob->stats.type[drp->frameType].end_calls++;
-        FFusionDebugPrint("%p EndBand #%d.\n", glob, myDrp->frameNumber);
+        
-    return noErr;
-}