]> git.sesse.net Git - vlc/blobdiff - modules/audio_output/alsa.c
aout: move packet FIFO management to plugins that need it
[vlc] / modules / audio_output / alsa.c
index a1d770dd3ffb681c02522b0c2d4ce7d04c086f6a..b0f91667d5df87d58e96bbe7eb6e41b8e2303d3e 100644 (file)
@@ -1,8 +1,8 @@
 /*****************************************************************************
  * alsa.c : alsa plugin for vlc
  *****************************************************************************
- * Copyright (C) 2000-2001 the VideoLAN team
- * $Id$
+ * Copyright (C) 2000-2010 the VideoLAN team
+ * Copyright (C) 2009-2011 RĂ©mi Denis-Courmont
  *
  * Authors: Henri Fallon <henri@videolan.org> - Original Author
  *          Jeffrey Baker <jwbaker@acm.org> - Port to ALSA 1.0 API
 # include "config.h"
 #endif
 
+#include <assert.h>
+
 #include <vlc_common.h>
 #include <vlc_plugin.h>
-
-#include <errno.h>                                                 /* ENOMEM */
 #include <vlc_dialog.h>
-
 #include <vlc_aout.h>
+#include <vlc_cpu.h>
 
-/* ALSA part
-   Note: we use the new API which is available since 0.9.0beta10a. */
-#define ALSA_PCM_NEW_HW_PARAMS_API
-#define ALSA_PCM_NEW_SW_PARAMS_API
 #include <alsa/asoundlib.h>
+#include <alsa/version.h>
 
 /*#define ALSA_DEBUG*/
 
@@ -55,6 +52,7 @@
  *****************************************************************************/
 struct aout_sys_t
 {
+    aout_packet_t packet;
     snd_pcm_t         * p_snd_pcm;
     unsigned int                 i_period_time;
 
@@ -62,13 +60,9 @@ struct aout_sys_t
     snd_output_t      * p_snd_stderr;
 #endif
 
-    bool b_playing;  /* playing status */
-    mtime_t start_date;
-
-    vlc_mutex_t lock;
-    vlc_cond_t  wait ;
-
-    snd_pcm_status_t *p_status;
+    mtime_t      start_date;
+    vlc_thread_t thread;
+    vlc_sem_t    wait;
 };
 
 #define A52_FRAME_NB 1536
@@ -85,30 +79,45 @@ struct aout_sys_t
 /* Why not ? --Bozo */
 /* Right. --Meuuh */
 
-#define DEFAULT_ALSA_DEVICE N_("default")
+#define DEFAULT_ALSA_DEVICE "default"
 
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
 static int   Open         ( vlc_object_t * );
 static void  Close        ( vlc_object_t * );
-static void  Play         ( aout_instance_t * );
-static void* ALSAThread   ( vlc_object_t * );
-static void  ALSAFill     ( aout_instance_t * );
+static void  Play         ( audio_output_t *, block_t * );
+static void* ALSAThread   ( void * );
+static void  ALSAFill     ( audio_output_t * );
 static int FindDevicesCallback( vlc_object_t *p_this, char const *psz_name,
                                 vlc_value_t newval, vlc_value_t oldval, void *p_unused );
+static void GetDevices( vlc_object_t *, module_config_t * );
 
 /*****************************************************************************
  * Module descriptor
  *****************************************************************************/
-static const char *const ppsz_devices[] = { "default" };
-static const char *const ppsz_devices_text[] = { N_("Default") };
+static const char *const ppsz_devices[] = {
+    "default", "plug:front",
+    "plug:side", "plug:rear", "plug:center_lfe",
+    "plug:surround40", "plug:surround41",
+    "plug:surround50", "plug:surround51",
+    "plug:surround71",
+    "hdmi", "iec958",
+};
+static const char *const ppsz_devices_text[] = {
+    N_("Default"), N_("Front speakers"),
+    N_("Side speakers"), N_("Rear speakers"), N_("Center and subwoofer"),
+    N_("Surround 4.0"), N_("Surround 4.1"),
+    N_("Surround 5.0"), N_("Surround 5.1"),
+    N_("Surround 7.1"),
+    N_("HDMI"), N_("S/PDIF"),
+};
 vlc_module_begin ()
     set_shortname( "ALSA" )
     set_description( N_("ALSA audio output") )
     set_category( CAT_AUDIO )
     set_subcategory( SUBCAT_AUDIO_AOUT )
