]> git.sesse.net Git - vlc/blobdiff - modules/control/netsync.c
Netsync: kill 2 warnings
[vlc] / modules / control / netsync.c
index 8be09774b1e4104c626d0226ab742345829795a1..852ef7848b98dffd021bc7201e6e7e488dbf3b39 100644 (file)
@@ -1,10 +1,11 @@
 /*****************************************************************************
  * netsync.c: synchronisation between several network clients.
  *****************************************************************************
- * Copyright (C) 2004 the VideoLAN team
+ * Copyright (C) 2004-2009 the VideoLAN team
  * $Id$
  *
  * Authors: Gildas Bazin <gbazin@videolan.org>
+ *          Jean-Paul Saman <jpsaman@videolan.org>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
 #ifdef HAVE_CONFIG_H
 # include "config.h"
 #endif
+#include <assert.h>
 
 #include <vlc_common.h>
 #include <vlc_plugin.h>
 #include <vlc_interface.h>
 #include <vlc_input.h>
-#include <vlc_es_out.h>
+#include <vlc_playlist.h>
 
 #ifdef HAVE_UNISTD_H
 #    include <unistd.h>
 #endif
-#ifdef HAVE_SYS_TIME_H
-#    include <sys/time.h>
-#endif
-#ifdef HAVE_SYS_TYPES_H
-#   include <sys/types.h>
-#endif
+#include <sys/types.h>
 #ifdef HAVE_POLL
 #   include <poll.h>
 #endif
 
 #define NETSYNC_PORT 9875
 
-/* Needed for Solaris */
-#ifndef INADDR_NONE
-#define INADDR_NONE 0xffffffff
-#endif
-
-/* FIXME: UGLY UGLY !! Netsync should be totally reworked */
-#include "../../src/input/input_internal.h"
-
 /*****************************************************************************
  * Module descriptor
  *****************************************************************************/
-static int  Activate( vlc_object_t * );
-static void Close   ( vlc_object_t * );
-
-static mtime_t GetClockRef( intf_thread_t *, mtime_t );
-
-/// \bug [String] This string is BAD.
-#define NETSYNC_TEXT N_( "Act as master" )
-#define NETSYNC_LONGTEXT N_( "Should " \
-  "act as the master client for the network synchronisation?" )
-
-/// \bug [String] This string is BAD.
-#define MIP_TEXT N_( "Master client ip address" )
-#define MIP_LONGTEXT N_( "IP address of " \
-  "the master client used for the network synchronisation." )
-
-vlc_module_begin();
-    set_shortname( N_("Network Sync"));
-    set_description( N_("Network synchronisation") );
-    set_category( CAT_ADVANCED );
-    set_subcategory( SUBCAT_ADVANCED_MISC );
-
-    add_bool( "netsync-master", 0, NULL,
-              NETSYNC_TEXT, NETSYNC_LONGTEXT, true );
-    add_string( "netsync-master-ip", NULL, NULL, MIP_TEXT, MIP_LONGTEXT,
-                true );
-
-    set_capability( "interface", 0 );
-    set_callbacks( Activate, Close );
-vlc_module_end();
-
-struct intf_sys_t
-{
-    input_thread_t *p_input;
-};
+static int  Open (vlc_object_t *);
+static void Close(vlc_object_t *);
+
+#define NETSYNC_TEXT N_("Network master clock")
+#define NETSYNC_LONGTEXT N_("When set then " \
+  "this vlc instance shall dictate its clock for synchronisation" \
+  "over clients listening on the masters network ip address")
+
+#define MIP_TEXT N_("Master server ip address")
+#define MIP_LONGTEXT N_("The IP address of " \
+  "the network master clock to use for clock synchronisation.")
+
+#define NETSYNC_TIMEOUT_TEXT N_("UDP timeout (in ms)")
+#define NETSYNC_TIMEOUT_LONGTEXT N_("Amount of time (in ms) " \
+  "to wait before aborting network reception of data.")
+
+vlc_module_begin()
+    set_shortname(N_("Network Sync"))
+    set_description(N_("Network synchronisation"))
+    set_category(CAT_ADVANCED)
+    set_subcategory(SUBCAT_ADVANCED_MISC)
+
+    add_bool("netsync-master", false, NULL,
+              NETSYNC_TEXT, NETSYNC_LONGTEXT, true)
+    add_string("netsync-master-ip", NULL, NULL, MIP_TEXT, MIP_LONGTEXT,
+                true)
+    add_integer("netsync-timeout", 500, NULL,
+                 NETSYNC_TIMEOUT_TEXT, NETSYNC_TIMEOUT_LONGTEXT, true)
+
+    set_capability("interface", 0)
+    set_callbacks(Open, Close)
+vlc_module_end()
 
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
-static void Run( intf_thread_t *p_intf );
+struct intf_sys_t {
+    int            fd;
+    int            timeout;
+    bool           is_master;
+    playlist_t     *playlist;
+
+    /* */
+    input_thread_t *input;
+    vlc_thread_t   thread;
+};
+
+static int PlaylistEvent(vlc_object_t *, char const *cmd,
+                         vlc_value_t oldval, vlc_value_t newval, void *data);
 
 /*****************************************************************************
  * Activate: initialize and create stuff
  *****************************************************************************/
