gst-indent
[platform/upstream/gst-plugins-good.git] / sys / oss / gstosssink.c
index 56b95e9..053eba4 100644 (file)
  * Boston, MA 02111-1307, USA.
  */
 
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
 #include <sys/ioctl.h>
 #include <sys/soundcard.h>
+#include <errno.h>
+#include <unistd.h>
+#include <string.h>
 
-#include <gstosssink.h>
+#include "gstosssink.h"
 
 /* elementfactory information */
-static GstElementDetails gst_osssink_details = {  
-  "Audio Sink (OSS)",
-  "Sink/Audio",
-  "LGPL",
-  "Output to a sound card via OSS",
-  VERSION,
-  "Erik Walthinsen <omega@cse.ogi.edu>, "
-  "Wim Taymans <wim.taymans@chello.be>",
-  "(C) 1999",
-};
-
-static void                    gst_osssink_class_init          (GstOssSinkClass *klass);
-static void                    gst_osssink_init                (GstOssSink *osssink);
-static void                    gst_osssink_finalize            (GObject *object);
-
-static GstElementStateReturn   gst_osssink_change_state        (GstElement *element);
-static void                    gst_osssink_set_clock           (GstElement *element, GstClock *clock);
-static GstClock*               gst_osssink_get_clock           (GstElement *element);
-static GstClockTime            gst_osssink_get_time            (GstClock *clock, gpointer data);
-
-static const GstFormat*        gst_osssink_get_formats         (GstPad *pad);
-static gboolean                gst_osssink_convert             (GstPad *pad, GstFormat src_format, gint64 src_value,
-                                                                GstFormat *dest_format, gint64 *dest_value);
-static const GstQueryType*     gst_osssink_get_query_types     (GstPad *pad);
-static gboolean                gst_osssink_query               (GstElement *element, GstQueryType type, 
-                                                                GstFormat *format, gint64 *value);
-static gboolean                gst_osssink_sink_query          (GstPad *pad, GstQueryType type,
-                                                                GstFormat *format, gint64 *value);
-
-static GstPadConnectReturn     gst_osssink_sinkconnect         (GstPad *pad, GstCaps *caps);
-
-static void                    gst_osssink_set_property        (GObject *object, guint prop_id, const GValue *value, 
-                                                                GParamSpec *pspec);
-static void                    gst_osssink_get_property        (GObject *object, guint prop_id, GValue *value, 
-                                                                GParamSpec *pspec);
-
-static void                    gst_osssink_chain               (GstPad *pad,GstBuffer *buf);
+static GstElementDetails gst_osssink_details =
+GST_ELEMENT_DETAILS ("Audio Sink (OSS)",
+    "Sink/Audio",
+    "Output to a sound card via OSS",
+    "Erik Walthinsen <omega@cse.ogi.edu>, "
+    "Wim Taymans <wim.taymans@chello.be>");
+
+static void gst_osssink_base_init (gpointer g_class);
+static void gst_osssink_class_init (GstOssSinkClass * klass);
+static void gst_osssink_init (GstOssSink * osssink);
+static void gst_osssink_dispose (GObject * object);
+
+static GstElementStateReturn gst_osssink_change_state (GstElement * element);
+static void gst_osssink_set_clock (GstElement * element, GstClock * clock);
+static GstClock *gst_osssink_get_clock (GstElement * element);
+static GstClockTime gst_osssink_get_time (GstClock * clock, gpointer data);
+
+static const GstFormat *gst_osssink_get_formats (GstPad * pad);
+static gboolean gst_osssink_convert (GstPad * pad, GstFormat src_format,
+    gint64 src_value, GstFormat * dest_format, gint64 * dest_value);
+static const GstQueryType *gst_osssink_get_query_types (GstPad * pad);
+static gboolean gst_osssink_query (GstElement * element, GstQueryType type,
+    GstFormat * format, gint64 * value);
+static gboolean gst_osssink_sink_query (GstPad * pad, GstQueryType type,
+    GstFormat * format, gint64 * value);
+
+static GstCaps *gst_osssink_sink_fixate (GstPad * pad, const GstCaps * caps);
+static GstPadLinkReturn gst_osssink_sinkconnect (GstPad * pad,
+    const GstCaps * caps);
+
+static void gst_osssink_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec);
+static void gst_osssink_get_property (GObject * object, guint prop_id,
+    GValue * value, GParamSpec * pspec);
+
+static void gst_osssink_chain (GstPad * pad, GstData * _data);
 
 /* OssSink signals and args */
