<div dir="ltr"><div dir="ltr">On Wed, May 11, 2022 at 8:54 AM Fridrich Maximilian <<a href="mailto:M.Fridrich@commend.com">M.Fridrich@commend.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
I am currently working on fixing [ASTERISK-30051] res_pjsip: No video<br>
after un-hold with moh_passthrough=yes. [1]<br>
<br>
I have debugged the code with DEBUG_THREADS enabled (which resolves the issue)<br>
and without and compared the two. The issue clearly seems to be that<br>
chan_pjsip:chan_pjsip_indicate() is called in the wrong order:<br>
<br>
1. It is called to indicate AST_CONTROL_STREAM_TOPOLOGY_REQUEST_CHANGE<br>
2. It is called to indicate AST_CONTROL_UNHOLD<br>
<br>
With DEBUG_THREADS enabled, AST_CONTROL_UNHOLD is always called first and video<br>
works as expected.<br>
<br>
Before I try to fix the underlying issue (which I think I have identified - see<br>
below) I want to understand why AST_CONTROL_UNHOLD needs to be indicated before<br>
AST_CONTROL_STREAM_TOPOLOGY_REQUEST_CHANGE.<br>
<br>
Does anyone have any idea why it has to be called in that order?<br></blockquote><div><br></div><div>You're in off-nominal untested un thought of territory, so the code behavior probably reflects that. Specifically having both audio and video, and doing hold/unhold. Audio hold is special and separate from stream negotiation, while video isn't, so things probably don't handle that.</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="font-family:tahoma,sans-serif"><font color="#073763">Joshua C. Colp</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Asterisk Technical Lead</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Sangoma Technologies</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Check us out at <a href="http://www.sangoma.com" target="_blank">www.sangoma.com</a> and <a href="http://www.asterisk.org" target="_blank">www.asterisk.org</a></font><br></div></div></div></div></div></div></div></div></div></div></div>