<div>
Hi,
</div><div><br></div><div>1) when use switch_rtp without a session SWITCH_RTP_FLAG_DEBUG_RTP_READ complaining no session attached because it use SWITCH_CHANNEL_SESSION_LOG.</div><div><br></div><div>Suggested fix:</div><div><br></div><div>use SWITCH_CHANNEL_SESSION_LOG where possible and SWITCH_CHANNEL_LOG when no session.</div><div><br></div><div>also, I think the following code is wrong because there's no rtp_session->session:</div><div><br></div><div>if (!rtp_session->session) {</div><div><div><span style="white-space:pre"> </span>rtp_session->flags[SWITCH_RTP_FLAG_DEBUG_RTP_READ] = 0;</div><div><span style="white-space:pre"> </span>switch_log_printf(SWITCH_CHANNEL_SESSION_LOG(rtp_session->session), SWITCH_LOG_CRIT, "RTP HAS NO SESSION!\n");</div><div><span style="white-space:pre"> </span>} else {</div></div><div><span style="white-space:pre"> </span>...</div><div><br></div><div>I can fix and push if confirmed.</div><div><br></div><div>2) switch_rtp_zerocopy_read_frame() blocks forever if no RTP comes. Anyway to make it non block? I tried the following code it blocks even with SWITCH_IO_FLAG_NOBLOCK.</div><div><br></div><div>I tried the following code in mod_skel, I can make a diff if necessary.</div><div><br></div><div><br></div><div><div><span style="white-space:pre"> </span>rtp_flag[SWITCH_RTP_FLAG_IO] = 1;</div><div><span style="white-space:pre"> </span>rtp_flag[SWITCH_RTP_FLAG_AUTOADJ] = 1;</div><div><span style="white-space:pre"> </span>rtp_flag[SWITCH_RTP_FLAG_DATAWAIT] = 0;</div><div><span style="white-space:pre"> </span>rtp_flag[SWITCH_RTP_FLAG_VIDEO] = 1;</div><div><span style="white-space:pre"> </span>rtp_flag[SWITCH_RTP_FLAG_NOBLOCK] = 1;</div><div><span style="white-space:pre"> </span>rtp_flag[SWITCH_RTP_FLAG_AUTO_CNG] = 1;</div><div><span style="white-space:pre"> </span>// rtp_flag[SWITCH_RTP_FLAG_DEBUG_RTP_READ] = 1;</div><div><span style="white-space:pre"> </span>// rtp_flag[SWITCH_RTP_FLAG_DEBUG_RTP_WRITE] = 1;</div><div><br></div><div><span style="white-space:pre"> </span>rtp_session = switch_rtp_new(IP, //local</div><div><span style="white-space:pre"> </span>50002, //recv port</div><div><span style="white-space:pre"> </span>IP,</div><div><span style="white-space:pre"> </span>42000, //remote, doesn't matter for us, but can't be 0</div><div><span style="white-space:pre"> </span>96, // payload_type</div><div><span style="white-space:pre"> </span>1,</div><div><span style="white-space:pre"> </span>90000,</div><div><span style="white-space:pre"> </span>rtp_flag,</div><div><span style="white-space:pre"> </span>NULL, &err, pool);</div><div><br></div><div><span style="white-space:pre"> </span>if (!rtp_session) {</div><div><span style="white-space:pre"> </span>switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_ERROR, "RTP Error: %s\n", err);</div><div><span style="white-space:pre"> </span>goto done;</div><div><span style="white-space:pre"> </span>}</div><div><br></div><div><span style="white-space:pre"> </span>while(!done) {</div><div><span style="white-space:pre"> </span>// status = switch_rtp_zerocopy_read_frame(rtp_session, &read_frame, SWITCH_IO_FLAG_NOBLOCK);</div><div><span style="white-space:pre"> </span>status = switch_rtp_zerocopy_read_frame(rtp_session, &read_frame, SWITCH_IO_FLAG_NONE);</div><div><span style="white-space:pre"> </span>switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_INFO, "read %d\n", read_frame.packetlen);</div><div><span style="white-space:pre"> </span>switch_yield(100000);</div><div><span style="white-space:pre"> </span>}</div></div><div><br></div>
<div><div>Thanks for help.</div><div><br></div><div><br></div><div>-- </div><div>Seven Du</div><div><span style="font-size: 10pt; "><a href="http://www.freeswitch.org.cn">http://www.freeswitch.org.cn</a></span></div><div><a href="http://about.me/dujinfang">http://about.me/dujinfang</a></div><div><div><a href="http://www.dujinfang.com">http://www.dujinfang.com</a></div></div><div><br></div><div>Sent with <a href="http://www.sparrowmailapp.com/?sig">Sparrow</a></div><div><br></div></div>