Add support for uncompressed video instead of H.264 (while still storing H.264 to...

[nageru] / mixer.cpp

diff --git a/mixer.cpp b/mixer.cpp
index 2802329b136c60c69b9f760ce812150bb7fa49a7..0817ff9a15b49829c27bb7e80346d25faf52256f 100644 (file)
--- a/mixer.cpp
+++ b/mixer.cpp
@@ -114,16 +114,20 @@ string generate_local_dump_filename(int frame)
 void QueueLengthPolicy::update_policy(int queue_length)
 {
        if (queue_length < 0) {  // Starvation.
-               if (safe_queue_length < 5) {
+               if (been_at_safe_point_since_last_starvation && safe_queue_length < 5) {
                        ++safe_queue_length;
                        fprintf(stderr, "Card %u: Starvation, increasing safe limit to %u frames\n",
                                card_index, safe_queue_length);
                }
                frames_with_at_least_one = 0;
+               been_at_safe_point_since_last_starvation = false;
                return;
        }
        if (queue_length > 0) {
-               if (++frames_with_at_least_one >= 50) {
+               if (queue_length >= int(safe_queue_length)) {
+                       been_at_safe_point_since_last_starvation = true;
+               }
+               if (++frames_with_at_least_one >= 50 && safe_queue_length > 0) {
                        --safe_queue_length;
                        fprintf(stderr, "Card %u: Spare frames for more than 50 frames, reducing safe limit to %u frames\n",
                                card_index, safe_queue_length);
@@ -473,7 +477,6 @@ void Mixer::bm_frame(unsigned card_index, uint16_t timecode,
                        new_frame.frame = RefCountedFrame(FrameAllocator::Frame());
                        new_frame.length = frame_length;
                        new_frame.interlaced = false;
-                       new_frame.ready_fence = nullptr;
                        new_frame.dropped_frames = dropped_frames;
                        card->new_frames.push(move(new_frame));
                        card->new_frames_changed.notify_all();
@@ -530,9 +533,9 @@ void Mixer::bm_frame(unsigned card_index, uint16_t timecode,
 
                GLuint pbo = userdata->pbo;
                check_error();
-               glBindBuffer(GL_PIXEL_UNPACK_BUFFER_ARB, pbo);
+               glBindBuffer(GL_PIXEL_UNPACK_BUFFER, pbo);
                check_error();
-               glMemoryBarrier(GL_CLIENT_MAPPED_BUFFER_BARRIER_BIT);
+               glFlushMappedBufferRange(GL_PIXEL_UNPACK_BUFFER, 0, video_frame.size);
                check_error();
 
                glBindTexture(GL_TEXTURE_2D, userdata->tex_cbcr[field]);
@@ -545,11 +548,11 @@ void Mixer::bm_frame(unsigned card_index, uint16_t timecode,
                check_error();
                glBindTexture(GL_TEXTURE_2D, 0);
                check_error();
-               glBindBuffer(GL_PIXEL_UNPACK_BUFFER_ARB, 0);
+               glBindBuffer(GL_PIXEL_UNPACK_BUFFER, 0);
                check_error();
-               GLsync fence = glFenceSync(GL_SYNC_GPU_COMMANDS_COMPLETE, /*flags=*/0);
+               RefCountedGLsync fence(GL_SYNC_GPU_COMMANDS_COMPLETE, /*flags=*/0);
                check_error();
-               assert(fence != nullptr);
+               assert(fence.get() != nullptr);
 
                if (field == 1) {
                        // Don't upload the second field as fast as we can; wait until
@@ -647,9 +650,9 @@ void Mixer::thread_func()
                        // The new texture might still be uploaded,
                        // tell the GPU to wait until it's there.
                        if (new_frame->ready_fence) {
-                               glWaitSync(new_frame->ready_fence, /*flags=*/0, GL_TIMEOUT_IGNORED);
+                               glWaitSync(new_frame->ready_fence.get(), /*flags=*/0, GL_TIMEOUT_IGNORED);
                                check_error();
-                               glDeleteSync(new_frame->ready_fence);
+                               new_frame->ready_fence.reset();
                                check_error();
                        }
                }