Re: RTCP XR not updated

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ming
Thanks for looking into this. My issue was indeed more related to the rtcp_xr_interval and can be resolved without any modification to the library.
Best,
Henoc


On 1 Aug 2016 03:40, "Ming" <ming@xxxxxxxxx> wrote:
Hi Henoc,

Why is "with_xr" never set?
 if (timestamp - stream->rtcp_xr_last_tx >= stream->rtcp_xr_interval),
which it eventually will, then it will be set (unless there's
something wrong with the timestamp, or rtcp_xr_interval is too big).
In those cases, it would be better to fix the problem itself
(timestamp/interval), instead of forcing to send RTCP XR everytime,
which is what you proposed.

Regards,
Ming

On Fri, Jul 29, 2016 at 6:37 PM, Henoc Agbota <henoc@xxxxxxxxxxxx> wrote:
> Hi
>
> Just want to report that in pjsip 2.4.5 even when enabled, the RTCP XR
> reports are not sent. The function check_tx_rtcp() in pjmedia/stream.c which
> is used to transmit periodic RTCP SR/RR reports could be used for that
> purpose but there appears to be a bug preventing that. Basically, the
> "with_xr" flag is never set. The issue is fixed by replacing
>
> #if defined(PJMEDIA_HAS_RTCP_XR) && (PJMEDIA_HAS_RTCP_XR != 0)
> if (stream->rtcp.xr_enabled) {
>    if (stream->rtcp_xr_last_tx == 0) {
> stream->rtcp_xr_last_tx = timestamp;
>    } else if (timestamp - stream->rtcp_xr_last_tx >=
> stream->rtcp_xr_interval)
>    {
> with_xr = PJ_TRUE;
>
> /* Update last tx RTCP XR */
> stream->rtcp_xr_last_tx = timestamp;
>    }
> }
> #endif
>
> by
>
> #if defined(PJMEDIA_HAS_RTCP_XR) && (PJMEDIA_HAS_RTCP_XR != 0)
> if (stream->rtcp.xr_enabled) {
>    if (stream->rtcp_xr_last_tx == 0) {
> stream->rtcp_xr_last_tx = timestamp;
>    } else if (timestamp - stream->rtcp_xr_last_tx >=
> stream->rtcp_xr_interval)
>    {
> /* Update last tx RTCP XR */
> stream->rtcp_xr_last_tx = timestamp;
>    }
>
> with_xr = PJ_TRUE;
> }
> #endif
>
> Thanks,
> Henoc
> --
> Henoc Agbota
> SamKnows Limited
> E-mail          henoc@xxxxxxxxxxxx
> Office          +44 (0) 20 3111 4336
> Web            www.samknows.com
>
> _______________________________________________
> Visit our blog: http://blog.pjsip.org
>
> pjsip mailing list
> pjsip@xxxxxxxxxxxxxxx
> http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org
>

_______________________________________________
Visit our blog: http://blog.pjsip.org

pjsip mailing list
pjsip@xxxxxxxxxxxxxxx
http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org
_______________________________________________
Visit our blog: http://blog.pjsip.org

pjsip mailing list
pjsip@xxxxxxxxxxxxxxx
http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org

[Index of Archives]     [Asterisk Users]     [Asterisk App Development]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [Linux API]
  Powered by Linux