aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-04-10only log actual access control class ramping changesStefan Sperling1-2/+4
Silence log messages about no-op changes to access granted to access control classes. For example, these always occur while configuration files are being loaded. Change-Id: I37580f4058f6e3a21d338234b1e22ea46f9538c6 Related: OS#2591
2018-04-09fix initialization of acc rampingStefan Sperling1-10/+4
Remove a redundant call to acc_ramp_init() during bootstrap_bts(). ACC ramping state is already initialized during VTY config parsing, and bootstrap_bts() accidentally disabled ACC ramping again even if it was enabled in the configuration. This bug was introduced in last-minute refactoring during review of https://gerrit.osmocom.org/#/c/6324/ when the acc_ramping_enabled flag was moved from struct gsm_bts to the acc_ramp structure itself. Also remove an acc_ramp_init() call in bootstrap_rsl(). It is no longer needed as it serves no purpose other than initializing the bitmasks of barred ACCs. To ensure that ACC ramping configuration provided to the BTS via system information stays correct, we move the call to acc_ramp_start(), which has the same effect on barred ACCs, further up. Change-Id: Ib7d99c9d3d5d424e21bb7710f719c42f9aef3e95 Fixes: 6442e4327b1100ecfed7a242e397ac37b7529bea Related: OS#2591
2018-04-09fix a format string error in bts_update_t3122_chan_load()Stefan Sperling1-1/+2
In a debug log message, an unsigned 64-bit value was printed with %lu but it should be printed with PRIu64 from inttypes.h instead. Change-Id: I09bb519e68886e650647b1369fd49d1964e76c7e
2018-04-07"show timeslot": Show dynamic PDCH state also for Osmocom-style dyn PDCHHarald Welte1-1/+4
So far we only used to show the current mode of the timeslot for IPA style dynamic PDCH, but not for osmocom-style. Old output for osmocom-style dyn PDCH: BTS 0, TRX 0, Timeslot 6, phys cfg TCH/F_TCH/H_PDCH, TSC 7 NM State: Oper 'Enabled', Admin 'Unlocked', Avail 'OK' new output with this patch: BTS 0, TRX 0, Timeslot 6, phys cfg TCH/F_TCH/H_PDCH, TSC 7 (PDCH mode) NM State: Oper 'Enabled', Admin 'Unlocked', Avail 'OK' where "PDCH mode" changes from "NONE mode" to "TCH/F mode" to "TCH/H mode" depending on the current state. Change-Id: If5044d6bec76b869bd6a7c0260a4c77e229dcd0d Related: OS#3099
2018-04-07Start Dynamic PDCH Initialization after RSL is upHarald Welte3-13/+3
We used to start the dynamic PDCH initialization right after the respective OML initialization of the related timeslot. However, this is problematic as the RSL link may very well not even be up yet at that point. So let's do this at RSL link esetablishment time, not from OML. Change-Id: I8ac30ca075a1db4b624dc38a423d844d705fda7e Closes: OS#1841
2018-04-07Generate the S_L_INP_TEI_UP signal earlier.Stefan Sperling1-13/+12
The S_L_INP_TEI_UP signal was generated when the first message from a BTS arrives on the OML/RSL link, rather than when the OML/RSL link comes up. Instead, generate this signal when the link is brought up, so we intitialize state regardless of how a particular BTS behaves. Tested with osmo-bts-virtual and virtphy/mobile programs, and with a sysmobts. This was already committed in faf0982ae20001519cf20c5d6345dad490a135f2 but subsequently reverted in 383a059a123b1e0e5aab76423db47846e329f095 because it introduced a regression. The underlying problem causing this regression has now been addressed by https://gerrit.osmocom.org/#/c/7462/ (libosmo-abis commit 49917c129b1456585258b0ebe89a513ecef823a8). Change-Id: I78a152671a00b95c997830f7161fca92bb898fa3 Depends: I52f7c903212b38e9c87e4d45e52b231b6f1ae9f5 Related: OS#2719
2018-04-05vty: re-add 'timeout-ping' and 'timeout-pong' as dummy commandsNeels Hofmeyr1-0/+21
Legacy VTY commands were removed in commit c74a5616bf1c77ac40ddd92f4927dca7a1b45bc8 == I5cf3fec31cc774c902f3cfe6d16fb85ef301694a Removing the dead VTY commands currently breaks our ttcn3-bts-test and is likely to create problems with users of osmo-bsc, when their config osmo-bsc stops working with a mere upgrade. Instead, add deprecated dummy commands that don't do anything, to not break existing configs. Catch all of these legacy commands: timeout-ping <number> timeout-pong <number> timeout-ping advanced no timeout-ping no timeout-ping advanced by defining timeout-ping ARG timeout-pong ARG no timeout-ping [ARG] I verified manually that starting osmo-bsc with all the abovementioned VTY commands in the config file works (and produces the deprecation messages). Change-Id: I95a5bad1ade66ded849cfc20bebb7fc522aecc38
2018-04-02doc/examples: use NECI = 1 by defaultVadim Yanitskiy2-2/+2
Every setup should run with NECI = 1. NECI = 0 is an ancient legacy setting. Change-Id: I564cac73e610d71df5ab244e04d08dd2c5ff2324
2018-04-02cosmetic: Add fixme note for OS#3112Philipp Maier1-0/+5
Change-Id: I0c3ffb567aff08014f8fb96928077afb8c2f229c
2018-04-02cosmetic: remove dead codePhilipp Maier6-634/+0
There is a lot of dead code in osmo_bsc_msc.c that used to handle the IPA multiplexed SSCP lite A interface. - remove portions of the dead code - remove IPA Ping related VTY commands Change-Id: I5cf3fec31cc774c902f3cfe6d16fb85ef301694a
2018-03-28ctx cleanup: use non-NULL talloc ctx for osmo_init_logging2()Neels Hofmeyr11-19/+41
Fix various sanitizer complaints about memory leaks using a sanitizer build with gcc (Debian 7.3.0-12) 7.3.0. Also fix deprecation warnings on osmo_init_logging(). Depends: I216837780e9405fdaec8059c63d10699c695b360 (libosmocore) Change-Id: I970c6f8a0e36a8b63e42349dbc92baff649e5cef
2018-03-28cosmetic: gsm0408_test: drop unused arg from bts_init()Neels Hofmeyr1-7/+8
Change-Id: I0c59d7b3240cf7a8ae1677d237dca9dd364188cb
2018-03-28move init from gsm_bts_alloc_register() to gsm_bts_alloc(); fix gsm0408_testNeels Hofmeyr2-36/+36
gsm_bts_alloc() ought to do all static initialization, so that e.g. unit tests can use an initialized bts struct that is fully usable, just not registered at the gsm_network. For example, two INIT_LLIST_HEAD() and various system information data pointers (pointing to proper places within the same bts struct) should definitely never be omitted. This fixes the remaining errors of gsm0408_test's SI tests, as revealed by a sanitizer build using gcc (Debian 7.3.0-12) 7.3.0. Fixes: ../../../../src/osmo-bsc/src/libbsc/system_information.c:613:3: runtime error: null pointer passed as argument 1, which is declared to never be null Initializing the bts->si_common.rach_control results in different expected bytes for SI2. The initialized values are bts->si_common.rach_control.re = 1; /* no re-establishment */ bts->si_common.rach_control.tx_integer = 9; /* 12 slots spread - 217/115 slots delay */ bts->si_common.rach_control.max_trans = 3; /* 7 retransmissions */ bts->si_common.rach_control.t2 = 4; /* no emergency calls */ Causing a change to e5 04. 04 is t2 = 4, and the remaining values are verified by: uint8_t re :1, cell_bar :1, tx_integer :4, max_trans :2; >>> hex(1 + (9 << 2) + (3 << 6)) '0xe5' Change-Id: I13ef24bbe0f1fbe49465e67a0b67d6fd69fff3b1
2018-03-28fix gsm0408_test: properly free bts struct after each testNeels Hofmeyr2-10/+23
Add missing deletion of osmo_stat_item_group_free(bts->bts_statg). Add missing bts free after test_si_ba_ind(). Log deallocation, update expected test output. This fixes some of the errors of gsm0408_test's SI tests, as revealed by a sanitizer build using gcc (Debian 7.3.0-12) 7.3.0. Fixes: ================================================================= ==19273==ERROR: AddressSanitizer: heap-use-after-free on address 0x60d0000004b8 at pc 0x7f1561a94621 bp 0x7ffe7a7a64a0 sp 0x7ffe7a7a6498 WRITE of size 8 at 0x60d0000004b8 thread T0 #0 0x7f1561a94620 in __llist_add ../../../src/libosmocore/include/osmocom/core/linuxlist.h:75 #1 0x7f1561a94620 in llist_add ../../../src/libosmocore/include/osmocom/core/linuxlist.h:90 #2 0x7f1561a94620 in osmo_stat_item_group_alloc ../../../src/libosmocore/src/stat_item.c:141 #3 0x5574f6a19935 in gsm_bts_alloc ../../../../src/osmo-bsc/src/libbsc/gsm_data.c:728 #4 0x5574f6a12920 in bts_init ../../../../src/osmo-bsc/tests/gsm0408/gsm0408_test.c:123 #5 0x5574f6a0ddb3 in test_si2q_e ../../../../src/osmo-bsc/tests/gsm0408/gsm0408_test.c:192 #6 0x5574f6a0ddb3 in main ../../../../src/osmo-bsc/tests/gsm0408/gsm0408_test.c:824 #7 0x7f156061aa86 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x21a86) #8 0x5574f6a10339 in _start (/n/s/dev/make/osmo-bsc/tests/gsm0408/gsm0408_test+0xfc339) Change-Id: I1ebbd8cc0622ce1a061e933813829b1f770072dc
2018-03-28range_enc_arfcns: avoid runtime error on zero sizeNeels Hofmeyr1-18/+24
If size <= 1, avoid allocating arfcns_left[size / 2], which results in a zero size and causes, with gcc 7.3.0 sanitizer, runtime errors: ../../../../src/osmo-bsc/src/libbsc/arfcn_range_encode.c:95:6: runtime error: variable length array bound evaluates to non-positive value 0 ../../../../src/osmo-bsc/src/libbsc/arfcn_range_encode.c:96:6: runtime error: variable length array bound evaluates to non-positive value 0 This fixes some of the errors of gsm0408_test, as revealed by a sanitizer build using gcc (Debian 7.3.0-12) 7.3.0. Change-Id: Idab2a194fb9d7c41ed3367f935080eaae4ce367f
2018-03-27change return type of page_subscriber() to voidStefan Sperling2-21/+12
We deliberately ignore errors from page_subscriber() so there is no point in having a non-void return value. Provide more context in the error message logged if paging failed. Add a comment in an implementation override of base_grace_paging_request() in the test suite to make return value semantics more clear. Change-Id: Ie18c2ba53d2055d3eaff8c9ed939eb844af6dd2e Related: I48f5efbcddd98e15256edfca06ba0ae6acb5bab1
2018-03-27gscon: fix illegal state transitionsPhilipp Maier1-2/+1
The GSCON fsm has two illegal state transions. The transions themselves are logically legal but theur respective entries in out_state_mask are missing. - remove unnecessary transition from ST_ACTIVE to ST_ACTIVE - Add ST_CLEARING to the out_state_mask of ST_WAIT_HO_COMPL Change-Id: I165c3e8b4b268b50e5f4a482b0fe890e940637b3 Closes: OS#3109
2018-03-26bssap: Log non handled paging requestsPau Espin Pedrol2-1/+3
Return code in bssap_test is changed to ack it was sent and prevent the log from being printed. Change-Id: Ie8075d076cc34570fefce3beb577567707a29d4e
2018-03-26paging: paging_request_bts: Fix wrong return valuePau Espin Pedrol1-1/+1
Change-Id: I48f5efbcddd98e15256edfca06ba0ae6acb5bab1
2018-03-26bsc_vty: Fix uninitialized var false positive on gcc 7.3.1Pau Espin Pedrol1-3/+3
Change-Id: If2b315d76081f7ef5f2faf07c502f20196ba6a09
2018-03-26libbsc: nokia_site: Fix uninitialized return valPau Espin Pedrol1-1/+2
Fixes following compilation warning: bts_nokia_site.c:1248:9: error: ‘ret’ may be used uninitialized in this function [-Werror=maybe-uninitialized] return ret; This could happen if parameter len is passed with value 0. If we reach the end of the function, it means everything went fine, so we can return 0 instead of using ret, which is only used to store error code before returning. Change-Id: Ic60c8c632a6ddadb8f197d56cc68c2dbfc121d05
2018-03-23contrib: osmo-bsc.service: Fix osmo-mgw.service dependencyPau Espin Pedrol1-1/+1
Since a while ago, osmo-bsc works only with osmo-mgw. Change-Id: If95d20db4d379b9632342f7e552237066124ac40
2018-03-23contrib: osmo-bsc.service: Update descriptionPau Espin Pedrol1-1/+1
Change-Id: I99d02bd8bd5842580518b34865dd0cbd0433bf27
2018-03-23cosmetic: remove dead code: osmo_bsc_reset.cPhilipp Maier1-190/+0
Change-Id: I22f92541982489a0948bd1dedc008439981fb33e Related: OS#3102
2018-03-22bs11: make bs11_config build againPhilipp Maier2-0/+17
The BS-11 related tool bs11_config is currently not built due to dependancy issues added by GSCON FSM. - Re-Add bs11_config to makefile.am - Add dependencies and stubs Change-Id: I77ee8d2d36813ad98542317d003a9057c4f84e75 Related: OS#2823
2018-03-22ipaccess: make ipaccess-config build againPhilipp Maier2-0/+16
The nanobts related tool ipaccess-config is currently not built due to dependancy issues added by GSCON FSM. - Re-Add ipacces-config to makefile.am - Add dependencies and stubs Change-Id: I1d0a45bed61b45352ed305b6b1fd8d9b3d3cecae Related: OS#2823
2018-03-22gsm0408_test: drop LAI encoding testNeels Hofmeyr2-28/+0
The LAI encoding test in gsm0408_test.c uses deprecated API, and a more thorough test for LAI encoding has been added in libosmocore/test/gsm0408/. Change-Id: I8a55d717ab15ba23943a2639accaadc33a4f731a
2018-03-22vty: drop unused vty definitions (*_NODE, msc_*)Neels Hofmeyr1-14/+0
Change-Id: I2a6516756bada8cc9375422e4f93891ab8d65ca5
2018-03-22bssmap: State correct speech codec in ASSIGNMENT COMPLETEHarald Welte5-110/+91
Correctly compute the TS 48.008 "speech mode" (codec) for AMR on TCH/F. There are way too many different ways how to express a given voice codec. There are two different schemes in TS 48.008 alone, plus one on TS 48.058 and one in 04.08 / 44.018. Let's avoid unneeded conversion (that we might get wrong) and avoid storing information in a sub-struct of the lchan if we can simply derive it from the lchan at the time we need it. Also, move BSSAP related encoding/conversion functions closer to the user (osmo_bsc_bssap), rather than in libbsc. Without this patch, TCH/F with AMR was erroneously reported as TCH/H with AMR in the BSSMAP ASSIGNMENT COMPLETE. After this patch, it's reported correctly. Change-Id: I6feebfae77fdc93a7ce333a25dd9b9267c5a4a2e Related: OS#3094 Related: OS#3095 Related: OS#3096
2018-03-22GSCON FSM: Fix argument order when calling gsm0808_assign_req()Harald Welte1-2/+3
The first argument is the channel mode (codec), the second one is a flag whether or not it's a full-rate channel. Before this patch the two arguments are reversed, resulting in * all assignments being TCH/F in general * all assignments have FR1 as codec, as the full_Rate=1 is interpreted as FR1 Change-Id: I80f2a016a7646252bce8d86ed0bab236f3adbfca Related: OS#3094
2018-03-22BSSAP: document match_codec_pref() more thoroughlyHarald Welte1-2/+8
Change-Id: If08c2c2db674d6ddc43339744a78b1632d075d4a
2018-03-22BSSAP: Fix test_codec_pref() implementation for AMRHarald Welte1-1/+1
In AMR-type cahnels, the gsm0808_speech_codec doesn't only include the codec (like EFR/FR/AMR) but also the specific codec configuration, i.e. the sub-set of AMR modes. Hence, we cannot convert from a channel type (just the codec) to the full gsm0808_speech_codec and then memcmp() that with the codec config received from the MSC. Rather, we must only compare the *type*. Change-Id: I0a3f362667a689135d5b62d151d491490dfd2976 Related: OS#3094
2018-03-22cosmetic: remove old, already commented-out codePhilipp Maier1-2/+0
Change-Id: Ief994b840e3fe694b37330c46c3899746d810a39 Related: OS#2823
2018-03-22cosmetic: remove dead code and obsolete fixmesPhilipp Maier1-17/+2
Change-Id: Ib171f4b5697c04603a20ffc6ebd617260a38ccb4 Related: OS#2823
2018-03-22Cosmetic: fix missing semicolon after osmo-assertPhilipp Maier1-3/+3
- add the missing semicolong, - correct wrong indentation caused by the missing semicolon Change-Id: I9a1144b70ab82156175c6cdee32a6d22b4b53539 Related: OS#2823
2018-03-22cosmetic: fix incomplete sentence in comment.Philipp Maier1-2/+2
Change-Id: I6a04811704750cf715e72eba2959d08e761ced29 Related: OS#2823
2018-03-22cosmetic: remove needless fixme note.Philipp Maier1-8/+0
Change-Id: I4860412f8228756f7cd5669c4f6c7d845904e36b Related: OS#2823
2018-03-22cosmetic: fix argument order of forward_dtap()Philipp Maier1-9/+8
The order of the arguments for forward_dtap() differs from submit_dtap() and sigtran_send() - bring arguments into a uniform order for all three functions Change-Id: Ida77c82e600f99d690ffa1850450925359e33ae8 Related: OS#2823
2018-03-22cosmetic: fix typoPhilipp Maier1-1/+1
Change-Id: I36ed82dcb6010b331fe247a60f373d8e2e8a4bfc Related: OS#2823
2018-03-22cosmetic: remove unused enum membersPhilipp Maier2-8/+0
The enum gscon_fsm_event has three unused members. - remove GSCON_EV_RR_HO_ACC, GSCON_EV_RR_HO_COMPL, GSCON_EV_RR_HO_FAIL Change-Id: I301f7160b9ec8380849f9c9906b41c121a54f49f Related: OS#2823
2018-03-22fixup: apply mnc3 change also in ipaccess/network_listen.cNeels Hofmeyr1-3/+3
Preceding commit I38ac98a4d25159cfd4f686efbfbaf8f00625a6d8 changed the ipac_bcch_info.cgi type to osmo_cell_global_id, but did not carry out corresponding changes in ipaccess/network_listen.c. The reason this was not caught is that recent commit I68286d26e2014048b054f39ef29c35fef420cc97 adding the new gscon FSM actually disables ipaccess-config in the build. Fix the build, given that ipaccess-config were re-enabled. Change-Id: I15cd58e8ba0563ce4e42a61e79d01394b61593dd
2018-03-22cosmetic: abis_nm: use osmo_cell_global_id, parse 3-digit MNCNeels Hofmeyr2-24/+5
Instead of a local redefinition, use osmo_cell_global_id. This change is cosmetic because the decoded PLMN is currently actually never used. Change-Id: I38ac98a4d25159cfd4f686efbfbaf8f00625a6d8
2018-03-19fix an error message in bssmap_handle_paging()Stefan Sperling1-1/+1
This error message suggested that parsing a cell identifier list failed because the list was too short. While, in fact, this code has no insight into the reason behind the parsing failure. A generic error message is more appropriate. Change-Id: I033747e2183984159f1505e772d7c9494b759058 Related: OS#3073
2018-03-18contrib: jenkins.sh: Add --enable-werror flagPau Espin Pedrol1-2/+2
Change-Id: Ife9ca2a5b835dfd6d03c981dcf0e48265eb861eb
2018-03-18pcu_sock.c: Avoid breaking strict-aliasing on ptr derreferencePau Espin Pedrol1-1/+1
Fixes following warning: pcu_sock.c: In function 'pcu_rx_data_req': pcu_sock.c:406:3: error: dereferencing type-punned pointer will break strict-aliasing rules [-Werror=strict-aliasing] tlli = *((uint32_t *)data_req->data); ^ Change-Id: I5c93487c72d15b061e73ed4f9acbf2e77dec967a
2018-03-17RR: Send RR STATUS in case of unsupported/unknown messageHarald Welte3-0/+33
This fixes BSC_Tests.TC_err_84_unknown_msg Related: OS#2903 Change-Id: I7ecb48971c6a162c7f4c749d128c31d0dbc4916f
2018-03-17paging: Unify formatting of log messages with (bts=%d) prefixHarald Welte1-7/+10
Change-Id: I3eabe8b7553711a4de02fe87994163bfe8955509
2018-03-17abis_nm: Improve and fix OML loggingHarald Welte1-87/+68
This code is among the oldest surviving parts of old bsc_hack/OpenBSC, and it shows. We used arcane constructs to puzzle together log messages, attempting to switch the log level between a LOGP and LOPGC [which doesn't work], not properly used log levels in general, ... Let's fix this up by using the new abis_nm_dump_foh() function introduced in libosmocore I9b2c2afec28882b817d104d5b062651ade7aadd8 Change-Id: Ibd757836c39edd2d1ce59e528342f8923a9e75e4
2018-03-17cosmetic: Fix infinite number of formatting errors in gscon_fsm_statesHarald Welte1-64/+64
I'm not sure how this got past our review :/ Guess I need to be extra careful. Change-Id: I8724cadda774c96f13f957d6d5e708bebb81e3ba
2018-03-17tests: handover_test.c: Add missing headerPau Espin Pedrol1-0/+2
Fixes following compilation warning: osmo-bsc/tests/handover/handover_test.c:48:83: warning: ‘struct mgcp_conn_peer’ declared inside parameter list will not be visible outside of this definition or declaration int __real_mgcp_conn_modify(struct osmo_fsm_inst *fi, uint32_t parent_evt, struct mgcp_conn_peer *conn_peer); ^~~~~~~~~~~~~~ Change-Id: I1b5e91adee3daaec71122a7691a214aa8240bce2
generated by cgit v1.2.3 (git 2.25.1) at 2024-09-25 04:46:51 +0000