-    add_string( "alsa-audio-device", DEFAULT_ALSA_DEVICE, aout_FindAndRestart,
+    add_string( "alsa-audio-device", DEFAULT_ALSA_DEVICE,
                 N_("ALSA Device Name"), NULL, false )
         add_deprecated_alias( "alsadev" )   /* deprecated since 0.9.3 */
         change_string_list( ppsz_devices, ppsz_devices_text, FindDevicesCallback )
@@ -118,163 +127,34 @@ vlc_module_begin ()
     set_callbacks( Open, Close )
 vlc_module_end ()
 
-/*****************************************************************************
- * Probe: probe the audio device for available formats and channels
- *****************************************************************************/
-static void Probe( aout_instance_t * p_aout,
-                   const char * psz_device, const char * psz_iec_device,
-                   int *pi_snd_pcm_format )
+/* VLC will insert a resampling filter in any case, so it is best to turn off
+ * ALSA (plug) resampling. */
+static const int mode = SND_PCM_NO_AUTO_RESAMPLE
+/* ALSA just discards extra channels. Not good. Disable it. */
+                      | SND_PCM_NO_AUTO_CHANNELS
+/* VLC is currently unable to leverage ALSA softvol. Disable it. */
+                      | SND_PCM_NO_SOFTVOL;
+
+/**
+ * Initializes list of devices.
+ */
+static void Probe (vlc_object_t *obj)
 {
-    struct aout_sys_t * p_sys = p_aout->output.p_sys;
-    vlc_value_t val, text;
-    int i_ret;
-
-    var_Create ( p_aout, "audio-device", VLC_VAR_INTEGER | VLC_VAR_HASCHOICE );
-    text.psz_string = _("Audio Device");
-    var_Change( p_aout, "audio-device", VLC_VAR_SETTEXT, &text, NULL );
-
-    /* We'll open the audio device in non blocking mode so we can just exit
-     * when it is already in use, but for the real stuff we'll still use
-     * the blocking mode */
-
-    /* Now test linear PCM capabilities */
-    if ( !(i_ret = snd_pcm_open( &p_sys->p_snd_pcm, psz_device,
-                                 SND_PCM_STREAM_PLAYBACK,
-                                 SND_PCM_NONBLOCK ) ) )
-    {
-        int i_channels;
-        snd_pcm_hw_params_t * p_hw;
-        snd_pcm_hw_params_alloca (&p_hw);
-
-        if ( snd_pcm_hw_params_any( p_sys->p_snd_pcm, p_hw ) < 0 )
-        {
-            msg_Warn( p_aout, "unable to retrieve initial hardware parameters"
-                              ", disabling linear PCM audio" );
-            snd_pcm_close( p_sys->p_snd_pcm );
-            var_Destroy( p_aout, "audio-device" );
-            return;
-        }
-
-        if ( snd_pcm_hw_params_set_format( p_sys->p_snd_pcm, p_hw,
-                                           *pi_snd_pcm_format ) < 0 )
-        {
-            int i_snd_rc = -1;
-
-            if( *pi_snd_pcm_format != SND_PCM_FORMAT_S16 )
-            {
-                *pi_snd_pcm_format = SND_PCM_FORMAT_S16;
-                i_snd_rc = snd_pcm_hw_params_set_format( p_sys->p_snd_pcm,
-                                                    p_hw, *pi_snd_pcm_format );
-            }
-            if ( i_snd_rc < 0 )
-            {
-                msg_Warn( p_aout, "unable to set stream sample size and "
-                          "word order, disabling linear PCM audio" );
-                snd_pcm_close( p_sys->p_snd_pcm );
-                var_Destroy( p_aout, "audio-device" );
-                return;
-            }
-        }
-
-        i_channels = aout_FormatNbChannels( &p_aout->output.output );
-
-        while ( i_channels > 0 )
-        {
-            if ( !snd_pcm_hw_params_test_channels( p_sys->p_snd_pcm, p_hw,
-                                                   i_channels ) )
-            {
-                switch ( i_channels )
-                {
-                case 1:
-                    val.i_int = AOUT_VAR_MONO;
-                    text.psz_string = _("Mono");
-                    var_Change( p_aout, "audio-device",
-                                VLC_VAR_ADDCHOICE, &val, &text );
-                    break;
-                case 2:
-                    val.i_int = AOUT_VAR_STEREO;
-                    text.psz_string = _("Stereo");
-                    var_Change( p_aout, "audio-device",
-                                VLC_VAR_ADDCHOICE, &val, &text );
-                    var_Set( p_aout, "audio-device", val );
-                    break;
-                case 4:
-                    val.i_int = AOUT_VAR_2F2R;
-                    text.psz_string = _("2 Front 2 Rear");
-                    var_Change( p_aout, "audio-device",
-                                VLC_VAR_ADDCHOICE, &val, &text );
-                    break;
-                case 6:
-                    val.i_int = AOUT_VAR_5_1;
-                    text.psz_string = "5.1";
-                    var_Change( p_aout, "audio-device",
-                                VLC_VAR_ADDCHOICE, &val, &text );
-                    break;
-                }
-            }
-
-            --i_channels;
-        }
-
-        /* Special case for mono on stereo only boards */
-        i_channels = aout_FormatNbChannels( &p_aout->output.output );
-        var_Change( p_aout, "audio-device", VLC_VAR_CHOICESCOUNT, &val, NULL );
-        if( val.i_int <= 0 && i_channels == 1 )
-        {
-            if ( !snd_pcm_hw_params_test_channels( p_sys->p_snd_pcm, p_hw, 2 ))
-            {
-                val.i_int = AOUT_VAR_STEREO;
-                text.psz_string = (char*)N_("Stereo");
-                var_Change( p_aout, "audio-device",
-                            VLC_VAR_ADDCHOICE, &val, &text );
-                var_Set( p_aout, "audio-device", val );
-            }
-        }
+    /* Due to design bug in audio output core, this hack is required: */
+    if (var_Type (obj, "audio-device"))
+        return;
 
-        /* Close the previously opened device */
-        snd_pcm_close( p_sys->p_snd_pcm );
-    }
-    else if ( i_ret == -EBUSY )
-    {
-        msg_Warn( p_aout, "audio device: %s is already in use", psz_device );
-    }
+    /* The variable does not exist - first call. */
+    vlc_value_t text;
 
-    /* Test for S/PDIF device if needed */
-    if ( psz_iec_device )
-    {
-        /* Opening the device should be enough */
-        if ( !(i_ret = snd_pcm_open( &p_sys->p_snd_pcm, psz_iec_device,
-                                     SND_PCM_STREAM_PLAYBACK,
-                                     SND_PCM_NONBLOCK ) ) )
-        {
-            val.i_int = AOUT_VAR_SPDIF;
-            text.psz_string = (char*)N_("A/52 over S/PDIF");
-            var_Change( p_aout, "audio-device",
-                        VLC_VAR_ADDCHOICE, &val, &text );
-            if( config_GetInt( p_aout, "spdif" ) )
-                var_Set( p_aout, "audio-device", val );
-
-            snd_pcm_close( p_sys->p_snd_pcm );
-        }
-        else if ( i_ret == -EBUSY )
-        {
-            msg_Warn( p_aout, "audio device: %s is already in use",
-                      psz_iec_device );
-        }
-    }
+    var_Create (obj, "audio-device", VLC_VAR_STRING | VLC_VAR_HASCHOICE);
+    text.psz_string = _("Audio Device");
+    var_Change (obj, "audio-device", VLC_VAR_SETTEXT, &text, NULL);
 
-    var_Change( p_aout, "audio-device", VLC_VAR_CHOICESCOUNT, &val, NULL );
-    if( val.i_int <= 0 )
-    {
-        /* Probe() has failed. */
-        msg_Dbg( p_aout, "failed to find a usable alsa configuration" );
-        var_Destroy( p_aout, "audio-device" );
-        return;
-    }
+    GetDevices (obj, NULL);
 
-    /* Add final settings to the variable */
-    var_AddCallback( p_aout, "audio-device", aout_ChannelsRestart, NULL );
-    var_SetBool( p_aout, "intf-change", true );
+    var_AddCallback (obj, "audio-device", aout_ChannelsRestart, NULL);
+    var_TriggerCallback (obj, "intf-change");
 }
 
 /*****************************************************************************
@@ -285,364 +165,315 @@ static void Probe( aout_instance_t * p_aout,
  * Note: the only heap-allocated string is psz_device. All the other pointers
  * are references to psz_device or to stack-allocated data.
  *****************************************************************************/
-static int Open( vlc_object_t *p_this )
+static int Open (vlc_object_t *obj)
 {
-    aout_instance_t * p_aout = (aout_instance_t *)p_this;
-    struct aout_sys_t * p_sys;
-    vlc_value_t val;
-
-    char psz_default_iec_device[128]; /* Buffer used to store the default
-                                         S/PDIF device */
-    char * psz_device, * psz_iec_device; /* device names for PCM and S/PDIF
-                                            output */
-
-    int i_vlc_pcm_format; /* Audio format for VLC's data */
-    int i_snd_pcm_format; /* Audio format for ALSA's data */
-
-    snd_pcm_uframes_t i_buffer_size = 0;
-    snd_pcm_uframes_t i_period_size = 0;
-    int i_channels = 0;
+    audio_output_t * p_aout = (audio_output_t *)obj;
 
-    snd_pcm_hw_params_t *p_hw;
-    snd_pcm_sw_params_t *p_sw;
-
-    int i_snd_rc = -1;
-    unsigned int i_old_rate;
-    bool b_retry = true;
+    /* Get device name */
+    char *psz_device;
 
-    /* Allocate structures */
-    p_aout->output.p_sys = p_sys = malloc( sizeof( aout_sys_t ) );
-    if( p_sys == NULL )
+    if (var_Type (p_aout, "audio-device"))
+        psz_device = var_GetString (p_aout, "audio-device");
+    else
+        psz_device = var_InheritString( p_aout, "alsa-audio-device" );
+    if (unlikely(psz_device == NULL))
         return VLC_ENOMEM;
-    p_sys->b_playing = false;
-    p_sys->start_date = 0;
-    vlc_cond_init( &p_sys->wait );
-    vlc_mutex_init( &p_sys->lock );
 
-    /* Get device name */
-    if( (psz_device = config_GetPsz( p_aout, "alsa-audio-device" )) == NULL )
+    snd_pcm_format_t pcm_format; /* ALSA sample format */
+    vlc_fourcc_t fourcc = p_aout->format.i_format;
+    bool spdif = false;
+
+    switch (fourcc)
     {
-        msg_Err( p_aout, "no audio device given (maybe \"default\" ?)" );
-        dialog_Fatal( p_aout, _("No Audio Device"), "%s",
-                        _("No audio device name was given. You might want to " \
-                          "enter \"default\".") );
-        free( p_sys );
-        return VLC_EGENERIC;
+        case VLC_CODEC_F64B:
+            pcm_format = SND_PCM_FORMAT_FLOAT64_BE;
+            break;
+        case VLC_CODEC_F64L:
+            pcm_format = SND_PCM_FORMAT_FLOAT64_LE;
+            break;
+        case VLC_CODEC_F32B:
+            pcm_format = SND_PCM_FORMAT_FLOAT_BE;
+            break;
+        case VLC_CODEC_F32L:
+            pcm_format = SND_PCM_FORMAT_FLOAT_LE;
+            break;
+        case VLC_CODEC_FI32:
+            fourcc = VLC_CODEC_FL32;
+            pcm_format = SND_PCM_FORMAT_FLOAT;
+            break;
+        case VLC_CODEC_S32B:
+            pcm_format = SND_PCM_FORMAT_S32_BE;
+            break;
+        case VLC_CODEC_S32L:
+            pcm_format = SND_PCM_FORMAT_S32_LE;
+            break;
+        case VLC_CODEC_S24B:
+            pcm_format = SND_PCM_FORMAT_S24_3BE;
+            break;
+        case VLC_CODEC_S24L:
+            pcm_format = SND_PCM_FORMAT_S24_3LE;
+            break;
+        case VLC_CODEC_U24B:
+            pcm_format = SND_PCM_FORMAT_U24_3BE;
+            break;
+        case VLC_CODEC_U24L:
+            pcm_format = SND_PCM_FORMAT_U24_3LE;
+            break;
+        case VLC_CODEC_S16B:
+            pcm_format = SND_PCM_FORMAT_S16_BE;
+            break;
+        case VLC_CODEC_S16L:
+            pcm_format = SND_PCM_FORMAT_S16_LE;
+            break;
+        case VLC_CODEC_U16B:
+            pcm_format = SND_PCM_FORMAT_U16_BE;
+            break;
+        case VLC_CODEC_U16L:
+            pcm_format = SND_PCM_FORMAT_U16_LE;
+            break;
+        case VLC_CODEC_S8:
+            pcm_format = SND_PCM_FORMAT_S8;
+            break;
+        case VLC_CODEC_U8:
+            pcm_format = SND_PCM_FORMAT_U8;
+            break;
+        default:
+            if (AOUT_FMT_NON_LINEAR(&p_aout->format))
+                spdif = var_InheritBool (p_aout, "spdif");
+            if (HAVE_FPU)
+            {
+                fourcc = VLC_CODEC_FL32;
+                pcm_format = SND_PCM_FORMAT_FLOAT;
+            }
+            else
+            {
+                fourcc = VLC_CODEC_S16N;
+                pcm_format = SND_PCM_FORMAT_S16;
+            }
     }
 
     /* Choose the IEC device for S/PDIF output:
-       if the device is overriden by the user then it will be the one
+       if the device is overridden by the user then it will be the one
        otherwise we compute the default device based on the output format. */
-    if( AOUT_FMT_NON_LINEAR( &p_aout->output.output )
-        && !strcmp( psz_device, DEFAULT_ALSA_DEVICE ) )
-    {
-        snprintf( psz_default_iec_device, sizeof(psz_default_iec_device),
-                  "iec958:AES0=0x%x,AES1=0x%x,AES2=0x%x,AES3=0x%x",
-                  IEC958_AES0_CON_EMPHASIS_NONE | IEC958_AES0_NONAUDIO,
-                  IEC958_AES1_CON_ORIGINAL | IEC958_AES1_CON_PCM_CODER,
-                  0,
-                  ( p_aout->output.output.i_rate == 48000 ?
-                    IEC958_AES3_CON_FS_48000 :
-                    ( p_aout->output.output.i_rate == 44100 ?
-                      IEC958_AES3_CON_FS_44100 : IEC958_AES3_CON_FS_32000 ) ) );
-        psz_iec_device = psz_default_iec_device;
-    }
-    else if( AOUT_FMT_NON_LINEAR( &p_aout->output.output ) )
-    {
-        psz_iec_device = psz_device;
-    }
-    else
+    if (spdif && !strcmp (psz_device, DEFAULT_ALSA_DEVICE))
     {
-        psz_iec_device = NULL;
-    }
-
-    /* Choose the linear PCM format (read the comment above about FPU
-       and float32) */
-    if( vlc_CPU() & CPU_CAPABILITY_FPU )
-    {
-        i_vlc_pcm_format = VLC_CODEC_FL32;
-        i_snd_pcm_format = SND_PCM_FORMAT_FLOAT;
-    }
-    else
-    {
-        i_vlc_pcm_format = VLC_CODEC_S16N;
-        i_snd_pcm_format = SND_PCM_FORMAT_S16;
-    }
+        unsigned aes3;
 
-    /* If the variable doesn't exist then it's the first time we're called
-       and we have to probe the available audio formats and channels */
-    if ( var_Type( p_aout, "audio-device" ) == 0 )
-    {
-        Probe( p_aout, psz_device, psz_iec_device, &i_snd_pcm_format );
-    }
+        switch (p_aout->format.i_rate)
+        {
+#define FS(freq) \
+            case freq: aes3 = IEC958_AES3_CON_FS_ ## freq; break;
+            FS( 44100) /* def. */ FS( 48000) FS( 32000)
+            FS( 22050)            FS( 24000)
+            FS( 88200) FS(768000) FS( 96000)
+            FS(176400)            FS(192000)
+#undef FS
+            default:
+                aes3 = IEC958_AES3_CON_FS_NOTID;
+                break;
+        }
 
-    if ( var_Get( p_aout, "audio-device", &val ) < 0 )
-    {
-        free( p_sys );
-        free( psz_device );
-        return VLC_EGENERIC;
+        free (psz_device);
+        if (asprintf (&psz_device,
+                      "iec958:AES0=0x%x,AES1=0x%x,AES2=0x%x,AES3=0x%x",
+                      IEC958_AES0_CON_EMPHASIS_NONE | IEC958_AES0_NONAUDIO,
+                      IEC958_AES1_CON_ORIGINAL | IEC958_AES1_CON_PCM_CODER,
+                      0, aes3) == -1)
+            return VLC_ENOMEM;
     }
 
-    p_aout->output.output.i_format = i_vlc_pcm_format;
-    if ( val.i_int == AOUT_VAR_5_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_LFE;
-        free( psz_device );
-        psz_device = strdup( "surround51" );
-    }
-    else if ( val.i_int == AOUT_VAR_2F2R )
-    {
-        p_aout->output.output.i_physical_channels
-            = AOUT_CHAN_LEFT | AOUT_CHAN_RIGHT
-               | AOUT_CHAN_REARLEFT | AOUT_CHAN_REARRIGHT;
-        free( psz_device );
-        psz_device = strdup( "surround40" );
-    }
-    else if ( val.i_int == AOUT_VAR_STEREO )
-    {
-        p_aout->output.output.i_physical_channels
-            = AOUT_CHAN_LEFT | AOUT_CHAN_RIGHT;
-    }
-    else if ( val.i_int == AOUT_VAR_MONO )
-    {
-        p_aout->output.output.i_physical_channels = AOUT_CHAN_CENTER;
-    }
-    else if( val.i_int != AOUT_VAR_SPDIF )
+    /* Allocate structures */
+    aout_sys_t *p_sys = malloc (sizeof (*p_sys));
+    if (unlikely(p_sys == NULL))
     {
-        /* This should not happen ! */
-        msg_Err( p_aout, "internal: can't find audio-device (%i)", val.i_int );
-        free( p_sys );
-        free( psz_device );
-        return VLC_EGENERIC;
+        free (psz_device);
+        return VLC_ENOMEM;
     }
+    p_aout->sys = p_sys;
 
 #ifdef ALSA_DEBUG
     snd_output_stdio_attach( &p_sys->p_snd_stderr, stderr, 0 );
 #endif
 
     /* Open the device */
-    if ( val.i_int == AOUT_VAR_SPDIF )
+    msg_Dbg( p_aout, "opening ALSA device `%s'", psz_device );
+    int val = snd_pcm_open (&p_sys->p_snd_pcm, psz_device,
+                            SND_PCM_STREAM_PLAYBACK, mode);
+#if (SND_LIB_VERSION <= 0x010015)
+# warning Please update alsa-lib to version > 1.0.21a.
+    var_Create (p_aout->p_libvlc, "alsa-working", VLC_VAR_BOOL);
+    if (val != 0 && var_GetBool (p_aout->p_libvlc, "alsa-working"))
+        dialog_Fatal (p_aout, "ALSA version problem",
+            "VLC failed to re-initialize your audio output device.\n"
+            "Please update alsa-lib to version 1.0.22 or higher "
+            "to fix this issue.");
+    var_SetBool (p_aout->p_libvlc, "alsa-working", !val);
+#endif
+    if (val != 0)
     {
-        if ( ( i_snd_rc = snd_pcm_open( &p_sys->p_snd_pcm, psz_iec_device,
-                            SND_PCM_STREAM_PLAYBACK, 0 ) ) < 0 )
+#if (SND_LIB_VERSION <= 0x010017)
+# warning Please update alsa-lib to version > 1.0.23.
+        var_Create (p_aout->p_libvlc, "alsa-broken", VLC_VAR_BOOL);
+        if (!var_GetBool (p_aout->p_libvlc, "alsa-broken"))
         {
-            msg_Err( p_aout, "cannot open ALSA device `%s' (%s)",
-                             psz_iec_device, snd_strerror( i_snd_rc ) );
-            dialog_Fatal( p_aout, _("Audio output failed"),
-                            _("VLC could not open the ALSA device \"%s\" (%s)."),
-                            psz_iec_device, snd_strerror( i_snd_rc ) );
-            free( p_sys );
-            free( psz_device );
-            return VLC_EGENERIC;
+            var_SetBool (p_aout->p_libvlc, "alsa-broken", true);
+            dialog_Fatal (p_aout, "Potential ALSA version problem",
+                "VLC failed to initialize your audio output device (if any).\n"
+                "Please update alsa-lib to version 1.0.24 or higher "
+                "to try to fix this issue.");
         }
+#endif
+        msg_Err (p_aout, "cannot open ALSA device `%s' (%s)",
+                 psz_device, snd_strerror (val));
+        dialog_Fatal (p_aout, _("Audio output failed"),
+                      _("The audio device \"%s\" could not be used:\n%s."),
+                      psz_device, snd_strerror (val));
+        free (psz_device);
+        free (p_sys);
+        return VLC_EGENERIC;
+    }
+    free( psz_device );
+
+    snd_pcm_uframes_t i_buffer_size;
+    snd_pcm_uframes_t i_period_size;
+    unsigned channels;
+
+    if (spdif)
+    {
+        fourcc = VLC_CODEC_SPDIFL;
         i_buffer_size = ALSA_SPDIF_BUFFER_SIZE;
-        i_snd_pcm_format = SND_PCM_FORMAT_S16;
-        i_channels = 2;
+        pcm_format = SND_PCM_FORMAT_S16;
+        channels = 2;
 
-        i_vlc_pcm_format = VLC_CODEC_SPDIFL;
-        p_aout->output.i_nb_samples = i_period_size = ALSA_SPDIF_PERIOD_SIZE;
-        p_aout->output.output.i_bytes_per_frame = AOUT_SPDIF_SIZE;
-        p_aout->output.output.i_frame_length = A52_FRAME_NB;
+        i_period_size = ALSA_SPDIF_PERIOD_SIZE;
+        p_aout->format.i_bytes_per_frame = AOUT_SPDIF_SIZE;
+        p_aout->format.i_frame_length = A52_FRAME_NB;
 
         aout_VolumeNoneInit( p_aout );
     }
     else
     {
-        int i;
-
-        msg_Dbg( p_aout, "opening ALSA device `%s'", psz_device );
-
-        /* Since it seems snd_pcm_close hasn't really released the device at
-          the time it returns, probe if the device is available in loop for 1s.
-          We cannot use blocking mode since the we would wait indefinitely when
-          switching from a dmx device to surround51. */
-
-        for( i = 10; i >= 0; i-- )
-        {
-            if ( ( i_snd_rc = snd_pcm_open( &p_sys->p_snd_pcm, psz_device,
-                   SND_PCM_STREAM_PLAYBACK, SND_PCM_NONBLOCK ) ) == -EBUSY )
-            {
-                if( i ) msleep( 100000 /* 100ms */ );
-                else
-                {
-                    msg_Err( p_aout, "audio device: %s is already in use",
-                              psz_device );
-                    dialog_Fatal( p_aout, _("Audio output failed"),
-                                    _("The audio device \"%s\" is already in use."),
-                                    psz_device );
-                }
-                continue;
-            }
-            break;
-        }
-        if( i_snd_rc < 0 )
-        {
-            msg_Err( p_aout, "cannot open ALSA device `%s' (%s)",
-                             psz_device, snd_strerror( i_snd_rc ) );
-            dialog_Fatal( p_aout, _("Audio output failed"),
-                            _("VLC could not open the ALSA device \"%s\" (%s)."),
-                            psz_device, snd_strerror( i_snd_rc ) );
-            free( p_sys );
-            free( psz_device );
-            return VLC_EGENERIC;
-        }
-
-        /* We want blocking mode */
-        snd_pcm_nonblock( p_sys->p_snd_pcm, 0 );
-
         i_buffer_size = ALSA_DEFAULT_BUFFER_SIZE;
-        i_channels = aout_FormatNbChannels( &p_aout->output.output );
+        channels = aout_FormatNbChannels( &p_aout->format );
 
-        p_aout->output.i_nb_samples = i_period_size = ALSA_DEFAULT_PERIOD_SIZE;
+        i_period_size = ALSA_DEFAULT_PERIOD_SIZE;
 
         aout_VolumeSoftInit( p_aout );
     }
 
-    /* Free psz_device so that all the remaining data is stack-allocated */
-    free( psz_device );
+    p_aout->pf_play = Play;
+    p_aout->pf_pause = aout_PacketPause;
+    p_aout->pf_flush = aout_PacketFlush;
 
-    p_aout->output.pf_play = Play;
+    snd_pcm_hw_params_t *p_hw;
+    snd_pcm_sw_params_t *p_sw;
 
     snd_pcm_hw_params_alloca(&p_hw);
     snd_pcm_sw_params_alloca(&p_sw);
 
-    /* Due to some bugs in alsa with some drivers, we need to retry in s16l
-       if snd_pcm_hw_params fails in fl32 */
-    while ( b_retry )
+    /* Get Initial hardware parameters */
+    val = snd_pcm_hw_params_any( p_sys->p_snd_pcm, p_hw );
+    if( val < 0 )
     {
-        b_retry = false;
-
-        /* Get Initial hardware parameters */
-        if ( ( i_snd_rc = snd_pcm_hw_params_any( p_sys->p_snd_pcm, p_hw ) ) < 0 )
-        {
-            msg_Err( p_aout, "unable to retrieve initial hardware parameters (%s)",
-                         snd_strerror( i_snd_rc ) );
-            goto error;
-        }
+        msg_Err( p_aout, "unable to retrieve hardware parameters (%s)",
+                snd_strerror( val ) );
+        goto error;
+    }
 
-        /* Set format. */
-        if ( ( i_snd_rc = snd_pcm_hw_params_set_format( p_sys->p_snd_pcm, p_hw,
-                                                    i_snd_pcm_format ) ) < 0 )
-        {
-            if( i_snd_pcm_format != SND_PCM_FORMAT_S16 )
-            {
-                i_snd_pcm_format = SND_PCM_FORMAT_S16;
-                i_snd_rc = snd_pcm_hw_params_set_format( p_sys->p_snd_pcm,
-                                                     p_hw, i_snd_pcm_format );
-            }
-            if ( i_snd_rc < 0 )
-            {
-                msg_Err( p_aout, "unable to set stream sample size and "
-                     "word order (%s)", snd_strerror( i_snd_rc ) );
-                goto error;
-            }
-        }
-        if( i_vlc_pcm_format != VLC_CODEC_SPDIFL )
-        switch( i_snd_pcm_format )
-        {
-        case SND_PCM_FORMAT_FLOAT:
-            i_vlc_pcm_format = VLC_CODEC_FL32;
-            break;
-        case SND_PCM_FORMAT_S16:
-            i_vlc_pcm_format = VLC_CODEC_S16N;
-            break;
-        }
-        p_aout->output.output.i_format = i_vlc_pcm_format;
+    /* Set format. */
+    val = snd_pcm_hw_params_set_format (p_sys->p_snd_pcm, p_hw, pcm_format);
+    if( val < 0 )
+    {
+        msg_Err (p_aout, "cannot set sample format: %s", snd_strerror (val));
+        goto error;
+    }
 
-        if ( ( i_snd_rc = snd_pcm_hw_params_set_access( p_sys->p_snd_pcm, p_hw,
-                                    SND_PCM_ACCESS_RW_INTERLEAVED ) ) < 0 )
-        {
-            msg_Err( p_aout, "unable to set interleaved stream format (%s)",
-                             snd_strerror( i_snd_rc ) );
-            goto error;
-        }
+    val = snd_pcm_hw_params_set_access( p_sys->p_snd_pcm, p_hw,
+                                        SND_PCM_ACCESS_RW_INTERLEAVED );
+    if( val < 0 )
+    {
+        msg_Err( p_aout, "unable to set interleaved stream format (%s)",
+                 snd_strerror( val ) );
+        goto error;
+    }
 
-        /* Set channels. */
-        if ( ( i_snd_rc = snd_pcm_hw_params_set_channels( p_sys->p_snd_pcm, p_hw,
-                                                      i_channels ) ) < 0 )
-        {
-            msg_Err( p_aout, "unable to set number of output channels (%s)",
-                             snd_strerror( i_snd_rc ) );
-            goto error;
-        }
+    /* Set channels. */
+    val = snd_pcm_hw_params_set_channels (p_sys->p_snd_pcm, p_hw, channels);
+    if (val < 0 && channels > 2) /* Fallback to stereo */
+    {
+        val = snd_pcm_hw_params_set_channels (p_sys->p_snd_pcm, p_hw, 2);
+        channels = 2;
+    }
+    if (val < 0)
+    {
+        msg_Err( p_aout, "unable to set number of output channels (%s)",
+                 snd_strerror( val ) );
+        goto error;
+    }
 
-        /* Set rate. */
-        i_old_rate = p_aout->output.output.i_rate;
-        i_snd_rc = snd_pcm_hw_params_set_rate_near( p_sys->p_snd_pcm, p_hw,
-                                                &p_aout->output.output.i_rate,
-                                                NULL );
-        if( i_snd_rc < 0 || p_aout->output.output.i_rate != i_old_rate )
-        {
-            msg_Warn( p_aout, "The rate %d Hz is not supported by your " \
-                "hardware. Using %d Hz instead.\n", i_old_rate, \
-                p_aout->output.output.i_rate );
-        }
+    /* Set rate. */
+    unsigned rate = p_aout->format.i_rate;
+    val = snd_pcm_hw_params_set_rate_near (p_sys->p_snd_pcm, p_hw, &rate,
+                                           NULL);
+    if (val < 0)
+    {
+        msg_Err (p_aout, "unable to set sampling rate (%s)",
+                 snd_strerror (val));
+        goto error;
+    }
+    if (p_aout->format.i_rate != rate)
+        msg_Warn (p_aout, "resampling from %d Hz to %d Hz",
+                  p_aout->format.i_rate, rate);
 
-        /* Set period size. */
-        if ( ( i_snd_rc = snd_pcm_hw_params_set_period_size_near( p_sys->p_snd_pcm,
-                                    p_hw, &i_period_size, NULL ) ) < 0 )
-        {
-            msg_Err( p_aout, "unable to set period size (%s)",
-                         snd_strerror( i_snd_rc ) );
-            goto error;
-        }
-        p_aout->output.i_nb_samples = i_period_size;
+    /* Set period size. */
+    val = snd_pcm_hw_params_set_period_size_near( p_sys->p_snd_pcm, p_hw,
+                                                  &i_period_size, NULL );
+    if( val < 0 )
+    {
+        msg_Err( p_aout, "unable to set period size (%s)",
+                 snd_strerror( val ) );
+        goto error;
+    }
 
-/* Set buffer size. */
-        if ( ( i_snd_rc = snd_pcm_hw_params_set_buffer_size_near( p_sys->p_snd_pcm,
-                                    p_hw, &i_buffer_size ) ) < 0 )
-        {
-            msg_Err( p_aout, "unable to set buffer size (%s)",
-                         snd_strerror( i_snd_rc ) );
-            goto error;
-        }
+    /* Set buffer size. */
+    val = snd_pcm_hw_params_set_buffer_size_near( p_sys->p_snd_pcm, p_hw,
+                                                  &i_buffer_size );
+    if( val )
+    {
+        msg_Err( p_aout, "unable to set buffer size (%s)",
+                 snd_strerror( val ) );
+        goto error;
+    }
 
-        /* Commit hardware parameters. */
-        if ( ( i_snd_rc = snd_pcm_hw_params( p_sys->p_snd_pcm, p_hw ) ) < 0 )
-        {
-            if ( b_retry == false &&
-                                i_snd_pcm_format == SND_PCM_FORMAT_FLOAT)
-            {
-                b_retry = true;
-                i_snd_pcm_format = SND_PCM_FORMAT_S16;
-                p_aout->output.output.i_format = VLC_CODEC_S16N;
-                msg_Warn( p_aout, "unable to commit hardware configuration "
-                                  "with fl32 samples. Retrying with s16l (%s)",                                     snd_strerror( i_snd_rc ) );
-            }
-            else
-            {
-                msg_Err( p_aout, "unable to commit hardware configuration (%s)",
-                         snd_strerror( i_snd_rc ) );
-                goto error;
-            }
-        }
+    /* Commit hardware parameters. */
+    val = snd_pcm_hw_params( p_sys->p_snd_pcm, p_hw );
+    if( val < 0 )
+    {
+        msg_Err( p_aout, "unable to commit hardware configuration (%s)",
+                 snd_strerror( val ) );
+        goto error;
     }
 
-    if( ( i_snd_rc = snd_pcm_hw_params_get_period_time( p_hw,
-                                    &p_sys->i_period_time, NULL ) ) < 0 )
+    val = snd_pcm_hw_params_get_period_time( p_hw, &p_sys->i_period_time,
+                                             NULL );
+    if( val < 0 )
     {
         msg_Err( p_aout, "unable to get period time (%s)",
-                         snd_strerror( i_snd_rc ) );
+                 snd_strerror( val ) );
         goto error;
     }
 
     /* Get Initial software parameters */
     snd_pcm_sw_params_current( p_sys->p_snd_pcm, p_sw );
 
-    i_snd_rc = snd_pcm_sw_params_set_sleep_min( p_sys->p_snd_pcm, p_sw, 0 );
-
-    i_snd_rc = snd_pcm_sw_params_set_avail_min( p_sys->p_snd_pcm, p_sw,
-                                                p_aout->output.i_nb_samples );
+    snd_pcm_sw_params_set_avail_min( p_sys->p_snd_pcm, p_sw, i_period_size );
     /* start playing when one period has been written */
-    i_snd_rc = snd_pcm_sw_params_set_start_threshold( p_sys->p_snd_pcm, p_sw,
-                                                      ALSA_DEFAULT_PERIOD_SIZE);
-    if( i_snd_rc < 0 )
+    val = snd_pcm_sw_params_set_start_threshold( p_sys->p_snd_pcm, p_sw,
+                                                 ALSA_DEFAULT_PERIOD_SIZE);
+    if( val < 0 )
     {
         msg_Err( p_aout, "unable to set start threshold (%s)",
-                          snd_strerror( i_snd_rc ) );
+                 snd_strerror( val ) );
         goto error;
     }
 
