summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--libhb/fifo.c15
-rw-r--r--libhb/internal.h1
-rw-r--r--libhb/muxcommon.c24
3 files changed, 32 insertions, 8 deletions
diff --git a/libhb/fifo.c b/libhb/fifo.c
index 42af6d9eb..7b5660673 100644
--- a/libhb/fifo.c
+++ b/libhb/fifo.c
@@ -196,6 +196,21 @@ void hb_buffer_realloc( hb_buffer_t * b, int size )
}
}
+void hb_buffer_reduce( hb_buffer_t * b, int size )
+{
+ if ( size < b->alloc / 8 || b->data == NULL )
+ {
+ uint32_t orig = b->alloc;
+ size = size_to_pool( size )->buffer_size;
+ b->data = realloc( b->data, size );
+ b->alloc = size;
+
+ hb_lock(buffers.lock);
+ buffers.allocated += size - orig;
+ hb_unlock(buffers.lock);
+ }
+}
+
// Frees the specified buffer list.
void hb_buffer_close( hb_buffer_t ** _b )
{
diff --git a/libhb/internal.h b/libhb/internal.h
index ee6bdccb2..00a7b3eb9 100644
--- a/libhb/internal.h
+++ b/libhb/internal.h
@@ -111,6 +111,7 @@ void hb_buffer_pool_free( void );
hb_buffer_t * hb_buffer_init( int size );
void hb_buffer_realloc( hb_buffer_t *, int size );
+void hb_buffer_reduce( hb_buffer_t * b, int size );
void hb_buffer_close( hb_buffer_t ** );
void hb_buffer_copy_settings( hb_buffer_t * dst,
const hb_buffer_t * src );
diff --git a/libhb/muxcommon.c b/libhb/muxcommon.c
index 98a978d0d..39f56dc6b 100644
--- a/libhb/muxcommon.c
+++ b/libhb/muxcommon.c
@@ -6,6 +6,7 @@
#include "hb.h"
+#define MIN_BUFFERING (1024*1024*10)
#define MAX_BUFFERING (1024*1024*50)
struct hb_mux_object_s
@@ -48,6 +49,7 @@ typedef struct
// allocated but the eof & rdy logic has to
// be changed to handle more than 32 tracks
// anyway so we keep it simple and fast.
+ int buffered_size;
} hb_mux_t;
struct hb_work_private_s
@@ -120,6 +122,7 @@ static void mf_push( hb_mux_t * mux, int tk, hb_buffer_t *buf )
uint32_t mask = track->mf.flen - 1;
uint32_t in = track->mf.in;
+ hb_buffer_reduce( buf, buf->size );
if ( track->buffered_size > MAX_BUFFERING )
{
mux->rdy = mux->allRdy;
@@ -152,11 +155,13 @@ static void mf_push( hb_mux_t * mux, int tk, hb_buffer_t *buf )
}
track->mf.fifo[in & mask] = buf;
track->mf.in = in + 1;
- track->buffered_size += buf->alloc;
+ track->buffered_size += buf->size;
+ mux->buffered_size += buf->size;
}
-static hb_buffer_t *mf_pull( hb_track_t *track )
+static hb_buffer_t *mf_pull( hb_mux_t * mux, int tk )
{
+ hb_track_t *track =mux->track[tk];
hb_buffer_t *b = NULL;
if ( track->mf.out != track->mf.in )
{
@@ -164,7 +169,8 @@ static hb_buffer_t *mf_pull( hb_track_t *track )
b = track->mf.fifo[track->mf.out & (track->mf.flen - 1)];
++track->mf.out;
- track->buffered_size -= b->alloc;
+ track->buffered_size -= b->size;
+ mux->buffered_size -= b->size;
}
return b;
}
@@ -190,13 +196,14 @@ static void MoveToInternalFifos( int tk, hb_mux_t *mux, hb_buffer_t * buf )
}
}
-static void OutputTrackChunk( hb_mux_t *mux, hb_track_t *track, hb_mux_object_t *m )
+static void OutputTrackChunk( hb_mux_t *mux, int tk, hb_mux_object_t *m )
{
+ hb_track_t *track = mux->track[tk];
hb_buffer_t *buf;
while ( ( buf = mf_peek( track ) ) != NULL && buf->start < mux->pts )
{
- buf = mf_pull( track );
+ buf = mf_pull( mux, tk );
track->frames += 1;
track->bytes += buf->size;
m->mux( m, track->mux_data, buf );
@@ -247,14 +254,15 @@ static int muxWork( hb_work_object_t * w, hb_buffer_t ** buf_in,
int more = mux->rdy;
// all tracks have at least 'interleave' ticks of data. Output
// all that we can in 'interleave' size chunks.
- while ( (( mux->rdy & mux->allRdy ) == mux->allRdy && more) ||
+ while ( (( mux->rdy & mux->allRdy ) == mux->allRdy &&
+ more && mux->buffered_size > MIN_BUFFERING ) ||
( mux->eof == mux->allEof ) )
{
more = 0;
for ( i = 0; i < mux->ntracks; ++i )
{
track = mux->track[i];
- OutputTrackChunk( mux, track, mux->m );
+ OutputTrackChunk( mux, i, mux->m );
if ( mf_full( track ) )
{
// If the track's fifo is still full, advance
@@ -375,7 +383,7 @@ void muxClose( hb_work_object_t * w )
{
hb_buffer_t * b;
track = mux->track[i];
- while ( (b = mf_pull( track )) != NULL )
+ while ( (b = mf_pull( mux, i )) != NULL )
{
hb_buffer_close( &b );
}