]> granicus.if.org Git - libvpx/commitdiff
Fix rare hang in multi-thread encoder on Windows
authorYunqing Wang <yunqingwang@google.com>
Thu, 5 May 2011 14:42:29 +0000 (10:42 -0400)
committerYunqing Wang <yunqingwang@google.com>
Thu, 5 May 2011 14:42:29 +0000 (10:42 -0400)
This patch is to fix a rare hang in multi-thread encoder that was
only seen on Windows. Thanks for John's help in debugging the
problem. More test is needed.

Change-Id: Idb11c6d344c2082362a032b34c5a602a1eea62fc

vp8/encoder/ethreading.c
vp8/encoder/onyx_if.c
vp8/encoder/onyx_int.h

index c00494dcff1ecdcdb8e34f19f845ae797f95e3a2..c92a366e8145d2fc31b907a4bf3de65418f87c76 100644 (file)
@@ -514,6 +514,7 @@ void vp8cx_create_encoder_threads(VP8_COMP *cpi)
             LPFTHREAD_DATA * lpfthd = &cpi->lpf_thread_data;
 
             sem_init(&cpi->h_event_start_lpf, 0, 0);
+            sem_init(&cpi->h_event_end_picklpf, 0, 0);
             sem_init(&cpi->h_event_end_lpf, 0, 0);
 
             lpfthd->ptr1 = (void *)cpi;
@@ -547,6 +548,7 @@ void vp8cx_remove_encoder_threads(VP8_COMP *cpi)
 
         sem_destroy(&cpi->h_event_end_encoding);
         sem_destroy(&cpi->h_event_end_lpf);
+        sem_destroy(&cpi->h_event_end_picklpf);
         sem_destroy(&cpi->h_event_start_lpf);
 
         //free thread related resources
index 1738e569986c148380b76dfc54e3941213a1734e..78f86557d1b52deb51ae66b32571ad4d7f0a26bb 100644 (file)
@@ -3140,7 +3140,7 @@ void loopfilter_frame(VP8_COMP *cpi, VP8_COMMON *cm)
 
 #if CONFIG_MULTITHREAD
     if (cpi->b_multi_threaded)
-        sem_post(&cpi->h_event_end_lpf); /* signal that we have set filter_level */
+        sem_post(&cpi->h_event_end_picklpf); /* signal that we have set filter_level */
 #endif
 
     if (cm->filter_level > 0)
@@ -4135,7 +4135,7 @@ static void encode_frame_to_data_rate
 #if CONFIG_MULTITHREAD
     /* wait that filter_level is picked so that we can continue with stream packing */
     if (cpi->b_multi_threaded)
-        sem_wait(&cpi->h_event_end_lpf);
+        sem_wait(&cpi->h_event_end_picklpf);
 #endif
 
     // build the bitstream
index e2e6b367c64758d8ffc4ece7c40aca6b14f3e08b..c2fcff88cdf461fcbd9631512555dcc1c5831bcc 100644 (file)
@@ -590,6 +590,7 @@ typedef struct
     sem_t *h_event_start_encoding;
     sem_t h_event_end_encoding;
     sem_t h_event_start_lpf;
+    sem_t h_event_end_picklpf;
     sem_t h_event_end_lpf;
 #endif