Grzegorz Sluja authored 3 years ago and Sukru Senli committed 3 years ago

This commit includes revert of:
commit d178f497
"res_pjsip:  Filter out non SIP(S) requests"

Yalu Zhang authored 3 years ago and Sukru Senli committed 3 years ago

This solves the issue of Unknown state of a device.

Grzegorz Sluja authored 3 years ago and Sukru Senli committed 3 years ago

After 'from_user' config has been removed from pjsip_endpoint config the
user in FROM header was wrong. Fix it with using session->id.number
instead of connected_id.number (which is wrong in this case).

Yalu Zhang authored 3 years ago and Sukru Senli committed 3 years ago

Don't destroy the connection when the peer sub-channel is in dialing or ringback state.

Grzegorz Sluja authored 3 years ago and Sukru Senli committed 3 years ago

There was a possibility that during attended call transfer, deadlock
happened in locking bridge_channel and channel.
bridge_channel_queue_deferred_frames() was waiting for brcm_indicate()
to unlock the channel, while brcm_indicate() which called
ast_bridge_channel_queue_frame() was waiting for
bridge_channel_queue_deferred_frames() to unlock the bridge.

Creating new macro to lock both channel and bridge_channel at the same
time and sched_yield() -> retry if failed and use it in
bridge_channel_queue_deferred_frames() avoiding the deadlock.

Yalu Zhang authored 3 years ago and Sukru Senli committed 3 years ago

Those two lines are included in brcm_signal_dialtone() which is about to invoke.

Grzegorz Sluja authored 3 years ago and Sukru Senli committed 3 years ago

Call Status for each line can be get by:
ubus call asterisk call_status '{ "line" : X }', where X is line:
{0, 1, 2, or 3}

callStatus is set in chan_brcm according to TR-104 requirements:
Idle: no ongoing calls
Dialing: the process of collecting digits for the calling number in an outgoing call, and ringing in outgoing call.
Delivered: N/A
Connected: a call is established
Alerting: ringing when an incoming call is received
Disconnected: the remote party has ended a call. but the local party has not gone on-hook.

Yalu Zhang authored 3 years ago and Sukru Senli committed 3 years ago

- asterisk.telephony -> voice.line, "line" -> "id"
- asterisk.sip -> voice.sip.client, "line" -> "uri"
- asterisk.mwi -> voice.mwi

Yalu Zhang authored 3 years ago and Sukru Senli committed 3 years ago

Also remove unused parameter "data":0 in asterisk.telephony ubus events.

Yalu Zhang authored 3 years ago and Sukru Senli committed 3 years ago

Also fix some miscellaneous compiling warnings.

Grzegorz Sluja authored 3 years ago and Sukru Senli committed 3 years ago

Signed-off-by: Grzegorz Sluja <grzegorz.sluja@iopsys.eu>

Grzegorz Sluja authored 3 years ago and Sukru Senli committed 3 years ago

When chan_brcm.so module is unloading we are trying to stop the ubus
thread. With pthread_cancel() called before pthread_join() we are hanging
for a long time since the thread has not set cancellation type. The
cancellation type 'deferred' is set by default what means that
cancellation is delayed until the thread next call, that takes time.
Using pthread_kill() instead of pthread_cancel() makes the signal is sent
to the thread immediately causing thread exits/joins suddenly.

Grzegorz Sluja authored 3 years ago and Sukru Senli committed 3 years ago

Signed-off-by: Grzegorz Sluja <grzegorz.sluja@iopsys.eu>

Grzegorz Sluja authored 3 years ago and Sukru Senli committed 3 years ago

This commit includes 3 fixes:
- change place of get/set rtp statistics to have them properly updated in cdr
for both incoming and outgoing call
- do not create additional "hangup" call log record
- save rtp stats counters from endptmngr into brcm_subchannel instead of
one global structure to support concurrent calls simultaneously

Signed-off-by: Grzegorz Sluja <grzegorz.sluja@iopsys.eu>

Yalu Zhang authored 3 years ago and Sukru Senli committed 3 years ago

For an unanswered inbound call, the connection has not been created in endptmngr when
the call terminates so the RTP statistics are not available.