aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2021-06-28cosmetic: gsm_12_21.h: Fix trailing whitespacePau Espin Pedrol1-2/+2
Change-Id: Ifdcdcfe30c41da233899101e1057bfca7d645a66
2021-06-25bts_features: add feature for BCCH carrier power reduction modeVadim Yanitskiy2-0/+3
Change-Id: I69283b3f35988fc7a1a1dcf1a1ad3b67f08ec716 Related: SYS#4919
2021-06-25gsm0808: add Kc128 to Handover RequestNeels Hofmeyr2-1/+9
Change-Id: Ieb6e43eef9e57281d54d4b7c63664668df5aef3e
2021-06-25gprs_ns2_vc_fsm: reset the ALIVE response time when stopping testAlexander Couzens1-0/+1
Prevent showing a response time when the NSVC has failed. Related: OS#5089 Change-Id: I5435077d35ff1ae3d743ea1b1b8ace7f0fb913cf
2021-06-25gprs_ns2: fix missing notify towards the NSE when NSVC become blockedAlexander Couzens3-0/+73
The NSE wasn't notified when a NSVC went into the BLOCKED state from an UNBLOCKED state. Related: OS#5182 Change-Id: I09634e414e9bb966e6b5809b7de1b59fbabd413d
2021-06-25gprs_ns2: use llist_add_tail to keep orderAlexander Couzens3-10/+10
When configuring multiple NSE/BINDs the order of the configuration should be keeped. Related: OS#5181 Change-Id: Ibbc03f0780b49543b5bd97ee059f11cfd6c2a126
2021-06-21add Kc128 to gsm0808 Create Ciphering CommandNeels Hofmeyr5-10/+73
Prepare for A5/4 support in osmo-msc. Add new function gsm0808_create_cipher2() which takes a struct as argument instead of individual fields. This is akin to e.g. gsm0808_create_handover_request() below in the file, and allows backwards compatibly extending the argument list without needing a new function signature every time. Add struct gsm0808_cipher_mode_command, as argument list for gsm0808_create_cipher2(), with kc128 included. Encode the Kc128 IE in gsm0808_create_cipher2(). Implement gsm0808_create_cipher() by calling gsm0808_create_cipher2(). Change-Id: Ib3906085e0c6e5a496a9f755f0f786238a86ca34
2021-06-18vty: make function cmd_range_match() publicPhilipp Maier2-4/+6
Certain control interface commands also may require to verfy a range in their verify function. cmd_range_match() from the VTY does exactly that and the range can be specified as string, the same way as we would specify it in the VTY. Change-Id: I53fc207677f52b1dc748b01d58424839cdba807c related: SYS#5369
2021-06-18osmo_select_shutdown_request(): allow finishing pending writes on SIGTERMNeels Hofmeyr2-1/+72
Allow telling osmo_select_main* to only service pending writes (shutdown mode). Introduce API fuctions to indicate a shutdown request, and find out whether shutdown is complete. Some osmo programs have a curious sleep of few seconds upon receiving SIGTERM. The idea presumably was to finish off pending writes before halting the program. But a sleep() on program exit is annoying, especially when there usually are no pending writes, and when osmo-bsc is launched numerous times for tests. Change-Id: Ib94d4316924103459577087c2214188679db2227
2021-06-16stats_vty: Improve generation of osmo countersDaniel Willmann1-11/+10
Use section for all counter headings, the idea is that the generated doc is included below a chapter describing the implemented counters which can optionally describe individual items. If not osmo_counters are present simply omit the whole section to tidy up the manual. Those are deprecated anyway and not used in the osmo-* projects that I checked. Change-Id: I126b39cb9458e0f62efa338967e07151a0eece0f Related: SYS#5005, OS#4523
2021-06-16control_cmd: fix typoPhilipp Maier1-1/+1
Change-Id: I9deaee42aeb0ab0549cc7c4f180ad9c216b9df39
2021-06-15ctrl: Support recovering from short writePau Espin Pedrol1-1/+7
osmo_wqueue has support for it, so simply handle it correctly in the callback (updating buffer and returning -EAGAIN). Related: OS#5169 Change-Id: I6cbc7ec6ae6832e61cddf4402332ba09b142a7d4
2021-06-15ctrl: Pre-calculate required size before allocating msgbPau Espin Pedrol2-62/+34
This commit fixes crash when response is more than ~4096 chars. Furthermore, we now allocate only the required memory, not 4096 for all messages, which usually don't require it. Test needs to be adapted since it assumed there was more available space at the end of the msgb. Related: OS#5169 Change-Id: I0b8f370f7b08736207f9efed13a0663b5e482824
2021-06-13gprs_ns2_sns: bss: set gss->familyAlexander Couzens1-0/+1
gss->family (and gss->ip) was never set. So IPv6 should have never worked. Fixes: e769f5226be6 ("gprs_ns2_sns: rework IP-SNS initial remote") Related: OS#5036 Change-Id: I4e39dc5c7f766a7040645ceb62afdf6a9cfad00f
2021-06-13gprs_ns2_sns: replace ns2_sns_type with address familyAlexander Couzens1-28/+24
Reduces the code as the conversion between address family (bind) and ns2_sns_type is not needed anymore. Related: OS#5036 Change-Id: I1bcdd43af34c926d4b88491d00669422c299bef7
2021-06-13gprs_ns2_sns: refactor nss_weight_sum_data -> ip46_weight_sum_dataAlexander Couzens1-7/+7
Allow to use ip46_weight_sum_data for local endpoints as well. Related: OS#5036 Change-Id: I75d88f3da89ad13e34a3fd5ae72bd967d81f4abc
2021-06-13gprs_ns2_sns: refactor ip4_weight_sum/ip6_weight_sumAlexander Couzens1-16/+14
Use the new introduced struct ns2_sns_elems instead of the elems seperate. Related: OS#5036 Change-Id: I0956ab6085554210569188f52cae121e32fca19b
2021-06-13gprs_ns2_sns: add check for duplicates to add_ip6_elem()Alexander Couzens1-3/+8
The ip4 version already checks for duplicates. Related: OS#5036 Change-Id: I4a1528ea3427946ce465085919b2b522d3cedd07
2021-06-13msgb_alloc_headroom: Change size args to be uint16_tPau Espin Pedrol1-2/+2
Underlaying APIs (msgb_alloc) use a uint16_t as a type, which means until now passing a value > 2^16 would succeed providing a msgb with less space than requested. Since those are static inline, there's no symbols used by apps, so we should be safe enough changing the type to be uint16_t, since change would only be applied at re-compile time. Change-Id: I83c8222484e4856c68134a1a9d8cf96eb91af1b8
2021-06-11bts_feature: Introduce feature to speak to PCUPau Espin Pedrol2-0/+3
This feature provides the BSC with information on whether the BTS talks the IPAC_PROTO_EXT_PCU osmocom extension over the underlying IPA multiplex of the OML link. Related: SYS#5303 Change-Id: Id62421f7f5540875ac877a187757f2cf0556bd02
2021-06-11ipaccess: Add new osmo extension IPAC_PROTO_EXT_PCUPau Espin Pedrol1-0/+1
This new extension protocol is used to forward Osmocom PCUIF messages BSC<->BTS<->PCU. It will be sent re-using the IPA multiplex of the OML link between BSC and BTS. BTS is responsible for forwarding the message over the unix socket to the PCU. Related: SYS#5303 Change-Id: I68b04def49946b6915dbd4e476999f249751cd28
2021-06-10ns2: use same name in ctr_group as stat_item_groupMichael Iedema1-0/+1
Change-Id: I7d2dcb6c7ddf72a0d06ff19f5d0e3dc740bd19ff
2021-06-08kdf: add key derivation functionsEric16-1/+1752
generic sha code from git://w1.fi/hostap.git commit 5ea93947ca67ba83529798b806a15b247cdb2e93 which also happens to be the source of our milenage code. Related: SYS#5324 Change-Id: Ibf2e49edada944d91ceba62bd0d6b6ce69261fcd
2021-06-07stats_vty: also show rate counter group name (if present)Vadim Yanitskiy1-1/+4
Change-Id: I358f52e81a85f041fc21960634d04d18e7883dd5
2021-06-07gprs_ns2_sns: move gss->remote specific check out of add_ip4_elem/add_ip6_elemAlexander Couzens1-8/+8
Allow to use the add_ip4_elem also with local elemens. Related: OS#5036 Change-Id: Ib48dfd7567467e60c5af2348924ece5cc6124206
2021-06-07gprs_ns2_sns: use struct ns2_sns_elems in add/update/remove remote_elemsAlexander Couzens1-41/+47
Prepare to use the function also with local elems. Related: OS#5036 Change-Id: I7db43c8dbd5bd7e7b07a7d629d2615bfa18623db
2021-06-07gprs_ns2_sns: refactor ns2_clear_ipv46_entries_local to use new elems functionsAlexander Couzens1-19/+10
Related: OS#5036 Change-Id: I7225f7a4215842f0ea601ce6d9d38220f98f808d
2021-06-07gprs_ns2_sns: refactor local and remote entries into a structAlexander Couzens1-105/+102
The IPv4/IPv6 elemens are the same for local and remote. Refactor the entries into a struct to use function to manipulate them with a single function. Related: OS#5036 Change-Id: I05e053a9eb3328655502dfe2981c8f402104e292
2021-06-07gprs_ns2_sns: bss: improve validation of configurationAlexander Couzens1-1/+49
Some combination of IPv4 and IPv6 result in incomplete combination. E.g. IPv6 binds, but only IPv4 endpoints and vice versa. Related: OS#5036 Change-Id: I2fcf67bc2431ddac23c4ae23cebbb29771e573f1
2021-06-07gprs_ns2_sns: move selection of the next bind into own functionAlexander Couzens1-12/+10
It will be also used by del_bind() when removing an active bind Related: OS#5036 Change-Id: Ic39f0e5474ecc055d9a1b6a7b30777574d8b741d
2021-06-05ns2: Use NSVC bufid in stats reportPau Espin Pedrol1-0/+2
Use the new stats API to set a meaningful name for its stats group too, so that they are easily to identify when reported. Example on VTY "show stats": """ NSVC Peer Statistics (0)('UDP-NSE01800-NSVC01800'): ALIVE response time : 0 ms NS Bind Statistics: Transmit backlog length: 0 packets """ Example reporting to statsd: """ my-stats-reporter-prefix.ns.nsvc.UDP-NSE01800-NSVC01800.alive.delay:0|g """ Related: SYS#5456 Change-Id: I4c696c615ba3416f8524d0797ce06c0ecd3a18f2
2021-06-05stat,rate_ctr: Allow setting group name and use it at report timePau Espin Pedrol7-35/+67
This patch adds a new field "name" to the rate_ctr and osmo_stat_item_group structs, together with an API to set it. This new field allows for easy identification of specific group instances when several of them exists, rather than using a sometimes random/increasing index value. If set, this name (string) is used instead of the index by the stats reporter. The name, if set, is also printed during "show stats" VTY commands. It's up to the user or application to set up unique or meaningful names to fullfill one's needs. WARNING: this commit breaks ABI and possibly creates unexpected behavior when run with non-rebuilt apps which use the modified structs directly to get the coutners, or if use the static inline API rate_ctr_inc2(). Existing users of these structs should migrate to use new APIs introduced in follow-up commits instead of accessing the field directly. Related: SYS#5456 Change-Id: I0dc510783dd9ae8436dae8005a7b3330e80d36f3
2021-06-05gprs_ns2: rework id strings of nsvcsAlexander Couzens3-11/+19
Ensure all nsvcs ids are unique as UDP ids might not be unique when multiple NSVCs connect to the same remote endpoint (multiple binds). Change the format of all ids to look similiar. FR: NSE11-NSVC23-FR-fr0-DLCI13. UDP: NSE11-NSVC-UDP-10.0.0.1:23000-192.168.1.1:24000 UDP: NSE11-NSVC23-UDP-10.0.0.1:23000-192.168.1.1:24000 UDP: NSE11-NSVC66-UDP-[fd01::1]:23000-[fd03::2]:24000 Change-Id: I618e263e73fcc64d4e46b57aa3a2cb2783837907
2021-06-04fsm: cosmetic: fix weird spacing in osmo_fsm_inst_alloc()Vadim Yanitskiy1-2/+2
Change-Id: I199085e717c6ebdc9061a0c9fa9a96bfbc11a74a
2021-06-04Use new stat item/ctr getter APIsPau Espin Pedrol10-93/+93
Generated with spatch: """ @@ expression E1, E2; @@ - &E2->ctr[E1] + rate_ctr_group_get_ctr(E2, E1) """ """ @@ expression E1, E2, E3; @@ - E2->items[E1] + osmo_stat_item_group_get_item(E2, E1) """ Change-Id: I41297a8df68e28dfc6016330ac82b0ed5dd0ebc1
2021-06-03gsm_08_58.h: add mask for Osmocom specific VAMOS C-bitsVadim Yanitskiy1-0/+1
Applying and reverting this mask allows one to quickly convert between VAMOS and non-VAMOS variants of Bm/Lm C-bits. Change-Id: Ia0bd8695a3f12331b696fe69117189cdd48b584d Related: SYS#4895, OS#4941, SYS#5315, OS#4940
2021-06-01stat,rate_ctr: Introduce new API to get counter at given indexPau Espin Pedrol4-3/+25
Having this API and forcing apps to use it will allow easily adding new members to the group structure without having so much impact in users of this struct. Related: SYS#5456 Change-Id: Iebbf401f11e36645f8964d389460918eb9e0910e
2021-06-01sim: Add osim_card_{reset,close}() APIHarald Welte4-0/+56
This is required to reset and close a card under software control after opening it with osim_card_open() Change-Id: Ie9ec66db4d54fdb1331f4ae05ca3ca4274912e9d
2021-06-01range_enc_determine_range(): Don't dereference array on size=0Pau Espin Pedrol1-0/+4
cherry-picked from osmo-bsc.git d4c391eab03ce768d7eaf483368d9cda4f9668df Change-Id: Iae058c35506bc25c9f4790889b89ac46aea664b6 Fixes: Coverity CID#182710 (osmo-bsc.git)
2021-05-29fixup for gsm48_chan_mode_to_non_vamos()Neels Hofmeyr1-5/+2
When modifying chan modes, I first thought rather always fail if there is no equivalent mode. That is true for gsm48_chan_mode_to_vamos(), but for a change to non-VAMOS, rather return the unchanged mode for non-VAMOS modes, so that gsm48_chan_mode_to_non_vamos(GSM_CMODE_SIGN) works without failure. This makes more convenient checking, e.g. in osmo-bsc's lchan_fsm.c making sure that a non-VAMOS lchan has a non-VAMOS chan_mode, for all types of lchans. Change-Id: Ibf20f04d167e0e0599012ff530bc17ba8c8ab562
2021-05-26gsm_08_58.h: add Osmocom specific Bm/Lm CBITs for VAMOSVadim Yanitskiy3-0/+19
Change-Id: If33c1695922d110c0d2c60d5c0136caf2587194e Related: SYS#4895, OS#4941, SYS#5315, OS#4940
2021-05-26rsl: make rsl_dec_chan_nr() more readable, use RSL_CHAN_NR_MASKVadim Yanitskiy1-31/+23
Change-Id: I1650e3b02e79461e42499268fa8ed30bff667489
2021-05-23gprs_ns2: vty: fix removing a bind from a SNSAlexander Couzens1-0/+1
Return 0 after removing a bind. Otherwise the vty would return a wrong error to the user. Change-Id: I088c18abafc82ae7bd7b8c487ad9e9b448c42ba9
2021-05-23gprs_ns2: sns: fix del bind()Alexander Couzens1-0/+1
When the bind to be removed is not the last entry, it would remove the wrong SNS bind resulting in use-after-free memory. Change-Id: I79062d404ebba9d5c8f7f209bebde146fa08c71f
2021-05-19add RSL_CMOD_CRT_OSMO_TCH_VAMOS_Bm, RSL_CMOD_CRT_OSMO_TCH_VAMOS_LmNeels Hofmeyr1-0/+2
These indicate VAMOS mode to osmo-bts during channel activation and channel mode modification. Change-Id: Ib4ea11c870683a58f0bbc8c9cd942faceba8c4d9
2021-05-19add RSL_IE_OSMO_TRAINING_SEQUENCENeels Hofmeyr2-0/+2
This IE is explained in the Abis manual in osmo-bts commit Ie583bd5969fdb45f772af198033e58d7c4077306 http://git.osmocom.org/osmo-bts/commit/?id=823cd529649b667af20bfa4e383255a8ed5931f1 Change-Id: I573c26b1431ee789e42f6fa620947b003c384dc0
2021-05-18vty: Introduce API vty_read_config_filepPau Espin Pedrol3-5/+11
This new API doesn't use host_config_set(), and allows passing a FILE* from any source, not only a filesystem path. Related: SYS#5369 Change-Id: I720ac04386261628c0798a1bfcaa91e2490a86c3
2021-05-17vty: Implement missing public API host_config_file()Pau Espin Pedrol2-1/+6
The API was defined in the header file but not implemented. Change-Id: I69de9864fa7cd421557b9fc630beefea6d90b4cc
2021-05-08osmo_sock_*_ofd(): Mark OSMO_FD_WRITE on non-blocking connect()Harald Welte1-5/+14
When we perform a non-blocking connect, the completion of the connect will be signaled by marking the fd as WRITE-able. So we should automatically set OSMO_FD_WRITE to make sure the user gets notified on completion. Change-Id: I22964c5d5da849abcd97a900bd86ab5b4ada05da
2021-05-06gb: Fix missing exporting symbol bssgp_encode_rim_pduPau Espin Pedrol1-0/+1
The symbol was not in the list of exported symbols. Change-Id: I4c3792c3b22989571548e40aa4fbf26912e699df