@@ -661,14 +492,25 @@ static int Open( vlc_object_t *p_this )
     snd_output_printf( p_sys->p_snd_stderr, "\n" );
 #endif
 
+    p_sys->start_date = 0;
+    vlc_sem_init( &p_sys->wait, 0 );
+    aout_PacketInit (p_aout, &p_sys->packet, i_period_size);
+
     /* Create ALSA thread and wait for its readiness. */
-    if( vlc_thread_create( p_aout, "aout", ALSAThread,
-                           VLC_THREAD_PRIORITY_OUTPUT ) )
+    if( vlc_clone( &p_sys->thread, ALSAThread, p_aout,
+                   VLC_THREAD_PRIORITY_OUTPUT ) )
     {
         msg_Err( p_aout, "cannot create ALSA thread (%m)" );
+        vlc_sem_destroy( &p_sys->wait );
         goto error;
     }
 
+    p_aout->format.i_format = fourcc;
+    p_aout->format.i_rate = rate;
+    if (channels == 2)
+        p_aout->format.i_physical_channels = AOUT_CHAN_LEFT|AOUT_CHAN_RIGHT;
+
+    Probe (obj);
     return 0;
 
 error:
@@ -681,109 +523,78 @@ error:
 }
 
 /*****************************************************************************
- * Play: nothing to do
+ * Play: start playback
  *****************************************************************************/
