X-Git-Url: http://git.osdn.jp/view?a=blobdiff_plain;f=libhb%2Freader.c;h=c93c2a0ccc2d022b3edd2743267b50aab0bfcade;hb=533776bbad20db93fe964bc69975f108b2a30888;hp=9ce2a806af28c1cbccfe43696aa1511bd2d6a32e;hpb=13b046adca50f4a130b567bdcfadbddf5fd76cc9;p=handbrake-jp%2Fhandbrake-jp-git.git diff --git a/libhb/reader.c b/libhb/reader.c index 9ce2a806..c93c2a0c 100644 --- a/libhb/reader.c +++ b/libhb/reader.c @@ -8,17 +8,27 @@ typedef struct { + double average; // average time between packets + int64_t last; // last timestamp seen on this stream + int id; // stream id +} stream_timing_t; + +typedef struct +{ hb_job_t * job; hb_title_t * title; volatile int * die; hb_dvd_t * dvd; - hb_buffer_t * ps; hb_stream_t * stream; + stream_timing_t *stream_timing; + int64_t scr_offset; hb_psdemux_t demux; + int scr_changes; uint sequence; int saw_video; + int st_slots; // size (in slots) of stream_timing array } hb_reader_t; /*********************************************************************** @@ -43,31 +53,24 @@ hb_thread_t * hb_reader_init( hb_job_t * job ) r->die = job->die; r->sequence = 0; - /* - * when the scr changes we need to base the timing offset change on the - * end of the current video frame otherwise we'll map the first frame - * following the change over the current frame & it will be discarded. - * Since the PTS only gives the start of the frame we need the average - * frame duration to get its end. See the comments in the init_delay - * setup in libhb/encx264.c to understand the following code. - */ - r->demux.frame_duration = 90000. * (double)job->vrate_base / (double)job->vrate; - if ( r->demux.frame_duration == 3753 ) - { - r->demux.frame_duration = 4506; - } - r->demux.frame_duration += 3; + r->st_slots = 4; + r->stream_timing = calloc( sizeof(stream_timing_t), r->st_slots ); + r->stream_timing[0].id = r->title->video_id; + r->stream_timing[0].average = 90000. * (double)job->vrate_base / + (double)job->vrate; + r->stream_timing[0].last = -r->stream_timing[0].average; + r->stream_timing[1].id = -1; return hb_thread_init( "reader", ReaderFunc, r, HB_NORMAL_PRIORITY ); } -static void push_buf( hb_reader_t *r, hb_fifo_t *fifo, hb_buffer_t *buf ) +static void push_buf( const hb_reader_t *r, hb_fifo_t *fifo, hb_buffer_t *buf ) { while( !*r->die && !r->job->done && hb_fifo_is_full( fifo ) ) { /* - * Loop until the incoming fifo is reaqdy to receive + * Loop until the incoming fifo is ready to receive * this buffer. */ hb_snooze( 50 ); @@ -75,6 +78,84 @@ static void push_buf( hb_reader_t *r, hb_fifo_t *fifo, hb_buffer_t *buf ) hb_fifo_push( fifo, buf ); } +// The MPEG STD (Standard Target Decoder) essentially requires that we keep +// per-stream timing so that when there's a timing discontinuity we can +// seemlessly join packets on either side of the discontinuity. This join +// requires that we know the timestamp of the previous packet and the +// average inter-packet time (since we position the new packet at the end +// of the previous packet). The next four routines keep track of this +// per-stream timing. + +// find the per-stream timing state for 'buf' + +static stream_timing_t *find_st( hb_reader_t *r, const hb_buffer_t *buf ) +{ + stream_timing_t *st = r->stream_timing; + for ( ; st->id != -1; ++st ) + { + if ( st->id == buf->id ) + return st; + } + return NULL; +} + +// find or create the per-stream timing state for 'buf' + +static stream_timing_t *id_to_st( hb_reader_t *r, const hb_buffer_t *buf ) +{ + stream_timing_t *st = r->stream_timing; + while ( st->id != buf->id && st->id != -1) + { + ++st; + } + // if we haven't seen this stream add it. + if ( st->id == -1 ) + { + // we keep the steam timing info in an array with some power-of-two + // number of slots. If we don't have two slots left (one for our new + // entry plus one for the "-1" eol) we need to expand the array. + int slot = st - r->stream_timing; + if ( slot + 1 >= r->st_slots ) + { + r->st_slots *= 2; + r->stream_timing = realloc( r->stream_timing, r->st_slots * + sizeof(*r->stream_timing) ); + st = r->stream_timing + slot; + } + st->id = buf->id; + st->average = 30.*90.; + st->last = buf->renderOffset - st->average; + + st[1].id = -1; + } + return st; +} + +// update the average inter-packet time of the stream associated with 'buf' +// using a recursive low-pass filter with a 16 packet time constant. + +static void update_ipt( hb_reader_t *r, const hb_buffer_t *buf ) +{ + stream_timing_t *st = id_to_st( r, buf ); + double dt = buf->renderOffset - st->last; + st->average += ( dt - st->average ) * (1./32.); + st->last = buf->renderOffset; +} + +// use the per-stream state associated with 'buf' to compute a new scr_offset +// such that 'buf' will follow the previous packet of this stream separated +// by the average packet time of the stream. + +static void new_scr_offset( hb_reader_t *r, hb_buffer_t *buf ) +{ + stream_timing_t *st = id_to_st( r, buf ); + int64_t nxt = st->last + st->average; + r->scr_offset = buf->renderOffset - nxt; + buf->renderOffset = nxt; + r->scr_changes = r->demux.scr_changes; + st->last = buf->renderOffset; +} + /*********************************************************************** * ReaderFunc *********************************************************************** @@ -127,7 +208,8 @@ static void ReaderFunc( void * _r ) } list = hb_list_init(); - r->ps = hb_buffer_init( HB_DVD_READ_BUFFER_SIZE ); + hb_buffer_t *ps = hb_buffer_init( HB_DVD_READ_BUFFER_SIZE ); + r->demux.flaky_clock = r->title->flaky_clock; while( !*r->die && !r->job->done ) { @@ -150,14 +232,14 @@ static void ReaderFunc( void * _r ) if (r->dvd) { - if( !hb_dvd_read( r->dvd, r->ps ) ) + if( !hb_dvd_read( r->dvd, ps ) ) { break; } } else if (r->stream) { - if ( !hb_stream_read( r->stream, r->ps ) ) + if ( !hb_stream_read( r->stream, ps ) ) { break; } @@ -187,11 +269,11 @@ static void ReaderFunc( void * _r ) if ( r->title->demuxer == HB_NULL_DEMUXER ) { - hb_demux_null( r->ps, list, &r->demux ); + hb_demux_null( ps, list, &r->demux ); } else { - hb_demux_ps( r->ps, list, &r->demux ); + hb_demux_ps( ps, list, &r->demux ); } while( ( buf = hb_list_item( list, 0 ) ) ) @@ -202,13 +284,14 @@ static void ReaderFunc( void * _r ) if ( ! r->saw_video ) { /* The first video packet defines 'time zero' so discard - data until we get a video packet with a PTS */ - if ( buf->id == r->title->video_id && buf->start != -1 ) + data until we get a video packet with a PTS & DTS */ + if ( buf->id == r->title->video_id && buf->start != -1 && + buf->renderOffset != -1 ) { + // force a new scr offset computation + r->scr_changes = r->demux.scr_changes - 1; r->saw_video = 1; - r->demux.scr_offset = buf->renderOffset; - hb_log( "reader: first SCR %llu scr_offset %llu", - r->demux.last_scr, r->demux.scr_offset ); + hb_log( "reader: first SCR %lld", r->demux.last_scr ); } else { @@ -217,15 +300,59 @@ static void ReaderFunc( void * _r ) } if( fifos ) { - if ( buf->start != -1 ) + if ( buf->renderOffset != -1 ) { - /* this packet has a PTS - correct it for the initial - video time offset & any timing discontinuities so that - everything after this sees a continuous clock with 0 - being the time of the first video packet. */ - buf->start -= r->demux.scr_offset; - buf->renderOffset -= r->demux.scr_offset; + if ( r->scr_changes == r->demux.scr_changes ) + { + // This packet is referenced to the same SCR as the last. + // Adjust timestamp to remove the System Clock Reference + // offset then update the average inter-packet time + // for this stream. + buf->renderOffset -= r->scr_offset; + update_ipt( r, buf ); + } + else + { + // This is the first audio or video packet after an SCR + // change. Compute a new scr offset that would make this + // packet follow the last of this stream with the correct + // average spacing. + stream_timing_t *st = find_st( r, buf ); + + if ( st ) + { + // if this isn't the video stream or we don't + // have audio yet then generate a new scr + if ( st != r->stream_timing || + r->stream_timing[1].id == -1 ) + { + new_scr_offset( r, buf ); + } + else + { + // defer the scr change until we get some + // audio since audio has a timestamp per + // frame but video doesn't. Clear the timestamps + // so the decoder will regenerate them from + // the frame durations. + buf->start = -1; + buf->renderOffset = -1; + } + } + else + { + // we got a new scr at the same time as the first + // packet of a stream we've never seen before. We + // have no idea what the timing should be so toss + // this buffer & wait for a stream we've already seen. + hb_buffer_close( &buf ); + continue; + } + } } + if ( buf->start != -1 ) + buf->start -= r->scr_offset; + buf->sequence = r->sequence++; /* if there are mutiple output fifos, send a copy of the * buffer down all but the first (we have to not ship the @@ -247,22 +374,38 @@ static void ReaderFunc( void * _r ) } } - /* send an empty buffer upstream to signal we're done */ - hb_fifo_push( r->job->fifo_mpeg2, hb_buffer_init(0) ); + // send empty buffers downstream to video & audio decoders to signal we're done. + push_buf( r, r->job->fifo_mpeg2, hb_buffer_init(0) ); + + hb_audio_t *audio; + for( n = 0; ( audio = hb_list_item( r->job->title->list_audio, n ) ); ++n ) + { + if ( audio->priv.fifo_in ) + push_buf( r, audio->priv.fifo_in, hb_buffer_init(0) ); + } hb_list_empty( &list ); - hb_buffer_close( &r->ps ); + hb_buffer_close( &ps ); if (r->dvd) { - hb_dvd_stop( r->dvd ); - hb_dvd_close( &r->dvd ); + hb_dvd_stop( r->dvd ); + hb_dvd_close( &r->dvd ); } else if (r->stream) { - hb_stream_close(&r->stream); + hb_stream_close(&r->stream); + } + + if ( r->stream_timing ) + { + free( r->stream_timing ); } hb_log( "reader: done. %d scr changes", r->demux.scr_changes ); + if ( r->demux.dts_drops ) + { + hb_log( "reader: %d drops because DTS out of range", r->demux.dts_drops ); + } free( r ); _r = NULL;