SDP: Add get/set option calls for RTP sched context per type.
authorRichard Mudgett <rmudgett@digium.com>
Thu, 8 Jun 2017 16:38:33 +0000 (11:38 -0500)
committerRichard Mudgett <rmudgett@digium.com>
Thu, 15 Jun 2017 14:42:15 +0000 (09:42 -0500)
Change-Id: I82dc75c63c48904e9e5a49e2205dcc06e88487e4

include/asterisk/sdp_options.h
main/sdp_options.c
main/sdp_private.h
main/sdp_state.c

index f49b794..b8c1bbd 100644 (file)
@@ -522,4 +522,29 @@ void ast_sdp_options_set_ssrc(struct ast_sdp_options *options, unsigned int ssrc
  */
 unsigned int ast_sdp_options_get_ssrc(const struct ast_sdp_options *options);
 
+/*!
+ * \brief Set the SDP options scheduler context used to create new streams of the type.
+ * \since 15.0.0
+ *
+ * \param options SDP Options
+ * \param type Media type the scheduler context is for.
+ * \param sched Scheduler context to use for the specified media type.
+ *
+ * \return Nothing
+ */
+void ast_sdp_options_set_sched_type(struct ast_sdp_options *options,
+       enum ast_media_type type, struct ast_sched_context *sched);
+
+/*!
+ * \brief Get the SDP options scheduler context used to create new streams of the type.
+ * \since 15.0.0
+ *
+ * \param options SDP Options
+ * \param type Media type the format cap represents.
+ *
+ * \return The stored scheduler context to create new streams of the type.
+ */
+struct ast_sched_context *ast_sdp_options_get_sched_type(const struct ast_sdp_options *options,
+       enum ast_media_type type);
+
 #endif /* _ASTERISK_SDP_OPTIONS_H */
index ab8fb29..a938583 100644 (file)
@@ -77,6 +77,39 @@ DEFINE_GETTERS_SETTERS_FOR(enum ast_sdp_options_impl, impl);
 DEFINE_GETTERS_SETTERS_FOR(enum ast_sdp_options_encryption, encryption);
 DEFINE_GETTERS_SETTERS_FOR(unsigned int, ssrc);
 
+struct ast_sched_context *ast_sdp_options_get_sched_type(const struct ast_sdp_options *options, enum ast_media_type type)
+{
+       struct ast_sched_context *sched = NULL;
+
+       switch (type) {
+       case AST_MEDIA_TYPE_AUDIO:
+       case AST_MEDIA_TYPE_VIDEO:
+       case AST_MEDIA_TYPE_IMAGE:
+       case AST_MEDIA_TYPE_TEXT:
+               sched = options->sched[type];
+               break;
+       case AST_MEDIA_TYPE_UNKNOWN:
+       case AST_MEDIA_TYPE_END:
+               break;
+       }
+       return sched;
+}
+
+void ast_sdp_options_set_sched_type(struct ast_sdp_options *options, enum ast_media_type type, struct ast_sched_context *sched)
+{
+       switch (type) {
+       case AST_MEDIA_TYPE_AUDIO:
+       case AST_MEDIA_TYPE_VIDEO:
+       case AST_MEDIA_TYPE_IMAGE:
+       case AST_MEDIA_TYPE_TEXT:
+               options->sched[type] = sched;
+               break;
+       case AST_MEDIA_TYPE_UNKNOWN:
+       case AST_MEDIA_TYPE_END:
+               break;
+       }
+}
+
 static void set_defaults(struct ast_sdp_options *options)
 {
        options->dtmf = DEFAULT_DTMF;
index a0b63df..62228a5 100644 (file)
@@ -35,6 +35,8 @@ struct ast_sdp_options {
                /*! RTP Engine Name */
                AST_STRING_FIELD(rtp_engine);
        );
+       /*! Scheduler context for the media stream types (Mainly for RTP) */
+       struct ast_sched_context *sched[AST_MEDIA_TYPE_END];
        struct {
                unsigned int rtp_symmetric:1;
                unsigned int udptl_symmetric:1;
index 0f06bf9..99421ad 100644 (file)
@@ -151,12 +151,6 @@ static void sdp_state_capabilities_free(struct sdp_state_capabilities *capabilit
        ast_free(capabilities);
 }
 
-/* TODO
- * This isn't set anywhere yet.
- */
-/*! \brief Scheduler for RTCP purposes */
-static struct ast_sched_context *sched;
-
 /*! \brief Internal function which creates an RTP instance */
 static struct sdp_state_rtp *create_rtp(const struct ast_sdp_options *options,
        enum ast_media_type media_type)
@@ -185,7 +179,8 @@ static struct sdp_state_rtp *create_rtp(const struct ast_sdp_options *options,
                return NULL;
        }
 
-       rtp->instance = ast_rtp_instance_new(options->rtp_engine, sched, media_address, NULL);
+       rtp->instance = ast_rtp_instance_new(options->rtp_engine,
+               ast_sdp_options_get_sched_type(options, media_type), media_address, NULL);
        if (!rtp->instance) {
                ast_log(LOG_ERROR, "Unable to create RTP instance using RTP engine '%s'\n",
                        options->rtp_engine);