-static void Play( aout_instance_t *p_aout )
+static void Play( audio_output_t *p_aout, block_t *block )
 {
-    if( !p_aout->output.p_sys->b_playing )
-    {
-        p_aout->output.p_sys->b_playing = true;
+    /* get the playing date of the first aout buffer */
+    p_aout->sys->start_date = block->i_pts;
 
-        /* get the playing date of the first aout buffer */
-        p_aout->output.p_sys->start_date =
-            aout_FifoFirstDate( p_aout, &p_aout->output.fifo );
+    aout_PacketPlay( p_aout, block );
+    p_aout->pf_play = aout_PacketPlay;
 
-        /* wake up the audio output thread */
-        vlc_mutex_lock( &p_aout->output.p_sys->lock );
-        vlc_cond_signal( &p_aout->output.p_sys->wait );
-        vlc_mutex_unlock( &p_aout->output.p_sys->lock );
-    }
+    /* wake up the audio output thread */
+    sem_post( &p_aout->sys->wait );
 }
 
 /*****************************************************************************
  * Close: close the ALSA device
  *****************************************************************************/
-static void Close( vlc_object_t *p_this )
+static void Close (vlc_object_t *obj)
 {
-    aout_instance_t *p_aout = (aout_instance_t *)p_this;
-    struct aout_sys_t * p_sys = p_aout->output.p_sys;
-    int i_snd_rc;
+    audio_output_t *p_aout = (audio_output_t *)obj;
+    struct aout_sys_t * p_sys = p_aout->sys;
 
     /* Make sure that the thread will stop once it is waken up */
-    vlc_object_kill( p_aout );
-
-    /* make sure the audio output thread is waken up */
-    vlc_mutex_lock( &p_aout->output.p_sys->lock );
-    vlc_cond_signal( &p_aout->output.p_sys->wait );
-    vlc_mutex_unlock( &p_aout->output.p_sys->lock );
-
-    /* */
-    vlc_thread_join( p_aout );
-    p_aout->b_die = false;
-
-    i_snd_rc = snd_pcm_close( p_sys->p_snd_pcm );
-
-    if( i_snd_rc > 0 )
-    {
-        msg_Err( p_aout, "failed closing ALSA device (%s)",
-                         snd_strerror( i_snd_rc ) );
-    }
+    vlc_cancel( p_sys->thread );
+    vlc_join( p_sys->thread, NULL );
+    vlc_sem_destroy( &p_sys->wait );
+    aout_PacketDestroy (p_aout);
 
+    snd_pcm_drop( p_sys->p_snd_pcm );
+    snd_pcm_close( p_sys->p_snd_pcm );
 #ifdef ALSA_DEBUG
     snd_output_close( p_sys->p_snd_stderr );
 #endif
-
     free( p_sys );
 }
 
 /*****************************************************************************
  * ALSAThread: asynchronous thread used to DMA the data to the device
  *****************************************************************************/