-enum {
+enum
+{
   SIGNAL_HANDOFF,
   LAST_SIGNAL
 };
 
-enum {
+enum
+{
   ARG_0,
-  ARG_DEVICE,
   ARG_MUTE,
   ARG_FRAGMENT,
   ARG_BUFFER_SIZE,
-  ARG_SYNC
+  ARG_SYNC,
+  ARG_CHUNK_SIZE,
   /* FILL ME */
 };
 
-GST_PAD_TEMPLATE_FACTORY (osssink_sink_factory,
-  "sink",
-  GST_PAD_SINK,
-  GST_PAD_ALWAYS,
-  GST_CAPS_NEW (
-    "osssink_sink",
-    "audio/raw",
-      "format",     GST_PROPS_STRING ("int"),   /* hack */
-      "law",        GST_PROPS_INT (0),
-      "endianness", GST_PROPS_INT (G_BYTE_ORDER),
-      "signed",     GST_PROPS_LIST (
-                     GST_PROPS_BOOLEAN (FALSE),
-                     GST_PROPS_BOOLEAN (TRUE)
-                   ),
-      "width",      GST_PROPS_LIST (
-                     GST_PROPS_INT (8),
-                     GST_PROPS_INT (16)
-                   ),
-      "depth",      GST_PROPS_LIST (
-                     GST_PROPS_INT (8),
-                     GST_PROPS_INT (16)
-                   ),
-      "rate",       GST_PROPS_INT_RANGE (1000, 48000),
-      "channels",   GST_PROPS_INT_RANGE (1, 2)
-  )
-);
+static GstStaticPadTemplate osssink_sink_factory =
+GST_STATIC_PAD_TEMPLATE ("sink",
+    GST_PAD_SINK,
+    GST_PAD_ALWAYS,
+    GST_STATIC_CAPS ("audio/x-raw-int, "
+       "endianness = (int) BYTE_ORDER, "
+       "signed = (boolean) { TRUE, FALSE }, "
+       "width = (int) { 8, 16 }, "
+       "depth = (int) { 8, 16 }, "
+       "rate = (int) [ 1000, 48000 ], " "channels = (int) [ 1, 2 ]")
+    );
 
 static GstElementClass *parent_class = NULL;
 static guint gst_osssink_signals[LAST_SIGNAL] = { 0 };
 
 GType
-gst_osssink_get_type (void) 
+gst_osssink_get_type (void)
 {
   static GType osssink_type = 0;
 
   if (!osssink_type) {
     static const GTypeInfo osssink_info = {
-      sizeof(GstOssSinkClass),
-      NULL,
+      sizeof (GstOssSinkClass),
+      gst_osssink_base_init,
       NULL,
-      (GClassInitFunc)gst_osssink_class_init,
+      (GClassInitFunc) gst_osssink_class_init,
       NULL,
       NULL,
-      sizeof(GstOssSink),
+      sizeof (GstOssSink),
       0,
-      (GInstanceInitFunc)gst_osssink_init,
+      (GInstanceInitFunc) gst_osssink_init,
     };
-    osssink_type = g_type_register_static (GST_TYPE_ELEMENT, "GstOssSink", &osssink_info, 0);
+    osssink_type =
+       g_type_register_static (GST_TYPE_OSSELEMENT, "GstOssSink",
+       &osssink_info, 0);
   }
 
   return osssink_type;
 }
 
-static GstBufferPool*
-gst_osssink_get_bufferpool (GstPad *pad)
+static void
+gst_osssink_dispose (GObject * object)
 {
-  GstOssSink *oss;
-  
-  oss = GST_OSSSINK (gst_pad_get_parent(pad));
+  GstOssSink *osssink = (GstOssSink *) object;
 
-  /* 6 buffers per chunk by default */
-  if (!oss->sinkpool)
-    oss->sinkpool = gst_buffer_pool_get_default (oss->bufsize, 6);
+  gst_object_unparent (GST_OBJECT (osssink->provided_clock));
 
-  return oss->sinkpool;
+  G_OBJECT_CLASS (parent_class)->dispose (object);
 }
 
 static void
-gst_osssink_finalize (GObject *object)
+gst_osssink_base_init (gpointer g_class)
 {
-  GstOssSink *osssink = (GstOssSink *) object;
+  GstElementClass *element_class = GST_ELEMENT_CLASS (g_class);
 
-  g_free (osssink->common.device);
-
-  G_OBJECT_CLASS (parent_class)->finalize (object);
+  gst_element_class_set_details (element_class, &gst_osssink_details);
+  gst_element_class_add_pad_template (element_class,
+      gst_static_pad_template_get (&osssink_sink_factory));
 }
-
 static void
-gst_osssink_class_init (GstOssSinkClass *klass) 
+gst_osssink_class_init (GstOssSinkClass * klass)
 {
   GObjectClass *gobject_class;
   GstElementClass *gstelement_class;
 
-  gobject_class = (GObjectClass*)klass;
-  gstelement_class = (GstElementClass*)klass;
+  gobject_class = (GObjectClass *) klass;
+  gstelement_class = (GstElementClass *) klass;
 
-  parent_class = g_type_class_ref(GST_TYPE_ELEMENT);
+  parent_class = g_type_class_ref (GST_TYPE_OSSELEMENT);
 
-  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_DEVICE,
-    g_param_spec_string ("device", "Device", "The device to use for output",
-                         "/dev/dsp", G_PARAM_READWRITE)); /* CHECKME! */
   g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_MUTE,
-    g_param_spec_boolean ("mute", "Mute", "Mute the audio",
-                          TRUE, G_PARAM_READWRITE)); 
+      g_param_spec_boolean ("mute", "Mute", "Mute the audio",
+         FALSE, G_PARAM_READWRITE));
   g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_SYNC,
-    g_param_spec_boolean ("sync", "Sync", "If syncing on timestamps should be enabled",
-                          TRUE, G_PARAM_READWRITE)); 
+      g_param_spec_boolean ("sync", "Sync",
+         "If syncing on timestamps should be enabled", TRUE,
+         G_PARAM_READWRITE));
   g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_FRAGMENT,
-    g_param_spec_int ("fragment", "Fragment", 
-                     "The fragment as 0xMMMMSSSS (MMMM = total fragments, 2^SSSS = fragment size)",
-                      0, G_MAXINT, 6, G_PARAM_READWRITE));
+      g_param_spec_int ("fragment", "Fragment",
+         "The fragment as 0xMMMMSSSS (MMMM = total fragments, 2^SSSS = fragment size)",
+         0, G_MAXINT, 6, G_PARAM_READWRITE));
   g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_BUFFER_SIZE,
-    g_param_spec_int ("buffer_size", "Buffer size", "The buffer size",
-                      0, G_MAXINT, 4096, G_PARAM_READWRITE));
+      g_param_spec_uint ("buffer_size", "Buffer size",
+         "Size of buffers in osssink's bufferpool (bytes)", 0, G_MAXINT, 4096,
+         G_PARAM_READWRITE));
+  g_object_class_install_property (G_OBJECT_CLASS (klass), ARG_CHUNK_SIZE,
+      g_param_spec_uint ("chunk_size", "Chunk size",
+         "Write data in chunk sized buffers", 0, G_MAXUINT, 4096,
+         G_PARAM_READWRITE));
 
   gst_osssink_signals[SIGNAL_HANDOFF] =
-    g_signal_new ("handoff", G_TYPE_FROM_CLASS (klass), G_SIGNAL_RUN_LAST,
-                  G_STRUCT_OFFSET (GstOssSinkClass, handoff), NULL, NULL,
-                  g_cclosure_marshal_VOID__VOID, G_TYPE_NONE, 0);
-  
+      g_signal_new ("handoff", G_TYPE_FROM_CLASS (klass), G_SIGNAL_RUN_LAST,
+      G_STRUCT_OFFSET (GstOssSinkClass, handoff), NULL, NULL,
+      g_cclosure_marshal_VOID__VOID, G_TYPE_NONE, 0);
+
   gobject_class->set_property = gst_osssink_set_property;
   gobject_class->get_property = gst_osssink_get_property;
-  gobject_class->finalize     = gst_osssink_finalize;
-  
+  gobject_class->dispose = gst_osssink_dispose;
+
   gstelement_class->change_state = GST_DEBUG_FUNCPTR (gst_osssink_change_state);
-  gstelement_class->query       = GST_DEBUG_FUNCPTR (gst_osssink_query);
-  gstelement_class->set_clock   = gst_osssink_set_clock;
-  gstelement_class->get_clock   = gst_osssink_get_clock;
-  
+  gstelement_class->query = GST_DEBUG_FUNCPTR (gst_osssink_query);
+  gstelement_class->set_clock = gst_osssink_set_clock;
+  gstelement_class->get_clock = gst_osssink_get_clock;
+
 }
 
-static void 
-gst_osssink_init (GstOssSink *osssink) 
+static void
+gst_osssink_init (GstOssSink * osssink)
 {
-  osssink->sinkpad = gst_pad_new_from_template (
-                 GST_PAD_TEMPLATE_GET (osssink_sink_factory), "sink");
+  osssink->sinkpad =
+      gst_pad_new_from_template (gst_static_pad_template_get
+      (&osssink_sink_factory), "sink");
   gst_element_add_pad (GST_ELEMENT (osssink), osssink->sinkpad);
   gst_pad_set_link_function (osssink->sinkpad, gst_osssink_sinkconnect);
-  gst_pad_set_bufferpool_function (osssink->sinkpad, gst_osssink_get_bufferpool);
+  gst_pad_set_fixate_function (osssink->sinkpad, gst_osssink_sink_fixate);
   gst_pad_set_convert_function (osssink->sinkpad, gst_osssink_convert);
   gst_pad_set_query_function (osssink->sinkpad, gst_osssink_sink_query);
-  gst_pad_set_query_type_function (osssink->sinkpad, gst_osssink_get_query_types);
+  gst_pad_set_query_type_function (osssink->sinkpad,
+      gst_osssink_get_query_types);
   gst_pad_set_formats_function (osssink->sinkpad, gst_osssink_get_formats);
 
   gst_pad_set_chain_function (osssink->sinkpad, gst_osssink_chain);
 
-  gst_osscommon_init (&osssink->common);
-
+  GST_DEBUG ("initializing osssink");
   osssink->bufsize = 4096;
-  osssink->resync = FALSE;
+  osssink->chunk_size = 4096;
+  osssink->mute = FALSE;
   osssink->sync = TRUE;
-  osssink->sinkpool = NULL;
-  osssink->provided_clock = GST_CLOCK (gst_oss_clock_new ("ossclock", gst_osssink_get_time, osssink));
+  osssink->resync = TRUE;
+  osssink->provided_clock =
+      gst_audio_clock_new ("ossclock", gst_osssink_get_time, osssink);
+  gst_object_set_parent (GST_OBJECT (osssink->provided_clock),
+      GST_OBJECT (osssink));
   osssink->handled = 0;
 
   GST_FLAG_SET (osssink, GST_ELEMENT_THREAD_SUGGESTED);
   GST_FLAG_SET (osssink, GST_ELEMENT_EVENT_AWARE);
 }
 
+static GstCaps *
+gst_osssink_sink_fixate (GstPad * pad, const GstCaps * caps)
+{
+  GstCaps *newcaps;
+  GstStructure *structure;
+
+  newcaps =
+      gst_caps_new_full (gst_structure_copy (gst_caps_get_structure (caps, 0)),
+      NULL);
+  structure = gst_caps_get_structure (newcaps, 0);
+
+  if (gst_caps_structure_fixate_field_nearest_int (structure, "rate", 44100)) {
+    return newcaps;
+  }
+  if (gst_caps_structure_fixate_field_nearest_int (structure, "depth", 16)) {
+    return newcaps;
+  }
+  if (gst_caps_structure_fixate_field_nearest_int (structure, "width", 16)) {
+    return newcaps;
+  }
+  if (gst_caps_structure_fixate_field_nearest_int (structure, "channels", 2)) {
+    return newcaps;
+  }
+
+  gst_caps_free (newcaps);
+
+  return NULL;
+}
 
