prototype change; it looks like we won't need it after all.
* ./src/audio_output/output.c: if audio output is too slow, but we
know there is no next packet, then we play it anyway. Rationale behind
this is: better play a slightly wrongly dated packet than play silence,
because it might be wrongly dated but there are chances it won't break
audio continuity.
* ./modules/audio_output/sdl.c: workaround for SDL's crap audio output
system (callback gets called at random times, and no way to know the
latency of the samples being played). Audio output is now rather good
here, but please give feedback.
* ./modules/audio_output/oss.c: we delay a bit before asking for a new
buffer to avoid starvation.
* aout_internal.h : internal defines for audio output
*****************************************************************************
* Copyright (C) 2002 VideoLAN
- * $Id: aout_internal.h,v 1.11 2002/08/24 10:19:42 sam Exp $
+ * $Id: aout_internal.h,v 1.12 2002/08/25 09:39:59 sam Exp $
*
* Authors: Christophe Massiot <massiot@via.ecp.fr>
*
audio_sample_format_t * p_format );
void aout_OutputPlay( aout_instance_t * p_aout, aout_buffer_t * p_buffer );
void aout_OutputDelete( aout_instance_t * p_aout );
-VLC_EXPORT( aout_buffer_t *, aout_OutputNextBuffer, ( aout_instance_t *, mtime_t, mtime_t, vlc_bool_t ) );
+VLC_EXPORT( aout_buffer_t *, aout_OutputNextBuffer, ( aout_instance_t *, mtime_t, vlc_bool_t ) );
void aout_FormatPrepare( audio_sample_format_t * p_format );
void aout_FifoInit( aout_instance_t *, aout_fifo_t *, u32 );
{
aout_buffer_t * (* aout_BufferNew_inner) ( aout_instance_t *, aout_input_t *, size_t ) ;
aout_buffer_t * (* aout_FifoPop_inner) ( aout_instance_t * p_aout, aout_fifo_t * p_fifo ) ;
- aout_buffer_t * (* aout_OutputNextBuffer_inner) ( aout_instance_t *, mtime_t, mtime_t, vlc_bool_t ) ;
+ aout_buffer_t * (* aout_OutputNextBuffer_inner) ( aout_instance_t *, mtime_t, vlc_bool_t ) ;
aout_input_t * (* __aout_InputNew_inner) ( vlc_object_t *, aout_instance_t **, audio_sample_format_t * ) ;
aout_instance_t * (* __aout_NewInstance_inner) ( vlc_object_t * ) ;
char * (* __config_GetPsz_inner) (vlc_object_t *, const char *) ;
* alsa.c : alsa plugin for vlc
*****************************************************************************
* Copyright (C) 2000-2001 VideoLAN
- * $Id: alsa.c,v 1.7 2002/08/24 10:19:42 sam Exp $
+ * $Id: alsa.c,v 1.8 2002/08/25 09:39:59 sam Exp $
*
* Authors: Henri Fallon <henri@videolan.org> - Original Author
* Jeffrey Baker <jwbaker@acm.org> - Port to ALSA 1.0 API
snd_pcm_status_get_tstamp( p_status, &ts_next );
next_date = (mtime_t)ts_next.tv_sec * 1000000 + ts_next.tv_usec;
- p_buffer = aout_OutputNextBuffer( p_aout, next_date, 0,
+ p_buffer = aout_OutputNextBuffer( p_aout, next_date,
p_sys->b_can_sleek );
/* Audio output buffer shortage -> stop the fill process and
/* Get the presentation date of the next write() operation. It
* is equal to the current date + latency */
p_buffer = aout_OutputNextBuffer( p_aout, mdate() + p_sys->latency,
- 0, VLC_FALSE );
+ VLC_FALSE );
if ( p_buffer != NULL )
{
* esd.c : EsounD module
*****************************************************************************
* Copyright (C) 2000, 2001 VideoLAN
- * $Id: esd.c,v 1.9 2002/08/24 10:19:42 sam Exp $
+ * $Id: esd.c,v 1.10 2002/08/25 09:39:59 sam Exp $
*
* Authors: Samuel Hocevar <sam@zoy.org>
*
/* Get the presentation date of the next write() operation. It
* is equal to the current date + buffered samples + esd latency */
p_buffer = aout_OutputNextBuffer( p_aout, mdate() + p_sys->latency,
- 0, VLC_FALSE );
+ VLC_FALSE );
if ( p_buffer != NULL )
{
* oss.c : OSS /dev/dsp module for vlc
*****************************************************************************
* Copyright (C) 2000-2002 VideoLAN
- * $Id: oss.c,v 1.17 2002/08/24 21:11:21 sam Exp $
+ * $Id: oss.c,v 1.18 2002/08/25 09:40:00 sam Exp $
*
* Authors: Michel Kaempf <maxx@via.ecp.fr>
* Samuel Hocevar <sam@zoy.org>
if ( p_aout->output.output.i_format != AOUT_FMT_SPDIF )
{
- mtime_t buffered = (mtime_t)GetBufInfo( p_aout ) * 1000000
- / p_aout->output.output.i_bytes_per_frame
- / p_aout->output.output.i_rate
- * p_aout->output.output.i_frame_length;
+ mtime_t buffered;
- while( buffered > 50000 )
+ do
{
- msleep( buffered / 2 - 10000 );
buffered = (mtime_t)GetBufInfo( p_aout ) * 1000000
/ p_aout->output.output.i_bytes_per_frame
/ p_aout->output.output.i_rate
* p_aout->output.output.i_frame_length;
- }
+ if( buffered < 50000 )
+ {
+ break;
+ }
+ msleep( buffered / 2 - 10000 );
+
+ } while( VLC_TRUE );
- /* Next buffer will be played at mdate()+buffered, and we tell
- * the audio output that it can wait for a new packet for
- * 20000 microseconds. */
+ /* Next buffer will be played at mdate()+buffered */
p_buffer = aout_OutputNextBuffer( p_aout, mdate() + buffered,
- 20000, VLC_FALSE );
+ VLC_FALSE );
}
else
{
- p_buffer = aout_OutputNextBuffer( p_aout, 0, 0, VLC_TRUE );
+ p_buffer = aout_OutputNextBuffer( p_aout, 0, VLC_TRUE );
}
if ( p_buffer != NULL )
* sdl.c : SDL audio output plugin for vlc
*****************************************************************************
* Copyright (C) 2000-2002 VideoLAN
- * $Id: sdl.c,v 1.5 2002/08/24 10:19:42 sam Exp $
+ * $Id: sdl.c,v 1.6 2002/08/25 09:40:00 sam Exp $
*
* Authors: Michel Kaempf <maxx@via.ecp.fr>
* Samuel Hocevar <sam@zoy.org>
#include SDL_INCLUDE_FILE
-#define FRAME_SIZE 2048*2
+#define FRAME_SIZE 2048
+
+/*****************************************************************************
+ * aout_sys_t: SDL audio output method descriptor
+ *****************************************************************************
+ * This structure is part of the audio output thread descriptor.
+ * It describes the specific properties of an audio device.
+ *****************************************************************************/
+struct aout_sys_t
+{
+ mtime_t call_time;
+ mtime_t buffer_time;
+};
/*****************************************************************************
* Local prototypes
static int Open ( vlc_object_t *p_this )
{
aout_instance_t *p_aout = (aout_instance_t *)p_this;
+ aout_sys_t * p_sys;
Uint32 i_flags = SDL_INIT_AUDIO;
if( SDL_WasInit( i_flags ) )
{
- return 1;
+ return VLC_EGENERIC;
+ }
+
+ /* Allocate structure */
+ p_aout->output.p_sys = p_sys = malloc( sizeof( aout_sys_t ) );
+ if( p_sys == NULL )
+ {
+ msg_Err( p_aout, "out of memory" );
+ return VLC_ENOMEM;
}
p_aout->output.pf_setformat = SetFormat;
if( SDL_Init( i_flags ) < 0 )
{
msg_Err( p_aout, "cannot initialize SDL (%s)", SDL_GetError() );
- return 1;
+ free( p_sys );
+ return VLC_EGENERIC;
}
- return 0;
+ return VLC_SUCCESS;
}
/*****************************************************************************
*****************************************************************************/
static int SetFormat( aout_instance_t *p_aout )
{
+ aout_sys_t * p_sys = p_aout->output.p_sys;
+
/* TODO: finish and clean this */
SDL_AudioSpec desired;
/* Open the sound device - FIXME : get the "natural" parameters */
if( SDL_OpenAudio( &desired, NULL ) < 0 )
{
- return -1;
+ return VLC_EGENERIC;
}
p_aout->output.output.i_format = AOUT_FMT_S16_NE;
p_aout->output.i_nb_samples = FRAME_SIZE;
+ p_sys->call_time = 0;
+ p_sys->buffer_time = (mtime_t)FRAME_SIZE * 1000000
+ / p_aout->output.output.i_rate;
+
SDL_PauseAudio( 0 );
- return 0;
+ return VLC_SUCCESS;
}
/*****************************************************************************
*****************************************************************************/
static void Close ( vlc_object_t *p_this )
{
+ aout_instance_t *p_aout = (aout_instance_t *)p_this;
+ aout_sys_t * p_sys = p_aout->output.p_sys;
+
SDL_PauseAudio( 1 );
SDL_CloseAudio();
SDL_QuitSubSystem( SDL_INIT_AUDIO );
+
+ free( p_sys );
}
/*****************************************************************************
static void SDLCallback( void * _p_aout, byte_t * p_stream, int i_len )
{
aout_instance_t * p_aout = (aout_instance_t *)_p_aout;
-//static mtime_t old = 0;
-//static mtime_t diff = 0;
-//mtime_t foo = mdate();
+ aout_sys_t * p_sys = p_aout->output.p_sys;
+
aout_buffer_t * p_buffer;
-//if(old) diff = (9 * diff + (foo-old))/10;
- /* FIXME : take into account SDL latency instead of mdate() */
- p_buffer = aout_OutputNextBuffer( p_aout, mdate(), 0, VLC_TRUE );
- //p_buffer = aout_OutputNextBuffer( p_aout, foo - diff, 0, VLC_TRUE );
-//fprintf(stderr, "foo - old : %lli, diff : %lli\n", foo-old, diff);
-//old=foo;
+
+ /* We try to stay around call_time + buffer_time/2. This is kludgy but
+ * unavoidable because SDL is completely unable to 1. tell us about its
+ * latency, and 2. call SDLCallback at regular intervals. */
+ if( mdate() < p_sys->call_time + p_sys->buffer_time / 2 )
+ {
+ /* We can't wait too much, because SDL will be lost, and we can't
+ * wait too little, because we are not sure that there will be
+ * samples in the queue. */
+ mwait( p_sys->call_time + p_sys->buffer_time / 4 );
+ p_sys->call_time += p_sys->buffer_time;
+ }
+ else
+ {
+ p_sys->call_time = mdate() + p_sys->buffer_time / 4;
+ }
+
+ /* Tell the output we're playing samples at call_time + 2*buffer_time */
+ p_buffer = aout_OutputNextBuffer( p_aout, p_sys->call_time
+ + 2 * p_sys->buffer_time, VLC_TRUE );
if ( i_len != FRAME_SIZE * sizeof(s16)
* p_aout->output.output.i_channels )
if ( p_buffer != NULL )
{
-//fprintf(stderr, "got buffer %lli\n", p_buffer->end_date - p_buffer->start_date);
-
p_aout->p_vlc->pf_memcpy( p_stream, p_buffer->p_buffer, i_len );
aout_BufferFree( p_buffer );
}
else
{
-//fprintf(stderr, "NO BUFFER !\n");
p_aout->p_vlc->pf_memset( p_stream, 0, i_len );
}
}
* waveout.c : Windows waveOut plugin for vlc
*****************************************************************************
* Copyright (C) 2001 VideoLAN
- * $Id: waveout.c,v 1.4 2002/08/19 21:31:11 massiot Exp $
+ * $Id: waveout.c,v 1.5 2002/08/25 09:40:00 sam Exp $
*
* Authors: Gildas Bazin <gbazin@netcourrier.com>
*
aout_BufferFree( (aout_buffer_t *)p_waveheader->dwUser );
/* FIXME : take into account WaveOut latency instead of mdate() */
- p_buffer = aout_OutputNextBuffer( p_aout, mdate(), 0 );
+ p_buffer = aout_OutputNextBuffer( p_aout, mdate(), VLC_FALSE );
PlayWaveOut( p_aout, h_waveout, p_waveheader, p_buffer );
}
* aout.m: CoreAudio output plugin
*****************************************************************************
* Copyright (C) 2002 VideoLAN
- * $Id: aout.m,v 1.6 2002/08/24 10:19:42 sam Exp $
+ * $Id: aout.m,v 1.7 2002/08/25 09:40:00 sam Exp $
*
* Authors: Colin Delacroix <colin@zoy.org>
* Jon Lech Johansen <jon-vl@nanocrew.net>
current_date = p_sys->clock_diff
+ AudioConvertHostTimeToNanos(host_time.mHostTime) / 1000;
- p_buffer = aout_OutputNextBuffer( p_aout, current_date, 0, VLC_FALSE );
+ p_buffer = aout_OutputNextBuffer( p_aout, current_date, VLC_FALSE );
/* move data into output data buffer */
if ( p_buffer != NULL )
* p_aout->output.output.i_frame_length;
next_date += mdate();
- p_buffer = aout_OutputNextBuffer( p_aout, next_date, 0, VLC_FALSE );
+ p_buffer = aout_OutputNextBuffer( p_aout, next_date, VLC_FALSE );
}
else
{
- p_buffer = aout_OutputNextBuffer( p_aout, 0, 0, VLC_TRUE );
+ p_buffer = aout_OutputNextBuffer( p_aout, 0, VLC_TRUE );
}
if ( p_buffer != NULL )
* aout.c: Windows DirectX audio output method
*****************************************************************************
* Copyright (C) 2001 VideoLAN
- * $Id: aout.c,v 1.6 2002/08/24 10:19:42 sam Exp $
+ * $Id: aout.c,v 1.7 2002/08/25 09:40:00 sam Exp $
*
* Authors: Gildas Bazin <gbazin@netcourrier.com>
*
}
/* FIXME : take into account DirectSound latency instead of mdate() */
- p_buffer = aout_OutputNextBuffer( p_aout, mdate(), 0, VLC_FALSE );
+ p_buffer = aout_OutputNextBuffer( p_aout, mdate(), VLC_FALSE );
/* Now do the actual memcpy into the circular buffer */
if ( l_bytes1 != p_notif->i_buffer_size )
* output.c : internal management of output streams for the audio output
*****************************************************************************
* Copyright (C) 2002 VideoLAN
- * $Id: output.c,v 1.10 2002/08/24 10:19:43 sam Exp $
+ * $Id: output.c,v 1.11 2002/08/25 09:40:00 sam Exp $
*
* Authors: Christophe Massiot <massiot@via.ecp.fr>
*
*****************************************************************************/
aout_buffer_t * aout_OutputNextBuffer( aout_instance_t * p_aout,
mtime_t start_date,
- mtime_t timeout,
vlc_bool_t b_can_sleek )
{
aout_buffer_t * p_buffer;
- mtime_t now = mdate();
vlc_mutex_lock( &p_aout->mixer_lock );
- timeout += now;
-
- while( p_aout->output.fifo.p_first == NULL && timeout > now )
- {
- vlc_mutex_unlock( &p_aout->mixer_lock );
- msleep( AOUT_PTS_TOLERANCE / 2 );
- vlc_mutex_lock( &p_aout->mixer_lock );
- now = mdate();
- }
-
p_buffer = p_aout->output.fifo.p_first;
- while ( p_buffer != NULL && p_buffer->start_date < start_date )
+ while ( p_buffer && p_buffer->start_date < start_date && p_buffer->p_next )
{
- msg_Dbg( p_aout, "audio output is too slow (%lld), trashing %lldms",
+ msg_Dbg( p_aout, "audio output is too slow (%lld), trashing %lldus",
start_date - p_buffer->start_date,
p_buffer->end_date - p_buffer->start_date );
p_buffer = p_buffer->p_next;
/* Set date to 0, to allow the mixer to send a new buffer ASAP */
aout_FifoSet( p_aout, &p_aout->output.fifo, 0 );
vlc_mutex_unlock( &p_aout->mixer_lock );
- msg_Dbg( p_aout, "audio output is starving, waited too long" );
+ msg_Dbg( p_aout,
+ "audio output is starving (no input), playing silence" );
return NULL;
}
+ (p_buffer->end_date - p_buffer->start_date) )
{
vlc_mutex_unlock( &p_aout->mixer_lock );
- msg_Dbg( p_aout, "audio output is starving (%lld)",
+ msg_Dbg( p_aout, "audio output is starving (%lld), playing silence",
p_buffer->start_date - start_date );
return NULL;
}