Attention is currently required from: jolly.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33603 )
Change subject: ASCI: Make bssmap_handle_ass_req_ct_speech() publically available.
......................................................................
Patch Set 2:
(2 comments)
Patchset:
PS2:
This patch really needs further explanation.
File src/osmo-bsc/osmo_bsc_bssap.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/33603/comment/0d82ec99_40636a6f
PS2, Line 744: static int select_codecs(struct assignment_request *req, const struct gsm0808_channel_type *ct,
why this change? bts can already be obtained through conn like before?
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33603
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ifc1e315d5282f01f8d1bd600d62476c2ae74eca9
Gerrit-Change-Number: 33603
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Fri, 07 Jul 2023 11:53:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: jolly.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33602 )
Change subject: ASCI: Make function to add OSMUX IE public
......................................................................
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33602
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1452cabb142f9e7a169f4ddfeac85908abaf8dfc
Gerrit-Change-Number: 33602
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Fri, 07 Jul 2023 11:51:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: jolly.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33597 )
Change subject: Cleanup code style of rate counters in osmo_bsc_msc.c
......................................................................
Patch Set 2: Code-Review+1
(1 comment)
Patchset:
PS2:
Sounds like this should be done before adding new counters :)
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33597
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I7cfd4f20ebfee0c8df9305372977d6628d6337c5
Gerrit-Change-Number: 33597
Gerrit-PatchSet: 2
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Fri, 07 Jul 2023 11:46:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: jolly.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/33596 )
Change subject: ASCI: Add new rate counters to support VGCS/VBS messages
......................................................................
Patch Set 1:
(2 comments)
Patchset:
PS1:
Another case of "first adding definitions in one patch and then using them in some other patch". Let's please add the definition in the same patch where they are used, to avoid ending with dangling counters not ticked, either now or upon revert, backport, etc.
File src/osmo-bsc/osmo_bsc_msc.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/33596/comment/f82a7c3b_418b28ae
PS1, Line 149: [MSC_CTR_BSSMAP_TX_DT1_VGCS_VBS_SETUP_ACK] = {
Same, format change.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/33596
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I18e4ca3599e480de2d0f64cc1b6f4bb6ce8020d4
Gerrit-Change-Number: 33596
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Comment-Date: Fri, 07 Jul 2023 11:45:29 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment