multifdsink: put back multifdsink before refactoring
authorThomas Vander Stichele <thomas (at) apestaart (dot) org>
Thu, 26 Jan 2012 09:08:47 +0000 (10:08 +0100)
committerThomas Vander Stichele <thomas (at) apestaart (dot) org>
Sun, 12 Feb 2012 21:07:29 +0000 (22:07 +0100)
gst/tcp/Makefile.am
gst/tcp/gstmultifdsink.c [new file with mode: 0644]
gst/tcp/gstmultifdsink.h [new file with mode: 0644]
tests/check/Makefile.am
tests/check/elements/multifdsink.c [new file with mode: 0644]

index 074c970..b94682c 100644 (file)
@@ -16,6 +16,7 @@ BUILT_SOURCES = $(built_sources) $(built_headers)
 libgsttcp_la_SOURCES = \
        gsttcpplugin.c \
        gsttcpclientsrc.c gsttcpclientsink.c \
+       gstmultifdsink.c  \
        gstmultisocketsink.c  \
        gsttcpserversrc.c gsttcpserversink.c
 
@@ -30,6 +31,7 @@ libgsttcp_la_LIBTOOLFLAGS = --tag=disable-static
 noinst_HEADERS = \
   gsttcp.h \
   gsttcpclientsrc.h gsttcpclientsink.h \
+  gstmultifdsink.h  \
   gstmultisocketsink.h  \
   gsttcpserversrc.h gsttcpserversink.h
 