-static void* ALSAThread( vlc_object_t* p_this )
+static void* ALSAThread( void *data )
 {
-    aout_instance_t * p_aout = (aout_instance_t*)p_this;
-    struct aout_sys_t * p_sys = p_aout->output.p_sys;
-    int canc = vlc_savecancel ();
-    p_sys->p_status = (snd_pcm_status_t *)malloc(snd_pcm_status_sizeof());
+    audio_output_t * p_aout = data;
+    struct aout_sys_t * p_sys = p_aout->sys;
 
     /* Wait for the exact time to start playing (avoids resampling) */
-    vlc_mutex_lock( &p_sys->lock );
-    while( !p_sys->start_date && vlc_object_alive (p_aout) )
-        vlc_cond_wait( &p_sys->wait, &p_sys->lock );
-    vlc_mutex_unlock( &p_sys->lock );
-
-    if( !vlc_object_alive (p_aout) )
-        goto cleanup;
+    vlc_sem_wait( &p_sys->wait );
+    mwait( p_sys->start_date - AOUT_MAX_PTS_ADVANCE / 4 );
+#warning Should wait for buffer availability instead!
 
-    mwait( p_sys->start_date - AOUT_PTS_TOLERANCE / 4 );
-
-    while ( vlc_object_alive (p_aout) )
-    {
+    for(;;)
         ALSAFill( p_aout );
-    }
 
-cleanup:
-    snd_pcm_drop( p_sys->p_snd_pcm );
-    free( p_aout->output.p_sys->p_status );
-    vlc_restorecancel (canc);
-    return NULL;
+    assert(0);
 }
 
 /*****************************************************************************
  * ALSAFill: function used to fill the ALSA buffer as much as possible
  *****************************************************************************/
