Change in osmo-bsc[master]: vty: check with is_ipaccess_bts() before using IPACC

[dexter]

dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/18971 )

Change subject: vty: check with is_ipaccess_bts() before using IPACC
......................................................................


Patch Set 2:

(4 comments)

(ensuring all comments to be sent!)

https://gerrit.osmocom.org/c/osmo-bsc/+/18971/1//COMMIT_MSG 
Commit Message:

https://gerrit.osmocom.org/c/osmo-bsc/+/18971/1//COMMIT_MSG@10 
PS1, Line 10: negotiate ip address and port for RTP/VoIP. This protocol is BTS
> to negotiate
Done


https://gerrit.osmocom.org/c/osmo-bsc/+/18971/1//COMMIT_MSG@12 
PS1, Line 12: all locations in osmo-bts check the bts type before using IPACC, lets
> let's
Done


https://gerrit.osmocom.org/c/osmo-bsc/+/18971/1/src/osmo-bsc/bsc_vty.c 
File src/osmo-bsc/bsc_vty.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/18971/1/src/osmo-bsc/bsc_vty.c@5138 
PS1, Line 5138: 	if (is_ipaccess_bts(lchan->ts->trx->bts))
> so afaiu after your patch, if the bts is not an IPA based one, this VTY cmd does nothing. […]
Done


https://gerrit.osmocom.org/c/osmo-bsc/+/18971/1/src/osmo-bsc/lchan_rtp_fsm.c 
File src/osmo-bsc/lchan_rtp_fsm.c:

https://gerrit.osmocom.org/c/osmo-bsc/+/18971/1/src/osmo-bsc/lchan_rtp_fsm.c@347 
PS1, Line 347: 	if (is_ipaccess_bts(lchan->ts->trx->bts)) {
> So if it's not an IPACC, why is it in a "wait_ipacc_mdcx_ack" state in first place? This patch looks […]
I am sorry, I will revert the changes in lchan_rtp_fsm.c. You are right, it makes no sense to check this here. I wrote a message, it was a bit late yesterday.



-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/18971
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I9ee78b6b1d342abaccc09a87dee6af79e76e5468
Gerrit-Change-Number: 18971
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-Comment-Date: Tue, 23 Jun 2020 14:54:24 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin at sysmocom.de>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200623/9215424c/attachment.htm>