-static int Activate( vlc_object_t *p_this )
+static int Open(vlc_object_t *object)
 {
-    intf_thread_t *p_intf = (intf_thread_t*)p_this;
+    intf_thread_t *intf = (intf_thread_t*)object;
+    intf_sys_t    *sys;
+    int fd;
+
+    if (!var_InheritBool(intf, "netsync-master")) {
+        char *psz_master = var_InheritString(intf, "netsync-master-ip");
+        if (psz_master == NULL) {
+            msg_Err(intf, "master address not specified");
+            return VLC_EGENERIC;
+        }
+        fd = net_ConnectUDP(VLC_OBJECT(intf), psz_master, NETSYNC_PORT, -1);
+        free(psz_master);
+    } else {
+        fd = net_ListenUDP1(VLC_OBJECT(intf), NULL, NETSYNC_PORT);
+    }
+
+    if (fd == -1) {
+        msg_Err(intf, "Netsync socket failure");
+        return VLC_EGENERIC;
+    }
 
-    p_intf->p_sys = malloc( sizeof( intf_sys_t ) );
-    if( !p_intf->p_sys )
-    {
-        msg_Err( p_intf, "no memory" );
+    intf->pf_run = NULL;
+    intf->p_sys = sys = malloc(sizeof(*sys));
+    if (!sys) {
+        net_Close(fd);
         return VLC_ENOMEM;
     }
 
-    p_intf->p_sys->p_input = NULL;
+    sys->fd = fd;
+    sys->is_master = var_InheritBool(intf, "netsync-master");
+    sys->timeout = var_InheritInteger(intf, "netsync-timeout");
+    if (sys->timeout < 500)
+        sys->timeout = 500;
+    sys->playlist = pl_Get(intf);
+    sys->input = NULL;
 
-    p_intf->pf_run = Run;
+    var_AddCallback(sys->playlist, "input-current", PlaylistEvent, intf);
     return VLC_SUCCESS;
 }
 
 /*****************************************************************************
  * Close: destroy interface
  *****************************************************************************/
-void Close( vlc_object_t *p_this )
+void Close(vlc_object_t *object)
 {
-    intf_thread_t *p_intf = (intf_thread_t*)p_this;
+    intf_thread_t *intf = (intf_thread_t*)object;
+    intf_sys_t *sys = intf->p_sys;
 
-    free( p_intf->p_sys );
+    assert(sys->input == NULL);
+    var_DelCallback(sys->playlist, "input-current", PlaylistEvent, intf);
+    net_Close(sys->fd);
+    free(sys);
 }
 
-/*****************************************************************************
- * Run: interface thread
- *****************************************************************************/
-static void Run( intf_thread_t *p_intf )
+static mtime_t GetPcrSystem(input_thread_t *input)
 {
-#define MAX_MSG_LENGTH (2 * sizeof(int64_t))
-
-    bool b_master = config_GetInt( p_intf, "netsync-master" );
-    char *psz_master = NULL;
-    char p_data[MAX_MSG_LENGTH];
-    int i_socket;
-
-    if( !b_master )
-    {
-        psz_master = config_GetPsz( p_intf, "netsync-master-ip" );
-        if( psz_master == NULL )
-        {
-            msg_Err( p_intf, "master address not specified" );
-            return;
-        }
-    }
-
-    if( b_master )
-        i_socket = net_ListenUDP1( VLC_OBJECT(p_intf), NULL, NETSYNC_PORT );
-    else
-        i_socket = net_ConnectUDP( VLC_OBJECT(p_intf), psz_master, NETSYNC_PORT, 0 );
-
-    free( psz_master );
-
-    if( i_socket < 0 )
-    {
-        msg_Err( p_intf, "failed opening UDP socket" ); /* str review: is this good enough? */
-        return;
-    }
+    int canc = vlc_savecancel();
+    /* TODO use the delay */
+    mtime_t system;
+    if (input_GetPcrSystem(input, &system, NULL))
+        system = -1;
+    vlc_restorecancel(canc);
+
+    return system;
+}
 
-    /* High priority thread */
-    vlc_thread_set_priority( p_intf, VLC_THREAD_PRIORITY_INPUT );
-
-    while( !intf_ShouldDie( p_intf ) )
-    {
-        /* Update the input */
-        if( p_intf->p_sys->p_input == NULL )
-        {
-            p_intf->p_sys->p_input =
-                (input_thread_t *)vlc_object_find( p_intf, VLC_OBJECT_INPUT,
-                                                   FIND_ANYWHERE );
-        }
-        else if( p_intf->p_sys->p_input->b_dead )
-        {
-            vlc_object_release( p_intf->p_sys->p_input );
-            p_intf->p_sys->p_input = NULL;
-        }
+static void *Master(void *handle)
+{
+    intf_thread_t *intf = handle;
+    intf_sys_t *sys = intf->p_sys;
+    for (;;) {
+        struct pollfd ufd = { .fd = sys->fd, .events = POLLIN, };
+        uint64_t data[2];
 
-        if( p_intf->p_sys->p_input == NULL )
-        {
-            /* Wait a bit */
-            msleep( INTF_IDLE_SLEEP );
+        if (poll(&ufd, 1, -1) <= 0)
             continue;
-        }
-
-        /*
-         * We now have an input
-         */
-
-        /* Initialize file descriptor set and timeout (0.5s) */
-        /* FIXME: arbitrary tick */
-        struct pollfd ufd = { .fd = i_socket, .events = POLLIN, };
-
-        if( b_master )
-        {
-            struct sockaddr_storage from;
-            mtime_t i_date, i_clockref, i_master_clockref;
-            int i_struct_size, i_read, i_ret;
-
-            /* Don't block */
-            i_ret = poll( &ufd, 1, 500 );
-            if( i_ret == 0 ) continue;
-            if( i_ret < 0 )
-            {
-                /* Wait a bit */
-                msleep( INTF_IDLE_SLEEP );
-                continue;
-            }
-
-            /* We received something */
-            i_struct_size = sizeof( from );
-            i_read = recvfrom( i_socket, p_data, MAX_MSG_LENGTH, 0,
-                               (struct sockaddr*)&from,
-                               (unsigned int *)&i_struct_size );
 
-            i_clockref = ntoh64(*(int64_t *)p_data);
+        /* We received something */
+        struct sockaddr_storage from;
+        unsigned struct_size = sizeof(from);
+        recvfrom(sys->fd, data, sizeof(data), 0,
+                 (struct sockaddr*)&from, &struct_size);
 
-            i_date = mdate();
-            *(int64_t *)p_data = hton64( i_date );
-
-            i_master_clockref = GetClockRef( p_intf, i_clockref );
-            *(((int64_t *)p_data)+1) = hton64( i_master_clockref );
+        mtime_t master_system = GetPcrSystem(sys->input);
+        if (master_system < 0)
+            continue;
 
-            /* Reply to the sender */
-            sendto( i_socket, p_data, 2 * sizeof(int64_t), 0,
-                    (struct sockaddr *)&from, i_struct_size );
+        data[0] = hton64(mdate());
+        data[1] = hton64(master_system);
 
+        /* Reply to the sender */
+        sendto(sys->fd, data, sizeof(data), 0,
+               (struct sockaddr *)&from, struct_size);
 #if 0
-            msg_Dbg( p_intf, "Master clockref: %"PRId64" -> %"PRId64", from %s "
-                     "(date: %"PRId64")", i_clockref, i_master_clockref,
-                     from.ss_family == AF_INET
-                     ? inet_ntoa(((struct sockaddr_in *)&from)->sin_addr)
-                     : "non-IPv4", i_date );
+        /* not sure we need the client information to sync,
+           since we are the master anyway */
+        mtime_t client_system = ntoh64(data[0]);
+        msg_Dbg(intf, "Master clockref: %"PRId64" -> %"PRId64", from %s "
+                 "(date: %"PRId64")", client_system, master_system,
+                 (from.ss_family == AF_INET) ? inet_ntoa(((struct sockaddr_in *)&from)->sin_addr)
+                 : "non-IPv4", /*date*/ 0);
 #endif
-        }
-        else
-        {
-            mtime_t i_send_date, i_receive_date, i_master_date, i_diff_date;
-            mtime_t i_master_clockref, i_client_clockref, i_drift;
-            mtime_t i_clockref = 0;
-            int i_sent, i_read, i_ret;
-
-            /* Send clock request to the master */
-            *(int64_t *)p_data = hton64( i_clockref );
-            i_send_date = mdate();
-
-            i_sent = send( i_socket, p_data, sizeof(int64_t), 0 );
-            if( i_sent <= 0 )
-            {
-                /* Wait a bit */
-                msleep( INTF_IDLE_SLEEP );
-                continue;
-            }
-
-            /* Don't block */
-            i_ret = poll( &ufd, 1, 500 );
-            if( i_ret == 0 ) continue;
-            if( i_ret < 0 )
-            {
-                /* Wait a bit */
-                msleep( INTF_IDLE_SLEEP );
-                continue;
-            }
-
-            i_receive_date = mdate();
+    }
+    return NULL;
+}
 
-            i_read = recv( i_socket, p_data, MAX_MSG_LENGTH, 0 );
-            if( i_read <= 0 )
-            {
-                /* Wait a bit */
-                msleep( INTF_IDLE_SLEEP );
-                continue;
-            }
+static void *Slave(void *handle)
+{
+    intf_thread_t *intf = handle;
+    intf_sys_t *sys = intf->p_sys;
 
-            i_master_date = ntoh64(*(int64_t *)p_data);
-            i_master_clockref = ntoh64(*(((int64_t *)p_data)+1));
+    for (;;) {
+        struct pollfd ufd = { .fd = sys->fd, .events = POLLIN, };
+        uint64_t data[2];
 
-            i_diff_date = i_receive_date -
-                          ((i_receive_date - i_send_date) / 2 + i_master_date);
+        mtime_t system = GetPcrSystem(sys->input);
+        if (system < 0)
+            goto wait;
 
-            i_client_clockref = i_drift = 0;
-            if( p_intf->p_sys->p_input && i_master_clockref )
-            {
-                i_client_clockref = GetClockRef( p_intf, i_clockref );
-                i_drift = i_client_clockref - i_master_clockref - i_diff_date;
+        /* Send clock request to the master */
+        data[0] = hton64(system);
 
-                /* Update our clock to match the master's one */
-                if( i_client_clockref )
-                    p_intf->p_sys->p_input->i_pts_delay -= i_drift;
-            }
+        const mtime_t send_date = mdate();
+        if (send(sys->fd, data, sizeof(data[0]), 0) <= 0)
+            goto wait;
 
+        /* Don't block */
+        int ret = poll(&ufd, 1, sys->timeout);
+        if (ret == 0)
+            continue;
+        if (ret < 0)
+            goto wait;
+
+        const mtime_t receive_date = mdate();
+        if (recv(sys->fd, data, sizeof(data), 0) <= 0)
+            goto wait;
+
+        const mtime_t master_date   = ntoh64(data[0]);
+        const mtime_t master_system = ntoh64(data[1]);
+        const mtime_t diff_date = receive_date -
+                                  ((receive_date - send_date) / 2 + master_date);
+
+        if (master_system > 0) {
+            int canc = vlc_savecancel();
+
+            mtime_t client_system;
+            if (!input_GetPcrSystem(sys->input, &client_system, NULL)) {
+                const mtime_t diff_system = client_system - master_system - diff_date;
+                if (diff_system != 0) {
+                    input_ModifyPcrSystem(sys->input, true, master_system - diff_date);
 #if 0
-            msg_Dbg( p_intf, "Slave clockref: %"PRId64" -> %"PRId64" -> %"PRId64", "
-                     "clock diff: %"PRId64" drift: %"PRId64,
-                     i_clockref, i_master_clockref,
-                     i_client_clockref, i_diff_date, i_drift );
+                    msg_Dbg(intf, "Slave clockref: %"PRId64" -> %"PRId64" -> %"PRId64","
+                             " clock diff: %"PRId64", diff: %"PRId64"",
+                             system, master_system, client_system,
+                             diff_system, diff_date);
 #endif
-
-            /* Wait a bit */
-            msleep( INTF_IDLE_SLEEP );
+                }
+            }
+            vlc_restorecancel(canc);
         }
+    wait:
+        msleep(INTF_IDLE_SLEEP);
     }
-
-    if( p_intf->p_sys->p_input ) vlc_object_release( p_intf->p_sys->p_input );
-    net_Close( i_socket );
+    return NULL;
 }
 
-static mtime_t GetClockRef( intf_thread_t *p_intf, mtime_t i_pts )
+static int InputEvent(vlc_object_t *object, char const *cmd,
+                      vlc_value_t oldval, vlc_value_t newval, void *data)
 {
-    input_thread_t *p_input = p_intf->p_sys->p_input;
-    mtime_t i_ts;
-
-    if( !p_input || !p_input->p->p_es_out ) return 0;
-
-    if( es_out_Control( p_input->p->p_es_out, ES_OUT_GET_TS, i_pts, &i_ts ) ==
-        VLC_SUCCESS )
-    {
-        return i_ts;
+    VLC_UNUSED(cmd); VLC_UNUSED(oldval); VLC_UNUSED(object);
+    intf_thread_t  *intf = data;
+    intf_sys_t     *sys = intf->p_sys;
+
+    if (newval.i_int == INPUT_EVENT_DEAD && sys->input) {
+        msg_Err(intf, "InputEvent DEAD");
+        vlc_cancel(sys->thread);
+        vlc_join(sys->thread, NULL);
+        vlc_object_release(sys->input);
+        sys->input = NULL;
     }
+    return VLC_SUCCESS;
+}
 
-    return 0;
+static int PlaylistEvent(vlc_object_t *object, char const *cmd,
+                         vlc_value_t oldval, vlc_value_t newval, void *data)
+{
+    VLC_UNUSED(cmd); VLC_UNUSED(oldval); VLC_UNUSED(object);
+    intf_thread_t  *intf = data;
+    intf_sys_t     *sys = intf->p_sys;
+
+    input_thread_t *input = newval.p_address;
+    assert(sys->input == NULL);
+    sys->input = vlc_object_hold(input);
+    if (vlc_clone(&sys->thread, sys->is_master ? Master : Slave, intf,
+                  VLC_THREAD_PRIORITY_INPUT)) {
+        vlc_object_release(input);
+        return VLC_SUCCESS;
+    }
+    var_AddCallback(input, "intf-event", InputEvent, intf);
+    return VLC_SUCCESS;
 }
+