XA: group ADPCM frames per ~50ms before sending to decoder
authorRémi Denis-Courmont <remi@remlab.net>
Sat, 5 Sep 2009 18:40:29 +0000 (21:40 +0300)
committerRémi Denis-Courmont <remi@remlab.net>
Sat, 5 Sep 2009 18:41:44 +0000 (21:41 +0300)
Sending one frame per block had way to high CPU overhead.

modules/demux/xa.c

index 65a3f2e..d1ffcc2 100644 (file)
@@ -62,6 +62,7 @@ struct demux_sys_t
 
     int64_t         i_data_offset;
     unsigned int    i_data_size;
 
     int64_t         i_data_offset;
     unsigned int    i_data_size;
+    unsigned int    i_block_frames;
 
     date_t          pts;
 };
 
     date_t          pts;
 };
@@ -79,7 +80,6 @@ typedef struct xa_header_t
     uint16_t wBitsPerSample;
 } xa_header_t;
 
     uint16_t wBitsPerSample;
 } xa_header_t;
 
-
 /*****************************************************************************
  * Open: check file and initializes structures
  *****************************************************************************/
 /*****************************************************************************
  * Open: check file and initializes structures
  *****************************************************************************/
@@ -129,6 +129,8 @@ static int Open( vlc_object_t * p_this )
     p_sys->i_data_offset = stream_Tell( p_demux->s );
     /* FIXME: better computation */
     p_sys->i_data_size = p_xa.iSize * 15 / 56;
     p_sys->i_data_offset = stream_Tell( p_demux->s );
     /* FIXME: better computation */
     p_sys->i_data_size = p_xa.iSize * 15 / 56;
+    /* How many frames per block (1:1 is too CPU intensive) */
+    p_sys->i_block_frames = p_sys->fmt.audio.i_rate / (28 * 20) + 1;
 
     msg_Dbg( p_demux, "fourcc: %4.4s, channels: %d, "
              "freq: %d Hz, bitrate: %dKo/s, blockalign: %d",
 
     msg_Dbg( p_demux, "fourcc: %4.4s, channels: %d, "
              "freq: %d Hz, bitrate: %dKo/s, blockalign: %d",
@@ -154,6 +156,7 @@ static int Demux( demux_t *p_demux )
     demux_sys_t *p_sys = p_demux->p_sys;
     block_t     *p_block;
     int64_t     i_offset;
     demux_sys_t *p_sys = p_demux->p_sys;
     block_t     *p_block;
     int64_t     i_offset;
+    unsigned    i_frames = p_sys->i_block_frames;
 
     i_offset = stream_Tell( p_demux->s );
 
 
     i_offset = stream_Tell( p_demux->s );
 
@@ -164,20 +167,20 @@ static int Demux( demux_t *p_demux )
         return 0;
     }
 
         return 0;
     }
 
-    p_block = stream_Block( p_demux->s, p_sys->fmt.audio.i_bytes_per_frame );
+    p_block = stream_Block( p_demux->s, p_sys->fmt.audio.i_bytes_per_frame *
+                            i_frames );
     if( p_block == NULL )
     {
         msg_Warn( p_demux, "cannot read data" );
         return 0;
     }
 
     if( p_block == NULL )
     {
         msg_Warn( p_demux, "cannot read data" );
         return 0;
     }
 
+    i_frames = p_block->i_buffer / p_sys->fmt.audio.i_bytes_per_frame;
     p_block->i_dts = p_block->i_pts =
     p_block->i_dts = p_block->i_pts =
-        date_Increment( &p_sys->pts, p_sys->fmt.audio.i_frame_length );
-
+        date_Increment( &p_sys->pts,
+                        i_frames * p_sys->fmt.audio.i_frame_length );
     es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block->i_pts );
     es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block->i_pts );
-
     es_out_Send( p_demux->out, p_sys->p_es, p_block );
     es_out_Send( p_demux->out, p_sys->p_es, p_block );
-
     return 1;
 }
 
     return 1;
 }