-static GstPadConnectReturn 
-gst_osssink_sinkconnect (GstPad *pad, GstCaps *caps) 
+static GstPadLinkReturn
+gst_osssink_sinkconnect (GstPad * pad, const GstCaps * caps)
 {
   GstOssSink *osssink = GST_OSSSINK (gst_pad_get_parent (pad));
 
-  if (!GST_CAPS_IS_FIXED (caps))
-    return GST_PAD_LINK_DELAYED;
-
-  if (!gst_osscommon_parse_caps (&osssink->common, caps))
+  if (!gst_osselement_parse_caps (GST_OSSELEMENT (osssink), caps))
     return GST_PAD_LINK_REFUSED;
 
-  if (!gst_osscommon_sync_parms (&osssink->common)) {
+  if (!gst_osselement_sync_parms (GST_OSSELEMENT (osssink))) {
     return GST_PAD_LINK_REFUSED;
   }
 
   return GST_PAD_LINK_OK;
 }
 
-static inline gint64 
-gst_osssink_get_delay (GstOssSink *osssink) 
+static inline gint
+gst_osssink_get_delay (GstOssSink * osssink)
 {
   gint delay = 0;
+  gint ret;
 
-  if (osssink->common.fd == -1)
+  if (GST_OSSELEMENT (osssink)->fd == -1)
     return 0;
 
-  if (ioctl (osssink->common.fd, SNDCTL_DSP_GETODELAY, &delay) < 0) {
+#ifdef SNDCTL_DSP_GETODELAY
+  ret = ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_GETODELAY, &delay);
+#else
+  ret = -1;
+#endif
+  if (ret < 0) {
     audio_buf_info info;
-    if (ioctl (osssink->common.fd, SNDCTL_DSP_GETOSPACE, &info) < 0) {
+
+    if (ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_GETOSPACE, &info) < 0) {
       delay = 0;
-    }
-    else {
-      delay = (info.fragstotal * info.fragsize) - info.bytes;    
+    } else {
+      delay = (info.fragstotal * info.fragsize) - info.bytes;
     }
   }
+
   return delay;
 }
 
-static GstClockTime 
-gst_osssink_get_time (GstClock *clock, gpointer data) 
+static GstClockTime
+gst_osssink_get_time (GstClock * clock, gpointer data)
 {
   GstOssSink *osssink = GST_OSSSINK (data);
   gint delay;
-  GstClockTime res;
+  GstClockTimeDiff res;
 
-  if (!osssink->common.bps)
+  if (!GST_OSSELEMENT (osssink)->bps)
     return 0;
 
   delay = gst_osssink_get_delay (osssink);
 
-  /* sometimes delay is bigger than the number of bytes sent to the device, 
-   * which screws up this calculation, we assume that everything is still 
-   * in the device then */
-  if (((guint64)delay) > osssink->handled) {
+  /* sometimes delay is bigger than the number of bytes sent to the device,
+   * which screws up this calculation, we assume that everything is still
+   * in the device then
+   * thomas: with proper handling of the return value, this doesn't seem to
+   * happen anymore, so remove the second code path after april 2004 */
+  if (delay > (gint64) osssink->handled) {
+    /*g_warning ("Delay %d > osssink->handled %" G_GUINT64_FORMAT
+       ", setting to osssink->handled",
+       delay, osssink->handled); */
     delay = osssink->handled;
   }
-  res =  (osssink->handled - delay) * GST_SECOND / osssink->common.bps;
+  res =
+      ((gint64) osssink->handled -
+      delay) * GST_SECOND / GST_OSSELEMENT (osssink)->bps;
+  if (res < 0)
+    res = 0;
 
   return res;
 }
 
-static GstClock*
-gst_osssink_get_clock (GstElement *element)
+static GstClock *
+gst_osssink_get_clock (GstElement * element)
 {
   GstOssSink *osssink;
-           
+
   osssink = GST_OSSSINK (element);
 
   return GST_CLOCK (osssink->provided_clock);
 }
 
 static void
-gst_osssink_set_clock (GstElement *element, GstClock *clock)
+gst_osssink_set_clock (GstElement * element, GstClock * clock)
 {
   GstOssSink *osssink;
-  
+
   osssink = GST_OSSSINK (element);
 
-  osssink->clock = clock;  
+  osssink->clock = clock;
 }
 
-static void 
-gst_osssink_chain (GstPad *pad, GstBuffer *buf) 
+static void
+gst_osssink_chain (GstPad * pad, GstData * _data)
 {
+  GstBuffer *buf = GST_BUFFER (_data);
   GstOssSink *osssink;
-  GstClockTime buftime;
+  GstClockTimeDiff buftime, soundtime, elementtime;
+  guchar *data;
+  guint to_write;
+  gint delay;
 
   /* this has to be an audio buffer */
   osssink = GST_OSSSINK (gst_pad_get_parent (pad));
@@ -325,95 +367,122 @@ gst_osssink_chain (GstPad *pad, GstBuffer *buf)
 
     switch (GST_EVENT_TYPE (event)) {
       case GST_EVENT_EOS:
-        ioctl (osssink->common.fd, SNDCTL_DSP_SYNC);
-       gst_oss_clock_set_active (osssink->provided_clock, FALSE);
+       ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_SYNC, 0);
+       gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock),
+           FALSE);
        gst_pad_event_default (pad, event);
-        return;
-      case GST_EVENT_NEW_MEDIA:
-       g_print ("new media\n");
-        break;
+       return;
       case GST_EVENT_DISCONTINUOUS:
-      {
-       gint64 value;
-
-        ioctl (osssink->common.fd, SNDCTL_DSP_RESET);
-       if (gst_event_discont_get_value (event, GST_FORMAT_TIME, &value)) {
-          if (!gst_clock_handle_discont (osssink->clock, value))
-           gst_oss_clock_set_active (osssink->provided_clock, FALSE);
-         osssink->handled = 0;
-       }
        osssink->resync = TRUE;
-        break;
-      }
+       /* pass-through */
       default:
        gst_pad_event_default (pad, event);
-        return;
+       return;
     }
-    gst_event_unref (event);
-    return;
+    g_assert_not_reached ();
   }
 