diff --git a/gst/tcp/gstmultifdsink.c b/gst/tcp/gstmultifdsink.c
new file mode 100644 (file)
index 0000000..061fd68
--- /dev/null
@@ -0,0 +1,2911 @@
+/* GStreamer
+ * Copyright (C) <1999> Erik Walthinsen <omega@cse.ogi.edu>
+ * Copyright (C) <2004> Thomas Vander Stichele <thomas at apestaart dot org>
+ * Copyright (C) 2006 Wim Taymans <wim at fluendo dot com>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ * Boston, MA 02111-1307, USA.
+ */
+
+/**
+ * SECTION:element-multifdsink
+ * @see_also: tcpserversink
+ *
+ * This plugin writes incoming data to a set of file descriptors. The
+ * file descriptors can be added to multifdsink by emitting the #GstMultiFdSink::add signal. 
+ * For each descriptor added, the #GstMultiFdSink::client-added signal will be called.
+ *
+ * As of version 0.10.8, a client can also be added with the #GstMultiFdSink::add-full signal
+ * that allows for more control over what and how much data a client 
+ * initially receives.
+ *
+ * Clients can be removed from multifdsink by emitting the #GstMultiFdSink::remove signal. For
+ * each descriptor removed, the #GstMultiFdSink::client-removed signal will be called. The
+ * #GstMultiFdSink::client-removed signal can also be fired when multifdsink decides that a
+ * client is not active anymore or, depending on the value of the
+ * #GstMultiFdSink:recover-policy property, if the client is reading too slowly.
+ * In all cases, multifdsink will never close a file descriptor itself.
+ * The user of multifdsink is responsible for closing all file descriptors.
+ * This can for example be done in response to the #GstMultiFdSink::client-fd-removed signal.
+ * Note that multifdsink still has a reference to the file descriptor when the
+ * #GstMultiFdSink::client-removed signal is emitted, so that "get-stats" can be performed on
+ * the descriptor; it is therefore not safe to close the file descriptor in
+ * the #GstMultiFdSink::client-removed signal handler, and you should use the 
+ * #GstMultiFdSink::client-fd-removed signal to safely close the fd.
+ *
+ * Multifdsink internally keeps a queue of the incoming buffers and uses a
+ * separate thread to send the buffers to the clients. This ensures that no
+ * client write can block the pipeline and that clients can read with different
+ * speeds.
+ *
+ * When adding a client to multifdsink, the #GstMultiFdSink:sync-method property will define
+ * which buffer in the queued buffers will be sent first to the client. Clients 
+ * can be sent the most recent buffer (which might not be decodable by the 
+ * client if it is not a keyframe), the next keyframe received in 
+ * multifdsink (which can take some time depending on the keyframe rate), or the
+ * last received keyframe (which will cause a simple burst-on-connect). 
+ * Multifdsink will always keep at least one keyframe in its internal buffers
+ * when the sync-mode is set to latest-keyframe.
+ *
+ * As of version 0.10.8, there are additional values for the #GstMultiFdSink:sync-method 
+ * property to allow finer control over burst-on-connect behaviour. By selecting
+ * the 'burst' method a minimum burst size can be chosen, 'burst-keyframe'
+ * additionally requires that the burst begin with a keyframe, and 
+ * 'burst-with-keyframe' attempts to burst beginning with a keyframe, but will
+ * prefer a minimum burst size even if it requires not starting with a keyframe.
+ *
+ * Multifdsink can be instructed to keep at least a minimum amount of data
+ * expressed in time or byte units in its internal queues with the 
+ * #GstMultiFdSink:time-min and #GstMultiFdSink:bytes-min properties respectively.
+ * These properties are useful if the application adds clients with the 
+ * #GstMultiFdSink::add-full signal to make sure that a burst connect can
+ * actually be honored. 
+ *
+ * When streaming data, clients are allowed to read at a different rate than
+ * the rate at which multifdsink receives data. If the client is reading too
+ * fast, no data will be send to the client until multifdsink receives more
+ * data. If the client, however, reads too slowly, data for that client will be 
+ * queued up in multifdsink. Two properties control the amount of data 
+ * (buffers) that is queued in multifdsink: #GstMultiFdSink:buffers-max and 
+ * #GstMultiFdSink:buffers-soft-max. A client that falls behind by
+ * #GstMultiFdSink:buffers-max is removed from multifdsink forcibly.
+ *
+ * A client with a lag of at least #GstMultiFdSink:buffers-soft-max enters the recovery
+ * procedure which is controlled with the #GstMultiFdSink:recover-policy property.
+ * A recover policy of NONE will do nothing, RESYNC_LATEST will send the most recently
+ * received buffer as the next buffer for the client, RESYNC_SOFT_LIMIT
+ * positions the client to the soft limit in the buffer queue and
+ * RESYNC_KEYFRAME positions the client at the most recent keyframe in the
+ * buffer queue.
+ *
+ * multifdsink will by default synchronize on the clock before serving the 
+ * buffers to the clients. This behaviour can be disabled by setting the sync 
+ * property to FALSE. Multifdsink will by default not do QoS and will never
+ * drop late buffers.
+ *
+ * Last reviewed on 2006-09-12 (0.10.10)
+ */
+
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <gst/gst-i18n-plugin.h>
+
+#include <sys/ioctl.h>
+
+#ifdef HAVE_UNISTD_H
+#include <unistd.h>
+#endif
+
+#include <fcntl.h>
+#include <sys/types.h>
+#include <sys/socket.h>
+#include <sys/stat.h>
+#include <netinet/in.h>
+
+#ifdef HAVE_FIONREAD_IN_SYS_FILIO
+#include <sys/filio.h>
+#endif
+
+#include "gstmultifdsink.h"
+#include "gstmultifdsink-marshal.h"
+
+#define NOT_IMPLEMENTED 0
+
+static GstStaticPadTemplate sinktemplate = GST_STATIC_PAD_TEMPLATE ("sink",
+    GST_PAD_SINK,
+    GST_PAD_ALWAYS,
+    GST_STATIC_CAPS_ANY);
+
+GST_DEBUG_CATEGORY_STATIC (multifdsink_debug);
+#define GST_CAT_DEFAULT (multifdsink_debug)
+
+/* MultiFdSink signals and args */
+enum
+{
+  /* methods */
+  SIGNAL_ADD,
+  SIGNAL_ADD_BURST,
+  SIGNAL_REMOVE,
+  SIGNAL_REMOVE_FLUSH,
+  SIGNAL_CLEAR,
+  SIGNAL_GET_STATS,
+
+  /* signals */
+  SIGNAL_CLIENT_ADDED,
+  SIGNAL_CLIENT_REMOVED,
+  SIGNAL_CLIENT_FD_REMOVED,
+
+  LAST_SIGNAL
+};
+
+
+/* this is really arbitrarily chosen */
+#define DEFAULT_MODE                    1
+#define DEFAULT_BUFFERS_MAX             -1
+#define DEFAULT_BUFFERS_SOFT_MAX        -1
+#define DEFAULT_TIME_MIN                -1
+#define DEFAULT_BYTES_MIN               -1
+#define DEFAULT_BUFFERS_MIN             -1
+#define DEFAULT_UNIT_TYPE               GST_TCP_UNIT_TYPE_BUFFERS
+#define DEFAULT_UNITS_MAX               -1
+#define DEFAULT_UNITS_SOFT_MAX          -1
+#define DEFAULT_RECOVER_POLICY          GST_RECOVER_POLICY_NONE
+#define DEFAULT_TIMEOUT                 0
+#define DEFAULT_SYNC_METHOD             GST_SYNC_METHOD_LATEST
+
+#define DEFAULT_BURST_UNIT              GST_TCP_UNIT_TYPE_UNDEFINED
+#define DEFAULT_BURST_VALUE             0
+
+#define DEFAULT_QOS_DSCP                -1
+#define DEFAULT_HANDLE_READ             TRUE
+
+#define DEFAULT_RESEND_STREAMHEADER      TRUE
+
+enum
+{
+  PROP_0,
+  PROP_MODE,
+  PROP_BUFFERS_QUEUED,
+  PROP_BYTES_QUEUED,
+  PROP_TIME_QUEUED,
+
+  PROP_UNIT_TYPE,
+  PROP_UNITS_MAX,
+  PROP_UNITS_SOFT_MAX,
+
+  PROP_BUFFERS_MAX,
+  PROP_BUFFERS_SOFT_MAX,
+
+  PROP_TIME_MIN,
+  PROP_BYTES_MIN,
+  PROP_BUFFERS_MIN,
+
+  PROP_RECOVER_POLICY,
+  PROP_TIMEOUT,
+  PROP_SYNC_METHOD,
+  PROP_BYTES_TO_SERVE,
+  PROP_BYTES_SERVED,
+
+  PROP_BURST_UNIT,
+  PROP_BURST_VALUE,
+
+  PROP_QOS_DSCP,
+
+  PROP_HANDLE_READ,
+
+  PROP_RESEND_STREAMHEADER,
+
+  PROP_NUM_FDS,
+
+  PROP_LAST
+};
+
+/* For backward compat, we can't really select the poll mode anymore with
+ * GstPoll. */
+#define GST_TYPE_FDSET_MODE (gst_fdset_mode_get_type())
+static GType
+gst_fdset_mode_get_type (void)
+{
+  static GType fdset_mode_type = 0;
+  static const GEnumValue fdset_mode[] = {
+    {0, "Select", "select"},
+    {1, "Poll", "poll"},
+    {2, "EPoll", "epoll"},
+    {0, NULL, NULL},
+  };
+
+  if (!fdset_mode_type) {
+    fdset_mode_type = g_enum_register_static ("GstFDSetMode", fdset_mode);
+  }
+  return fdset_mode_type;
+}
+
+#define GST_TYPE_RECOVER_POLICY (gst_recover_policy_get_type())
+static GType
+gst_recover_policy_get_type (void)
+{
+  static GType recover_policy_type = 0;
+  static const GEnumValue recover_policy[] = {
+    {GST_RECOVER_POLICY_NONE,
+        "Do not try to recover", "none"},
+    {GST_RECOVER_POLICY_RESYNC_LATEST,
+        "Resync client to latest buffer", "latest"},
+    {GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT,
+        "Resync client to soft limit", "soft-limit"},
+    {GST_RECOVER_POLICY_RESYNC_KEYFRAME,
+        "Resync client to most recent keyframe", "keyframe"},
+    {0, NULL, NULL},
+  };
+
+  if (!recover_policy_type) {
+    recover_policy_type =
+        g_enum_register_static ("GstRecoverPolicy", recover_policy);
+  }
+  return recover_policy_type;
+}
+
+#define GST_TYPE_SYNC_METHOD (gst_sync_method_get_type())
+static GType
+gst_sync_method_get_type (void)
+{
+  static GType sync_method_type = 0;
+  static const GEnumValue sync_method[] = {
+    {GST_SYNC_METHOD_LATEST,
+        "Serve starting from the latest buffer", "latest"},
+    {GST_SYNC_METHOD_NEXT_KEYFRAME,
+        "Serve starting from the next keyframe", "next-keyframe"},
+    {GST_SYNC_METHOD_LATEST_KEYFRAME,
+          "Serve everything since the latest keyframe (burst)",
+        "latest-keyframe"},
+    {GST_SYNC_METHOD_BURST, "Serve burst-value data to client", "burst"},
+    {GST_SYNC_METHOD_BURST_KEYFRAME,
+          "Serve burst-value data starting on a keyframe",
+        "burst-keyframe"},
+    {GST_SYNC_METHOD_BURST_WITH_KEYFRAME,
+          "Serve burst-value data preferably starting on a keyframe",
+        "burst-with-keyframe"},
+    {0, NULL, NULL},
+  };
+
+  if (!sync_method_type) {
+    sync_method_type = g_enum_register_static ("GstSyncMethod", sync_method);
+  }
+  return sync_method_type;
+}
+
+#define GST_TYPE_UNIT_TYPE (gst_unit_type_get_type())
+static GType
+gst_unit_type_get_type (void)
+{
+  static GType unit_type_type = 0;
+  static const GEnumValue unit_type[] = {
+    {GST_TCP_UNIT_TYPE_UNDEFINED, "Undefined", "undefined"},
+    {GST_TCP_UNIT_TYPE_BUFFERS, "Buffers", "buffers"},
+    {GST_TCP_UNIT_TYPE_BYTES, "Bytes", "bytes"},
+    {GST_TCP_UNIT_TYPE_TIME, "Time", "time"},
+    {0, NULL, NULL},
+  };
+
+  if (!unit_type_type) {
+    unit_type_type = g_enum_register_static ("GstTCPUnitType", unit_type);
+  }
+  return unit_type_type;
+}
+
+#define GST_TYPE_CLIENT_STATUS (gst_client_status_get_type())
+static GType
+gst_client_status_get_type (void)
+{
+  static GType client_status_type = 0;
+  static const GEnumValue client_status[] = {
+    {GST_CLIENT_STATUS_OK, "ok", "ok"},
+    {GST_CLIENT_STATUS_CLOSED, "Closed", "closed"},
+    {GST_CLIENT_STATUS_REMOVED, "Removed", "removed"},
+    {GST_CLIENT_STATUS_SLOW, "Too slow", "slow"},
+    {GST_CLIENT_STATUS_ERROR, "Error", "error"},
+    {GST_CLIENT_STATUS_DUPLICATE, "Duplicate", "duplicate"},
+    {GST_CLIENT_STATUS_FLUSHING, "Flushing", "flushing"},
+    {0, NULL, NULL},
+  };
+
+  if (!client_status_type) {
+    client_status_type =
+        g_enum_register_static ("GstClientStatus", client_status);
+  }
+  return client_status_type;
+}
+
+static void gst_multi_fd_sink_finalize (GObject * object);
+
+static void gst_multi_fd_sink_remove_client_link (GstMultiFdSink * sink,
+    GList * link);
+
+static GstFlowReturn gst_multi_fd_sink_render (GstBaseSink * bsink,
+    GstBuffer * buf);
+static GstStateChangeReturn gst_multi_fd_sink_change_state (GstElement *
+    element, GstStateChange transition);
+
+static void gst_multi_fd_sink_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec);
+static void gst_multi_fd_sink_get_property (GObject * object, guint prop_id,
+    GValue * value, GParamSpec * pspec);
+
+#define gst_multi_fd_sink_parent_class parent_class
+G_DEFINE_TYPE (GstMultiFdSink, gst_multi_fd_sink, GST_TYPE_BASE_SINK);
+
+static guint gst_multi_fd_sink_signals[LAST_SIGNAL] = { 0 };
+
+static void
+gst_multi_fd_sink_class_init (GstMultiFdSinkClass * klass)
+{
+  GObjectClass *gobject_class;
+  GstElementClass *gstelement_class;
+  GstBaseSinkClass *gstbasesink_class;
+
+  gobject_class = (GObjectClass *) klass;
+  gstelement_class = (GstElementClass *) klass;
+  gstbasesink_class = (GstBaseSinkClass *) klass;
+
+  gobject_class->set_property = gst_multi_fd_sink_set_property;
+  gobject_class->get_property = gst_multi_fd_sink_get_property;
+  gobject_class->finalize = gst_multi_fd_sink_finalize;
+
+  /**
+   * GstMultiFdSink::mode
+   *
+   * The mode for selecting activity on the fds. 
+   *
+   * This property is deprecated since 0.10.18, if will now automatically
+   * select and use the most optimal method.
+   */
+  g_object_class_install_property (gobject_class, PROP_MODE,
+      g_param_spec_enum ("mode", "Mode",
+          "The mode for selecting activity on the fds (deprecated)",
+          GST_TYPE_FDSET_MODE, DEFAULT_MODE,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_BUFFERS_MAX,
+      g_param_spec_int ("buffers-max", "Buffers max",
+          "max number of buffers to queue for a client (-1 = no limit)", -1,
+          G_MAXINT, DEFAULT_BUFFERS_MAX,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BUFFERS_SOFT_MAX,
+      g_param_spec_int ("buffers-soft-max", "Buffers soft max",
+          "Recover client when going over this limit (-1 = no limit)", -1,
+          G_MAXINT, DEFAULT_BUFFERS_SOFT_MAX,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_BYTES_MIN,
+      g_param_spec_int ("bytes-min", "Bytes min",
+          "min number of bytes to queue (-1 = as little as possible)", -1,
+          G_MAXINT, DEFAULT_BYTES_MIN,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_TIME_MIN,
+      g_param_spec_int64 ("time-min", "Time min",
+          "min number of time to queue (-1 = as little as possible)", -1,
+          G_MAXINT64, DEFAULT_TIME_MIN,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BUFFERS_MIN,
+      g_param_spec_int ("buffers-min", "Buffers min",
+          "min number of buffers to queue (-1 = as few as possible)", -1,
+          G_MAXINT, DEFAULT_BUFFERS_MIN,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_UNIT_TYPE,
+      g_param_spec_enum ("unit-type", "Units type",
+          "The unit to measure the max/soft-max/queued properties",
+          GST_TYPE_UNIT_TYPE, DEFAULT_UNIT_TYPE,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_UNITS_MAX,
+      g_param_spec_int64 ("units-max", "Units max",
+          "max number of units to queue (-1 = no limit)", -1, G_MAXINT64,
+          DEFAULT_UNITS_MAX, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_UNITS_SOFT_MAX,
+      g_param_spec_int64 ("units-soft-max", "Units soft max",
+          "Recover client when going over this limit (-1 = no limit)", -1,
+          G_MAXINT64, DEFAULT_UNITS_SOFT_MAX,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_BUFFERS_QUEUED,
+      g_param_spec_uint ("buffers-queued", "Buffers queued",
+          "Number of buffers currently queued", 0, G_MAXUINT, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+#if NOT_IMPLEMENTED
+  g_object_class_install_property (gobject_class, PROP_BYTES_QUEUED,
+      g_param_spec_uint ("bytes-queued", "Bytes queued",
+          "Number of bytes currently queued", 0, G_MAXUINT, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_TIME_QUEUED,
+      g_param_spec_uint64 ("time-queued", "Time queued",
+          "Number of time currently queued", 0, G_MAXUINT64, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+#endif
+
+  g_object_class_install_property (gobject_class, PROP_RECOVER_POLICY,
+      g_param_spec_enum ("recover-policy", "Recover Policy",
+          "How to recover when client reaches the soft max",
+          GST_TYPE_RECOVER_POLICY, DEFAULT_RECOVER_POLICY,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_TIMEOUT,
+      g_param_spec_uint64 ("timeout", "Timeout",
+          "Maximum inactivity timeout in nanoseconds for a client (0 = no limit)",
+          0, G_MAXUINT64, DEFAULT_TIMEOUT,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_SYNC_METHOD,
+      g_param_spec_enum ("sync-method", "Sync Method",
+          "How to sync new clients to the stream", GST_TYPE_SYNC_METHOD,
+          DEFAULT_SYNC_METHOD, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BYTES_TO_SERVE,
+      g_param_spec_uint64 ("bytes-to-serve", "Bytes to serve",
+          "Number of bytes received to serve to clients", 0, G_MAXUINT64, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BYTES_SERVED,
+      g_param_spec_uint64 ("bytes-served", "Bytes served",
+          "Total number of bytes send to all clients", 0, G_MAXUINT64, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_BURST_UNIT,
+      g_param_spec_enum ("burst-unit", "Burst unit",
+          "The format of the burst units (when sync-method is burst[[-with]-keyframe])",
+          GST_TYPE_UNIT_TYPE, DEFAULT_BURST_UNIT,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BURST_VALUE,
+      g_param_spec_uint64 ("burst-value", "Burst value",
+          "The amount of burst expressed in burst-unit", 0, G_MAXUINT64,
+          DEFAULT_BURST_VALUE, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_QOS_DSCP,
+      g_param_spec_int ("qos-dscp", "QoS diff srv code point",
+          "Quality of Service, differentiated services code point (-1 default)",
+          -1, 63, DEFAULT_QOS_DSCP,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  /**
+   * GstMultiFdSink::handle-read
+   *
+   * Handle read requests from clients and discard the data.
+   *
+   * Since: 0.10.23
+   */
+  g_object_class_install_property (gobject_class, PROP_HANDLE_READ,
+      g_param_spec_boolean ("handle-read", "Handle Read",
+          "Handle client reads and discard the data",
+          DEFAULT_HANDLE_READ, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  /**
+   * GstMultiFdSink::resend-streamheader
+   *
+   * Resend the streamheaders to existing clients when they change.
+   *
+   * Since: 0.10.23
+   */
+  g_object_class_install_property (gobject_class, PROP_RESEND_STREAMHEADER,
+      g_param_spec_boolean ("resend-streamheader", "Resend streamheader",
+          "Resend the streamheader if it changes in the caps",
+          DEFAULT_RESEND_STREAMHEADER,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_NUM_FDS,
+      g_param_spec_uint ("num-fds", "Number of fds",
+          "The current number of client file descriptors.",
+          0, G_MAXUINT, 0, G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+
+  /**
+   * GstMultiFdSink::add:
+   * @gstmultifdsink: the multifdsink element to emit this signal on
+   * @fd:             the file descriptor to add to multifdsink
+   *
+   * Hand the given open file descriptor to multifdsink to write to.
+   */
+  gst_multi_fd_sink_signals[SIGNAL_ADD] =
+      g_signal_new ("add", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
+          add), NULL, NULL, g_cclosure_marshal_VOID__INT, G_TYPE_NONE, 1,
+      G_TYPE_INT);
+  /**
+   * GstMultiFdSink::add-full:
+   * @gstmultifdsink: the multifdsink element to emit this signal on
+   * @fd:             the file descriptor to add to multifdsink
+   * @sync:           the sync method to use
+   * @unit_type_min:  the unit-type of @value_min
+   * @value_min:      the minimum amount of data to burst expressed in
+   *                  @unit_type_min units.
+   * @unit_type_max:  the unit-type of @value_max
+   * @value_max:      the maximum amount of data to burst expressed in
+   *                  @unit_type_max units.
+   *
+   * Hand the given open file descriptor to multifdsink to write to and
+   * specify the burst parameters for the new connection.
+   */
+  gst_multi_fd_sink_signals[SIGNAL_ADD_BURST] =
+      g_signal_new ("add-full", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
+          add_full), NULL, NULL,
+      gst_multi_fd_sink_marshal_VOID__INT_ENUM_INT_UINT64_INT_UINT64,
+      G_TYPE_NONE, 6, G_TYPE_INT, GST_TYPE_SYNC_METHOD, GST_TYPE_UNIT_TYPE,
+      G_TYPE_UINT64, GST_TYPE_UNIT_TYPE, G_TYPE_UINT64);
+  /**
+   * GstMultiFdSink::remove:
+   * @gstmultifdsink: the multifdsink element to emit this signal on
+   * @fd:             the file descriptor to remove from multifdsink
+   *
+   * Remove the given open file descriptor from multifdsink.
+   */
+  gst_multi_fd_sink_signals[SIGNAL_REMOVE] =
+      g_signal_new ("remove", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
+          remove), NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT, G_TYPE_NONE,
+      1, G_TYPE_INT);
+  /**
+   * GstMultiFdSink::remove-flush:
+   * @gstmultifdsink: the multifdsink element to emit this signal on
+   * @fd:             the file descriptor to remove from multifdsink
+   *
+   * Remove the given open file descriptor from multifdsink after flushing all
+   * the pending data to the fd.
+   */
+  gst_multi_fd_sink_signals[SIGNAL_REMOVE_FLUSH] =
+      g_signal_new ("remove-flush", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
+          remove_flush), NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT,
+      G_TYPE_NONE, 1, G_TYPE_INT);
+  /**
+   * GstMultiFdSink::clear:
+   * @gstmultifdsink: the multifdsink element to emit this signal on
+   *
+   * Remove all file descriptors from multifdsink.  Since multifdsink did not
+   * open fd's itself, it does not explicitly close the fd.  The application
+   * should do so by connecting to the client-fd-removed callback.
+   */
+  gst_multi_fd_sink_signals[SIGNAL_CLEAR] =
+      g_signal_new ("clear", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
+          clear), NULL, NULL, g_cclosure_marshal_VOID__VOID, G_TYPE_NONE, 0);
+
+  /**
+   * GstMultiFdSink::get-stats:
+   * @gstmultifdsink: the multifdsink element to emit this signal on
+   * @fd:             the file descriptor to get stats of from multifdsink
+   *
+   * Get statistics about @fd. This function returns a GValueArray to ease
+   * automatic wrapping for bindings.
+   *
+   * Returns: a GValueArray with the statistics. The array contains guint64
+   *     values that represent respectively: total number of bytes sent, time
+   *     when the client was added, time when the client was
+   *     disconnected/removed, time the client is/was active, last activity
+   *     time (in epoch seconds), number of buffers dropped.
+   *     All times are expressed in nanoseconds (GstClockTime).
+   *     The array can be 0-length if the client was not found.
+   */
+  gst_multi_fd_sink_signals[SIGNAL_GET_STATS] =
+      g_signal_new ("get-stats", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
+          get_stats), NULL, NULL, gst_multi_fd_sink_marshal_BOXED__INT,
+      G_TYPE_VALUE_ARRAY, 1, G_TYPE_INT);
+
+  /**
+   * GstMultiFdSink::client-added:
+   * @gstmultifdsink: the multifdsink element that emitted this signal
+   * @fd:             the file descriptor that was added to multifdsink
+   *
+   * The given file descriptor was added to multifdsink. This signal will
+   * be emitted from the streaming thread so application should be prepared
+   * for that.
+   */
+  gst_multi_fd_sink_signals[SIGNAL_CLIENT_ADDED] =
+      g_signal_new ("client-added", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiFdSinkClass, client_added),
+      NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT, G_TYPE_NONE, 1,
+      G_TYPE_INT);
+  /**
+   * GstMultiFdSink::client-removed:
+   * @gstmultifdsink: the multifdsink element that emitted this signal
+   * @fd:             the file descriptor that is to be removed from multifdsink
+   * @status:         the reason why the client was removed
+   *
+   * The given file descriptor is about to be removed from multifdsink. This
+   * signal will be emitted from the streaming thread so applications should
+   * be prepared for that.
+   *
+   * @gstmultifdsink still holds a handle to @fd so it is possible to call
+   * the get-stats signal from this callback. For the same reason it is
+   * not safe to close() and reuse @fd in this callback.
+   */
+  gst_multi_fd_sink_signals[SIGNAL_CLIENT_REMOVED] =
+      g_signal_new ("client-removed", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiFdSinkClass,
+          client_removed), NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT_ENUM,
+      G_TYPE_NONE, 2, G_TYPE_INT, GST_TYPE_CLIENT_STATUS);
+  /**
+   * GstMultiFdSink::client-fd-removed:
+   * @gstmultifdsink: the multifdsink element that emitted this signal
+   * @fd:             the file descriptor that was removed from multifdsink
+   *
+   * The given file descriptor was removed from multifdsink. This signal will
+   * be emitted from the streaming thread so applications should be prepared
+   * for that.
+   *
+   * In this callback, @gstmultifdsink has removed all the information
+   * associated with @fd and it is therefore not possible to call get-stats
+   * with @fd. It is however safe to close() and reuse @fd in the callback.
+   *
+   * Since: 0.10.7
+   */
+  gst_multi_fd_sink_signals[SIGNAL_CLIENT_FD_REMOVED] =
+      g_signal_new ("client-fd-removed", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiFdSinkClass,
+          client_fd_removed), NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT,
+      G_TYPE_NONE, 1, G_TYPE_INT);
+
+  gst_element_class_add_pad_template (gstelement_class,
+      gst_static_pad_template_get (&sinktemplate));
+
+  gst_element_class_set_details_simple (gstelement_class,
+      "Multi filedescriptor sink", "Sink/Network",
+      "Send data to multiple filedescriptors",
+      "Thomas Vander Stichele <thomas at apestaart dot org>, "
+      "Wim Taymans <wim@fluendo.com>");
+
+  gstelement_class->change_state =
+      GST_DEBUG_FUNCPTR (gst_multi_fd_sink_change_state);
+
+  gstbasesink_class->render = GST_DEBUG_FUNCPTR (gst_multi_fd_sink_render);
+
+  klass->add = GST_DEBUG_FUNCPTR (gst_multi_fd_sink_add);
+  klass->add_full = GST_DEBUG_FUNCPTR (gst_multi_fd_sink_add_full);
+  klass->remove = GST_DEBUG_FUNCPTR (gst_multi_fd_sink_remove);
+  klass->remove_flush = GST_DEBUG_FUNCPTR (gst_multi_fd_sink_remove_flush);
+  klass->clear = GST_DEBUG_FUNCPTR (gst_multi_fd_sink_clear);
+  klass->get_stats = GST_DEBUG_FUNCPTR (gst_multi_fd_sink_get_stats);
+
+  GST_DEBUG_CATEGORY_INIT (multifdsink_debug, "multifdsink", 0, "FD sink");
+}
+
+static void
+gst_multi_fd_sink_init (GstMultiFdSink * this)
+{
+  GST_OBJECT_FLAG_UNSET (this, GST_MULTI_FD_SINK_OPEN);
+
+  this->mode = DEFAULT_MODE;
+
+  CLIENTS_LOCK_INIT (this);
+  this->clients = NULL;
+  this->fd_hash = g_hash_table_new (g_int_hash, g_int_equal);
+
+  this->bufqueue = g_array_new (FALSE, TRUE, sizeof (GstBuffer *));
+  this->unit_type = DEFAULT_UNIT_TYPE;
+  this->units_max = DEFAULT_UNITS_MAX;
+  this->units_soft_max = DEFAULT_UNITS_SOFT_MAX;
+  this->time_min = DEFAULT_TIME_MIN;
+  this->bytes_min = DEFAULT_BYTES_MIN;
+  this->buffers_min = DEFAULT_BUFFERS_MIN;
+  this->recover_policy = DEFAULT_RECOVER_POLICY;
+
+  this->timeout = DEFAULT_TIMEOUT;
+  this->def_sync_method = DEFAULT_SYNC_METHOD;
+  this->def_burst_unit = DEFAULT_BURST_UNIT;
+  this->def_burst_value = DEFAULT_BURST_VALUE;
+
+  this->qos_dscp = DEFAULT_QOS_DSCP;
+  this->handle_read = DEFAULT_HANDLE_READ;
+
+  this->resend_streamheader = DEFAULT_RESEND_STREAMHEADER;
+
+  this->header_flags = 0;
+}
+
+static void
+gst_multi_fd_sink_finalize (GObject * object)
+{
+  GstMultiFdSink *this;
+
+  this = GST_MULTI_FD_SINK (object);
+
+  CLIENTS_LOCK_CLEAR (this);
+  g_hash_table_destroy (this->fd_hash);
+  g_array_free (this->bufqueue, TRUE);
+
+  G_OBJECT_CLASS (parent_class)->finalize (object);
+}
+
+static gint
+setup_dscp_client (GstMultiFdSink * sink, GstTCPClient * client)
+{
+  gint tos;
+  gint ret;
+  union gst_sockaddr
+  {
+    struct sockaddr sa;
+    struct sockaddr_in6 sa_in6;
+    struct sockaddr_storage sa_stor;
+  } sa;
+  socklen_t slen = sizeof (sa);
+  gint af;
+
+  /* don't touch */
+  if (sink->qos_dscp < 0)
+    return 0;
+
+  if ((ret = getsockname (client->fd.fd, &sa.sa, &slen)) < 0) {
+    GST_DEBUG_OBJECT (sink, "could not get sockname: %s", g_strerror (errno));
+    return ret;
+  }
+
+  af = sa.sa.sa_family;
+
+  /* if this is an IPv4-mapped address then do IPv4 QoS */
+  if (af == AF_INET6) {
+
+    GST_DEBUG_OBJECT (sink, "check IP6 socket");
+    if (IN6_IS_ADDR_V4MAPPED (&(sa.sa_in6.sin6_addr))) {
+      GST_DEBUG_OBJECT (sink, "mapped to IPV4");
+      af = AF_INET;
+    }
+  }
+
+  /* extract and shift 6 bits of the DSCP */
+  tos = (sink->qos_dscp & 0x3f) << 2;
+
+  switch (af) {
+    case AF_INET:
+      ret = setsockopt (client->fd.fd, IPPROTO_IP, IP_TOS, &tos, sizeof (tos));
+      break;
+    case AF_INET6:
+#ifdef IPV6_TCLASS
+      ret =
+          setsockopt (client->fd.fd, IPPROTO_IPV6, IPV6_TCLASS, &tos,
+          sizeof (tos));
+      break;
+#endif
+    default:
+      ret = 0;
+      GST_ERROR_OBJECT (sink, "unsupported AF");
+      break;
+  }
+  if (ret)
+    GST_DEBUG_OBJECT (sink, "could not set DSCP: %s", g_strerror (errno));
+
+  return ret;
+}
+
+
+static void
+setup_dscp (GstMultiFdSink * sink)
+{
+  GList *clients, *next;
+
+  CLIENTS_LOCK (sink);
+  for (clients = sink->clients; clients; clients = next) {
+    GstTCPClient *client;
+
+    client = (GstTCPClient *) clients->data;
+    next = g_list_next (clients);
+
+    setup_dscp_client (sink, client);
+  }
+  CLIENTS_UNLOCK (sink);
+}
+
+/* "add-full" signal implementation */
+void
+gst_multi_fd_sink_add_full (GstMultiFdSink * sink, int fd,
+    GstSyncMethod sync_method, GstTCPUnitType min_unit, guint64 min_value,
+    GstTCPUnitType max_unit, guint64 max_value)
+{
+  GstTCPClient *client;
+  GList *clink;
+  GTimeVal now;
+  gint flags;
+  struct stat statbuf;
+
+  GST_DEBUG_OBJECT (sink, "[fd %5d] adding client, sync_method %d, "
+      "min_unit %d, min_value %" G_GUINT64_FORMAT
+      ", max_unit %d, max_value %" G_GUINT64_FORMAT, fd, sync_method,
+      min_unit, min_value, max_unit, max_value);
+
+  /* do limits check if we can */
+  if (min_unit == max_unit) {
+    if (max_value != -1 && min_value != -1 && max_value < min_value)
+      goto wrong_limits;
+  }
+
+  /* create client datastructure */
+  client = g_new0 (GstTCPClient, 1);
+  client->fd.fd = fd;
+  client->status = GST_CLIENT_STATUS_OK;
+  client->bufpos = -1;
+  client->flushcount = -1;
+  client->bufoffset = 0;
+  client->sending = NULL;
+  client->bytes_sent = 0;
+  client->dropped_buffers = 0;
+  client->avg_queue_size = 0;
+  client->first_buffer_ts = GST_CLOCK_TIME_NONE;
+  client->last_buffer_ts = GST_CLOCK_TIME_NONE;
+  client->new_connection = TRUE;
+  client->burst_min_unit = min_unit;
+  client->burst_min_value = min_value;
+  client->burst_max_unit = max_unit;
+  client->burst_max_value = max_value;
+  client->sync_method = sync_method;
+  client->currently_removing = FALSE;
+
+  /* update start time */
+  g_get_current_time (&now);
+  client->connect_time = GST_TIMEVAL_TO_TIME (now);
+  client->disconnect_time = 0;
+  /* set last activity time to connect time */
+  client->last_activity_time = client->connect_time;
+
+  CLIENTS_LOCK (sink);
+
+  /* check the hash to find a duplicate fd */
+  clink = g_hash_table_lookup (sink->fd_hash, &client->fd.fd);
+  if (clink != NULL)
+    goto duplicate;
+
+  /* we can add the fd now */
+  clink = sink->clients = g_list_prepend (sink->clients, client);
+  g_hash_table_insert (sink->fd_hash, &client->fd.fd, clink);
+  sink->clients_cookie++;
+
+  /* set the socket to non blocking */
+  if (fcntl (fd, F_SETFL, O_NONBLOCK) < 0) {
+    GST_ERROR_OBJECT (sink, "failed to make socket %d non-blocking: %s", fd,
+        g_strerror (errno));
+  }
+
+  /* we always read from a client */
+  gst_poll_add_fd (sink->fdset, &client->fd);
+
+  /* we don't try to read from write only fds */
+  if (sink->handle_read) {
+    flags = fcntl (fd, F_GETFL, 0);
+    if ((flags & O_ACCMODE) != O_WRONLY) {
+      gst_poll_fd_ctl_read (sink->fdset, &client->fd, TRUE);
+    }
+  }
+  /* figure out the mode, can't use send() for non sockets */
+  if (fstat (fd, &statbuf) == 0 && S_ISSOCK (statbuf.st_mode)) {
+    client->is_socket = TRUE;
+    setup_dscp_client (sink, client);
+  }
+
+  gst_poll_restart (sink->fdset);
+
+  CLIENTS_UNLOCK (sink);
+
+  g_signal_emit (G_OBJECT (sink),
+      gst_multi_fd_sink_signals[SIGNAL_CLIENT_ADDED], 0, fd);
+
+  return;
+
+  /* errors */
+wrong_limits:
+  {
+    GST_WARNING_OBJECT (sink,
+        "[fd %5d] wrong values min =%" G_GUINT64_FORMAT ", max=%"
+        G_GUINT64_FORMAT ", unit %d specified when adding client", fd,
+        min_value, max_value, min_unit);
+    return;
+  }
+duplicate:
+  {
+    client->status = GST_CLIENT_STATUS_DUPLICATE;
+    CLIENTS_UNLOCK (sink);
+    GST_WARNING_OBJECT (sink, "[fd %5d] duplicate client found, refusing", fd);
+    g_signal_emit (G_OBJECT (sink),
+        gst_multi_fd_sink_signals[SIGNAL_CLIENT_REMOVED], 0, fd,
+        client->status);
+    g_free (client);
+    return;
+  }
+}
+
+/* "add" signal implementation */
+void
+gst_multi_fd_sink_add (GstMultiFdSink * sink, int fd)
+{
+  gst_multi_fd_sink_add_full (sink, fd, sink->def_sync_method,
+      sink->def_burst_unit, sink->def_burst_value, sink->def_burst_unit, -1);
+}
+
+/* "remove" signal implementation */
+void
+gst_multi_fd_sink_remove (GstMultiFdSink * sink, int fd)
+{
+  GList *clink;
+
+  GST_DEBUG_OBJECT (sink, "[fd %5d] removing client", fd);
+
+  CLIENTS_LOCK (sink);
+  clink = g_hash_table_lookup (sink->fd_hash, &fd);
+  if (clink != NULL) {
+    GstTCPClient *client = (GstTCPClient *) clink->data;
+
+    if (client->status != GST_CLIENT_STATUS_OK) {
+      GST_INFO_OBJECT (sink,
+          "[fd %5d] Client already disconnecting with status %d",
+          fd, client->status);
+      goto done;
+    }
+
+    client->status = GST_CLIENT_STATUS_REMOVED;
+    gst_multi_fd_sink_remove_client_link (sink, clink);
+    gst_poll_restart (sink->fdset);
+  } else {
+    GST_WARNING_OBJECT (sink, "[fd %5d] no client with this fd found!", fd);
+  }
+
+done:
+  CLIENTS_UNLOCK (sink);
+}
+
+/* "remove-flush" signal implementation */
+void
+gst_multi_fd_sink_remove_flush (GstMultiFdSink * sink, int fd)
+{
+  GList *clink;
+
+  GST_DEBUG_OBJECT (sink, "[fd %5d] flushing client", fd);
+
+  CLIENTS_LOCK (sink);
+  clink = g_hash_table_lookup (sink->fd_hash, &fd);
+  if (clink != NULL) {
+    GstTCPClient *client = (GstTCPClient *) clink->data;
+
+    if (client->status != GST_CLIENT_STATUS_OK) {
+      GST_INFO_OBJECT (sink,
+          "[fd %5d] Client already disconnecting with status %d",
+          fd, client->status);
+      goto done;
+    }
+
+    /* take the position of the client as the number of buffers left to flush.
+     * If the client was at position -1, we flush 0 buffers, 0 == flush 1
+     * buffer, etc... */
+    client->flushcount = client->bufpos + 1;
+    /* mark client as flushing. We can not remove the client right away because
+     * it might have some buffers to flush in the ->sending queue. */
+    client->status = GST_CLIENT_STATUS_FLUSHING;
+  } else {
+    GST_WARNING_OBJECT (sink, "[fd %5d] no client with this fd found!", fd);
+  }
+done:
+  CLIENTS_UNLOCK (sink);
+}
+
+/* can be called both through the signal (i.e. from any thread) or when 
+ * stopping, after the writing thread has shut down */
+void
+gst_multi_fd_sink_clear (GstMultiFdSink * sink)
+{
+  GList *clients, *next;
+  guint32 cookie;
+
+  GST_DEBUG_OBJECT (sink, "clearing all clients");
+
+  CLIENTS_LOCK (sink);
+restart:
+  cookie = sink->clients_cookie;
+  for (clients = sink->clients; clients; clients = next) {
+    GstTCPClient *client;
+
+    if (cookie != sink->clients_cookie) {
+      GST_DEBUG_OBJECT (sink, "cookie changed while removing all clients");
+      goto restart;
+    }
+
+    client = (GstTCPClient *) clients->data;
+    next = g_list_next (clients);
+
+    client->status = GST_CLIENT_STATUS_REMOVED;
+    gst_multi_fd_sink_remove_client_link (sink, clients);
+  }
+  gst_poll_restart (sink->fdset);
+  CLIENTS_UNLOCK (sink);
+}
+
+/* "get-stats" signal implementation
+ * the array returned contains:
+ *
+ * guint64 : bytes_sent
+ * guint64 : connect time (in nanoseconds, since Epoch)
+ * guint64 : disconnect time (in nanoseconds, since Epoch)
+ * guint64 : time the client is/was connected (in nanoseconds)
+ * guint64 : last activity time (in nanoseconds, since Epoch)
+ * guint64 : buffers dropped due to recovery
+ * guint64 : timestamp of the first buffer sent (in nanoseconds)
+ * guint64 : timestamp of the last buffer sent (in nanoseconds)
+ */
+GValueArray *
+gst_multi_fd_sink_get_stats (GstMultiFdSink * sink, int fd)
+{
+  GstTCPClient *client;
+  GValueArray *result = NULL;
+  GList *clink;
+
+  CLIENTS_LOCK (sink);
+  clink = g_hash_table_lookup (sink->fd_hash, &fd);
+  if (clink == NULL)
+    goto noclient;
+
+  client = (GstTCPClient *) clink->data;
+  if (client != NULL) {
+    GValue value = { 0 };
+    guint64 interval;
+
+    result = g_value_array_new (7);
+
+    g_value_init (&value, G_TYPE_UINT64);
+    g_value_set_uint64 (&value, client->bytes_sent);
+    result = g_value_array_append (result, &value);
+    g_value_unset (&value);
+    g_value_init (&value, G_TYPE_UINT64);
+    g_value_set_uint64 (&value, client->connect_time);
+    result = g_value_array_append (result, &value);
+    g_value_unset (&value);
+    if (client->disconnect_time == 0) {
+      GTimeVal nowtv;
+
+      g_get_current_time (&nowtv);
+
+      interval = GST_TIMEVAL_TO_TIME (nowtv) - client->connect_time;
+    } else {
+      interval = client->disconnect_time - client->connect_time;
+    }
+    g_value_init (&value, G_TYPE_UINT64);
+    g_value_set_uint64 (&value, client->disconnect_time);
+    result = g_value_array_append (result, &value);
+    g_value_unset (&value);
+    g_value_init (&value, G_TYPE_UINT64);
+    g_value_set_uint64 (&value, interval);
+    result = g_value_array_append (result, &value);
+    g_value_unset (&value);
+    g_value_init (&value, G_TYPE_UINT64);
+    g_value_set_uint64 (&value, client->last_activity_time);
+    result = g_value_array_append (result, &value);
+    g_value_unset (&value);
+    g_value_init (&value, G_TYPE_UINT64);
+    g_value_set_uint64 (&value, client->dropped_buffers);
+    result = g_value_array_append (result, &value);
+    g_value_unset (&value);
+    g_value_init (&value, G_TYPE_UINT64);
+    g_value_set_uint64 (&value, client->first_buffer_ts);
+    result = g_value_array_append (result, &value);
+    g_value_unset (&value);
+    g_value_init (&value, G_TYPE_UINT64);
+    g_value_set_uint64 (&value, client->last_buffer_ts);
+    result = g_value_array_append (result, &value);
+  }
+
+noclient:
+  CLIENTS_UNLOCK (sink);
+
+  /* python doesn't like a NULL pointer yet */
+  if (result == NULL) {
+    GST_WARNING_OBJECT (sink, "[fd %5d] no client with this found!", fd);
+    result = g_value_array_new (0);
+  }
+
+  return result;
+}
+
+/* should be called with the clientslock helt.
+ * Note that we don't close the fd as we didn't open it in the first
+ * place. An application should connect to the client-fd-removed signal and
+ * close the fd itself.
+ */
+static void
+gst_multi_fd_sink_remove_client_link (GstMultiFdSink * sink, GList * link)
+{
+  int fd;
+  GTimeVal now;
+  GstTCPClient *client = (GstTCPClient *) link->data;
+  GstMultiFdSinkClass *fclass;
+
+  fclass = GST_MULTI_FD_SINK_GET_CLASS (sink);
+
+  fd = client->fd.fd;
+
+  if (client->currently_removing) {
+    GST_WARNING_OBJECT (sink, "[fd %5d] client is already being removed", fd);
+    return;
+  } else {
+    client->currently_removing = TRUE;
+  }
+
+  /* FIXME: if we keep track of ip we can log it here and signal */
+  switch (client->status) {
+    case GST_CLIENT_STATUS_OK:
+      GST_WARNING_OBJECT (sink, "[fd %5d] removing client %p for no reason",
+          fd, client);
+      break;
+    case GST_CLIENT_STATUS_CLOSED:
+      GST_DEBUG_OBJECT (sink, "[fd %5d] removing client %p because of close",
+          fd, client);
+      break;
+    case GST_CLIENT_STATUS_REMOVED:
+      GST_DEBUG_OBJECT (sink,
+          "[fd %5d] removing client %p because the app removed it", fd, client);
+      break;
+    case GST_CLIENT_STATUS_SLOW:
+      GST_INFO_OBJECT (sink,
+          "[fd %5d] removing client %p because it was too slow", fd, client);
+      break;
+    case GST_CLIENT_STATUS_ERROR:
+      GST_WARNING_OBJECT (sink,
+          "[fd %5d] removing client %p because of error", fd, client);
+      break;
+    case GST_CLIENT_STATUS_FLUSHING:
+    default:
+      GST_WARNING_OBJECT (sink,
+          "[fd %5d] removing client %p with invalid reason %d", fd, client,
+          client->status);
+      break;
+  }
+
+  gst_poll_remove_fd (sink->fdset, &client->fd);
+
+  g_get_current_time (&now);
+  client->disconnect_time = GST_TIMEVAL_TO_TIME (now);
+
+  /* free client buffers */
+  g_slist_foreach (client->sending, (GFunc) gst_mini_object_unref, NULL);
+  g_slist_free (client->sending);
+  client->sending = NULL;
+
+  if (client->caps)
+    gst_caps_unref (client->caps);
+  client->caps = NULL;
+
+  /* unlock the mutex before signaling because the signal handler
+   * might query some properties */
+  CLIENTS_UNLOCK (sink);
+
+  g_signal_emit (G_OBJECT (sink),
+      gst_multi_fd_sink_signals[SIGNAL_CLIENT_REMOVED], 0, fd, client->status);
+
+  /* lock again before we remove the client completely */
+  CLIENTS_LOCK (sink);
+
+  /* fd cannot be reused in the above signal callback so we can safely
+   * remove it from the hashtable here */
+  if (!g_hash_table_remove (sink->fd_hash, &client->fd.fd)) {
+    GST_WARNING_OBJECT (sink,
+        "[fd %5d] error removing client %p from hash", client->fd.fd, client);
+  }
+  /* after releasing the lock above, the link could be invalid, more
+   * precisely, the next and prev pointers could point to invalid list
+   * links. One optimisation could be to add a cookie to the linked list
+   * and take a shortcut when it did not change between unlocking and locking
+   * our mutex. For now we just walk the list again. */
+  sink->clients = g_list_remove (sink->clients, client);
+  sink->clients_cookie++;
+
+  if (fclass->removed)
+    fclass->removed (sink, client->fd.fd);
+
+  g_free (client);
+  CLIENTS_UNLOCK (sink);
+
+  /* and the fd is really gone now */
+  g_signal_emit (G_OBJECT (sink),
+      gst_multi_fd_sink_signals[SIGNAL_CLIENT_FD_REMOVED], 0, fd);
+
+  CLIENTS_LOCK (sink);
+}
+
+/* handle a read on a client fd,
+ * which either indicates a close or should be ignored
+ * returns FALSE if some error occured or the client closed. */
+static gboolean
+gst_multi_fd_sink_handle_client_read (GstMultiFdSink * sink,
+    GstTCPClient * client)
+{
+  int avail, fd;
+  gboolean ret;
+
+  fd = client->fd.fd;
+
+  if (ioctl (fd, FIONREAD, &avail) < 0)
+    goto ioctl_failed;
+
+  GST_DEBUG_OBJECT (sink, "[fd %5d] select reports client read of %d bytes",
+      fd, avail);
+
+  ret = TRUE;
+
+  if (avail == 0) {
+    /* client sent close, so remove it */
+    GST_DEBUG_OBJECT (sink, "[fd %5d] client asked for close, removing", fd);
+    client->status = GST_CLIENT_STATUS_CLOSED;
+    ret = FALSE;
+  } else if (avail < 0) {
+    GST_WARNING_OBJECT (sink, "[fd %5d] avail < 0, removing", fd);
+    client->status = GST_CLIENT_STATUS_ERROR;
+    ret = FALSE;
+  } else {
+    guint8 dummy[512];
+    gint nread;
+
+    /* just Read 'n' Drop, could also just drop the client as it's not supposed
+     * to write to us except for closing the socket, I guess it's because we
+     * like to listen to our customers. */
+    do {
+      /* this is the maximum we can read */
+      gint to_read = MIN (avail, 512);
+
+      GST_DEBUG_OBJECT (sink, "[fd %5d] client wants us to read %d bytes",
+          fd, to_read);
+
+      nread = read (fd, dummy, to_read);
+      if (nread < -1) {
+        GST_WARNING_OBJECT (sink, "[fd %5d] could not read %d bytes: %s (%d)",
+            fd, to_read, g_strerror (errno), errno);
+        client->status = GST_CLIENT_STATUS_ERROR;
+        ret = FALSE;
+        break;
+      } else if (nread == 0) {
+        GST_WARNING_OBJECT (sink, "[fd %5d] 0 bytes in read, removing", fd);
+        client->status = GST_CLIENT_STATUS_ERROR;
+        ret = FALSE;
+        break;
+      }
+      avail -= nread;
+    }
+    while (avail > 0);
+  }
+  return ret;
+
+  /* ERRORS */
+ioctl_failed:
+  {
+    GST_WARNING_OBJECT (sink, "[fd %5d] ioctl failed: %s (%d)",
+        fd, g_strerror (errno), errno);
+    client->status = GST_CLIENT_STATUS_ERROR;
+    return FALSE;
+  }
+}
+
+static gboolean
+is_sync_frame (GstMultiFdSink * sink, GstBuffer * buffer)
+{
+  if (GST_BUFFER_FLAG_IS_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT)) {
+    return FALSE;
+  } else if (!GST_BUFFER_FLAG_IS_SET (buffer, GST_BUFFER_FLAG_IN_CAPS)) {
+    return TRUE;
+  }
+
+  return FALSE;
+}
+
+/* queue the given buffer for the given client */
+static gboolean
+gst_multi_fd_sink_client_queue_buffer (GstMultiFdSink * sink,
+    GstTCPClient * client, GstBuffer * buffer)
+{
+  GstCaps *caps;
+
+  /* TRUE: send them if the new caps have them */
+  gboolean send_streamheader = FALSE;
+  GstStructure *s;
+
+  /* before we queue the buffer, we check if we need to queue streamheader
+   * buffers (because it's a new client, or because they changed) */
+  caps = gst_pad_get_current_caps (GST_BASE_SINK_PAD (sink));
+
+  if (!client->caps) {
+    GST_DEBUG_OBJECT (sink,
+        "[fd %5d] no previous caps for this client, send streamheader",
+        client->fd.fd);
+    send_streamheader = TRUE;
+    client->caps = gst_caps_ref (caps);
+  } else {
+    /* there were previous caps recorded, so compare */
+    if (!gst_caps_is_equal (caps, client->caps)) {
+      const GValue *sh1, *sh2;
+
+      /* caps are not equal, but could still have the same streamheader */
+      s = gst_caps_get_structure (caps, 0);
+      if (!gst_structure_has_field (s, "streamheader")) {
+        /* no new streamheader, so nothing new to send */
+        GST_DEBUG_OBJECT (sink,
+            "[fd %5d] new caps do not have streamheader, not sending",
+            client->fd.fd);
+      } else {
+        /* there is a new streamheader */
+        s = gst_caps_get_structure (client->caps, 0);
+        if (!gst_structure_has_field (s, "streamheader")) {
+          /* no previous streamheader, so send the new one */
+          GST_DEBUG_OBJECT (sink,
+              "[fd %5d] previous caps did not have streamheader, sending",
+              client->fd.fd);
+          send_streamheader = TRUE;
+        } else {
+          /* both old and new caps have streamheader set */
+          if (!sink->resend_streamheader) {
+            GST_DEBUG_OBJECT (sink,
+                "[fd %5d] asked to not resend the streamheader, not sending",
+                client->fd.fd);
+            send_streamheader = FALSE;
+          } else {
+            sh1 = gst_structure_get_value (s, "streamheader");
+            s = gst_caps_get_structure (caps, 0);
+            sh2 = gst_structure_get_value (s, "streamheader");
+            if (gst_value_compare (sh1, sh2) != GST_VALUE_EQUAL) {
+              GST_DEBUG_OBJECT (sink,
+                  "[fd %5d] new streamheader different from old, sending",
+                  client->fd.fd);
+              send_streamheader = TRUE;
+            }
+          }
+        }
+      }
+    }
+    /* Replace the old caps */
+    gst_caps_unref (client->caps);
+    client->caps = gst_caps_ref (caps);
+  }
+
+  if (G_UNLIKELY (send_streamheader)) {
+    const GValue *sh;
+    GArray *buffers;
+    int i;
+
+    GST_LOG_OBJECT (sink,
+        "[fd %5d] sending streamheader from caps %" GST_PTR_FORMAT,
+        client->fd.fd, caps);
+    s = gst_caps_get_structure (caps, 0);
+    if (!gst_structure_has_field (s, "streamheader")) {
+      GST_DEBUG_OBJECT (sink,
+          "[fd %5d] no new streamheader, so nothing to send", client->fd.fd);
+    } else {
+      GST_LOG_OBJECT (sink,
+          "[fd %5d] sending streamheader from caps %" GST_PTR_FORMAT,
+          client->fd.fd, caps);
+      sh = gst_structure_get_value (s, "streamheader");
+      g_assert (G_VALUE_TYPE (sh) == GST_TYPE_ARRAY);
+      buffers = g_value_peek_pointer (sh);
+      GST_DEBUG_OBJECT (sink, "%d streamheader buffers", buffers->len);
+      for (i = 0; i < buffers->len; ++i) {
+        GValue *bufval;
+        GstBuffer *buffer;
+
+        bufval = &g_array_index (buffers, GValue, i);
+        g_assert (G_VALUE_TYPE (bufval) == GST_TYPE_BUFFER);
+        buffer = g_value_peek_pointer (bufval);
+        GST_DEBUG_OBJECT (sink,
+            "[fd %5d] queueing streamheader buffer of length %" G_GSIZE_FORMAT,
+            client->fd.fd, gst_buffer_get_size (buffer));
+        gst_buffer_ref (buffer);
+
+        client->sending = g_slist_append (client->sending, buffer);
+      }
+    }
+  }
+
+  gst_caps_unref (caps);
+  caps = NULL;
+
+  GST_LOG_OBJECT (sink, "[fd %5d] queueing buffer of length %" G_GSIZE_FORMAT,
+      client->fd.fd, gst_buffer_get_size (buffer));
+
+  gst_buffer_ref (buffer);
+  client->sending = g_slist_append (client->sending, buffer);
+
+  return TRUE;
+}
+
+/* find the keyframe in the list of buffers starting the
+ * search from @idx. @direction as -1 will search backwards, 
+ * 1 will search forwards.
+ * Returns: the index or -1 if there is no keyframe after idx.
+ */
+static gint
+find_syncframe (GstMultiFdSink * sink, gint idx, gint direction)
+{
+  gint i, len, result;
+
+  /* take length of queued buffers */
+  len = sink->bufqueue->len;
+
+  /* assume we don't find a keyframe */
+  result = -1;
+
+  /* then loop over all buffers to find the first keyframe */
+  for (i = idx; i >= 0 && i < len; i += direction) {
+    GstBuffer *buf;
+
+    buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+    if (is_sync_frame (sink, buf)) {
+      GST_LOG_OBJECT (sink, "found keyframe at %d from %d, direction %d",
+          i, idx, direction);
+      result = i;
+      break;
+    }
+  }
+  return result;
+}
+
+#define find_next_syncframe(s,i)       find_syncframe(s,i,1)
+#define find_prev_syncframe(s,i)       find_syncframe(s,i,-1)
+
+/* Get the number of buffers from the buffer queue needed to satisfy
+ * the maximum max in the configured units.
+ * If units are not BUFFERS, and there are insufficient buffers in the
+ * queue to satify the limit, return len(queue) + 1 */
+static gint
+get_buffers_max (GstMultiFdSink * sink, gint64 max)
+{
+  switch (sink->unit_type) {
+    case GST_TCP_UNIT_TYPE_BUFFERS:
+      return max;
+    case GST_TCP_UNIT_TYPE_TIME:
+    {
+      GstBuffer *buf;
+      int i;
+      int len;
+      gint64 diff;
+      GstClockTime first = GST_CLOCK_TIME_NONE;
+
+      len = sink->bufqueue->len;
+
+      for (i = 0; i < len; i++) {
+        buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+        if (GST_BUFFER_TIMESTAMP_IS_VALID (buf)) {
+          if (first == -1)
+            first = GST_BUFFER_TIMESTAMP (buf);
+
+          diff = first - GST_BUFFER_TIMESTAMP (buf);
+
+          if (diff > max)
+            return i + 1;
+        }
+      }
+      return len + 1;
+    }
+    case GST_TCP_UNIT_TYPE_BYTES:
+    {
+      GstBuffer *buf;
+      int i;
+      int len;
+      gint acc = 0;
+
+      len = sink->bufqueue->len;
+
+      for (i = 0; i < len; i++) {
+        buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+        acc += gst_buffer_get_size (buf);
+
+        if (acc > max)
+          return i + 1;
+      }
+      return len + 1;
+    }
+    default:
+      return max;
+  }
+}
+
+/* find the positions in the buffer queue where *_min and *_max
+ * is satisfied
+ */
+/* count the amount of data in the buffers and return the index
+ * that satifies the given limits.
+ *
+ * Returns: index @idx in the buffer queue so that the given limits are
+ * satisfied. TRUE if all the limits could be satisfied, FALSE if not
+ * enough data was in the queue.
+ *
+ * FIXME, this code might now work if any of the units is in buffers...
+ */
+static gboolean
+find_limits (GstMultiFdSink * sink,
+    gint * min_idx, gint bytes_min, gint buffers_min, gint64 time_min,
+    gint * max_idx, gint bytes_max, gint buffers_max, gint64 time_max)
+{
+  GstClockTime first, time;
+  gint i, len, bytes;
+  gboolean result, max_hit;
+
+  /* take length of queue */
+  len = sink->bufqueue->len;
+
+  /* this must hold */
+  g_assert (len > 0);
+
+  GST_LOG_OBJECT (sink,
+      "bytes_min %d, buffers_min %d, time_min %" GST_TIME_FORMAT
+      ", bytes_max %d, buffers_max %d, time_max %" GST_TIME_FORMAT, bytes_min,
+      buffers_min, GST_TIME_ARGS (time_min), bytes_max, buffers_max,
+      GST_TIME_ARGS (time_max));
+
+  /* do the trivial buffer limit test */
+  if (buffers_min != -1 && len < buffers_min) {
+    *min_idx = len - 1;
+    *max_idx = len - 1;
+    return FALSE;
+  }
+
+  result = FALSE;
+  /* else count bytes and time */
+  first = -1;
+  bytes = 0;
+  /* unset limits */
+  *min_idx = -1;
+  *max_idx = -1;
+  max_hit = FALSE;
+
+  i = 0;
+  /* loop through the buffers, when a limit is ok, mark it 
+   * as -1, we have at least one buffer in the queue. */
+  do {
+    GstBuffer *buf;
+
+    /* if we checked all min limits, update result */
+    if (bytes_min == -1 && time_min == -1 && *min_idx == -1) {
+      /* don't go below 0 */
+      *min_idx = MAX (i - 1, 0);
+    }
+    /* if we reached one max limit break out */
+    if (max_hit) {
+      /* i > 0 when we get here, we subtract one to get the position
+       * of the previous buffer. */
+      *max_idx = i - 1;
+      /* we have valid complete result if we found a min_idx too */
+      result = *min_idx != -1;
+      break;
+    }
+    buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+
+    bytes += gst_buffer_get_size (buf);
+
+    /* take timestamp and save for the base first timestamp */
+    if ((time = GST_BUFFER_TIMESTAMP (buf)) != -1) {
+      GST_LOG_OBJECT (sink, "Ts %" GST_TIME_FORMAT " on buffer",
+          GST_TIME_ARGS (time));
+      if (first == -1)
+        first = time;
+
+      /* increase max usage if we did not fill enough. Note that
+       * buffers are sorted from new to old, so the first timestamp is
+       * bigger than the next one. */
+      if (time_min != -1 && first - time >= time_min)
+        time_min = -1;
+      if (time_max != -1 && first - time >= time_max)
+        max_hit = TRUE;
+    } else {
+      GST_LOG_OBJECT (sink, "No timestamp on buffer");
+    }
+    /* time is OK or unknown, check and increase if not enough bytes */
+    if (bytes_min != -1) {
+      if (bytes >= bytes_min)
+        bytes_min = -1;
+    }
+    if (bytes_max != -1) {
+      if (bytes >= bytes_max) {
+        max_hit = TRUE;
+      }
+    }
+    i++;
+  }
+  while (i < len);
+
+  /* if we did not hit the max or min limit, set to buffer size */
+  if (*max_idx == -1)
+    *max_idx = len - 1;
+  /* make sure min does not exceed max */
+  if (*min_idx == -1)
+    *min_idx = *max_idx;
+
+  return result;
+}
+
+/* parse the unit/value pair and assign it to the result value of the
+ * right type, leave the other values untouched 
+ *
+ * Returns: FALSE if the unit is unknown or undefined. TRUE otherwise.
+ */
+static gboolean
+assign_value (GstTCPUnitType unit, guint64 value, gint * bytes, gint * buffers,
+    GstClockTime * time)
+{
+  gboolean res = TRUE;
+
+  /* set only the limit of the given format to the given value */
+  switch (unit) {
+    case GST_TCP_UNIT_TYPE_BUFFERS:
+      *buffers = (gint) value;
+      break;
+    case GST_TCP_UNIT_TYPE_TIME:
+      *time = value;
+      break;
+    case GST_TCP_UNIT_TYPE_BYTES:
+      *bytes = (gint) value;
+      break;
+    case GST_TCP_UNIT_TYPE_UNDEFINED:
+    default:
+      res = FALSE;
+      break;
+  }
+  return res;
+}
+
+/* count the index in the buffer queue to satisfy the given unit
+ * and value pair starting from buffer at index 0.
+ *
+ * Returns: TRUE if there was enough data in the queue to satisfy the
+ * burst values. @idx contains the index in the buffer that contains enough
+ * data to satisfy the limits or the last buffer in the queue when the
+ * function returns FALSE.
+ */
+static gboolean
+count_burst_unit (GstMultiFdSink * sink, gint * min_idx,
+    GstTCPUnitType min_unit, guint64 min_value, gint * max_idx,
+    GstTCPUnitType max_unit, guint64 max_value)
+{
+  gint bytes_min = -1, buffers_min = -1;
+  gint bytes_max = -1, buffers_max = -1;
+  GstClockTime time_min = GST_CLOCK_TIME_NONE, time_max = GST_CLOCK_TIME_NONE;
+
+  assign_value (min_unit, min_value, &bytes_min, &buffers_min, &time_min);
+  assign_value (max_unit, max_value, &bytes_max, &buffers_max, &time_max);
+
+  return find_limits (sink, min_idx, bytes_min, buffers_min, time_min,
+      max_idx, bytes_max, buffers_max, time_max);
+}
+
+/* decide where in the current buffer queue this new client should start
+ * receiving buffers from.
+ * This function is called whenever a client is connected and has not yet
+ * received a buffer.
+ * If this returns -1, it means that we haven't found a good point to
+ * start streaming from yet, and this function should be called again later
+ * when more buffers have arrived.
+ */
+static gint
+gst_multi_fd_sink_new_client (GstMultiFdSink * sink, GstTCPClient * client)
+{
+  gint result;
+
+  GST_DEBUG_OBJECT (sink,
+      "[fd %5d] new client, deciding where to start in queue", client->fd.fd);
+  GST_DEBUG_OBJECT (sink, "queue is currently %d buffers long",
+      sink->bufqueue->len);
+  switch (client->sync_method) {
+    case GST_SYNC_METHOD_LATEST:
+      /* no syncing, we are happy with whatever the client is going to get */
+      result = client->bufpos;
+      GST_DEBUG_OBJECT (sink,
+          "[fd %5d] SYNC_METHOD_LATEST, position %d", client->fd.fd, result);
+      break;
+    case GST_SYNC_METHOD_NEXT_KEYFRAME:
+    {
+      /* if one of the new buffers (between client->bufpos and 0) in the queue
+       * is a sync point, we can proceed, otherwise we need to keep waiting */
+      GST_LOG_OBJECT (sink,
+          "[fd %5d] new client, bufpos %d, waiting for keyframe", client->fd.fd,
+          client->bufpos);
+
+      result = find_prev_syncframe (sink, client->bufpos);
+      if (result != -1) {
+        GST_DEBUG_OBJECT (sink,
+            "[fd %5d] SYNC_METHOD_NEXT_KEYFRAME: result %d",
+            client->fd.fd, result);
+        break;
+      }
+
+      /* client is not on a syncbuffer, need to skip these buffers and
+       * wait some more */
+      GST_LOG_OBJECT (sink,
+          "[fd %5d] new client, skipping buffer(s), no syncpoint found",
+          client->fd.fd);
+      client->bufpos = -1;
+      break;
+    }
+    case GST_SYNC_METHOD_LATEST_KEYFRAME:
+    {
+      GST_DEBUG_OBJECT (sink,
+          "[fd %5d] SYNC_METHOD_LATEST_KEYFRAME", client->fd.fd);
+
+      /* for new clients we initially scan the complete buffer queue for
+       * a sync point when a buffer is added. If we don't find a keyframe,
+       * we need to wait for the next keyframe and so we change the client's
+       * sync method to GST_SYNC_METHOD_NEXT_KEYFRAME.
+       */
+      result = find_next_syncframe (sink, 0);
+      if (result != -1) {
+        GST_DEBUG_OBJECT (sink,
+            "[fd %5d] SYNC_METHOD_LATEST_KEYFRAME: result %d", client->fd.fd,
+            result);
+        break;
+      }
+
+      GST_DEBUG_OBJECT (sink,
+          "[fd %5d] SYNC_METHOD_LATEST_KEYFRAME: no keyframe found, "
+          "switching to SYNC_METHOD_NEXT_KEYFRAME", client->fd.fd);
+      /* throw client to the waiting state */
+      client->bufpos = -1;
+      /* and make client sync to next keyframe */
+      client->sync_method = GST_SYNC_METHOD_NEXT_KEYFRAME;
+      break;
+    }
+    case GST_SYNC_METHOD_BURST:
+    {
+      gboolean ok;
+      gint max;
+
+      /* move to the position where we satisfy the client's burst
+       * parameters. If we could not satisfy the parameters because there
+       * is not enough data, we just send what we have (which is in result).
+       * We use the max value to limit the search
+       */
+      ok = count_burst_unit (sink, &result, client->burst_min_unit,
+          client->burst_min_value, &max, client->burst_max_unit,
+          client->burst_max_value);
+      GST_DEBUG_OBJECT (sink,
+          "[fd %5d] SYNC_METHOD_BURST: burst_unit returned %d, result %d",
+          client->fd.fd, ok, result);
+
+      GST_LOG_OBJECT (sink, "min %d, max %d", result, max);
+
+      /* we hit the max and it is below the min, use that then */
+      if (max != -1 && max <= result) {
+        result = MAX (max - 1, 0);
+        GST_DEBUG_OBJECT (sink,
+            "[fd %5d] SYNC_METHOD_BURST: result above max, taken down to %d",
+            client->fd.fd, result);
+      }
+      break;
+    }
+    case GST_SYNC_METHOD_BURST_KEYFRAME:
+    {
+      gint min_idx, max_idx;
+      gint next_syncframe, prev_syncframe;
+
+      /* BURST_KEYFRAME:
+       *
+       * _always_ start sending a keyframe to the client. We first search
+       * a keyframe between min/max limits. If there is none, we send it the
+       * last keyframe before min. If there is none, the behaviour is like
+       * NEXT_KEYFRAME.
+       */
+      /* gather burst limits */
+      count_burst_unit (sink, &min_idx, client->burst_min_unit,
+          client->burst_min_value, &max_idx, client->burst_max_unit,
+          client->burst_max_value);
+
+      GST_LOG_OBJECT (sink, "min %d, max %d", min_idx, max_idx);
+
+      /* first find a keyframe after min_idx */
+      next_syncframe = find_next_syncframe (sink, min_idx);
+      if (next_syncframe != -1 && next_syncframe < max_idx) {
+        /* we have a valid keyframe and it's below the max */
+        GST_LOG_OBJECT (sink, "found keyframe in min/max limits");
+        result = next_syncframe;
+        break;
+      }
+
+      /* no valid keyframe, try to find one below min */
+      prev_syncframe = find_prev_syncframe (sink, min_idx);
+      if (prev_syncframe != -1) {
+        GST_WARNING_OBJECT (sink,
+            "using keyframe below min in BURST_KEYFRAME sync mode");
+        result = prev_syncframe;
+        break;
+      }
+
+      /* no prev keyframe or not enough data  */
+      GST_WARNING_OBJECT (sink,
+          "no prev keyframe found in BURST_KEYFRAME sync mode, waiting for next");
+
+      /* throw client to the waiting state */
+      client->bufpos = -1;
+      /* and make client sync to next keyframe */
+      client->sync_method = GST_SYNC_METHOD_NEXT_KEYFRAME;
+      result = -1;
+      break;
+    }
+    case GST_SYNC_METHOD_BURST_WITH_KEYFRAME:
+    {
+      gint min_idx, max_idx;
+      gint next_syncframe;
+
+      /* BURST_WITH_KEYFRAME:
+       *
+       * try to start sending a keyframe to the client. We first search
+       * a keyframe between min/max limits. If there is none, we send it the
+       * amount of data up 'till min.
+       */
+      /* gather enough data to burst */
+      count_burst_unit (sink, &min_idx, client->burst_min_unit,
+          client->burst_min_value, &max_idx, client->burst_max_unit,
+          client->burst_max_value);
+
+      GST_LOG_OBJECT (sink, "min %d, max %d", min_idx, max_idx);
+
+      /* first find a keyframe after min_idx */
+      next_syncframe = find_next_syncframe (sink, min_idx);
+      if (next_syncframe != -1 && next_syncframe < max_idx) {
+        /* we have a valid keyframe and it's below the max */
+        GST_LOG_OBJECT (sink, "found keyframe in min/max limits");
+        result = next_syncframe;
+        break;
+      }
+
+      /* no keyframe, send data from min_idx */
+      GST_WARNING_OBJECT (sink, "using min in BURST_WITH_KEYFRAME sync mode");
+
+      /* make sure we don't go over the max limit */
+      if (max_idx != -1 && max_idx <= min_idx) {
+        result = MAX (max_idx - 1, 0);
+      } else {
+        result = min_idx;
+      }
+
+      break;
+    }
+    default:
+      g_warning ("unknown sync method %d", client->sync_method);
+      result = client->bufpos;
+      break;
+  }
+  return result;
+}
+
+/* Handle a write on a client,
+ * which indicates a read request from a client.
+ *
+ * For each client we maintain a queue of GstBuffers that contain the raw bytes
+ * we need to send to the client.
+ *
+ * We first check to see if we need to send streamheaders. If so, we queue them.
+ *
+ * Then we run into the main loop that tries to send as many buffers as
+ * possible. It will first exhaust the client->sending queue and if the queue
+ * is empty, it will pick a buffer from the global queue.
+ *
+ * Sending the buffers from the client->sending queue is basically writing
+ * the bytes to the socket and maintaining a count of the bytes that were
+ * sent. When the buffer is completely sent, it is removed from the
+ * client->sending queue and we try to pick a new buffer for sending.
+ *
+ * When the sending returns a partial buffer we stop sending more data as
+ * the next send operation could block.
+ *
+ * This functions returns FALSE if some error occured.
+ */
+static gboolean
+gst_multi_fd_sink_handle_client_write (GstMultiFdSink * sink,
+    GstTCPClient * client)
+{
+  int fd = client->fd.fd;
+  gboolean more;
+  gboolean flushing;
+  GstClockTime now;
+  GTimeVal nowtv;
+
+  g_get_current_time (&nowtv);
+  now = GST_TIMEVAL_TO_TIME (nowtv);
+
+  flushing = client->status == GST_CLIENT_STATUS_FLUSHING;
+
+  more = TRUE;
+  do {
+    gint maxsize;
+
+    if (!client->sending) {
+      /* client is not working on a buffer */
+      if (client->bufpos == -1) {
+        /* client is too fast, remove from write queue until new buffer is
+         * available */
+        gst_poll_fd_ctl_write (sink->fdset, &client->fd, FALSE);
+        /* if we flushed out all of the client buffers, we can stop */
+        if (client->flushcount == 0)
+          goto flushed;
+
+        return TRUE;
+      } else {
+        /* client can pick a buffer from the global queue */
+        GstBuffer *buf;
+        GstClockTime timestamp;
+
+        /* for new connections, we need to find a good spot in the
+         * bufqueue to start streaming from */
+        if (client->new_connection && !flushing) {
+          gint position = gst_multi_fd_sink_new_client (sink, client);
+
+          if (position >= 0) {
+            /* we got a valid spot in the queue */
+            client->new_connection = FALSE;
+            client->bufpos = position;
+          } else {
+            /* cannot send data to this client yet */
+            gst_poll_fd_ctl_write (sink->fdset, &client->fd, FALSE);
+            return TRUE;
+          }
+        }
+
+        /* we flushed all remaining buffers, no need to get a new one */
+        if (client->flushcount == 0)
+          goto flushed;
+
+        /* grab buffer */
+        buf = g_array_index (sink->bufqueue, GstBuffer *, client->bufpos);
+        client->bufpos--;
+
+        /* update stats */
+        timestamp = GST_BUFFER_TIMESTAMP (buf);
+        if (client->first_buffer_ts == GST_CLOCK_TIME_NONE)
+          client->first_buffer_ts = timestamp;
+        if (timestamp != -1)
+          client->last_buffer_ts = timestamp;
+
+        /* decrease flushcount */
+        if (client->flushcount != -1)
+          client->flushcount--;
+
+        GST_LOG_OBJECT (sink, "[fd %5d] client %p at position %d",
+            fd, client, client->bufpos);
+
+        /* queueing a buffer will ref it */
+        gst_multi_fd_sink_client_queue_buffer (sink, client, buf);
+
+        /* need to start from the first byte for this new buffer */
+        client->bufoffset = 0;
+      }
+    }
+
+    /* see if we need to send something */
+    if (client->sending) {
+      ssize_t wrote;
+      GstBuffer *head;
+      guint8 *data;
+      gsize size;
+
+      /* pick first buffer from list */
+      head = GST_BUFFER (client->sending->data);
+
+      data = gst_buffer_map (head, &size, NULL, GST_MAP_READ);
+      maxsize = size - client->bufoffset;
+
+      /* try to write the complete buffer */
+#ifdef MSG_NOSIGNAL
+#define FLAGS MSG_NOSIGNAL
+#else
+#define FLAGS 0
+#endif
+      if (client->is_socket) {
+        wrote = send (fd, data + client->bufoffset, maxsize, FLAGS);
+      } else {
+        wrote = write (fd, data + client->bufoffset, maxsize);
+      }
+      gst_buffer_unmap (head, data, size);
+
+      if (wrote < 0) {
+        /* hmm error.. */
+        if (errno == EAGAIN) {
+          /* nothing serious, resource was unavailable, try again later */
+          more = FALSE;
+        } else if (errno == ECONNRESET) {
+          goto connection_reset;
+        } else {
+          goto write_error;
+        }
+      } else {
+        if (wrote < maxsize) {
+          /* partial write means that the client cannot read more and we should
+           * stop sending more */
+          GST_LOG_OBJECT (sink,
+              "partial write on %d of %" G_GSSIZE_FORMAT " bytes", fd, wrote);
+          client->bufoffset += wrote;
+          more = FALSE;
+        } else {
+          /* complete buffer was written, we can proceed to the next one */
+          client->sending = g_slist_remove (client->sending, head);
+          gst_buffer_unref (head);
+          /* make sure we start from byte 0 for the next buffer */
+          client->bufoffset = 0;
+        }
+        /* update stats */
+        client->bytes_sent += wrote;
+        client->last_activity_time = now;
+        sink->bytes_served += wrote;
+      }
+    }
+  } while (more);
+
+  return TRUE;
+
+  /* ERRORS */
+flushed:
+  {
+    GST_DEBUG_OBJECT (sink, "[fd %5d] flushed, removing", fd);
+    client->status = GST_CLIENT_STATUS_REMOVED;
+    return FALSE;
+  }
+connection_reset:
+  {
+    GST_DEBUG_OBJECT (sink, "[fd %5d] connection reset by peer, removing", fd);
+    client->status = GST_CLIENT_STATUS_CLOSED;
+    return FALSE;
+  }
+write_error:
+  {
+    GST_WARNING_OBJECT (sink,
+        "[fd %5d] could not write, removing client: %s (%d)", fd,
+        g_strerror (errno), errno);
+    client->status = GST_CLIENT_STATUS_ERROR;
+    return FALSE;
+  }
+}
+
+/* calculate the new position for a client after recovery. This function
+ * does not update the client position but merely returns the required
+ * position.
+ */
+static gint
+gst_multi_fd_sink_recover_client (GstMultiFdSink * sink, GstTCPClient * client)
+{
+  gint newbufpos;
+
+  GST_WARNING_OBJECT (sink,
+      "[fd %5d] client %p is lagging at %d, recover using policy %d",
+      client->fd.fd, client, client->bufpos, sink->recover_policy);
+
+  switch (sink->recover_policy) {
+    case GST_RECOVER_POLICY_NONE:
+      /* do nothing, client will catch up or get kicked out when it reaches
+       * the hard max */
+      newbufpos = client->bufpos;
+      break;
+    case GST_RECOVER_POLICY_RESYNC_LATEST:
+      /* move to beginning of queue */
+      newbufpos = -1;
+      break;
+    case GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT:
+      /* move to beginning of soft max */
+      newbufpos = get_buffers_max (sink, sink->units_soft_max);
+      break;
+    case GST_RECOVER_POLICY_RESYNC_KEYFRAME:
+      /* find keyframe in buffers, we search backwards to find the
+       * closest keyframe relative to what this client already received. */
+      newbufpos = MIN (sink->bufqueue->len - 1,
+          get_buffers_max (sink, sink->units_soft_max) - 1);
+
+      while (newbufpos >= 0) {
+        GstBuffer *buf;
+
+        buf = g_array_index (sink->bufqueue, GstBuffer *, newbufpos);
+        if (is_sync_frame (sink, buf)) {
+          /* found a buffer that is not a delta unit */
+          break;
+        }
+        newbufpos--;
+      }
+      break;
+    default:
+      /* unknown recovery procedure */
+      newbufpos = get_buffers_max (sink, sink->units_soft_max);
+      break;
+  }
+  return newbufpos;
+}
+
+/* Queue a buffer on the global queue.
+ *
+ * This function adds the buffer to the front of a GArray. It removes the
+ * tail buffer if the max queue size is exceeded, unreffing the queued buffer.
+ * Note that unreffing the buffer is not a problem as clients who
+ * started writing out this buffer will still have a reference to it in the
+ * client->sending queue.
+ *
+ * After adding the buffer, we update all client positions in the queue. If
+ * a client moves over the soft max, we start the recovery procedure for this
+ * slow client. If it goes over the hard max, it is put into the slow list
+ * and removed.
+ *
+ * Special care is taken of clients that were waiting for a new buffer (they
+ * had a position of -1) because they can proceed after adding this new buffer.
+ * This is done by adding the client back into the write fd_set and signaling
+ * the select thread that the fd_set changed.
+ */
+static void
+gst_multi_fd_sink_queue_buffer (GstMultiFdSink * sink, GstBuffer * buf)
+{
+  GList *clients, *next;
+  gint queuelen;
+  gboolean need_signal = FALSE;
+  gint max_buffer_usage;
+  gint i;
+  GTimeVal nowtv;
+  GstClockTime now;
+  gint max_buffers, soft_max_buffers;
+  guint cookie;
+
+  g_get_current_time (&nowtv);
+  now = GST_TIMEVAL_TO_TIME (nowtv);
+
+  CLIENTS_LOCK (sink);
+  /* add buffer to queue */
+  g_array_prepend_val (sink->bufqueue, buf);
+  queuelen = sink->bufqueue->len;
+
+  if (sink->units_max > 0)
+    max_buffers = get_buffers_max (sink, sink->units_max);
+  else
+    max_buffers = -1;
+
+  if (sink->units_soft_max > 0)
+    soft_max_buffers = get_buffers_max (sink, sink->units_soft_max);
+  else
+    soft_max_buffers = -1;
+  GST_LOG_OBJECT (sink, "Using max %d, softmax %d", max_buffers,
+      soft_max_buffers);
+
+  /* then loop over the clients and update the positions */
+  max_buffer_usage = 0;
+
+restart:
+  cookie = sink->clients_cookie;
+  for (clients = sink->clients; clients; clients = next) {
+    GstTCPClient *client;
+
+    if (cookie != sink->clients_cookie) {
+      GST_DEBUG_OBJECT (sink, "Clients cookie outdated, restarting");
+      goto restart;
+    }
+
+    client = (GstTCPClient *) clients->data;
+    next = g_list_next (clients);
+
+    client->bufpos++;
+    GST_LOG_OBJECT (sink, "[fd %5d] client %p at position %d",
+        client->fd.fd, client, client->bufpos);
+    /* check soft max if needed, recover client */
+    if (soft_max_buffers > 0 && client->bufpos >= soft_max_buffers) {
+      gint newpos;
+
+      newpos = gst_multi_fd_sink_recover_client (sink, client);
+      if (newpos != client->bufpos) {
+        client->dropped_buffers += client->bufpos - newpos;
+        client->bufpos = newpos;
+        client->discont = TRUE;
+        GST_INFO_OBJECT (sink, "[fd %5d] client %p position reset to %d",
+            client->fd.fd, client, client->bufpos);
+      } else {
+        GST_INFO_OBJECT (sink,
+            "[fd %5d] client %p not recovering position",
+            client->fd.fd, client);
+      }
+    }
+    /* check hard max and timeout, remove client */
+    if ((max_buffers > 0 && client->bufpos >= max_buffers) ||
+        (sink->timeout > 0
+            && now - client->last_activity_time > sink->timeout)) {
+      /* remove client */
+      GST_WARNING_OBJECT (sink, "[fd %5d] client %p is too slow, removing",
+          client->fd.fd, client);
+      /* remove the client, the fd set will be cleared and the select thread
+       * will be signaled */
+      client->status = GST_CLIENT_STATUS_SLOW;
+      /* set client to invalid position while being removed */
+      client->bufpos = -1;
+      gst_multi_fd_sink_remove_client_link (sink, clients);
+      need_signal = TRUE;
+      continue;
+    } else if (client->bufpos == 0 || client->new_connection) {
+      /* can send data to this client now. need to signal the select thread that
+       * the fd_set changed */
+      gst_poll_fd_ctl_write (sink->fdset, &client->fd, TRUE);
+      need_signal = TRUE;
+    }
+    /* keep track of maximum buffer usage */
+    if (client->bufpos > max_buffer_usage) {
+      max_buffer_usage = client->bufpos;
+    }
+  }
+
+  /* make sure we respect bytes-min, buffers-min and time-min when they are set */
+  {
+    gint usage, max;
+
+    GST_LOG_OBJECT (sink,
+        "extending queue %d to respect time_min %" GST_TIME_FORMAT
+        ", bytes_min %d, buffers_min %d", max_buffer_usage,
+        GST_TIME_ARGS (sink->time_min), sink->bytes_min, sink->buffers_min);
+
+    /* get index where the limits are ok, we don't really care if all limits
+     * are ok, we just queue as much as we need. We also don't compare against
+     * the max limits. */
+    find_limits (sink, &usage, sink->bytes_min, sink->buffers_min,
+        sink->time_min, &max, -1, -1, -1);
+
+    max_buffer_usage = MAX (max_buffer_usage, usage + 1);
+    GST_LOG_OBJECT (sink, "extended queue to %d", max_buffer_usage);
+  }
+
+  /* now look for sync points and make sure there is at least one
+   * sync point in the queue. We only do this if the LATEST_KEYFRAME or 
+   * BURST_KEYFRAME mode is selected */
+  if (sink->def_sync_method == GST_SYNC_METHOD_LATEST_KEYFRAME ||
+      sink->def_sync_method == GST_SYNC_METHOD_BURST_KEYFRAME) {
+    /* no point in searching beyond the queue length */
+    gint limit = queuelen;
+    GstBuffer *buf;
+
+    /* no point in searching beyond the soft-max if any. */
+    if (soft_max_buffers > 0) {
+      limit = MIN (limit, soft_max_buffers);
+    }
+    GST_LOG_OBJECT (sink,
+        "extending queue to include sync point, now at %d, limit is %d",
+        max_buffer_usage, limit);
+    for (i = 0; i < limit; i++) {
+      buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+      if (is_sync_frame (sink, buf)) {
+        /* found a sync frame, now extend the buffer usage to
+         * include at least this frame. */
+        max_buffer_usage = MAX (max_buffer_usage, i);
+        break;
+      }
+    }
+    GST_LOG_OBJECT (sink, "max buffer usage is now %d", max_buffer_usage);
+  }
+
+  GST_LOG_OBJECT (sink, "len %d, usage %d", queuelen, max_buffer_usage);
+
+  /* nobody is referencing units after max_buffer_usage so we can
+   * remove them from the queue. We remove them in reverse order as
+   * this is the most optimal for GArray. */
+  for (i = queuelen - 1; i > max_buffer_usage; i--) {
+    GstBuffer *old;
+
+    /* queue exceeded max size */
+    queuelen--;
+    old = g_array_index (sink->bufqueue, GstBuffer *, i);
+    sink->bufqueue = g_array_remove_index (sink->bufqueue, i);
+
+    /* unref tail buffer */
+    gst_buffer_unref (old);
+  }
+  /* save for stats */
+  sink->buffers_queued = max_buffer_usage;
+  CLIENTS_UNLOCK (sink);
+
+  /* and send a signal to thread if fd_set changed */
+  if (need_signal) {
+    gst_poll_restart (sink->fdset);
+  }
+}
+
+/* Handle the clients. Basically does a blocking select for one
+ * of the client fds to become read or writable. We also have a
+ * filedescriptor to receive commands on that we need to check.
+ *
+ * After going out of the select call, we read and write to all
+ * clients that can do so. Badly behaving clients are put on a
+ * garbage list and removed.
+ */
+static void
+gst_multi_fd_sink_handle_clients (GstMultiFdSink * sink)
+{
+  int result;
+  GList *clients, *next;
+  gboolean try_again;
+  GstMultiFdSinkClass *fclass;
+  guint cookie;
+
+  fclass = GST_MULTI_FD_SINK_GET_CLASS (sink);
+
+  do {
+    try_again = FALSE;
+
+    /* check for:
+     * - server socket input (ie, new client connections)
+     * - client socket input (ie, clients saying goodbye)
+     * - client socket output (ie, client reads)          */
+    GST_LOG_OBJECT (sink, "waiting on action on fdset");
+
+    result = gst_poll_wait (sink->fdset, sink->timeout != 0 ? sink->timeout :
+        GST_CLOCK_TIME_NONE);
+
+    /* Handle the special case in which the sink is not receiving more buffers
+     * and will not disconnect inactive client in the streaming thread. */
+    if (G_UNLIKELY (result == 0)) {
+      GstClockTime now;
+      GTimeVal nowtv;
+
+      g_get_current_time (&nowtv);
+      now = GST_TIMEVAL_TO_TIME (nowtv);
+
+      CLIENTS_LOCK (sink);
+      for (clients = sink->clients; clients; clients = next) {
+        GstTCPClient *client;
+
+        client = (GstTCPClient *) clients->data;
+        next = g_list_next (clients);
+        if (sink->timeout > 0
+            && now - client->last_activity_time > sink->timeout) {
+          client->status = GST_CLIENT_STATUS_SLOW;
+          gst_multi_fd_sink_remove_client_link (sink, clients);
+        }
+      }
+      CLIENTS_UNLOCK (sink);
+      return;
+    } else if (result < 0) {
+      GST_WARNING_OBJECT (sink, "wait failed: %s (%d)", g_strerror (errno),
+          errno);
+      if (errno == EBADF) {
+        /* ok, so one or more of the fds is invalid. We loop over them to find
+         * the ones that give an error to the F_GETFL fcntl. */
+        CLIENTS_LOCK (sink);
+      restart:
+        cookie = sink->clients_cookie;
+        for (clients = sink->clients; clients; clients = next) {
+          GstTCPClient *client;
+          int fd;
+          long flags;
+          int res;
+
+          if (cookie != sink->clients_cookie) {
+            GST_DEBUG_OBJECT (sink, "Cookie changed finding bad fd");
+            goto restart;
+          }
+
+          client = (GstTCPClient *) clients->data;
+          next = g_list_next (clients);
+
+          fd = client->fd.fd;
+
+          res = fcntl (fd, F_GETFL, &flags);
+          if (res == -1) {
+            GST_WARNING_OBJECT (sink, "fnctl failed for %d, removing: %s (%d)",
+                fd, g_strerror (errno), errno);
+            if (errno == EBADF) {
+              client->status = GST_CLIENT_STATUS_ERROR;
+              /* releases the CLIENTS lock */
+              gst_multi_fd_sink_remove_client_link (sink, clients);
+            }
+          }
+        }
+        CLIENTS_UNLOCK (sink);
+        /* after this, go back in the select loop as the read/writefds
+         * are not valid */
+        try_again = TRUE;
+      } else if (errno == EINTR) {
+        /* interrupted system call, just redo the wait */
+        try_again = TRUE;
+      } else if (errno == EBUSY) {
+        /* the call to gst_poll_wait() was flushed */
+        return;
+      } else {
+        /* this is quite bad... */
+        GST_ELEMENT_ERROR (sink, RESOURCE, READ, (NULL),
+            ("select failed: %s (%d)", g_strerror (errno), errno));
+        return;
+      }
+    } else {
+      GST_LOG_OBJECT (sink, "wait done: %d sockets with events", result);
+    }
+  } while (try_again);
+
+  /* subclasses can check fdset with this virtual function */
+  if (fclass->wait)
+    fclass->wait (sink, sink->fdset);
+
+  /* Check the clients */
+  CLIENTS_LOCK (sink);
+
+restart2:
+  cookie = sink->clients_cookie;
+  for (clients = sink->clients; clients; clients = next) {
+    GstTCPClient *client;
+
+    if (sink->clients_cookie != cookie) {
+      GST_DEBUG_OBJECT (sink, "Restarting loop, cookie out of date");
+      goto restart2;
+    }
+
+    client = (GstTCPClient *) clients->data;
+    next = g_list_next (clients);
+
+    if (client->status != GST_CLIENT_STATUS_FLUSHING
+        && client->status != GST_CLIENT_STATUS_OK) {
+      gst_multi_fd_sink_remove_client_link (sink, clients);
+      continue;
+    }
+
+    if (gst_poll_fd_has_closed (sink->fdset, &client->fd)) {
+      client->status = GST_CLIENT_STATUS_CLOSED;
+      gst_multi_fd_sink_remove_client_link (sink, clients);
+      continue;
+    }
+    if (gst_poll_fd_has_error (sink->fdset, &client->fd)) {
+      GST_WARNING_OBJECT (sink, "gst_poll_fd_has_error for %d", client->fd.fd);
+      client->status = GST_CLIENT_STATUS_ERROR;
+      gst_multi_fd_sink_remove_client_link (sink, clients);
+      continue;
+    }
+    if (gst_poll_fd_can_read (sink->fdset, &client->fd)) {
+      /* handle client read */
+      if (!gst_multi_fd_sink_handle_client_read (sink, client)) {
+        gst_multi_fd_sink_remove_client_link (sink, clients);
+        continue;
+      }
+    }
+    if (gst_poll_fd_can_write (sink->fdset, &client->fd)) {
+      /* handle client write */
+      if (!gst_multi_fd_sink_handle_client_write (sink, client)) {
+        gst_multi_fd_sink_remove_client_link (sink, clients);
+        continue;
+      }
+    }
+  }
+  CLIENTS_UNLOCK (sink);
+}
+
+/* we handle the client communication in another thread so that we do not block
+ * the gstreamer thread while we select() on the client fds */
+static gpointer
+gst_multi_fd_sink_thread (GstMultiFdSink * sink)
+{
+  while (sink->running) {
+    gst_multi_fd_sink_handle_clients (sink);
+  }
+  return NULL;
+}
+
+static GstFlowReturn
+gst_multi_fd_sink_render (GstBaseSink * bsink, GstBuffer * buf)
+{
+  GstMultiFdSink *sink;
+  gboolean in_caps;
+#if 0
+  GstCaps *bufcaps, *padcaps;
+#endif
+
+  sink = GST_MULTI_FD_SINK (bsink);
+
+  g_return_val_if_fail (GST_OBJECT_FLAG_IS_SET (sink, GST_MULTI_FD_SINK_OPEN),
+      GST_FLOW_WRONG_STATE);
+
+#if 0
+  /* since we check every buffer for streamheader caps, we need to make
+   * sure every buffer has caps set */
+  bufcaps = gst_buffer_get_caps (buf);
+  padcaps = GST_PAD_CAPS (GST_BASE_SINK_PAD (bsink));
+
+  /* make sure we have caps on the pad */
+  if (!padcaps && !bufcaps)
+    goto no_caps;
+#endif
+
+  /* get IN_CAPS first, code below might mess with the flags */
+  in_caps = GST_BUFFER_FLAG_IS_SET (buf, GST_BUFFER_FLAG_IN_CAPS);
+
+#if 0
+  /* stamp the buffer with previous caps if no caps set */
+  if (!bufcaps) {
+    if (!gst_buffer_is_writable (buf)) {
+      /* metadata is not writable, copy will be made and original buffer
+       * will be unreffed so we need to ref so that we don't lose the
+       * buffer in the render method. */
+      gst_buffer_ref (buf);
+      /* the new buffer is ours only, we keep it out of the scope of this
+       * function */
+      buf = gst_buffer_make_writable (buf);
+    } else {
+      /* else the metadata is writable, we ref because we keep the buffer
+       * out of the scope of this method */
+      gst_buffer_ref (buf);
+    }
+    /* buffer metadata is writable now, set the caps */
+    gst_buffer_set_caps (buf, padcaps);
+  } else {
+    gst_caps_unref (bufcaps);
+
+    /* since we keep this buffer out of the scope of this method */
+    gst_buffer_ref (buf);
+  }
+#endif
+
+  GST_LOG_OBJECT (sink, "received buffer %p, in_caps: %s, offset %"
+      G_GINT64_FORMAT ", offset_end %" G_GINT64_FORMAT
+      ", timestamp %" GST_TIME_FORMAT ", duration %" GST_TIME_FORMAT,
+      buf, in_caps ? "yes" : "no", GST_BUFFER_OFFSET (buf),
+      GST_BUFFER_OFFSET_END (buf),
+      GST_TIME_ARGS (GST_BUFFER_TIMESTAMP (buf)),
+      GST_TIME_ARGS (GST_BUFFER_DURATION (buf)));
+
+  /* if we get IN_CAPS buffers, but the previous buffer was not IN_CAPS,
+   * it means we're getting new streamheader buffers, and we should clear
+   * the old ones */
+  if (in_caps && sink->previous_buffer_in_caps == FALSE) {
+    GST_DEBUG_OBJECT (sink,
+        "receiving new IN_CAPS buffers, clearing old streamheader");
+    g_slist_foreach (sink->streamheader, (GFunc) gst_mini_object_unref, NULL);
+    g_slist_free (sink->streamheader);
+    sink->streamheader = NULL;
+  }
+
+  /* save the current in_caps */
+  sink->previous_buffer_in_caps = in_caps;
+
+  /* if the incoming buffer is marked as IN CAPS, then we assume for now
+   * it's a streamheader that needs to be sent to each new client, so we
+   * put it on our internal list of streamheader buffers.
+   * FIXME: we could check if the buffer's contents are in fact part of the
+   * current streamheader.
+   *
+   * We don't send the buffer to the client, since streamheaders are sent
+   * separately when necessary. */
+  if (in_caps) {
+    GST_DEBUG_OBJECT (sink, "appending IN_CAPS buffer with length %"
+        G_GSIZE_FORMAT " to streamheader", gst_buffer_get_size (buf));
+    sink->streamheader = g_slist_append (sink->streamheader, buf);
+  } else {
+    /* queue the buffer, this is a regular data buffer. */
+    gst_multi_fd_sink_queue_buffer (sink, buf);
+
+    sink->bytes_to_serve += gst_buffer_get_size (buf);
+  }
+  return GST_FLOW_OK;
+
+  /* ERRORS */
+#if 0
+no_caps:
+  {
+    GST_ELEMENT_ERROR (sink, CORE, NEGOTIATION, (NULL),
+        ("Received first buffer without caps set"));
+    return GST_FLOW_NOT_NEGOTIATED;
+  }
+#endif
+}
+
+static void
+gst_multi_fd_sink_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec)
+{
+  GstMultiFdSink *multifdsink;
+
+  multifdsink = GST_MULTI_FD_SINK (object);
+
+  switch (prop_id) {
+    case PROP_MODE:
+      multifdsink->mode = g_value_get_enum (value);
+      break;
+    case PROP_BUFFERS_MAX:
+      multifdsink->units_max = g_value_get_int (value);
+      break;
+    case PROP_BUFFERS_SOFT_MAX:
+      multifdsink->units_soft_max = g_value_get_int (value);
+      break;
+    case PROP_TIME_MIN:
+      multifdsink->time_min = g_value_get_int64 (value);
+      break;
+    case PROP_BYTES_MIN:
+      multifdsink->bytes_min = g_value_get_int (value);
+      break;
+    case PROP_BUFFERS_MIN:
+      multifdsink->buffers_min = g_value_get_int (value);
+      break;
+    case PROP_UNIT_TYPE:
+      multifdsink->unit_type = g_value_get_enum (value);
+      break;
+    case PROP_UNITS_MAX:
+      multifdsink->units_max = g_value_get_int64 (value);
+      break;
+    case PROP_UNITS_SOFT_MAX:
+      multifdsink->units_soft_max = g_value_get_int64 (value);
+      break;
+    case PROP_RECOVER_POLICY:
+      multifdsink->recover_policy = g_value_get_enum (value);
+      break;
+    case PROP_TIMEOUT:
+      multifdsink->timeout = g_value_get_uint64 (value);
+      break;
+    case PROP_SYNC_METHOD:
+      multifdsink->def_sync_method = g_value_get_enum (value);
+      break;
+    case PROP_BURST_UNIT:
+      multifdsink->def_burst_unit = g_value_get_enum (value);
+      break;
+    case PROP_BURST_VALUE:
+      multifdsink->def_burst_value = g_value_get_uint64 (value);
+      break;
+    case PROP_QOS_DSCP:
+      multifdsink->qos_dscp = g_value_get_int (value);
+      setup_dscp (multifdsink);
+      break;
+    case PROP_HANDLE_READ:
+      multifdsink->handle_read = g_value_get_boolean (value);
+      break;
+    case PROP_RESEND_STREAMHEADER:
+      multifdsink->resend_streamheader = g_value_get_boolean (value);
+      break;
+
+    default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
+      break;
+  }
+}
+
+static void
+gst_multi_fd_sink_get_property (GObject * object, guint prop_id, GValue * value,
+    GParamSpec * pspec)
+{
+  GstMultiFdSink *multifdsink;
+
+  multifdsink = GST_MULTI_FD_SINK (object);
+
+  switch (prop_id) {
+    case PROP_MODE:
+      g_value_set_enum (value, multifdsink->mode);
+      break;
+    case PROP_BUFFERS_MAX:
+      g_value_set_int (value, multifdsink->units_max);
+      break;
+    case PROP_BUFFERS_SOFT_MAX:
+      g_value_set_int (value, multifdsink->units_soft_max);
+      break;
+    case PROP_TIME_MIN:
+      g_value_set_int64 (value, multifdsink->time_min);
+      break;
+    case PROP_BYTES_MIN:
+      g_value_set_int (value, multifdsink->bytes_min);
+      break;
+    case PROP_BUFFERS_MIN:
+      g_value_set_int (value, multifdsink->buffers_min);
+      break;
+    case PROP_BUFFERS_QUEUED:
+      g_value_set_uint (value, multifdsink->buffers_queued);
+      break;
+    case PROP_BYTES_QUEUED:
+      g_value_set_uint (value, multifdsink->bytes_queued);
+      break;
+    case PROP_TIME_QUEUED:
+      g_value_set_uint64 (value, multifdsink->time_queued);
+      break;
+    case PROP_UNIT_TYPE:
+      g_value_set_enum (value, multifdsink->unit_type);
+      break;
+    case PROP_UNITS_MAX:
+      g_value_set_int64 (value, multifdsink->units_max);
+      break;
+    case PROP_UNITS_SOFT_MAX:
+      g_value_set_int64 (value, multifdsink->units_soft_max);
+      break;
+    case PROP_RECOVER_POLICY:
+      g_value_set_enum (value, multifdsink->recover_policy);
+      break;
+    case PROP_TIMEOUT:
+      g_value_set_uint64 (value, multifdsink->timeout);
+      break;
+    case PROP_SYNC_METHOD:
+      g_value_set_enum (value, multifdsink->def_sync_method);
+      break;
+    case PROP_BYTES_TO_SERVE:
+      g_value_set_uint64 (value, multifdsink->bytes_to_serve);
+      break;
+    case PROP_BYTES_SERVED:
+      g_value_set_uint64 (value, multifdsink->bytes_served);
+      break;
+    case PROP_BURST_UNIT:
+      g_value_set_enum (value, multifdsink->def_burst_unit);
+      break;
+    case PROP_BURST_VALUE:
+      g_value_set_uint64 (value, multifdsink->def_burst_value);
+      break;
+    case PROP_QOS_DSCP:
+      g_value_set_int (value, multifdsink->qos_dscp);
+      break;
+    case PROP_HANDLE_READ:
+      g_value_set_boolean (value, multifdsink->handle_read);
+      break;
+    case PROP_RESEND_STREAMHEADER:
+      g_value_set_boolean (value, multifdsink->resend_streamheader);
+      break;
+    case PROP_NUM_FDS:
+      g_value_set_uint (value, g_hash_table_size (multifdsink->fd_hash));
+      break;
+
+    default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
+      break;
+  }
+}
+
+
+/* create a socket for sending to remote machine */
+static gboolean
+gst_multi_fd_sink_start (GstBaseSink * bsink)
+{
+  GstMultiFdSinkClass *fclass;
+  GstMultiFdSink *this;
+
+  if (GST_OBJECT_FLAG_IS_SET (bsink, GST_MULTI_FD_SINK_OPEN))
+    return TRUE;
+
+  this = GST_MULTI_FD_SINK (bsink);
+  fclass = GST_MULTI_FD_SINK_GET_CLASS (this);
+
+  GST_INFO_OBJECT (this, "starting in mode %d", this->mode);
+  if ((this->fdset = gst_poll_new (TRUE)) == NULL)
+    goto socket_pair;
+
+  this->streamheader = NULL;
+  this->bytes_to_serve = 0;
+  this->bytes_served = 0;
+
+  if (fclass->init) {
+    fclass->init (this);
+  }
+
+  this->running = TRUE;
+
+#if !GLIB_CHECK_VERSION (2, 31, 0)
+  this->thread = g_thread_create ((GThreadFunc) gst_multi_fd_sink_thread,
+      this, TRUE, NULL);
+#else
+  this->thread = g_thread_new ("multifdsink",
+      (GThreadFunc) gst_multi_fd_sink_thread, this);
+#endif
+
+  GST_OBJECT_FLAG_SET (this, GST_MULTI_FD_SINK_OPEN);
+
+  return TRUE;
+
+  /* ERRORS */
+socket_pair:
+  {
+    GST_ELEMENT_ERROR (this, RESOURCE, OPEN_READ_WRITE, (NULL),
+        GST_ERROR_SYSTEM);
+    return FALSE;
+  }
+}
+
+static gboolean
+multifdsink_hash_remove (gpointer key, gpointer value, gpointer data)
+{
+  return TRUE;
+}
+
+static gboolean
+gst_multi_fd_sink_stop (GstBaseSink * bsink)
+{
+  GstMultiFdSinkClass *fclass;
+  GstMultiFdSink *this;
+  GstBuffer *buf;
+  int i;
+
+  this = GST_MULTI_FD_SINK (bsink);
+  fclass = GST_MULTI_FD_SINK_GET_CLASS (this);
+
+  if (!GST_OBJECT_FLAG_IS_SET (bsink, GST_MULTI_FD_SINK_OPEN))
+    return TRUE;
+
+  this->running = FALSE;
+
+  gst_poll_set_flushing (this->fdset, TRUE);
+  if (this->thread) {
+    GST_DEBUG_OBJECT (this, "joining thread");
+    g_thread_join (this->thread);
+    GST_DEBUG_OBJECT (this, "joined thread");
+    this->thread = NULL;
+  }
+
+  /* free the clients */
+  gst_multi_fd_sink_clear (this);
+
+  if (this->streamheader) {
+    g_slist_foreach (this->streamheader, (GFunc) gst_mini_object_unref, NULL);
+    g_slist_free (this->streamheader);
+    this->streamheader = NULL;
+  }
+
+  if (fclass->close)
+    fclass->close (this);
+
+  if (this->fdset) {
+    gst_poll_free (this->fdset);
+    this->fdset = NULL;
+  }
+  g_hash_table_foreach_remove (this->fd_hash, multifdsink_hash_remove, this);
+
+  /* remove all queued buffers */
+  if (this->bufqueue) {
+    GST_DEBUG_OBJECT (this, "Emptying bufqueue with %d buffers",
+        this->bufqueue->len);
+    for (i = this->bufqueue->len - 1; i >= 0; --i) {
+      buf = g_array_index (this->bufqueue, GstBuffer *, i);
+      GST_LOG_OBJECT (this, "Removing buffer %p (%d) with refcount %d", buf, i,
+          GST_MINI_OBJECT_REFCOUNT (buf));
+      gst_buffer_unref (buf);
+      this->bufqueue = g_array_remove_index (this->bufqueue, i);
+    }
+    /* freeing the array is done in _finalize */
+  }
+  GST_OBJECT_FLAG_UNSET (this, GST_MULTI_FD_SINK_OPEN);
+
+  return TRUE;
+}
+
+static GstStateChangeReturn
+gst_multi_fd_sink_change_state (GstElement * element, GstStateChange transition)
+{
+  GstMultiFdSink *sink;
+  GstStateChangeReturn ret;
+
+  sink = GST_MULTI_FD_SINK (element);
+
+  /* we disallow changing the state from the streaming thread */
+  if (g_thread_self () == sink->thread)
+    return GST_STATE_CHANGE_FAILURE;
+
+
+  switch (transition) {
+    case GST_STATE_CHANGE_NULL_TO_READY:
+      if (!gst_multi_fd_sink_start (GST_BASE_SINK (sink)))
+        goto start_failed;
+      break;
+    case GST_STATE_CHANGE_READY_TO_PAUSED:
+      break;
+    case GST_STATE_CHANGE_PAUSED_TO_PLAYING:
+      break;
+    default:
+      break;
+  }
+
+  ret = GST_ELEMENT_CLASS (parent_class)->change_state (element, transition);
+
+  switch (transition) {
+    case GST_STATE_CHANGE_PLAYING_TO_PAUSED:
+      break;
+    case GST_STATE_CHANGE_PAUSED_TO_READY:
+      break;
+    case GST_STATE_CHANGE_READY_TO_NULL:
+      gst_multi_fd_sink_stop (GST_BASE_SINK (sink));
+      break;
+    default:
+      break;
+  }
+  return ret;
+
+  /* ERRORS */
+start_failed:
+  {
+    /* error message was posted */
+    return GST_STATE_CHANGE_FAILURE;
+  }
+}
+
+static gboolean
+plugin_init (GstPlugin * plugin)
+{
+  if (!gst_element_register (plugin, "multifdsink", GST_RANK_NONE,
+          GST_TYPE_MULTI_FD_SINK))
+    return FALSE;
+
+  return TRUE;
+}
+
+GST_PLUGIN_DEFINE (GST_VERSION_MAJOR,
+    GST_VERSION_MINOR,
+    "multifdsink",
+    "transfer data to multiple fds",
+    plugin_init, VERSION, GST_LICENSE, GST_PACKAGE_NAME, GST_PACKAGE_ORIGIN)
diff --git a/gst/tcp/gstmultifdsink.h b/gst/tcp/gstmultifdsink.h
new file mode 100644 (file)
index 0000000..5ac0df7
--- /dev/null
@@ -0,0 +1,283 @@
+/* GStreamer
+ * Copyright (C) <1999> Erik Walthinsen <omega@cse.ogi.edu>
+ * Copyright (C) <2004> Thomas Vander Stichele <thomas at apestaart dot org>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ * Boston, MA 02111-1307, USA.
+ */
+
+
+#ifndef __GST_MULTI_FD_SINK_H__
+#define __GST_MULTI_FD_SINK_H__
+
+#include <gst/gst.h>
+#include <gst/base/gstbasesink.h>
+
+G_BEGIN_DECLS
+
+#define GST_TYPE_MULTI_FD_SINK \
+  (gst_multi_fd_sink_get_type())
+#define GST_MULTI_FD_SINK(obj) \
+  (G_TYPE_CHECK_INSTANCE_CAST((obj),GST_TYPE_MULTI_FD_SINK,GstMultiFdSink))
+#define GST_MULTI_FD_SINK_CLASS(klass) \
+  (G_TYPE_CHECK_CLASS_CAST((klass),GST_TYPE_MULTI_FD_SINK,GstMultiFdSinkClass))
+#define GST_IS_MULTI_FD_SINK(obj) \
+  (G_TYPE_CHECK_INSTANCE_TYPE((obj),GST_TYPE_MULTI_FD_SINK))
+#define GST_IS_MULTI_FD_SINK_CLASS(klass) \
+  (G_TYPE_CHECK_CLASS_TYPE((klass),GST_TYPE_MULTI_FD_SINK))
+#define GST_MULTI_FD_SINK_GET_CLASS(klass) \
+  (G_TYPE_INSTANCE_GET_CLASS ((klass), GST_TYPE_MULTI_FD_SINK, GstMultiFdSinkClass))
+
+
+typedef struct _GstMultiFdSink GstMultiFdSink;
+typedef struct _GstMultiFdSinkClass GstMultiFdSinkClass;
+
+typedef enum {
+  GST_MULTI_FD_SINK_OPEN             = (GST_ELEMENT_FLAG_LAST << 0),
+
+  GST_MULTI_FD_SINK_FLAG_LAST        = (GST_ELEMENT_FLAG_LAST << 2)
+} GstMultiFdSinkFlags;
+
+/**
+ * GstRecoverPolicy:
+ * @GST_RECOVER_POLICY_NONE             : no recovering is done
+ * @GST_RECOVER_POLICY_RESYNC_LATEST    : client is moved to last buffer
+ * @GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT: client is moved to the soft limit
+ * @GST_RECOVER_POLICY_RESYNC_KEYFRAME  : client is moved to latest keyframe
+ *
+ * Possible values for the recovery procedure to use when a client consumes
+ * data too slow and has a backlag of more that soft-limit buffers.
+ */
+typedef enum
+{
+  GST_RECOVER_POLICY_NONE,
+  GST_RECOVER_POLICY_RESYNC_LATEST,
+  GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT,
+  GST_RECOVER_POLICY_RESYNC_KEYFRAME
+} GstRecoverPolicy;
+
+/**
+ * GstSyncMethod:
+ * @GST_SYNC_METHOD_LATEST              : client receives most recent buffer
+ * @GST_SYNC_METHOD_NEXT_KEYFRAME       : client receives next keyframe
+ * @GST_SYNC_METHOD_LATEST_KEYFRAME     : client receives latest keyframe (burst)
+ * @GST_SYNC_METHOD_BURST               : client receives specific amount of data
+ * @GST_SYNC_METHOD_BURST_KEYFRAME      : client receives specific amount of data 
+ *                                        starting from latest keyframe
+ * @GST_SYNC_METHOD_BURST_WITH_KEYFRAME : client receives specific amount of data from
+ *                                        a keyframe, or if there is not enough data after
+ *                                        the keyframe, starting before the keyframe
+ *
+ * This enum defines the selection of the first buffer that is sent
+ * to a new client.
+ */
+typedef enum
+{
+  GST_SYNC_METHOD_LATEST,
+  GST_SYNC_METHOD_NEXT_KEYFRAME,
+  GST_SYNC_METHOD_LATEST_KEYFRAME,
+  GST_SYNC_METHOD_BURST,
+  GST_SYNC_METHOD_BURST_KEYFRAME,
+  GST_SYNC_METHOD_BURST_WITH_KEYFRAME
+} GstSyncMethod;
+
+/**
+ * GstTCPUnitType:
+ * @GST_TCP_UNIT_TYPE_UNDEFINED: undefined
+ * @GST_TCP_UNIT_TYPE_BUFFERS  : buffers
+ * @GST_TCP_UNIT_TYPE_TIME     : timeunits (in nanoseconds)
+ * @GST_TCP_UNIT_TYPE_BYTES    : bytes
+ *
+ * The units used to specify limits.
+ */
+typedef enum
+{
+  GST_TCP_UNIT_TYPE_UNDEFINED,
+  GST_TCP_UNIT_TYPE_BUFFERS,
+  GST_TCP_UNIT_TYPE_TIME,
+  GST_TCP_UNIT_TYPE_BYTES
+} GstTCPUnitType;
+
+/**
+ * GstClientStatus:
+ * @GST_CLIENT_STATUS_OK       : client is ok
+ * @GST_CLIENT_STATUS_CLOSED   : client closed the socket
+ * @GST_CLIENT_STATUS_REMOVED  : client is removed
+ * @GST_CLIENT_STATUS_SLOW     : client is too slow
+ * @GST_CLIENT_STATUS_ERROR    : client is in error
+ * @GST_CLIENT_STATUS_DUPLICATE: same client added twice
+ * @GST_CLIENT_STATUS_FLUSHING : client is flushing out the remaining buffers.
+ *
+ * This specifies the reason why a client was removed from
+ * multifdsink and is received in the "client-removed" signal.
+ */
+typedef enum
+{
+  GST_CLIENT_STATUS_OK          = 0,
+  GST_CLIENT_STATUS_CLOSED      = 1,
+  GST_CLIENT_STATUS_REMOVED     = 2,
+  GST_CLIENT_STATUS_SLOW        = 3,
+  GST_CLIENT_STATUS_ERROR       = 4,
+  GST_CLIENT_STATUS_DUPLICATE   = 5,
+  GST_CLIENT_STATUS_FLUSHING    = 6
+} GstClientStatus;
+
+/* structure for a client
+ */
+typedef struct {
+  GstPollFD fd;
+
+  gint bufpos;                  /* position of this client in the global queue */
+  gint flushcount;              /* the remaining number of buffers to flush out or -1 if the 
+                                   client is not flushing. */
+
+  GstClientStatus status;
+  gboolean is_socket;
+
+  GSList *sending;              /* the buffers we need to send */
+  gint bufoffset;               /* offset in the first buffer */
+
+  gboolean discont;
+
+  gboolean caps_sent;
+  gboolean new_connection;
+
+  gboolean currently_removing;
+
+  /* method to sync client when connecting */
+  GstSyncMethod sync_method;
+  GstTCPUnitType   burst_min_unit;
+  guint64       burst_min_value;
+  GstTCPUnitType   burst_max_unit;
+  guint64       burst_max_value;
+
+  GstCaps *caps;                /* caps of last queued buffer */
+
+  /* stats */
+  guint64 bytes_sent;
+  guint64 connect_time;
+  guint64 disconnect_time;
+  guint64 last_activity_time;
+  guint64 dropped_buffers;
+  guint64 avg_queue_size;
+  guint64 first_buffer_ts;
+  guint64 last_buffer_ts;
+} GstTCPClient;
+
+#define CLIENTS_LOCK_INIT(fdsink)       (g_rec_mutex_init(&fdsink->clientslock))
+#define CLIENTS_LOCK_CLEAR(fdsink)      (g_rec_mutex_clear(&fdsink->clientslock))
+#define CLIENTS_LOCK(fdsink)            (g_rec_mutex_lock(&fdsink->clientslock))
+#define CLIENTS_UNLOCK(fdsink)          (g_rec_mutex_unlock(&fdsink->clientslock))
+
+/**
+ * GstMultiFdSink:
+ *
+ * The multifdsink object structure.
+ */
+struct _GstMultiFdSink {
+  GstBaseSink element;
+
+  /*< private >*/
+  guint64 bytes_to_serve; /* how much bytes we must serve */
+  guint64 bytes_served; /* how much bytes have we served */
+
+  GRecMutex clientslock;  /* lock to protect the clients list */
+  GList *clients;       /* list of clients we are serving */
+  GHashTable *fd_hash;  /* index on fd to client */
+  guint clients_cookie; /* Cookie to detect changes to the clients list */
+
+  gint mode;
+  GstPoll *fdset;
+
+  GSList *streamheader; /* GSList of GstBuffers to use as streamheader */
+  gboolean previous_buffer_in_caps;
+
+  guint mtu;
+  gint qos_dscp;
+  gboolean handle_read;
+
+  GArray *bufqueue;     /* global queue of buffers */
+
+  gboolean running;     /* the thread state */
+  GThread *thread;      /* the sender thread */
+
+  /* these values are used to check if a client is reading fast
+   * enough and to control receovery */
+  GstTCPUnitType unit_type;/* the type of the units */
+  gint64 units_max;       /* max units to queue for a client */
+  gint64 units_soft_max;  /* max units a client can lag before recovery starts */
+  GstRecoverPolicy recover_policy;
+  GstClockTime timeout; /* max amount of nanoseconds to remain idle */
+
+  GstSyncMethod def_sync_method;    /* what method to use for connecting clients */
+  GstTCPUnitType   def_burst_unit;
+  guint64       def_burst_value;
+
+  /* these values are used to control the amount of data
+   * kept in the queues. It allows clients to perform a burst
+   * on connect. */
+  gint   bytes_min;    /* min number of bytes to queue */
+  gint64 time_min;     /* min time to queue */
+  gint   buffers_min;   /* min number of buffers to queue */
+
+  gboolean resend_streamheader; /* resend streamheader if it changes */
+
+  /* stats */
+  gint buffers_queued;  /* number of queued buffers */
+  gint bytes_queued;    /* number of queued bytes */
+  gint time_queued;     /* number of queued time */
+
+  guint8 header_flags;
+};
+
+struct _GstMultiFdSinkClass {
+  GstBaseSinkClass parent_class;
+
+  /* element methods */
+  void          (*add)          (GstMultiFdSink *sink, int fd);
+  void          (*add_full)     (GstMultiFdSink *sink, int fd, GstSyncMethod sync,
+                                GstTCPUnitType format, guint64 value, 
+                                GstTCPUnitType max_unit, guint64 max_value);
+  void          (*remove)       (GstMultiFdSink *sink, int fd);
+  void          (*remove_flush) (GstMultiFdSink *sink, int fd);
+  void          (*clear)        (GstMultiFdSink *sink);
+  GValueArray*  (*get_stats)    (GstMultiFdSink *sink, int fd);
+
+  /* vtable */
+  gboolean (*init)   (GstMultiFdSink *sink);
+  gboolean (*wait)   (GstMultiFdSink *sink, GstPoll *set);
+  gboolean (*close)  (GstMultiFdSink *sink);
+  void (*removed) (GstMultiFdSink *sink, int fd);
+
+  /* signals */
+  void (*client_added) (GstElement *element, gint fd);
+  void (*client_removed) (GstElement *element, gint fd, GstClientStatus status);
+  void (*client_fd_removed) (GstElement *element, gint fd);
+};
+
+GType gst_multi_fd_sink_get_type (void);
+
+void          gst_multi_fd_sink_add          (GstMultiFdSink *sink, int fd);
+void          gst_multi_fd_sink_add_full     (GstMultiFdSink *sink, int fd, GstSyncMethod sync, 
+                                              GstTCPUnitType min_unit, guint64 min_value,
+                                              GstTCPUnitType max_unit, guint64 max_value);
+void          gst_multi_fd_sink_remove       (GstMultiFdSink *sink, int fd);
+void          gst_multi_fd_sink_remove_flush (GstMultiFdSink *sink, int fd);
+void          gst_multi_fd_sink_clear        (GstMultiFdSink *sink);
+GValueArray*  gst_multi_fd_sink_get_stats    (GstMultiFdSink *sink, int fd);
+
+G_END_DECLS
+
+#endif /* __GST_MULTI_FD_SINK_H__ */
index d4d0b15..aeae28d 100644 (file)
@@ -132,6 +132,7 @@ check_PROGRAMS = \
        elements/audioresample      \
        elements/gdpdepay           \
        elements/gdppay             \
+       elements/multifdsink        \
        elements/multisocketsink    \
        elements/playbin            \
        elements/playbin-compressed \
diff --git a/tests/check/elements/multifdsink.c b/tests/check/elements/multifdsink.c
new file mode 100644 (file)
index 0000000..6d5a252
--- /dev/null
@@ -0,0 +1,866 @@
+/* GStreamer
+ *
+ * Copyright (C) 2006 Thomas Vander Stichele <thomas at apestaart dot org>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ * Boston, MA 02111-1307, USA.
+ */
+
+#include <unistd.h>
+#include <sys/ioctl.h>
+#ifdef HAVE_FIONREAD_IN_SYS_FILIO
+#include <sys/filio.h>
+#endif
+
+#include <gst/check/gstcheck.h>
+
+static GstPad *mysrcpad;
+
+static GstStaticPadTemplate srctemplate = GST_STATIC_PAD_TEMPLATE ("src",
+    GST_PAD_SRC,
+    GST_PAD_ALWAYS,
+    GST_STATIC_CAPS ("application/x-gst-check")
+    );
+
+static GstElement *
+setup_multifdsink (void)
+{
+  GstElement *multifdsink;
+
+  GST_DEBUG ("setup_multifdsink");
+  multifdsink = gst_check_setup_element ("multifdsink");
+  mysrcpad = gst_check_setup_src_pad (multifdsink, &srctemplate);
+
+  return multifdsink;
+}
+
+static void
+cleanup_multifdsink (GstElement * multifdsink)
+{
+  GST_DEBUG ("cleanup_multifdsink");
+
+  gst_check_teardown_src_pad (multifdsink);
+  gst_check_teardown_element (multifdsink);
+}
+
+static void
+wait_bytes_served (GstElement * sink, guint64 bytes)
+{
+  guint64 bytes_served = 0;
+
+  while (bytes_served != bytes) {
+    g_object_get (sink, "bytes-served", &bytes_served, NULL);
+  }
+}
+
+/* FIXME: possibly racy, since if it would write, we may not get it
+ * immediately ? */
+#define fail_if_can_read(msg,fd) \
+G_STMT_START { \
+  long avail; \
+\
+  fail_if (ioctl (fd, FIONREAD, &avail) < 0, "%s: could not ioctl", msg); \
+  fail_if (avail > 0, "%s: has bytes available to read"); \
+} G_STMT_END;
+
+
+GST_START_TEST (test_no_clients)
+{
+  GstElement *sink;
+  GstBuffer *buffer;
+  GstCaps *caps;
+
+  sink = setup_multifdsink ();
+
+  ASSERT_SET_STATE (sink, GST_STATE_PLAYING, GST_STATE_CHANGE_ASYNC);
+
+  caps = gst_caps_from_string ("application/x-gst-check");
+  buffer = gst_buffer_new_and_alloc (4);
+  gst_pad_set_caps (mysrcpad, caps);
+  gst_caps_unref (caps);
+  fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+
+  GST_DEBUG ("cleaning up multifdsink");
+  ASSERT_SET_STATE (sink, GST_STATE_NULL, GST_STATE_CHANGE_SUCCESS);
+  cleanup_multifdsink (sink);
+}
+
+GST_END_TEST;
+
+GST_START_TEST (test_add_client)
+{
+  GstElement *sink;
+  GstBuffer *buffer;
+  GstCaps *caps;
+  int pfd[2];
+  gchar data[4];
+
+  sink = setup_multifdsink ();
+
+  fail_if (pipe (pfd) == -1);
+
+  ASSERT_SET_STATE (sink, GST_STATE_PLAYING, GST_STATE_CHANGE_ASYNC);
+
+  /* add the client */
+  g_signal_emit_by_name (sink, "add", pfd[1]);
+
+  caps = gst_caps_from_string ("application/x-gst-check");
+  GST_DEBUG ("Created test caps %p %" GST_PTR_FORMAT, caps, caps);
+  buffer = gst_buffer_new_and_alloc (4);
+  gst_pad_set_caps (mysrcpad, caps);
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 2);
+  gst_buffer_fill (buffer, 0, "dead", 4);
+  fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+
+  GST_DEBUG ("reading");
+  fail_if (read (pfd[0], data, 4) < 4);
+  fail_unless (strncmp (data, "dead", 4) == 0);
+  wait_bytes_served (sink, 4);
+
+  GST_DEBUG ("cleaning up multifdsink");
+  ASSERT_SET_STATE (sink, GST_STATE_NULL, GST_STATE_CHANGE_SUCCESS);
+  cleanup_multifdsink (sink);
+
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
+  gst_caps_unref (caps);
+}
+
+GST_END_TEST;
+
+#define fail_unless_read(msg,fd,size,ref) \
+G_STMT_START { \
+  char data[size + 1]; \
+  int nbytes; \
+\
+  GST_DEBUG ("%s: reading %d bytes", msg, size); \
+  nbytes = read (fd, data, size); \
+  data[size] = 0; \
+  GST_DEBUG ("%s: read %d bytes", msg, nbytes); \
+  fail_if (nbytes < size); \
+  fail_unless (memcmp (data, ref, size) == 0, \
+      "data read '%s' differs from '%s'", data, ref); \
+} G_STMT_END;
+
+/* from the given two data buffers, create two streamheader buffers and
+ * some caps that match it, and store them in the given pointers
+ * returns  one ref to each of the buffers and the caps */
+static void
+gst_multifdsink_create_streamheader (const gchar * data1,
+    const gchar * data2, GstBuffer ** hbuf1, GstBuffer ** hbuf2,
+    GstCaps ** caps)
+{
+  GstBuffer *buf;
+  GValue array = { 0 };
+  GValue value = { 0 };
+  GstStructure *structure;
+  guint size1 = strlen (data1);
+  guint size2 = strlen (data2);
+
+  fail_if (hbuf1 == NULL);
+  fail_if (hbuf2 == NULL);
+  fail_if (caps == NULL);
+
+  /* create caps with streamheader, set the caps, and push the IN_CAPS
+   * buffers */
+  *hbuf1 = gst_buffer_new_and_alloc (size1);
+  GST_BUFFER_FLAG_SET (*hbuf1, GST_BUFFER_FLAG_IN_CAPS);
+  gst_buffer_fill (*hbuf1, 0, data1, size1);
+  *hbuf2 = gst_buffer_new_and_alloc (size2);
+  GST_BUFFER_FLAG_SET (*hbuf2, GST_BUFFER_FLAG_IN_CAPS);
+  gst_buffer_fill (*hbuf2, 0, data2, size2);
+
+  g_value_init (&array, GST_TYPE_ARRAY);
+
+  g_value_init (&value, GST_TYPE_BUFFER);
+  /* we take a copy, set it on the array (which refs it), then unref our copy */
+  buf = gst_buffer_copy (*hbuf1);
+  gst_value_set_buffer (&value, buf);
+  ASSERT_BUFFER_REFCOUNT (buf, "copied buffer", 2);
+  gst_buffer_unref (buf);
+  gst_value_array_append_value (&array, &value);
+  g_value_unset (&value);
+
+  g_value_init (&value, GST_TYPE_BUFFER);
+  buf = gst_buffer_copy (*hbuf2);
+  gst_value_set_buffer (&value, buf);
+  ASSERT_BUFFER_REFCOUNT (buf, "copied buffer", 2);
+  gst_buffer_unref (buf);
+  gst_value_array_append_value (&array, &value);
+  g_value_unset (&value);
+
+  *caps = gst_caps_from_string ("application/x-gst-check");
+  structure = gst_caps_get_structure (*caps, 0);
+
+  gst_structure_set_value (structure, "streamheader", &array);
+  g_value_unset (&array);
+  ASSERT_CAPS_REFCOUNT (*caps, "streamheader caps", 1);
+
+  /* we want to keep them around for the tests */
+  gst_buffer_ref (*hbuf1);
+  gst_buffer_ref (*hbuf2);
+
+  GST_DEBUG ("created streamheader caps %p %" GST_PTR_FORMAT, *caps, *caps);
+}
+
+
+/* this test:
+ * - adds a first client
+ * - sets streamheader caps on the pad
+ * - pushes the IN_CAPS buffers
+ * - pushes a buffer
+ * - verifies that the client received all the data correctly, and did not
+ *   get multiple copies of the streamheader
+ * - adds a second client
+ * - verifies that this second client receives the streamheader caps too, plus
+ * - the new buffer
+ */
+GST_START_TEST (test_streamheader)
+{
+  GstElement *sink;
+  GstBuffer *hbuf1, *hbuf2, *buf;
+  GstCaps *caps;
+  int pfd1[2], pfd2[2];
+
+  sink = setup_multifdsink ();
+
+  fail_if (pipe (pfd1) == -1);
+  fail_if (pipe (pfd2) == -1);
+
+  ASSERT_SET_STATE (sink, GST_STATE_PLAYING, GST_STATE_CHANGE_ASYNC);
+
+  /* add the first client */
+  g_signal_emit_by_name (sink, "add", pfd1[1]);
+
+  /* create caps with streamheader, set the caps, and push the IN_CAPS
+   * buffers */
+  gst_multifdsink_create_streamheader ("babe", "deadbeef", &hbuf1, &hbuf2,
+      &caps);
+  fail_unless (gst_pad_set_caps (mysrcpad, caps));
+  /* one is ours, two on the buffers, and one now on the pad */
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 4);
+
+  fail_unless (gst_pad_push (mysrcpad, hbuf1) == GST_FLOW_OK);
+  fail_unless (gst_pad_push (mysrcpad, hbuf2) == GST_FLOW_OK);
+
+  //FIXME:
+  //fail_if_can_read ("first client", pfd1[0]);
+
+  /* push a non-IN_CAPS buffer, this should trigger the client receiving the
+   * first three buffers */
+  buf = gst_buffer_new_and_alloc (4);
+  gst_buffer_fill (buf, 0, "f00d", 4);
+  gst_pad_push (mysrcpad, buf);
+
+  fail_unless_read ("first client", pfd1[0], 4, "babe");
+  fail_unless_read ("first client", pfd1[0], 8, "deadbeef");
+  fail_unless_read ("first client", pfd1[0], 4, "f00d");
+  wait_bytes_served (sink, 16);
+
+  /* now add the second client */
+  g_signal_emit_by_name (sink, "add", pfd2[1]);
+  //FIXME:
+  //fail_if_can_read ("second client", pfd2[0]);
+
+  /* now push another buffer, which will trigger streamheader for second
+   * client */
+  buf = gst_buffer_new_and_alloc (4);
+  gst_buffer_fill (buf, 0, "deaf", 4);
+  gst_pad_push (mysrcpad, buf);
+
+  fail_unless_read ("first client", pfd1[0], 4, "deaf");
+
+  fail_unless_read ("second client", pfd2[0], 4, "babe");
+  fail_unless_read ("second client", pfd2[0], 8, "deadbeef");
+  /* we missed the f00d buffer */
+  fail_unless_read ("second client", pfd2[0], 4, "deaf");
+  wait_bytes_served (sink, 36);
+
+  GST_DEBUG ("cleaning up multifdsink");
+
+  g_signal_emit_by_name (sink, "remove", pfd1[1]);
+  g_signal_emit_by_name (sink, "remove", pfd2[1]);
+
+  ASSERT_SET_STATE (sink, GST_STATE_NULL, GST_STATE_CHANGE_SUCCESS);
+  cleanup_multifdsink (sink);
+
+  ASSERT_BUFFER_REFCOUNT (hbuf1, "hbuf1", 1);
+  ASSERT_BUFFER_REFCOUNT (hbuf2, "hbuf2", 1);
+  gst_buffer_unref (hbuf1);
+  gst_buffer_unref (hbuf2);
+
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
+  gst_caps_unref (caps);
+}
+
+GST_END_TEST;
+
+/* this tests changing of streamheaders
+ * - set streamheader caps on the pad
+ * - pushes the IN_CAPS buffers
+ * - pushes a buffer
+ * - add a first client
+ * - verifies that this first client receives the first streamheader caps,
+ *   plus a new buffer
+ * - change streamheader caps
+ * - verify that the first client receives the new streamheader buffers as well
+ */
+GST_START_TEST (test_change_streamheader)
+{
+  GstElement *sink;
+  GstBuffer *hbuf1, *hbuf2, *buf;
+  GstCaps *caps;
+  int pfd1[2], pfd2[2];
+
+  sink = setup_multifdsink ();
+
+  fail_if (pipe (pfd1) == -1);
+  fail_if (pipe (pfd2) == -1);
+
+  ASSERT_SET_STATE (sink, GST_STATE_PLAYING, GST_STATE_CHANGE_ASYNC);
+
+  /* create caps with streamheader, set the caps, and push the IN_CAPS
+   * buffers */
+  gst_multifdsink_create_streamheader ("first", "header", &hbuf1, &hbuf2,
+      &caps);
+  fail_unless (gst_pad_set_caps (mysrcpad, caps));
+  /* one is ours, two on the buffers, and one now on the pad */
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 4);
+
+  /* one to hold for the test and one to give away */
+  ASSERT_BUFFER_REFCOUNT (hbuf1, "hbuf1", 2);
+  ASSERT_BUFFER_REFCOUNT (hbuf2, "hbuf2", 2);
+
+  fail_unless (gst_pad_push (mysrcpad, hbuf1) == GST_FLOW_OK);
+  fail_unless (gst_pad_push (mysrcpad, hbuf2) == GST_FLOW_OK);
+
+  /* add the first client */
+  g_signal_emit_by_name (sink, "add", pfd1[1]);
+
+  /* verify this hasn't triggered a write yet */
+  /* FIXME: possibly racy, since if it would write, we may not get it
+   * immediately ? */
+  //fail_if_can_read ("first client, no buffer", pfd1[0]);
+
+  /* now push a buffer and read */
+  buf = gst_buffer_new_and_alloc (4);
+  gst_buffer_fill (buf, 0, "f00d", 4);
+  gst_pad_push (mysrcpad, buf);
+
+  fail_unless_read ("change: first client", pfd1[0], 5, "first");
+  fail_unless_read ("change: first client", pfd1[0], 6, "header");
+  fail_unless_read ("change: first client", pfd1[0], 4, "f00d");
+  //wait_bytes_served (sink, 16);
+
+  /* now add the second client */
+  g_signal_emit_by_name (sink, "add", pfd2[1]);
+  //fail_if_can_read ("second client, no buffer", pfd2[0]);
+
+  /* change the streamheader */
+
+  /* before we change, multifdsink still has a list of the old streamheaders */
+  ASSERT_BUFFER_REFCOUNT (hbuf1, "hbuf1", 2);
+  ASSERT_BUFFER_REFCOUNT (hbuf2, "hbuf2", 2);
+  gst_buffer_unref (hbuf1);
+  gst_buffer_unref (hbuf2);
+
+  /* drop our ref to the previous caps */
+  gst_caps_unref (caps);
+
+  gst_multifdsink_create_streamheader ("second", "header", &hbuf1, &hbuf2,
+      &caps);
+  fail_unless (gst_pad_set_caps (mysrcpad, caps));
+  /* one to hold for the test and one to give away */
+  ASSERT_BUFFER_REFCOUNT (hbuf1, "hbuf1", 2);
+  ASSERT_BUFFER_REFCOUNT (hbuf2, "hbuf2", 2);
+
+  fail_unless (gst_pad_push (mysrcpad, hbuf1) == GST_FLOW_OK);
+  fail_unless (gst_pad_push (mysrcpad, hbuf2) == GST_FLOW_OK);
+
+  /* verify neither client has new data available to read */
+  //fail_if_can_read ("first client, changed streamheader", pfd1[0]);
+  //fail_if_can_read ("second client, changed streamheader", pfd2[0]);
+
+  /* now push another buffer, which will trigger streamheader for second
+   * client, but should also send new streamheaders to first client */
+  buf = gst_buffer_new_and_alloc (8);
+  gst_buffer_fill (buf, 0, "deadbabe", 8);
+  gst_pad_push (mysrcpad, buf);
+
+  fail_unless_read ("first client", pfd1[0], 6, "second");
+  fail_unless_read ("first client", pfd1[0], 6, "header");
+  fail_unless_read ("first client", pfd1[0], 8, "deadbabe");
+
+  /* new streamheader data */
+  fail_unless_read ("second client", pfd2[0], 6, "second");
+  fail_unless_read ("second client", pfd2[0], 6, "header");
+  /* we missed the f00d buffer */
+  fail_unless_read ("second client", pfd2[0], 8, "deadbabe");
+  //wait_bytes_served (sink, 36);
+
+  GST_DEBUG ("cleaning up multifdsink");
+  g_signal_emit_by_name (sink, "remove", pfd1[1]);
+  g_signal_emit_by_name (sink, "remove", pfd2[1]);
+  ASSERT_SET_STATE (sink, GST_STATE_NULL, GST_STATE_CHANGE_SUCCESS);
+
+  /* setting to NULL should have cleared the streamheader */
+  ASSERT_BUFFER_REFCOUNT (hbuf1, "hbuf1", 1);
+  ASSERT_BUFFER_REFCOUNT (hbuf2, "hbuf2", 1);
+  gst_buffer_unref (hbuf1);
+  gst_buffer_unref (hbuf2);
+  cleanup_multifdsink (sink);
+
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
+  gst_caps_unref (caps);
+}
+
+GST_END_TEST;
+
+/* keep 100 bytes and burst 80 bytes to clients */
+GST_START_TEST (test_burst_client_bytes)
+{
+  GstElement *sink;
+  GstBuffer *buffer;
+  GstCaps *caps;
+  int pfd1[2];
+  int pfd2[2];
+  int pfd3[2];
+  gchar data[16];
+  gint i;
+  guint buffers_queued;
+
+  sink = setup_multifdsink ();
+  /* make sure we keep at least 100 bytes at all times */
+  g_object_set (sink, "bytes-min", 100, NULL);
+  g_object_set (sink, "sync-method", 3, NULL);  /* 3 = burst */
+  g_object_set (sink, "burst-unit", 3, NULL);   /* 3 = bytes */
+  g_object_set (sink, "burst-value", (guint64) 80, NULL);
+
+  fail_if (pipe (pfd1) == -1);
+  fail_if (pipe (pfd2) == -1);
+  fail_if (pipe (pfd3) == -1);
+
+  ASSERT_SET_STATE (sink, GST_STATE_PLAYING, GST_STATE_CHANGE_ASYNC);
+
+  caps = gst_caps_from_string ("application/x-gst-check");
+  gst_pad_set_caps (mysrcpad, caps);
+  GST_DEBUG ("Created test caps %p %" GST_PTR_FORMAT, caps, caps);
+
+  /* push buffers in, 9 * 16 bytes = 144 bytes */
+  for (i = 0; i < 9; i++) {
+    gchar *data;
+
+    buffer = gst_buffer_new_and_alloc (16);
+
+    /* copy some id */
+    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
+    g_snprintf (data, 16, "deadbee%08x", i);
+    gst_buffer_unmap (buffer, data, 16);
+
+    fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+  }
+
+  /* check that at least 7 buffers (112 bytes) are in the queue */
+  g_object_get (sink, "buffers-queued", &buffers_queued, NULL);
+  fail_if (buffers_queued != 7);
+
+  /* now add the clients */
+  g_signal_emit_by_name (sink, "add", pfd1[1]);
+  g_signal_emit_by_name (sink, "add_full", pfd2[1], 3,
+      3, (guint64) 50, 3, (guint64) 200);
+  g_signal_emit_by_name (sink, "add_full", pfd3[1], 3,
+      3, (guint64) 50, 3, (guint64) 50);
+
+  /* push last buffer to make client fds ready for reading */
+  for (i = 9; i < 10; i++) {
+    gchar *data;
+
+    buffer = gst_buffer_new_and_alloc (16);
+
+    /* copy some id */
+    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
+    g_snprintf (data, 16, "deadbee%08x", i);
+    gst_buffer_unmap (buffer, data, 16);
+
+    fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+  }
+
+  /* now we should only read the last 5 buffers (5 * 16 = 80 bytes) */
+  GST_DEBUG ("Reading from client 1");
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000005", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000006", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000007", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  /* second client only bursts 50 bytes = 4 buffers (we get 4 buffers since
+   * the max alows it) */
+  GST_DEBUG ("Reading from client 2");
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000006", 16) == 0);
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000007", 16) == 0);
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  /* third client only bursts 50 bytes = 4 buffers, we can't send
+   * more than 50 bytes so we only get 3 buffers (48 bytes). */
+  GST_DEBUG ("Reading from client 3");
+  fail_if (read (pfd3[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000007", 16) == 0);
+  fail_if (read (pfd3[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd3[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  GST_DEBUG ("cleaning up multifdsink");
+  ASSERT_SET_STATE (sink, GST_STATE_NULL, GST_STATE_CHANGE_SUCCESS);
+  cleanup_multifdsink (sink);
+
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
+  gst_caps_unref (caps);
+}
+
+GST_END_TEST;
+
+/* keep 100 bytes and burst 80 bytes to clients */
+GST_START_TEST (test_burst_client_bytes_keyframe)
+{
+  GstElement *sink;
+  GstBuffer *buffer;
+  GstCaps *caps;
+  int pfd1[2];
+  int pfd2[2];
+  int pfd3[2];
+  gchar data[16];
+  gint i;
+  guint buffers_queued;
+
+  sink = setup_multifdsink ();
+  /* make sure we keep at least 100 bytes at all times */
+  g_object_set (sink, "bytes-min", 100, NULL);
+  g_object_set (sink, "sync-method", 4, NULL);  /* 3 = burst_keyframe */
+  g_object_set (sink, "burst-unit", 3, NULL);   /* 3 = bytes */
+  g_object_set (sink, "burst-value", (guint64) 80, NULL);
+
+  fail_if (pipe (pfd1) == -1);
+  fail_if (pipe (pfd2) == -1);
+  fail_if (pipe (pfd3) == -1);
+
+  ASSERT_SET_STATE (sink, GST_STATE_PLAYING, GST_STATE_CHANGE_ASYNC);
+
+  caps = gst_caps_from_string ("application/x-gst-check");
+  GST_DEBUG ("Created test caps %p %" GST_PTR_FORMAT, caps, caps);
+  gst_pad_set_caps (mysrcpad, caps);
+
+  /* push buffers in, 9 * 16 bytes = 144 bytes */
+  for (i = 0; i < 9; i++) {
+    gchar *data;
+
+    buffer = gst_buffer_new_and_alloc (16);
+
+    /* mark most buffers as delta */
+    if (i != 0 && i != 4 && i != 8)
+      GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
+
+    /* copy some id */
+    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
+    g_snprintf (data, 16, "deadbee%08x", i);
+    gst_buffer_unmap (buffer, data, 16);
+
+    fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+  }
+
+  /* check that at least 7 buffers (112 bytes) are in the queue */
+  g_object_get (sink, "buffers-queued", &buffers_queued, NULL);
+  fail_if (buffers_queued != 7);
+
+  /* now add the clients */
+  g_signal_emit_by_name (sink, "add", pfd1[1]);
+  g_signal_emit_by_name (sink, "add_full", pfd2[1], 4,
+      3, (guint64) 50, 3, (guint64) 90);
+  g_signal_emit_by_name (sink, "add_full", pfd3[1], 4,
+      3, (guint64) 50, 3, (guint64) 50);
+
+  /* push last buffer to make client fds ready for reading */
+  for (i = 9; i < 10; i++) {
+    gchar *data;
+
+    buffer = gst_buffer_new_and_alloc (16);
+    GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
+
+    /* copy some id */
+    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
+    g_snprintf (data, 16, "deadbee%08x", i);
+    gst_buffer_unmap (buffer, data, 16);
+
+    fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+  }
+
+  /* now we should only read the last 6 buffers (min 5 * 16 = 80 bytes),
+   * keyframe at buffer 4 */
+  GST_DEBUG ("Reading from client 1");
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000004", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000005", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000006", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000007", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  /* second client only bursts 50 bytes = 4 buffers, there is
+   * no keyframe above min and below max, so get one below min */
+  GST_DEBUG ("Reading from client 2");
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  /* third client only bursts 50 bytes = 4 buffers, we can't send
+   * more than 50 bytes so we only get 2 buffers (32 bytes). */
+  GST_DEBUG ("Reading from client 3");
+  fail_if (read (pfd3[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd3[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  GST_DEBUG ("cleaning up multifdsink");
+  ASSERT_SET_STATE (sink, GST_STATE_NULL, GST_STATE_CHANGE_SUCCESS);
+  cleanup_multifdsink (sink);
+
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
+  gst_caps_unref (caps);
+}
+
+GST_END_TEST;
+
+/* keep 100 bytes and burst 80 bytes to clients */
+GST_START_TEST (test_burst_client_bytes_with_keyframe)
+{
+  GstElement *sink;
+  GstBuffer *buffer;
+  GstCaps *caps;
+  int pfd1[2];
+  int pfd2[2];
+  int pfd3[2];
+  gchar data[16];
+  gint i;
+  guint buffers_queued;
+
+  sink = setup_multifdsink ();
+  /* make sure we keep at least 100 bytes at all times */
+  g_object_set (sink, "bytes-min", 100, NULL);
+  g_object_set (sink, "sync-method", 5, NULL);  /* 3 = burst_with_keyframe */
+  g_object_set (sink, "burst-unit", 3, NULL);   /* 3 = bytes */
+  g_object_set (sink, "burst-value", (guint64) 80, NULL);
+
+  fail_if (pipe (pfd1) == -1);
+  fail_if (pipe (pfd2) == -1);
+  fail_if (pipe (pfd3) == -1);
+
+  ASSERT_SET_STATE (sink, GST_STATE_PLAYING, GST_STATE_CHANGE_ASYNC);
+
+  caps = gst_caps_from_string ("application/x-gst-check");
+  gst_pad_set_caps (mysrcpad, caps);
+  GST_DEBUG ("Created test caps %p %" GST_PTR_FORMAT, caps, caps);
+
+  /* push buffers in, 9 * 16 bytes = 144 bytes */
+  for (i = 0; i < 9; i++) {
+    gchar *data;
+
+    buffer = gst_buffer_new_and_alloc (16);
+
+    /* mark most buffers as delta */
+    if (i != 0 && i != 4 && i != 8)
+      GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
+
+    /* copy some id */
+    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
+    g_snprintf (data, 16, "deadbee%08x", i);
+    gst_buffer_unmap (buffer, data, 16);
+
+    fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+  }
+
+  /* check that at least 7 buffers (112 bytes) are in the queue */
+  g_object_get (sink, "buffers-queued", &buffers_queued, NULL);
+  fail_if (buffers_queued != 7);
+
+  /* now add the clients */
+  g_signal_emit_by_name (sink, "add", pfd1[1]);
+  g_signal_emit_by_name (sink, "add_full", pfd2[1], 5,
+      3, (guint64) 50, 3, (guint64) 90);
+  g_signal_emit_by_name (sink, "add_full", pfd3[1], 5,
+      3, (guint64) 50, 3, (guint64) 50);
+
+  /* push last buffer to make client fds ready for reading */
+  for (i = 9; i < 10; i++) {
+    gchar *data;
+
+    buffer = gst_buffer_new_and_alloc (16);
+    GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
+
+    /* copy some id */
+    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
+    g_snprintf (data, 16, "deadbee%08x", i);
+    gst_buffer_unmap (buffer, data, 16);
+
+    fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+  }
+
+  /* now we should only read the last 6 buffers (min 5 * 16 = 80 bytes),
+   * keyframe at buffer 4 */
+  GST_DEBUG ("Reading from client 1");
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000004", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000005", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000006", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000007", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  /* second client only bursts 50 bytes = 4 buffers, there is
+   * no keyframe above min and below max, so send min */
+  GST_DEBUG ("Reading from client 2");
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000006", 16) == 0);
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000007", 16) == 0);
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd2[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  /* third client only bursts 50 bytes = 4 buffers, we can't send
+   * more than 50 bytes so we only get 3 buffers (48 bytes). */
+  GST_DEBUG ("Reading from client 3");
+  fail_if (read (pfd3[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000007", 16) == 0);
+  fail_if (read (pfd3[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000008", 16) == 0);
+  fail_if (read (pfd3[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000009", 16) == 0);
+
+  GST_DEBUG ("cleaning up multifdsink");
+  ASSERT_SET_STATE (sink, GST_STATE_NULL, GST_STATE_CHANGE_SUCCESS);
+  cleanup_multifdsink (sink);
+
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
+  gst_caps_unref (caps);
+}
+
+GST_END_TEST;
+
+/* Check that we can get data when multifdsink is configured in next-keyframe
+ * mode */
+GST_START_TEST (test_client_next_keyframe)
+{
+  GstElement *sink;
+  GstBuffer *buffer;
+  GstCaps *caps;
+  int pfd1[2];
+  gchar data[16];
+  gint i;
+
+  sink = setup_multifdsink ();
+  g_object_set (sink, "sync-method", 1, NULL);  /* 1 = next-keyframe */
+
+  fail_if (pipe (pfd1) == -1);
+
+  ASSERT_SET_STATE (sink, GST_STATE_PLAYING, GST_STATE_CHANGE_ASYNC);
+
+  caps = gst_caps_from_string ("application/x-gst-check");
+  gst_pad_set_caps (mysrcpad, caps);
+  GST_DEBUG ("Created test caps %p %" GST_PTR_FORMAT, caps, caps);
+
+  /* now add our client */
+  g_signal_emit_by_name (sink, "add", pfd1[1]);
+
+  /* push buffers in: keyframe, then non-keyframe */
+  for (i = 0; i < 2; i++) {
+    gchar *data;
+
+    buffer = gst_buffer_new_and_alloc (16);
+
+    /* copy some id */
+    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
+    g_snprintf (data, 16, "deadbee%08x", i);
+    gst_buffer_unmap (buffer, data, 16);
+    if (i > 0)
+      GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
+
+    fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
+  }
+
+  /* now we should be able to read some data */
+  GST_DEBUG ("Reading from client 1");
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000000", 16) == 0);
+  fail_if (read (pfd1[0], data, 16) < 16);
+  fail_unless (strncmp (data, "deadbee00000001", 16) == 0);
+
+  GST_DEBUG ("cleaning up multifdsink");
+  ASSERT_SET_STATE (sink, GST_STATE_NULL, GST_STATE_CHANGE_SUCCESS);
+  cleanup_multifdsink (sink);
+
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
+  gst_caps_unref (caps);
+}
+
+GST_END_TEST;
+
+/* FIXME: add test simulating chained oggs where:
+ * sync-method is burst-on-connect
+ * (when multifdsink actually does burst-on-connect based on byte size, not
+   "last keyframe" which any frame for audio :))
+ * an old client still needs to read from before the new streamheaders
+ * a new client gets the new streamheaders
+ */
+static Suite *
+multifdsink_suite (void)
+{
+  Suite *s = suite_create ("multifdsink");
+  TCase *tc_chain = tcase_create ("general");
+
+  suite_add_tcase (s, tc_chain);
+  tcase_add_test (tc_chain, test_no_clients);
+  tcase_add_test (tc_chain, test_add_client);
+  tcase_add_test (tc_chain, test_streamheader);
+  tcase_add_test (tc_chain, test_change_streamheader);
+  tcase_add_test (tc_chain, test_burst_client_bytes);
+  tcase_add_test (tc_chain, test_burst_client_bytes_keyframe);
+  tcase_add_test (tc_chain, test_burst_client_bytes_with_keyframe);
+  tcase_add_test (tc_chain, test_client_next_keyframe);
+
+  return s;
+}
+
+GST_CHECK_MAIN (multifdsink);