X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Faudio_output%2Fdirectx.c;h=6c7228e7dc1ae657d98e1b8c5bdac2532146a685;hb=688f0a8d8913f20ac5d3e27bf4f2bbe35c9f5de7;hp=5bed710d0c99b984f841e02a7e7b820d26aa86ad;hpb=a90a19a6b0468ea9fedadc27cfc1118d70295263;p=vlc diff --git a/modules/audio_output/directx.c b/modules/audio_output/directx.c index 5bed710d0c..6c7228e7dc 100644 --- a/modules/audio_output/directx.c +++ b/modules/audio_output/directx.c @@ -1,7 +1,7 @@ /***************************************************************************** * directx.c: Windows DirectX audio output method ***************************************************************************** - * Copyright (C) 2001 VideoLAN + * Copyright (C) 2001 the VideoLAN team * $Id$ * * Authors: Gildas Bazin @@ -18,32 +18,28 @@ * * You should have received a copy of the GNU General Public License * along with this program; if not, write to the Free Software - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111, USA. + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA. *****************************************************************************/ /***************************************************************************** * Preamble *****************************************************************************/ -#include /* ENOMEM */ -#include /* open(), O_WRONLY */ -#include /* strerror() */ -#include /* calloc(), malloc(), free() */ -#include -#include -#include "aout_internal.h" +#ifdef HAVE_CONFIG_H +# include "config.h" +#endif + +#include +#include +#include #include #include #include -#define FRAME_SIZE 2048 /* The size is in samples, not in bytes */ -#define FRAMES_NUM 8 - -/* frame buffer status */ -#define FRAME_QUEUED 0 -#define FRAME_EMPTY 1 +#define FRAME_SIZE ((int)p_aout->output.output.i_rate/20) /* Size in samples */ +#define FRAMES_NUM 8 /* Needs to be > 3 */ /***************************************************************************** * DirectSound GUIDs. @@ -51,7 +47,6 @@ * the linking stage. *****************************************************************************/ #include -DEFINE_GUID(IID_IDirectSoundNotify, 0xb0210783, 0x89cd, 0x11d0, 0xaf, 0x8, 0x0, 0xa0, 0xc9, 0x25, 0xcd, 0x16); /***************************************************************************** * Useful macros @@ -90,6 +85,9 @@ DEFINE_GUID(IID_IDirectSoundNotify, 0xb0210783, 0x89cd, 0x11d0, 0xaf, 0x8, 0x0, # define SPEAKER_RESERVED 0x80000000 #endif +#ifndef DSSPEAKER_DSSPEAKER_DIRECTOUT +# define DSSPEAKER_DSSPEAKER_DIRECTOUT 0x00000000 +#endif #ifndef DSSPEAKER_HEADPHONE # define DSSPEAKER_HEADPHONE 0x00000001 #endif @@ -108,6 +106,15 @@ DEFINE_GUID(IID_IDirectSoundNotify, 0xb0210783, 0x89cd, 0x11d0, 0xaf, 0x8, 0x0, #ifndef DSSPEAKER_5POINT1 # define DSSPEAKER_5POINT1 0x00000006 #endif +#ifndef DSSPEAKER_7POINT1 +# define DSSPEAKER_7POINT1 0x00000007 +#endif +#ifndef DSSPEAKER_7POINT1_SURROUND +# define DSSPEAKER_7POINT1_SURROUND 0x00000008 +#endif +#ifndef DSSPEAKER_7POINT1_WIDE +# define DSSPEAKER_7POINT1_WIDE DSSPEAKER_7POINT1 +#endif #ifndef _WAVEFORMATEXTENSIBLE_ typedef struct { @@ -134,12 +141,12 @@ typedef struct notification_thread_t { VLC_COMMON_MEMBERS - aout_instance_t * p_aout; - int i_frame_status[FRAMES_NUM]; /* status of each frame buffer */ - DSBPOSITIONNOTIFY p_events[FRAMES_NUM]; /* play notification events */ - int i_frame_size; /* Size in bytes of one frame */ + aout_instance_t *p_aout; + int i_frame_size; /* size in bytes of one frame */ + int i_write_slot; /* current write position in our circular buffer */ mtime_t start_date; + HANDLE event; } notification_thread_t; @@ -152,19 +159,22 @@ typedef struct notification_thread_t struct aout_sys_t { HINSTANCE hdsound_dll; /* handle of the opened dsound dll */ + + int i_device_id; /* user defined device */ + LPGUID p_device_guid; + LPDIRECTSOUND p_dsobject; /* main Direct Sound object */ LPDIRECTSOUNDBUFFER p_dsbuffer; /* the sound buffer we use (direct sound * takes care of mixing all the * secondary buffers into the primary) */ - LPDIRECTSOUNDNOTIFY p_dsnotify; /* the position notify interface */ notification_thread_t *p_notif; /* DirectSoundThread id */ int b_playing; /* playing status */ int i_frame_size; /* Size in bytes of one frame */ - vlc_bool_t b_chan_reorder; /* do we need channel reordering */ + bool b_chan_reorder; /* do we need channel reordering */ int pi_chan_table[AOUT_CHAN_MAX]; uint32_t i_channel_mask; uint32_t i_bits_per_sample; @@ -197,8 +207,8 @@ static void Play ( aout_instance_t * ); /* local functions */ static void Probe ( aout_instance_t * ); static int InitDirectSound ( aout_instance_t * ); -static int CreateDSBuffer ( aout_instance_t *, int, int, int, int, int, vlc_bool_t ); -static int CreateDSBufferPCM ( aout_instance_t *, int*, int, int, int, vlc_bool_t ); +static int CreateDSBuffer ( aout_instance_t *, int, int, int, int, int, bool ); +static int CreateDSBufferPCM ( aout_instance_t *, int*, int, int, int, bool ); static void DestroyDSBuffer ( aout_instance_t * ); static void DirectSoundThread ( notification_thread_t * ); static int FillBuffer ( aout_instance_t *, int, aout_buffer_t * ); @@ -206,12 +216,26 @@ static int FillBuffer ( aout_instance_t *, int, aout_buffer_t * ); /***************************************************************************** * Module descriptor *****************************************************************************/ +#define DEVICE_TEXT N_("Output device") +#define DEVICE_LONGTEXT N_( \ + "DirectX device number: 0 default device, 1..N device by number" \ + "(Note that the default device appears as 0 AND another number)." ) +#define FLOAT_TEXT N_("Use float32 output") +#define FLOAT_LONGTEXT N_( \ + "The option allows you to enable or disable the high-quality float32 " \ + "audio output mode (which is not well supported by some soundcards)." ) + vlc_module_begin(); - set_description( _("DirectX audio output") ); + set_description( N_("DirectX audio output") ); + set_shortname( "DirectX" ); set_capability( "audio output", 100 ); set_category( CAT_AUDIO ); set_subcategory( SUBCAT_AUDIO_AOUT ); add_shortcut( "directx" ); + add_integer( "directx-audio-device", 0, NULL, DEVICE_TEXT, + DEVICE_LONGTEXT, true ); + add_bool( "directx-audio-float32", 0, 0, FLOAT_TEXT, + FLOAT_LONGTEXT, true ); set_callbacks( OpenAudio, CloseAudio ); vlc_module_end(); @@ -224,28 +248,32 @@ static int OpenAudio( vlc_object_t *p_this ) { aout_instance_t * p_aout = (aout_instance_t *)p_this; vlc_value_t val; - int i; msg_Dbg( p_aout, "OpenAudio" ); /* Allocate structure */ p_aout->output.p_sys = malloc( sizeof( aout_sys_t ) ); if( p_aout->output.p_sys == NULL ) - { - msg_Err( p_aout, "out of memory" ); - return VLC_EGENERIC; - } + return VLC_ENOMEM; /* Initialize some variables */ p_aout->output.p_sys->p_dsobject = NULL; p_aout->output.p_sys->p_dsbuffer = NULL; - p_aout->output.p_sys->p_dsnotify = NULL; p_aout->output.p_sys->p_notif = NULL; p_aout->output.p_sys->b_playing = 0; p_aout->output.pf_play = Play; aout_VolumeSoftInit( p_aout ); + /* Retrieve config values */ + var_Create( p_aout, "directx-audio-float32", + VLC_VAR_BOOL | VLC_VAR_DOINHERIT ); + var_Create( p_aout, "directx-audio-device", + VLC_VAR_INTEGER|VLC_VAR_DOINHERIT ); + var_Get( p_aout, "directx-audio-device", &val ); + p_aout->output.p_sys->i_device_id = val.i_int; + p_aout->output.p_sys->p_device_guid = 0; + /* Initialise DirectSound */ if( InitDirectSound( p_aout ) ) { @@ -264,16 +292,6 @@ static int OpenAudio( vlc_object_t *p_this ) goto error; } - /* Now we need to setup our DirectSound play notification structure */ - p_aout->output.p_sys->p_notif = - vlc_object_create( p_aout, sizeof(notification_thread_t) ); - p_aout->output.p_sys->p_notif->p_aout = p_aout; - - /* Then create the notification events */ - for( i = 0; i < FRAMES_NUM; i++ ) - p_aout->output.p_sys->p_notif->p_events[i].hEventNotify = - CreateEvent( NULL, FALSE, FALSE, NULL ); - /* Open the device */ if( val.i_int == AOUT_VAR_SPDIF ) { @@ -290,7 +308,7 @@ static int OpenAudio( vlc_object_t *p_this ) p_aout->output.output.i_physical_channels, aout_FormatNbChannels( &p_aout->output.output ), p_aout->output.output.i_rate, - p_aout->output.p_sys->i_frame_size, VLC_FALSE ) + p_aout->output.p_sys->i_frame_size, false ) != VLC_SUCCESS ) { msg_Err( p_aout, "cannot open directx audio device" ); @@ -309,6 +327,14 @@ static int OpenAudio( vlc_object_t *p_this ) | AOUT_CHAN_REARLEFT | AOUT_CHAN_REARRIGHT | AOUT_CHAN_LFE; } + else if( val.i_int == AOUT_VAR_7_1 ) + { + p_aout->output.output.i_physical_channels + = AOUT_CHAN_LEFT | AOUT_CHAN_RIGHT | AOUT_CHAN_CENTER + | AOUT_CHAN_REARLEFT | AOUT_CHAN_REARRIGHT + | AOUT_CHAN_MIDDLELEFT | AOUT_CHAN_MIDDLERIGHT + | AOUT_CHAN_LFE; + } else if( val.i_int == AOUT_VAR_3F2R ) { p_aout->output.output.i_physical_channels @@ -334,7 +360,7 @@ static int OpenAudio( vlc_object_t *p_this ) if( CreateDSBufferPCM( p_aout, &p_aout->output.output.i_format, p_aout->output.output.i_physical_channels, aout_FormatNbChannels( &p_aout->output.output ), - p_aout->output.output.i_rate, VLC_FALSE ) + p_aout->output.output.i_rate, false ) != VLC_SUCCESS ) { msg_Err( p_aout, "cannot open directx audio device" ); @@ -345,20 +371,29 @@ static int OpenAudio( vlc_object_t *p_this ) /* Calculate the frame size in bytes */ p_aout->output.i_nb_samples = FRAME_SIZE; aout_FormatPrepare( &p_aout->output.output ); - p_aout->output.p_sys->i_frame_size = - FRAME_SIZE * p_aout->output.output.i_bytes_per_frame; - aout_VolumeSoftInit( p_aout ); } + /* Now we need to setup our DirectSound play notification structure */ + p_aout->output.p_sys->p_notif = + vlc_object_create( p_aout, sizeof(notification_thread_t) ); + p_aout->output.p_sys->p_notif->p_aout = p_aout; + + p_aout->output.p_sys->p_notif->event = CreateEvent( 0, FALSE, FALSE, 0 ); + p_aout->output.p_sys->p_notif->i_frame_size = + p_aout->output.p_sys->i_frame_size; + /* then launch the notification thread */ msg_Dbg( p_aout, "creating DirectSoundThread" ); if( vlc_thread_create( p_aout->output.p_sys->p_notif, "DirectSound Notification Thread", DirectSoundThread, - VLC_THREAD_PRIORITY_HIGHEST, VLC_FALSE ) ) + VLC_THREAD_PRIORITY_HIGHEST, false ) ) { msg_Err( p_aout, "cannot create DirectSoundThread" ); + CloseHandle( p_aout->output.p_sys->p_notif->event ); + vlc_object_release( p_aout->output.p_sys->p_notif ); + p_aout->output.p_sys->p_notif = NULL; goto error; } @@ -380,6 +415,7 @@ static void Probe( aout_instance_t * p_aout ) int i_format; unsigned int i_physical_channels; DWORD ui_speaker_config; + bool is_default_output_set = false; var_Create( p_aout, "audio-device", VLC_VAR_INTEGER | VLC_VAR_HASCHOICE ); text.psz_string = _("Audio Device"); @@ -392,17 +428,40 @@ static void Probe( aout_instance_t * p_aout ) if( p_aout->output.output.i_physical_channels == i_physical_channels ) { if( CreateDSBufferPCM( p_aout, &i_format, i_physical_channels, 6, - p_aout->output.output.i_rate, VLC_TRUE ) + p_aout->output.output.i_rate, true ) == VLC_SUCCESS ) { val.i_int = AOUT_VAR_5_1; - text.psz_string = N_("5.1"); + text.psz_string = "5.1"; var_Change( p_aout, "audio-device", VLC_VAR_ADDCHOICE, &val, &text ); + var_Change( p_aout, "audio-device", VLC_VAR_SETDEFAULT, &val, NULL ); + is_default_output_set = true; msg_Dbg( p_aout, "device supports 5.1 channels" ); } } + /* Test for 7.1 support */ + i_physical_channels = AOUT_CHAN_LEFT | AOUT_CHAN_RIGHT | + AOUT_CHAN_CENTER | AOUT_CHAN_REARLEFT | + AOUT_CHAN_MIDDLELEFT | AOUT_CHAN_MIDDLERIGHT | + AOUT_CHAN_REARRIGHT | AOUT_CHAN_LFE; + if( p_aout->output.output.i_physical_channels == i_physical_channels ) + { + if( CreateDSBufferPCM( p_aout, &i_format, i_physical_channels, 8, + p_aout->output.output.i_rate, true ) + == VLC_SUCCESS ) + { + val.i_int = AOUT_VAR_7_1; + text.psz_string = "7.1"; + var_Change( p_aout, "audio-device", + VLC_VAR_ADDCHOICE, &val, &text ); + var_Change( p_aout, "audio-device", VLC_VAR_SETDEFAULT, &val, NULL ); + is_default_output_set = true; + msg_Dbg( p_aout, "device supports 7.1 channels" ); + } + } + /* Test for 3 Front 2 Rear support */ i_physical_channels = AOUT_CHAN_LEFT | AOUT_CHAN_RIGHT | AOUT_CHAN_CENTER | AOUT_CHAN_REARLEFT | @@ -410,13 +469,18 @@ static void Probe( aout_instance_t * p_aout ) if( p_aout->output.output.i_physical_channels == i_physical_channels ) { if( CreateDSBufferPCM( p_aout, &i_format, i_physical_channels, 5, - p_aout->output.output.i_rate, VLC_TRUE ) + p_aout->output.output.i_rate, true ) == VLC_SUCCESS ) { val.i_int = AOUT_VAR_3F2R; text.psz_string = N_("3 Front 2 Rear"); var_Change( p_aout, "audio-device", VLC_VAR_ADDCHOICE, &val, &text ); + if(!is_default_output_set) + { + var_Change( p_aout, "audio-device", VLC_VAR_SETDEFAULT, &val, NULL ); + is_default_output_set = true; + } msg_Dbg( p_aout, "device supports 5 channels" ); } } @@ -428,13 +492,18 @@ static void Probe( aout_instance_t * p_aout ) == i_physical_channels ) { if( CreateDSBufferPCM( p_aout, &i_format, i_physical_channels, 4, - p_aout->output.output.i_rate, VLC_TRUE ) + p_aout->output.output.i_rate, true ) == VLC_SUCCESS ) { val.i_int = AOUT_VAR_2F2R; text.psz_string = N_("2 Front 2 Rear"); var_Change( p_aout, "audio-device", VLC_VAR_ADDCHOICE, &val, &text ); + if(!is_default_output_set) + { + var_Change( p_aout, "audio-device", VLC_VAR_SETDEFAULT, &val, NULL ); + is_default_output_set = true; + } msg_Dbg( p_aout, "device supports 4 channels" ); } } @@ -442,20 +511,25 @@ static void Probe( aout_instance_t * p_aout ) /* Test for stereo support */ i_physical_channels = AOUT_CHAN_LEFT | AOUT_CHAN_RIGHT; if( CreateDSBufferPCM( p_aout, &i_format, i_physical_channels, 2, - p_aout->output.output.i_rate, VLC_TRUE ) + p_aout->output.output.i_rate, true ) == VLC_SUCCESS ) { val.i_int = AOUT_VAR_STEREO; text.psz_string = N_("Stereo"); var_Change( p_aout, "audio-device", VLC_VAR_ADDCHOICE, &val, &text ); - var_Change( p_aout, "audio-device", VLC_VAR_SETDEFAULT, &val, NULL ); + if(!is_default_output_set) + { + var_Change( p_aout, "audio-device", VLC_VAR_SETDEFAULT, &val, NULL ); + is_default_output_set = true; + msg_Dbg( p_aout, "device supports 2 channels (DEFAULT!)" ); + } msg_Dbg( p_aout, "device supports 2 channels" ); } /* Test for mono support */ i_physical_channels = AOUT_CHAN_CENTER; if( CreateDSBufferPCM( p_aout, &i_format, i_physical_channels, 1, - p_aout->output.output.i_rate, VLC_TRUE ) + p_aout->output.output.i_rate, true ) == VLC_SUCCESS ) { val.i_int = AOUT_VAR_MONO; @@ -470,13 +544,20 @@ static void Probe( aout_instance_t * p_aout ) &ui_speaker_config ) ) { ui_speaker_config = DSSPEAKER_STEREO; + msg_Dbg( p_aout, "GetSpeakerConfig failed" ); } switch( DSSPEAKER_CONFIG(ui_speaker_config) ) { + case DSSPEAKER_7POINT1: + msg_Dbg( p_aout, "Windows says your SpeakerConfig is 7.1" ); + val.i_int = AOUT_VAR_7_1; + break; case DSSPEAKER_5POINT1: + msg_Dbg( p_aout, "Windows says your SpeakerConfig is 5.1" ); val.i_int = AOUT_VAR_5_1; break; case DSSPEAKER_QUAD: + msg_Dbg( p_aout, "Windows says your SpeakerConfig is Quad" ); val.i_int = AOUT_VAR_2F2R; break; #if 0 /* Lots of people just get their settings wrong and complain that @@ -486,8 +567,11 @@ static void Probe( aout_instance_t * p_aout ) break; #endif case DSSPEAKER_SURROUND: + msg_Dbg( p_aout, "Windows says your SpeakerConfig is surround" ); case DSSPEAKER_STEREO: + msg_Dbg( p_aout, "Windows says your SpeakerConfig is stereo" ); default: + /* If nothing else is found, choose stereo output */ val.i_int = AOUT_VAR_STEREO; break; } @@ -500,7 +584,7 @@ static void Probe( aout_instance_t * p_aout ) p_aout->output.output.i_physical_channels, aout_FormatNbChannels( &p_aout->output.output ), p_aout->output.output.i_rate, - AOUT_SPDIF_SIZE, VLC_TRUE ) + AOUT_SPDIF_SIZE, true ) == VLC_SUCCESS ) { msg_Dbg( p_aout, "device supports A/52 over S/PDIF" ); @@ -523,7 +607,7 @@ static void Probe( aout_instance_t * p_aout ) var_AddCallback( p_aout, "audio-device", aout_ChannelsRestart, NULL ); - val.b_bool = VLC_TRUE; + val.b_bool = true; var_Set( p_aout, "intf-change", val ); } @@ -537,6 +621,7 @@ static void Play( aout_instance_t *p_aout ) if( !p_aout->output.p_sys->b_playing ) { aout_buffer_t *p_buffer; + int i; p_aout->output.p_sys->b_playing = 1; @@ -545,11 +630,15 @@ static void Play( aout_instance_t *p_aout ) aout_FifoFirstDate( p_aout, &p_aout->output.fifo ); /* fill in the first samples */ - p_buffer = aout_FifoPop( p_aout, &p_aout->output.fifo ); - FillBuffer( p_aout, 0, p_buffer ); + for( i = 0; i < FRAMES_NUM; i++ ) + { + p_buffer = aout_FifoPop( p_aout, &p_aout->output.fifo ); + if( !p_buffer ) break; + FillBuffer( p_aout, i, p_buffer ); + } /* wake up the audio output thread */ - SetEvent( p_aout->output.p_sys->p_notif->p_events[0].hEventNotify ); + SetEvent( p_aout->output.p_sys->p_notif->event ); } } @@ -561,23 +650,18 @@ static void CloseAudio( 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; - msg_Dbg( p_aout, "CloseAudio" ); + msg_Dbg( p_aout, "closing audio device" ); /* kill the position notification thread, if any */ if( p_sys->p_notif ) { vlc_object_detach( p_sys->p_notif ); - if( p_sys->p_notif->b_thread ) - { - p_sys->p_notif->b_die = 1; + vlc_object_kill( p_sys->p_notif ); + /* wake up the audio thread if needed */ + if( !p_sys->b_playing ) SetEvent( p_sys->p_notif->event ); - if( !p_sys->b_playing ) - /* wake up the audio thread */ - SetEvent( p_sys->p_notif->p_events[0].hEventNotify ); - - vlc_thread_join( p_sys->p_notif ); - } - vlc_object_destroy( p_sys->p_notif ); + vlc_thread_join( p_sys->p_notif ); + vlc_object_release( p_sys->p_notif ); } /* release the secondary buffer */ @@ -585,19 +669,42 @@ static void CloseAudio( vlc_object_t *p_this ) /* finally release the DirectSound object */ if( p_sys->p_dsobject ) IDirectSound_Release( p_sys->p_dsobject ); - + /* free DSOUND.DLL */ if( p_sys->hdsound_dll ) FreeLibrary( p_sys->hdsound_dll ); + free( p_aout->output.p_sys->p_device_guid ); free( p_sys ); } +/***************************************************************************** + * CallBackDirectSoundEnum: callback to enumerate available devices + *****************************************************************************/ +static int CALLBACK CallBackDirectSoundEnum( LPGUID p_guid, LPCSTR psz_desc, + LPCSTR psz_mod, LPVOID _p_aout ) +{ + aout_instance_t *p_aout = (aout_instance_t *)_p_aout; + + msg_Dbg( p_aout, "found device: %s", psz_desc ); + + if( p_aout->output.p_sys->i_device_id == 0 && p_guid ) + { + p_aout->output.p_sys->p_device_guid = malloc( sizeof( GUID ) ); + *p_aout->output.p_sys->p_device_guid = *p_guid; + msg_Dbg( p_aout, "using device: %s", psz_desc ); + } + + p_aout->output.p_sys->i_device_id--; + return 1; +} + /***************************************************************************** * InitDirectSound: handle all the gory details of DirectSound initialisation *****************************************************************************/ static int InitDirectSound( aout_instance_t *p_aout ) { HRESULT (WINAPI *OurDirectSoundCreate)(LPGUID, LPDIRECTSOUND *, LPUNKNOWN); + HRESULT (WINAPI *OurDirectSoundEnumerate)(LPDSENUMCALLBACK, LPVOID); p_aout->output.p_sys->hdsound_dll = LoadLibrary("DSOUND.DLL"); if( p_aout->output.p_sys->hdsound_dll == NULL ) @@ -606,17 +713,32 @@ static int InitDirectSound( aout_instance_t *p_aout ) goto error; } - OurDirectSoundCreate = (void *)GetProcAddress( - p_aout->output.p_sys->hdsound_dll, - "DirectSoundCreate" ); + OurDirectSoundCreate = (void *) + GetProcAddress( p_aout->output.p_sys->hdsound_dll, + "DirectSoundCreate" ); if( OurDirectSoundCreate == NULL ) { msg_Warn( p_aout, "GetProcAddress FAILED" ); goto error; } + /* Get DirectSoundEnumerate */ + OurDirectSoundEnumerate = (void *) + GetProcAddress( p_aout->output.p_sys->hdsound_dll, + "DirectSoundEnumerateA" ); + if( OurDirectSoundEnumerate ) + { + /* Attempt enumeration */ + if( FAILED( OurDirectSoundEnumerate( CallBackDirectSoundEnum, + p_aout ) ) ) + { + msg_Dbg( p_aout, "enumeration of DirectSound devices failed" ); + } + } + /* Create the direct sound object */ - if FAILED( OurDirectSoundCreate( NULL, &p_aout->output.p_sys->p_dsobject, + if FAILED( OurDirectSoundCreate( p_aout->output.p_sys->p_device_guid, + &p_aout->output.p_sys->p_dsobject, NULL ) ) { msg_Warn( p_aout, "cannot create a direct sound device" ); @@ -667,7 +789,7 @@ static int InitDirectSound( aout_instance_t *p_aout ) *****************************************************************************/ static int CreateDSBuffer( aout_instance_t *p_aout, int i_format, int i_channels, int i_nb_channels, int i_rate, - int i_bytes_per_frame, vlc_bool_t b_probe ) + int i_bytes_per_frame, bool b_probe ) { WAVEFORMATEXTENSIBLE waveformat; DSBUFFERDESC dsbdesc; @@ -725,7 +847,6 @@ static int CreateDSBuffer( aout_instance_t *p_aout, int i_format, memset(&dsbdesc, 0, sizeof(DSBUFFERDESC)); dsbdesc.dwSize = sizeof(DSBUFFERDESC); dsbdesc.dwFlags = DSBCAPS_GETCURRENTPOSITION2/* Better position accuracy */ - | DSBCAPS_CTRLPOSITIONNOTIFY /* We need notification */ | DSBCAPS_GLOBALFOCUS; /* Allows background playing */ /* Only use the new WAVE_FORMAT_EXTENSIBLE format for multichannel audio */ @@ -777,40 +898,8 @@ static int CreateDSBuffer( aout_instance_t *p_aout, int i_format, return VLC_SUCCESS; } - /* Backup the size of a frame */ - p_aout->output.p_sys->p_notif->i_frame_size = i_bytes_per_frame; - - /* Now the secondary buffer is created, we need to setup its position - * notification */ - for( i = 0; i < FRAMES_NUM; i++ ) - { - p_aout->output.p_sys->p_notif->p_events[i].dwOffset = i * - p_aout->output.p_sys->p_notif->i_frame_size; - - p_aout->output.p_sys->p_notif->i_frame_status[i] = FRAME_EMPTY; - } - - /* Get the IDirectSoundNotify interface */ - if FAILED( IDirectSoundBuffer_QueryInterface( - p_aout->output.p_sys->p_dsbuffer, - &IID_IDirectSoundNotify, - (LPVOID *)&p_aout->output.p_sys->p_dsnotify ) ) - { - msg_Err( p_aout, "cannot get IDirectSoundNotify interface" ); - goto error; - } - - if FAILED( IDirectSoundNotify_SetNotificationPositions( - p_aout->output.p_sys->p_dsnotify, - FRAMES_NUM, - p_aout->output.p_sys->p_notif->p_events ) ) - { - msg_Err( p_aout, "cannot set position notification" ); - goto error; - } - + p_aout->output.p_sys->i_frame_size = i_bytes_per_frame; p_aout->output.p_sys->i_channel_mask = waveformat.dwChannelMask; - p_aout->output.p_sys->b_chan_reorder = aout_CheckChannelReorder( pi_channels_in, pi_channels_out, waveformat.dwChannelMask, i_nb_channels, @@ -822,10 +911,6 @@ static int CreateDSBuffer( aout_instance_t *p_aout, int i_format, } return VLC_SUCCESS; - - error: - DestroyDSBuffer( p_aout ); - return VLC_EGENERIC; } /***************************************************************************** @@ -836,11 +921,15 @@ static int CreateDSBuffer( aout_instance_t *p_aout, int i_format, ****************************************************************************/ static int CreateDSBufferPCM( aout_instance_t *p_aout, int *i_format, int i_channels, int i_nb_channels, int i_rate, - vlc_bool_t b_probe ) + bool b_probe ) { + vlc_value_t val; + + var_Get( p_aout, "directx-audio-float32", &val ); + /* Float32 audio samples are not supported for 5.1 output on the emu101k */ - if( i_nb_channels > 2 || + if( !val.b_bool || i_nb_channels > 2 || CreateDSBuffer( p_aout, VLC_FOURCC('f','l','3','2'), i_channels, i_nb_channels, i_rate, FRAME_SIZE * 4 * i_nb_channels, b_probe ) @@ -873,12 +962,6 @@ static int CreateDSBufferPCM( aout_instance_t *p_aout, int *i_format, *****************************************************************************/ static void DestroyDSBuffer( aout_instance_t *p_aout ) { - if( p_aout->output.p_sys->p_dsnotify ) - { - IDirectSoundNotify_Release( p_aout->output.p_sys->p_dsnotify ); - p_aout->output.p_sys->p_dsnotify = NULL; - } - if( p_aout->output.p_sys->p_dsbuffer ) { IDirectSoundBuffer_Release( p_aout->output.p_sys->p_dsbuffer ); @@ -944,8 +1027,7 @@ static int FillBuffer( aout_instance_t *p_aout, int i_frame, p_sys->i_bits_per_sample ); } - p_aout->p_vlc->pf_memcpy( p_write_position, p_buffer->p_buffer, - l_bytes1 ); + vlc_memcpy( p_write_position, p_buffer->p_buffer, l_bytes1 ); aout_BufferFree( p_buffer ); } @@ -953,26 +1035,23 @@ static int FillBuffer( aout_instance_t *p_aout, int i_frame, IDirectSoundBuffer_Unlock( p_sys->p_dsbuffer, p_write_position, l_bytes1, p_wrap_around, l_bytes2 ); + p_notif->i_write_slot = (i_frame + 1) % FRAMES_NUM; return VLC_SUCCESS; } /***************************************************************************** - * DirectSoundThread: this thread will capture play notification events. + * DirectSoundThread: this thread will capture play notification events. ***************************************************************************** * We use this thread to emulate a callback mechanism. The thread probes for * event notification and fills up the DS secondary buffer when needed. *****************************************************************************/ static void DirectSoundThread( notification_thread_t *p_notif ) { - HANDLE notification_events[FRAMES_NUM]; - HRESULT dsresult; aout_instance_t *p_aout = p_notif->p_aout; - int i, i_which_frame, i_last_frame, i_next_frame; - mtime_t mtime; - vlc_bool_t b_sleek; - - for( i = 0; i < FRAMES_NUM; i++ ) - notification_events[i] = p_notif->p_events[i].hEventNotify; + bool b_sleek; + mtime_t last_time; + HRESULT dsresult; + long l_queued = 0; /* We don't want any resampling when using S/PDIF output */ b_sleek = p_aout->output.output.i_format == VLC_FOURCC('s','p','d','i'); @@ -983,9 +1062,9 @@ static void DirectSoundThread( notification_thread_t *p_notif ) msg_Dbg( p_notif, "DirectSoundThread ready" ); /* Wait here until Play() is called */ - WaitForSingleObject( notification_events[0], INFINITE ); + WaitForSingleObject( p_notif->event, INFINITE ); - if( !p_notif->b_die ) + if( vlc_object_alive (p_notif) ) { mwait( p_notif->start_date - AOUT_PTS_TOLERANCE / 2 ); @@ -1008,88 +1087,64 @@ static void DirectSoundThread( notification_thread_t *p_notif ) msg_Err( p_aout, "cannot start playing buffer" ); } } + last_time = mdate(); - while( !p_notif->b_die ) + while( vlc_object_alive (p_notif) ) { - aout_buffer_t *p_buffer; - long l_latency; + long l_read, l_free_slots; + mtime_t mtime = mdate(); + int i; - /* wait for the position notification */ - i_which_frame = WaitForMultipleObjects( FRAMES_NUM, - notification_events, 0, - INFINITE ) - WAIT_OBJECT_0; + /* + * Fill in as much audio data as we can in our circular buffer + */ - if( p_notif->b_die ) - break; - - mtime = mdate(); - - /* We take into account the current latency */ - if SUCCEEDED( IDirectSoundBuffer_GetCurrentPosition( - p_aout->output.p_sys->p_dsbuffer, - &l_latency, NULL ) ) + /* Find out current play position */ + if FAILED( IDirectSoundBuffer_GetCurrentPosition( + p_aout->output.p_sys->p_dsbuffer, &l_read, NULL ) ) { - if( l_latency > (i_which_frame * FRAME_SIZE) - && l_latency < ((i_which_frame+1) * FRAME_SIZE) ) - { - l_latency = - ( l_latency / - p_aout->output.output.i_bytes_per_frame % - FRAME_SIZE ); - } - else - { - l_latency = FRAME_SIZE - ( l_latency / - p_aout->output.output.i_bytes_per_frame % - FRAME_SIZE ); - } + msg_Err( p_aout, "GetCurrentPosition() failed!" ); + l_read = 0; } - else + + /* Detect underruns */ + if( l_queued && mtime - last_time > + INT64_C(1000000) * l_queued / p_aout->output.output.i_rate ) { - l_latency = 0; + msg_Dbg( p_aout, "detected underrun!" ); } - - /* Mark last frame as empty */ - i_last_frame = (i_which_frame + FRAMES_NUM -1) % FRAMES_NUM; - i_next_frame = (i_which_frame + 1) % FRAMES_NUM; - p_notif->i_frame_status[i_last_frame] = FRAME_EMPTY; + last_time = mtime; /* Try to fill in as many frame buffers as possible */ - for( i = i_next_frame; (i % FRAMES_NUM) != i_which_frame; i++ ) - { - - /* Check if frame buf is already filled */ - if( p_notif->i_frame_status[i % FRAMES_NUM] == FRAME_QUEUED ) - continue; + l_read /= p_aout->output.output.i_bytes_per_frame; + l_queued = p_notif->i_write_slot * FRAME_SIZE - l_read; + if( l_queued < 0 ) l_queued += (FRAME_SIZE * FRAMES_NUM); + l_free_slots = (FRAMES_NUM * FRAME_SIZE - l_queued) / FRAME_SIZE; - p_buffer = aout_OutputNextBuffer( p_aout, - mtime + 1000000 / p_aout->output.output.i_rate * - ((i - i_next_frame + 1) * FRAME_SIZE + l_latency), b_sleek ); + for( i = 0; i < l_free_slots; i++ ) + { + aout_buffer_t *p_buffer = aout_OutputNextBuffer( p_aout, + mtime + INT64_C(1000000) * (i * FRAME_SIZE + l_queued) / + p_aout->output.output.i_rate, b_sleek ); /* If there is no audio data available and we have some buffered * already, then just wait for the next time */ - if( !p_buffer && (i != i_next_frame) ) - { - //msg_Err( p_aout, "only %i frame buffers filled!", - // i - i_next_frame ); - break; - } - - if( FillBuffer( p_aout, (i%FRAMES_NUM), p_buffer ) - != VLC_SUCCESS ) - break; + if( !p_buffer && (i || l_queued / FRAME_SIZE) ) break; - /* Mark the frame buffer as QUEUED */ - p_notif->i_frame_status[i%FRAMES_NUM] = FRAME_QUEUED; + if( FillBuffer( p_aout, p_notif->i_write_slot % FRAMES_NUM, + p_buffer ) != VLC_SUCCESS ) break; } + /* Sleep a reasonable amount of time */ + l_queued += (i * FRAME_SIZE); + msleep( INT64_C(1000000) * l_queued / p_aout->output.output.i_rate / 2 ); } /* make sure the buffer isn't playing */ IDirectSoundBuffer_Stop( p_aout->output.p_sys->p_dsbuffer ); - /* free the events */ - for( i = 0; i < FRAMES_NUM; i++ ) - CloseHandle( notification_events[i] ); + /* free the event */ + CloseHandle( p_notif->event ); msg_Dbg( p_notif, "DirectSoundThread exiting" ); }