-static void ALSAFill( aout_instance_t * p_aout )
+static void ALSAFill( audio_output_t * p_aout )
 {
-    struct aout_sys_t * p_sys = p_aout->output.p_sys;
-    aout_buffer_t * p_buffer;
-    snd_pcm_status_t * p_status = p_sys->p_status;
+    struct aout_sys_t * p_sys = p_aout->sys;
+    snd_pcm_t *p_pcm = p_sys->p_snd_pcm;
+    snd_pcm_status_t * p_status;
     int i_snd_rc;
     mtime_t next_date;
 
+    int canc = vlc_savecancel();
     /* Fill in the buffer until space or audio output buffer shortage */
 
     /* Get the status */
-    i_snd_rc = snd_pcm_status( p_sys->p_snd_pcm, p_status );
+    snd_pcm_status_alloca(&p_status);
+    i_snd_rc = snd_pcm_status( p_pcm, p_status );
     if( i_snd_rc < 0 )
     {
         msg_Err( p_aout, "cannot get device status" );
@@ -794,8 +605,7 @@ static void ALSAFill( aout_instance_t * p_aout )
     if( snd_pcm_status_get_state( p_status ) == SND_PCM_STATE_XRUN )
     {
         /* Prepare the device */
-        i_snd_rc = snd_pcm_prepare( p_sys->p_snd_pcm );
-
+        i_snd_rc = snd_pcm_prepare( p_pcm );
         if( i_snd_rc )
         {
             msg_Err( p_aout, "cannot recover from buffer underrun" );
@@ -805,7 +615,7 @@ static void ALSAFill( aout_instance_t * p_aout )
         msg_Dbg( p_aout, "recovered from buffer underrun" );
 
         /* Get the new status */
-        i_snd_rc = snd_pcm_status( p_sys->p_snd_pcm, p_status );
+        i_snd_rc = snd_pcm_status( p_pcm, p_status );
         if( i_snd_rc < 0 )
         {
             msg_Err( p_aout, "cannot get device status after recovery" );
@@ -820,73 +630,93 @@ static void ALSAFill( aout_instance_t * p_aout )
         /* Here the device should be in RUNNING state, p_status is valid. */
         snd_pcm_sframes_t delay = snd_pcm_status_get_delay( p_status );
         if( delay == 0 ) /* workaround buggy alsa drivers */
-            if( snd_pcm_delay( p_sys->p_snd_pcm, &delay ) < 0 )
+            if( snd_pcm_delay( p_pcm, &delay ) < 0 )
                 delay = 0; /* FIXME: use a positive minimal delay */
-        int i_bytes = snd_pcm_frames_to_bytes( p_sys->p_snd_pcm, delay );
-        next_date = mdate() + ( (mtime_t)i_bytes * 1000000
-                / p_aout->output.output.i_bytes_per_frame
-                / p_aout->output.output.i_rate
-                * p_aout->output.output.i_frame_length );
+
+        size_t i_bytes = snd_pcm_frames_to_bytes( p_pcm, delay );
+        mtime_t delay_us = CLOCK_FREQ * i_bytes
+                / p_aout->format.i_bytes_per_frame
+                / p_aout->format.i_rate
+                * p_aout->format.i_frame_length;
 
 #ifdef ALSA_DEBUG
         snd_pcm_state_t state = snd_pcm_status_get_state( p_status );
         if( state != SND_PCM_STATE_RUNNING )
             msg_Err( p_aout, "pcm status (%d) != RUNNING", state );
 
-        msg_Dbg( p_aout, "Delay is %ld frames (%d bytes)", delay, i_bytes );
+        msg_Dbg( p_aout, "Delay is %ld frames (%zu bytes)", delay, i_bytes );
 
-        msg_Dbg( p_aout, "Bytes per frame: %d", p_aout->output.output.i_bytes_per_frame );
-        msg_Dbg( p_aout, "Rate: %d", p_aout->output.output.i_rate );
-        msg_Dbg( p_aout, "Frame length: %d", p_aout->output.output.i_frame_length );
-
-        msg_Dbg( p_aout, "Next date is in %d microseconds", (int)(next_date - mdate()) );
+        msg_Dbg( p_aout, "Bytes per frame: %d", p_aout->format.i_bytes_per_frame );
+        msg_Dbg( p_aout, "Rate: %d", p_aout->format.i_rate );
+        msg_Dbg( p_aout, "Frame length: %d", p_aout->format.i_frame_length );
+        msg_Dbg( p_aout, "Next date: in %"PRId64" microseconds", delay_us );
 #endif
+        next_date = mdate() + delay_us;
     }
 
-    p_buffer = aout_OutputNextBuffer( p_aout, next_date,
-           (p_aout->output.output.i_format ==  VLC_CODEC_SPDIFL) );
+    block_t *p_buffer = aout_OutputNextBuffer( p_aout, next_date,
+           (p_aout->format.i_format ==  VLC_CODEC_SPDIFL) );
 
     /* Audio output buffer shortage -> stop the fill process and wait */
     if( p_buffer == NULL )
         goto error;
 
+    block_cleanup_push( p_buffer );
     for (;;)
     {
-        i_snd_rc = snd_pcm_writei( p_sys->p_snd_pcm, p_buffer->p_buffer,
-                                   p_buffer->i_nb_samples );
-        if( i_snd_rc != -ESTRPIPE )
-            break;
+        int n = snd_pcm_poll_descriptors_count(p_pcm);
+        struct pollfd ufd[n];
+        unsigned short revents;
+
+        snd_pcm_poll_descriptors(p_pcm, ufd, n);
+        do
+        {
+            vlc_restorecancel(canc);
+            poll(ufd, n, -1);
+            canc = vlc_savecancel();
+            snd_pcm_poll_descriptors_revents(p_pcm, ufd, n, &revents);
+        }
+        while(!revents);
+
+        if(revents & POLLOUT)
+        {
+            i_snd_rc = snd_pcm_writei( p_pcm, p_buffer->p_buffer,
+                                       p_buffer->i_nb_samples );
+            if( i_snd_rc != -ESTRPIPE )
+                break;
+        }
 
         /* a suspend event occurred
          * (stream is suspended and waiting for an application recovery) */
         msg_Dbg( p_aout, "entering in suspend mode, trying to resume..." );
 
-        while( vlc_object_alive (p_aout) && vlc_object_alive (p_aout->p_libvlc) &&
-               ( i_snd_rc = snd_pcm_resume( p_sys->p_snd_pcm ) ) == -EAGAIN )
+        while( ( i_snd_rc = snd_pcm_resume( p_pcm ) ) == -EAGAIN )
         {
-            msleep( 1000000 );
+            vlc_restorecancel(canc);
+            msleep(CLOCK_FREQ); /* device still suspended, wait... */
+            canc = vlc_savecancel();
         }
 
         if( i_snd_rc < 0 )
-            /* Device does not supprot resuming, restart it */
-            i_snd_rc = snd_pcm_prepare( p_sys->p_snd_pcm );
+            /* Device does not support resuming, restart it */
+            i_snd_rc = snd_pcm_prepare( p_pcm );
 
     }
 
     if( i_snd_rc < 0 )
         msg_Err( p_aout, "cannot write: %s", snd_strerror( i_snd_rc ) );
 
-    aout_BufferFree( p_buffer );
+    vlc_restorecancel(canc);
+    vlc_cleanup_run();
     return;
 
 error:
     if( i_snd_rc < 0 )
         msg_Err( p_aout, "ALSA error: %s", snd_strerror( i_snd_rc ) );
-    msleep( p_sys->i_period_time >> 1 );
-}
 
-static void GetDevicesForCard( module_config_t *p_item, int i_card );
-static void GetDevices( module_config_t *p_item );
+    vlc_restorecancel(canc);
+    msleep(p_sys->i_period_time / 2);
+}
 
 /*****************************************************************************
  * config variable callback
@@ -895,7 +725,6 @@ static int FindDevicesCallback( vlc_object_t *p_this, char const *psz_name,
                                vlc_value_t newval, vlc_value_t oldval, void *p_unused )
 {
     module_config_t *p_item;
-    int i;
     (void)newval;
     (void)oldval;
     (void)p_unused;
@@ -906,6 +735,8 @@ static int FindDevicesCallback( vlc_object_t *p_this, char const *psz_name,
     /* Clear-up the current list */
     if( p_item->i_list )
     {
+        int i;
+
         /* Keep the first entrie */
         for( i = 1; i < p_item->i_list; i++ )
         {
@@ -918,7 +749,7 @@ static int FindDevicesCallback( vlc_object_t *p_this, char const *psz_name,
     }
     p_item->i_list = 1;
 
-    GetDevices( p_item );
+    GetDevices( p_this, p_item );
 
     /* Signal change to the interface */
     p_item->b_dirty = true;
@@ -927,94 +758,65 @@ static int FindDevicesCallback( vlc_object_t *p_this, char const *psz_name,
 }
 
 
-static void GetDevicesForCard( module_config_t *p_item, int i_card )
+static void GetDevices (vlc_object_t *obj, module_config_t *item)
 {
-    int i_pcm_device = -1;
-    int i_err = 0;
-    snd_pcm_info_t *p_pcm_info;
-    snd_ctl_t *p_ctl;
-    char psz_dev[64];
-    char *psz_card_name;
+    void **hints;
 
-    sprintf( psz_dev, "hw:%i", i_card );
+    msg_Dbg(obj, "Available ALSA PCM devices:");
 
-    if( ( i_err = snd_ctl_open( &p_ctl, psz_dev, 0 ) ) < 0 )
+    if (snd_device_name_hint(-1, "pcm", &hints) < 0)
         return;
 
-    if( ( i_err = snd_card_get_name( i_card, &psz_card_name ) ) != 0)
-        psz_card_name = _("Unknown soundcard");
-
-    snd_pcm_info_alloca( &p_pcm_info );
-
-    for (;;)
+    for (size_t i = 0; hints[i] != NULL; i++)
     {
-        char *psz_device, *psz_descr;
-        if( ( i_err = snd_ctl_pcm_next_device( p_ctl, &i_pcm_device ) ) < 0 )
-            i_pcm_device = -1;
-        if( i_pcm_device < 0 )
-            break;
+        void *hint = hints[i];
+        char *dev;
 
-        snd_pcm_info_set_device( p_pcm_info, i_pcm_device );
-        snd_pcm_info_set_subdevice( p_pcm_info, 0 );
-        snd_pcm_info_set_stream( p_pcm_info, SND_PCM_STREAM_PLAYBACK );
-
-        if( ( i_err = snd_ctl_pcm_info( p_ctl, p_pcm_info ) ) < 0 )
+        char *name = snd_device_name_get_hint(hint, "NAME");
+        if (unlikely(name == NULL))
+            continue;
+        if (unlikely(asprintf (&dev, "plug:'%s'", name) == -1))
         {
-            if( i_err != -ENOENT )
-            {
-                /*printf( "get_devices_for_card(): "
-                         "snd_ctl_pcm_info() "
-                         "failed (%d:%d): %s.\n", i_card,
-                         i_pcm_device, snd_strerror( -i_err ) );*/
-            }
+            free(name);
             continue;
         }
 
-        if( asprintf( &psz_device, "hw:%d,%d", i_card, i_pcm_device ) == -1 )
-            break;
-        if( asprintf( &psz_descr, "%s: %s (%s)", psz_card_name,
-                  snd_pcm_info_get_name(p_pcm_info), psz_device ) == -1 )
+        char *desc = snd_device_name_get_hint(hint, "DESC");
+        if (desc != NULL)
+            for (char *lf = strchr(desc, '\n'); lf; lf = strchr(lf, '\n'))
+                 *lf = ' ';
+        msg_Dbg(obj, "%s (%s)", (desc != NULL) ? desc : name, name);
+
+        if (item != NULL)
         {
-            free( psz_device );
-            break;
+            item->ppsz_list = xrealloc(item->ppsz_list,
+                                       (item->i_list + 2) * sizeof(char *));
+            item->ppsz_list_text = xrealloc(item->ppsz_list_text,
+                                          (item->i_list + 2) * sizeof(char *));
+            item->ppsz_list[item->i_list] = dev;
+            if (desc == NULL)
+                desc = strdup(name);
+            item->ppsz_list_text[item->i_list] = desc;
+            item->i_list++;
+        }
+        else
+        {
+            vlc_value_t val, text;
+
+            val.psz_string = dev;
+            text.psz_string = desc;
+            var_Change(obj, "audio-device", VLC_VAR_ADDCHOICE, &val, &text);
+            free(desc);
+            free(dev);
+            free(name);
         }
-
-        p_item->ppsz_list =
-            (char **)realloc( p_item->ppsz_list,
-                              (p_item->i_list + 2) * sizeof(char *) );
-        p_item->ppsz_list_text =
-            (char **)realloc( p_item->ppsz_list_text,
-                              (p_item->i_list + 2) * sizeof(char *) );
-        p_item->ppsz_list[ p_item->i_list ] = psz_device;
-        p_item->ppsz_list_text[ p_item->i_list ] = psz_descr;
-        p_item->i_list++;
-        p_item->ppsz_list[ p_item->i_list ] = NULL;
-        p_item->ppsz_list_text[ p_item->i_list ] = NULL;
     }
 
-    snd_ctl_close( p_ctl );
-}
-
-
-
-static void GetDevices( module_config_t *p_item )
-{
-    int i_card = -1;
-    int i_err = 0;
-
-    if( ( i_err = snd_card_next( &i_card ) ) != 0 )
-    {
-        /*printf( "snd_card_next() failed: %s", snd_strerror( -i_err ) );*/
-        return;
-    }
+    snd_device_name_free_hint(hints);
 
-    while( i_card > -1 )
+    if (item != NULL)
     {
-        GetDevicesForCard( p_item, i_card );
-        if( ( i_err = snd_card_next( &i_card ) ) != 0 )
-        {
-            /*printf( "snd_card_next() failed: %s", snd_strerror( -i_err ) );*/
-            break;
-        }
+        item->ppsz_list[item->i_list] = NULL;
+        item->ppsz_list_text[item->i_list] = NULL;
     }
 }