-  if (!osssink->common.bps) {
+  if (!GST_OSSELEMENT (osssink)->bps) {
     gst_buffer_unref (buf);
-    gst_element_error (GST_ELEMENT (osssink), "capsnego was never performed, unknown data type");
+    GST_ELEMENT_ERROR (osssink, CORE, NEGOTIATION, (NULL),
+       ("format wasn't negotiated before chain function"));
     return;
   }
 
-  buftime = GST_BUFFER_TIMESTAMP (buf);
-
-  if (osssink->common.fd >= 0) {
-    if (!osssink->mute) {
-      guchar *data = GST_BUFFER_DATA (buf);
-      gint size = GST_BUFFER_SIZE (buf);
-
-      if (osssink->clock) {
-        gint delay = 0;
-       gint64 queued;
-       GstClockTimeDiff jitter;
-    
-       delay = gst_osssink_get_delay (osssink);
-       queued = delay * GST_SECOND / osssink->common.bps;
-
-       if  (osssink->resync && osssink->sync) {
-          GstClockID id = gst_clock_new_single_shot_id (osssink->clock, buftime - queued);
-
-         gst_element_clock_wait (GST_ELEMENT (osssink), id, &jitter);
-         gst_clock_id_free (id);
-
-         if (jitter >= 0) {
-            gst_clock_handle_discont (osssink->clock, buftime - queued + jitter);
-           write (osssink->common.fd, data, size);
-           gst_oss_clock_set_active (osssink->provided_clock, TRUE);
-           osssink->resync = FALSE;
-           osssink->handled += size;
+  data = GST_BUFFER_DATA (buf);
+  to_write = GST_BUFFER_SIZE (buf);
+  /* sync audio with buffers timestamp. elementtime is the *current* time.
+   * soundtime is the time if the soundcard has processed all queued data. */
+  elementtime = gst_element_get_time (GST_ELEMENT (osssink));
+  delay = gst_osssink_get_delay (osssink);
+  if (delay < 0)
+    delay = 0;
+  soundtime = elementtime + delay * GST_SECOND / GST_OSSELEMENT (osssink)->bps;
+  if (GST_BUFFER_TIMESTAMP_IS_VALID (buf)) {
+    buftime = GST_BUFFER_TIMESTAMP (buf);
+  } else {
+    buftime = soundtime;
+  }
+  GST_LOG_OBJECT (osssink,
+      "time: real %" G_GUINT64_FORMAT ", buffer: %" G_GUINT64_FORMAT, soundtime,
+      buftime);
+  if (MAX (buftime, soundtime) - MIN (buftime, soundtime) > (GST_SECOND / 10)) {
+    /* we need to adjust to the buffers here */
+    GST_INFO_OBJECT (osssink,
+       "need sync: real %" G_GUINT64_FORMAT ", buffer: %" G_GUINT64_FORMAT,
+       soundtime, buftime);
+    if (soundtime > buftime) {
+      /* do *not* throw frames out. It's useless. The next frame will come in
+       * too late. And the next one. And so on. We don't want to lose sound.
+       * This is a placeholder for what - some day - should become QoS, i.e.
+       * sending events upstream to drop buffers. */
+    } else {
+      guint64 to_handle =
+         (((buftime -
+                 soundtime) * GST_OSSELEMENT (osssink)->bps / GST_SECOND) /
+         ((GST_OSSELEMENT (osssink)->width / 8) *
+             GST_OSSELEMENT (osssink)->channels)) *
+         (GST_OSSELEMENT (osssink)->width / 8) *
+         GST_OSSELEMENT (osssink)->channels;
+
+      if (!osssink->resync) {
+       guint8 *buf = g_new (guint8, to_handle);
+
+       memset (buf, (GST_OSSELEMENT (osssink)->width == 8) ? 0 : 128,
+           to_handle);
+       while (to_handle > 0) {
+         gint done = write (GST_OSSELEMENT (osssink)->fd, buf,
+             MIN (to_handle, osssink->chunk_size));
+
+         if (done == -1 && errno != EINTR) {
+           break;
+         } else {
+           to_handle -= done;
+           osssink->handled += done;
          }
        }
-       else {
-         write (osssink->common.fd, data, size);
-         osssink->handled += size;
-        }
+       g_free (buf);
+      } else {
+       /* Timestamps at start-of-stream (MPEG) or after seek (hey,
+        * again MPEG!) can be borken, therefore this hacklet. */
+       osssink->handled += to_handle;
       }
-      /* no clock, try to be as fast as possible */
-      else {
-        audio_buf_info ospace;
+    }
+  }
 
-        ioctl (osssink->common.fd, SNDCTL_DSP_GETOSPACE, &ospace);
+  if (GST_OSSELEMENT (osssink)->fd >= 0 && to_write > 0) {
+    if (!osssink->mute) {
 
-        if (ospace.bytes >= size) {
-          write (osssink->common.fd, data, size);
+      while (to_write > 0) {
+       gint done = write (GST_OSSELEMENT (osssink)->fd, data,
+           MIN (to_write, osssink->chunk_size));
+
+       if (done == -1) {
+         if (errno != EINTR)
+           break;
+       } else {
+         to_write -= done;
+         data += done;
+         osssink->handled += done;
        }
       }
+    } else {
+      g_warning ("muting osssinks unimplemented wrt clocks!");
     }
   }
+
+  gst_audio_clock_update_time ((GstAudioClock *) osssink->provided_clock,
+      gst_osssink_get_time (osssink->provided_clock, osssink));
+
   gst_buffer_unref (buf);
 }
 
-static const GstFormat*
-gst_osssink_get_formats (GstPad *pad)
+static const GstFormat *
+gst_osssink_get_formats (GstPad * pad)
 {
   static const GstFormat formats[] = {
     GST_FORMAT_TIME,
-    GST_FORMAT_UNITS,
+    GST_FORMAT_DEFAULT,
     GST_FORMAT_BYTES,
     0
   };
@@ -421,19 +490,19 @@ gst_osssink_get_formats (GstPad *pad)
 }
 
 static gboolean
-gst_osssink_convert (GstPad *pad, GstFormat src_format, gint64 src_value,
-                    GstFormat *dest_format, gint64 *dest_value)
+gst_osssink_convert (GstPad * pad, GstFormat src_format, gint64 src_value,
+    GstFormat * dest_format, gint64 * dest_value)
 {
   GstOssSink *osssink;
 
   osssink = GST_OSSSINK (gst_pad_get_parent (pad));
-  
-  return gst_osscommon_convert (&osssink->common, src_format, src_value,
-                               dest_format, dest_value);
+
+  return gst_osselement_convert (GST_OSSELEMENT (osssink),
+      src_format, src_value, dest_format, dest_value);
 }
 
-static const GstQueryType*
-gst_osssink_get_query_types (GstPad *pad)
+static const GstQueryType *
+gst_osssink_get_query_types (GstPad * pad)
 {
   static const GstQueryType query_types[] = {
     GST_QUERY_LATENCY,
@@ -444,32 +513,33 @@ gst_osssink_get_query_types (GstPad *pad)
 }
 
 static gboolean
-gst_osssink_sink_query (GstPad *pad, GstQueryType type, GstFormat *format, gint64 *value) 
+gst_osssink_sink_query (GstPad * pad, GstQueryType type, GstFormat * format,
+    gint64 * value)
 {
   gboolean res = TRUE;
   GstOssSink *osssink;
 
   osssink = GST_OSSSINK (gst_pad_get_parent (pad));
-  
+
   switch (type) {
     case GST_QUERY_LATENCY:
-      if (!gst_osssink_convert (pad, 
-                               GST_FORMAT_BYTES, gst_osssink_get_delay (osssink),
-                               format, value)) 
-      {
-        res = FALSE;
+      if (!gst_osssink_convert (pad,
+             GST_FORMAT_BYTES, gst_osssink_get_delay (osssink),
+             format, value)) {
+       res = FALSE;
       }
       break;
     case GST_QUERY_POSITION:
-      if (!gst_osssink_convert (pad, 
-                               GST_FORMAT_TIME, gst_clock_get_time (osssink->provided_clock),
-                               format, value)) 
-      {
-        res = FALSE;
+      if (!gst_osssink_convert (pad,
+             GST_FORMAT_TIME, gst_element_get_time (GST_ELEMENT (osssink)),
+             format, value)) {
+       res = FALSE;
       }
       break;
     default:
-      res = gst_pad_query (gst_pad_get_peer (osssink->sinkpad), type, format, value);
+      res =
+         gst_pad_query (gst_pad_get_peer (osssink->sinkpad), type, format,
+         value);
       break;
   }
 
@@ -477,150 +547,111 @@ gst_osssink_sink_query (GstPad *pad, GstQueryType type, GstFormat *format, gint6
 }
 
 static gboolean
-gst_osssink_query (GstElement *element, GstQueryType type, GstFormat *format, gint64 *value) 
+gst_osssink_query (GstElement * element, GstQueryType type, GstFormat * format,
+    gint64 * value)
 {
   GstOssSink *osssink = GST_OSSSINK (element);
 
   return gst_osssink_sink_query (osssink->sinkpad, type, format, value);
 }
 
-static void 
-gst_osssink_set_property (GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec) 
+static void
+gst_osssink_set_property (GObject * object, guint prop_id, const GValue * value,
+    GParamSpec * pspec)
 {
   GstOssSink *osssink;
 
   osssink = GST_OSSSINK (object);
 
   switch (prop_id) {
-    case ARG_DEVICE:
-      /* disallow changing the device while it is opened
-         get_property("device") should return the right one */
-      if (!GST_FLAG_IS_SET (osssink, GST_OSSSINK_OPEN))
-      {
-       g_free (osssink->common.device);
-        osssink->common.device = g_strdup (g_value_get_string (value));
-       g_object_notify (object, "device");
-      }
-      break;
     case ARG_MUTE:
       osssink->mute = g_value_get_boolean (value);
       g_object_notify (G_OBJECT (osssink), "mute");
       break;
     case ARG_FRAGMENT:
-      osssink->common.fragment = g_value_get_int (value);
-      gst_osscommon_sync_parms (&osssink->common);
+      GST_OSSELEMENT (osssink)->fragment = g_value_get_int (value);
+      gst_osselement_sync_parms (GST_OSSELEMENT (osssink));
       break;
     case ARG_BUFFER_SIZE:
-      if (osssink->bufsize == g_value_get_int (value)) break;
-      osssink->bufsize = g_value_get_int (value);
-      osssink->sinkpool = gst_buffer_pool_get_default (osssink->bufsize, 6);
+      osssink->bufsize = g_value_get_uint (value);
       g_object_notify (object, "buffer_size");
       break;
     case ARG_SYNC:
       osssink->sync = g_value_get_boolean (value);
       g_object_notify (G_OBJECT (osssink), "sync");
       break;
+    case ARG_CHUNK_SIZE:
+      osssink->chunk_size = g_value_get_uint (value);
+      break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
   }
 }
 
-static void 
-gst_osssink_get_property (GObject *object, guint prop_id, GValue *value, GParamSpec *pspec) 
+static void
+gst_osssink_get_property (GObject * object, guint prop_id, GValue * value,
+    GParamSpec * pspec)
 {
   GstOssSink *osssink;
 
   osssink = GST_OSSSINK (object);
 
   switch (prop_id) {
-    case ARG_DEVICE:
-      g_value_set_string (value, osssink->common.device);
-      break;
     case ARG_MUTE:
       g_value_set_boolean (value, osssink->mute);
       break;
     case ARG_FRAGMENT:
-      g_value_set_int (value, osssink->common.fragment);
+      g_value_set_int (value, GST_OSSELEMENT (osssink)->fragment);
       break;
     case ARG_BUFFER_SIZE:
-      g_value_set_int (value, osssink->bufsize);
+      g_value_set_uint (value, osssink->bufsize);
       break;
     case ARG_SYNC:
       g_value_set_boolean (value, osssink->sync);
       break;
+    case ARG_CHUNK_SIZE:
+      g_value_set_uint (value, osssink->chunk_size);
+      break;
     default:
       G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
       break;
   }
 }
 
-static GstElementStateReturn 
-gst_osssink_change_state (GstElement *element) 
+static GstElementStateReturn
+gst_osssink_change_state (GstElement * element)
 {
   GstOssSink *osssink;
 
   osssink = GST_OSSSINK (element);
 
   switch (GST_STATE_TRANSITION (element)) {
-    case GST_STATE_NULL_TO_READY:
-      if (!GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN)) {
-       gchar *error;
-
-        if (!gst_osscommon_open_audio (&osssink->common, GST_OSSCOMMON_WRITE, &error)) {
-         gst_element_error (GST_ELEMENT (osssink), error);
-         g_free (error);
-          return GST_STATE_FAILURE;
-        }
-       GST_FLAG_SET (element, GST_OSSSINK_OPEN);
-      }
-      break;
     case GST_STATE_READY_TO_PAUSED:
       break;
     case GST_STATE_PAUSED_TO_PLAYING:
-      osssink->resync = TRUE;
+      gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock),
+         TRUE);
       break;
     case GST_STATE_PLAYING_TO_PAUSED:
-    {
-      if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN)) 
-        ioctl (osssink->common.fd, SNDCTL_DSP_RESET, 0);
-      gst_oss_clock_set_active (osssink->provided_clock, FALSE);
-      osssink->resync = TRUE;
+      if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN))
+       ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_RESET, 0);
+      gst_audio_clock_set_active (GST_AUDIO_CLOCK (osssink->provided_clock),
+         FALSE);
       break;
-    }
     case GST_STATE_PAUSED_TO_READY:
       if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN))
-        ioctl (osssink->common.fd, SNDCTL_DSP_RESET, 0);
-      gst_osscommon_reset (&osssink->common);
+       ioctl (GST_OSSELEMENT (osssink)->fd, SNDCTL_DSP_RESET, 0);
+      gst_osselement_reset (GST_OSSELEMENT (osssink));
+      osssink->handled = 0;
+      osssink->resync = TRUE;
       break;
-    case GST_STATE_READY_TO_NULL:
-      if (GST_FLAG_IS_SET (element, GST_OSSSINK_OPEN)) {
-        gst_osscommon_close_audio (&osssink->common);
-        GST_FLAG_UNSET (osssink, GST_OSSSINK_OPEN);
-
-        GST_INFO (GST_CAT_PLUGIN_INFO, "osssink: closed sound device");
-      }
+    default:
       break;
   }
-      
+
   if (GST_ELEMENT_CLASS (parent_class)->change_state)
     return GST_ELEMENT_CLASS (parent_class)->change_state (element);
 
   return GST_STATE_SUCCESS;
 }
-
-gboolean 
-gst_osssink_factory_init (GstPlugin *plugin) 
-{ 
-  GstElementFactory *factory;
-
-  factory = gst_element_factory_new ("osssink", GST_TYPE_OSSSINK, &gst_osssink_details);
-  g_return_val_if_fail (factory != NULL, FALSE);
-
-  gst_element_factory_add_pad_template (factory, GST_PAD_TEMPLATE_GET (osssink_sink_factory));
-
-  gst_plugin_add_feature (plugin, GST_PLUGIN_FEATURE (factory));
-
-  return TRUE;
-}
-