AgeCommit message (Collapse)AuthorFilesLines
2018-02-16HO: always do async handoverNeels Hofmeyr1-4/+2
We're really only interested in async handover so far. Set ho->async to true always. Change-Id: I48715864d4b953987c86d93fe1b3a5456f0c4e0f
2018-02-16HO: cosmetic: bsc_handover_start_lchan_change(): tweak local varsNeels Hofmeyr1-3/+11
To keep this cosmetic change apart from subsequent functional changes: have a local do_assigment flag and a local network pointer. Change-Id: I09684dcf2a00e3d930e058cc1c925ad6e59b1468
2018-02-16HO: add new_lchan_type arg to bsc_handover_start()Neels Hofmeyr4-13/+16
Upcoming handover_decision_2 will want to be able to handover to a differing TCH type, hence add a parameter to bsc_handover_start(); adjust current callers to pass the old lchan type. Tweak the 'bts' argument to 'new_bts'. Change-Id: I4478ebcaada00897cc38c5a299e07661139ed3c5
2018-02-16HO: cosmetic: bsc_handover_start(): "fix" memcpy for AMR configNeels Hofmeyr1-2/+2
memcpy obviously needs the byte size, not the ARRAY_SIZE. For the uint8_t arrays, these are incidentally the same number, but semantically it should be sizeof. Change-Id: I97b6ac05192ad477b246a23ce00c5c91abfd938a
2018-02-16HO: process_meas_rep: guard against modulo zeroNeels Hofmeyr1-0/+3
Fixes: CID#182318 Change-Id: I3ad9d01cbffa287adce55b3569667b58a2d3556b
2018-02-16handover_logic.c: on HO command, send new lchan's MS powerNeels Hofmeyr1-1/+1
Change-Id: I90eda8405f2d2eddfb041ca38b34fe30c397e09d
2018-02-16handover_logic.c: always do inter-cell channel activationNeels Hofmeyr1-3/+1
In commit [1], the channel activation kind was modified out-of-context to use Immediate Assignment in case of intra-cell re-assignment. This so far hasn't worked out in tests, so instead stay with the strategy to use RSL_ACT_INTER_ASYNC also for re-assignment within the same cell. We may move to RSL_ACT_INTRA_IMM_ASS in a separate patch after proper testing. [1]: 5eaa4fb821751b70ad2b864d8765ff96bd301996, I2d91765c1f9719c64fd99426a872cecc724215db "HO: add indicators for inter-cell and async ho, use for chan act type", Note that so far, no re-assignment within cells is triggered by the current handover decision algorithm, so the effect will be really noticeable with handover decision 2 that is coming up. See feature OS#2949 which asks for implementing actual RR Assignment to handover within a cell. Related: OS#2949 Change-Id: Ia3d979094fa16620f1acc5933418797829710ba8
2018-02-15bts chan_load: ignore unusable BTSNeels Hofmeyr1-0/+5
For unconnected BTS, the channel load would report a "bogus channel load sample" every second (on RLL debug). Instead, skip unusable BTS. This follows up on commit 6cee893a0f2c4e53155a2631aff12a5f615b973d / I57e38f6d6ba3b23cc6e1f9520b90261dbb1f1cec 'Make "waiting indicator" of IMMEDIATE ASSIGN REJECT dynamic.' Change-Id: I627fe10e1f5c1f2f23745c906320871024b5c8c3
2018-02-15vty: 'show bts': fix indentingNeels Hofmeyr1-19/+19
Add two-space indent to many items. Change-Id: I34e3c8928871fc41b9551ab0502a42140ccbdda8
2018-02-15vty: 'show bts': write '(none)' if none are found.Neels Hofmeyr1-0/+4
Change-Id: I77039824b85a14b9d7bdfe6d2f717679b6886915
2018-02-14gsm_network: drop unused subscr_epxire_timerNeels Hofmeyr1-3/+0
Change-Id: I2e34ffb35e244472f8bfc993facc8d6e130f10d9
2018-02-14drop unused common.hNeels Hofmeyr3-8/+0
Change-Id: I7cf4076d7e36ae71d88e70a86d5c2d0640c1146f
2018-02-14drop libcommon-cs completelyNeels Hofmeyr26-114/+2
Change-Id: I07d4a48af3154ee4d904686f230a51b8b8a94ff9
2018-02-14common_cs.h: mv gsm_encr to gsm_data.hNeels Hofmeyr2-8/+8
This leaves common_cs.h practically empty. Leave its removal to the next patch, which removes libcommon-cs entirely (I07d4a48af3154ee4d904686f230a51b8b8a94ff9). Change-Id: Ic3233f03580aa8c0ab178dfd33e68ecab5b9f042
2018-02-14libcommon-cs: move vty bits to libbsc/bsc_vty.cNeels Hofmeyr3-209/+197
The gsm_network VTY was partly shared between libmsc and libbsc in the old openbsc.git; now osmo-bsc.git has its own copy, so merge all of it into bsc_vty.c. This leaves common_cs_vty.c practically empty; leave removal of the file to later, when we drop the entire libcommon-cs in I07d4a48af3154ee4d904686f230a51b8b8a94ff9. Note that gsmnet_from_vty() is also already declared in bsc/vty.h. Change-Id: I6f3a596f31762b48afed39a85a343c400826300f
2018-02-14libcommon_cs: move gsm48 bits to libbscNeels Hofmeyr6-63/+65
These functions were originally shared between libmsc and libbsc in the old openbsc.git; now osmo-bsc.git has its own copies, so move them into libbsc. Change-Id: Ie411c2ce8008accee54782a442d6361e50777a54
2018-02-14gsm_network: drop unused trans_listNeels Hofmeyr2-7/+0
Change-Id: I7910ce8098d5431e41409bf09429ae4221efb360
2018-02-14libcommon-cs: move gsm_network_init() into bsc_network_init()Neels Hofmeyr3-38/+15
Some part of the network init was common between libbsc and libmsc in the old openbsc.git repository. Now osmo-bsc.git is independent with its own copy of the gsm_network initialization. So move it over to libbsc. Change-Id: I8968787a5f0b078619264f0cb42349a9bc7943af
2018-02-14libcommon-cs: move a_reset.c into libbscNeels Hofmeyr3-1/+1
a_reset.c was originally used by both libmsc and libbsc in the old openbsc.git repository. Now osmo-msc.git has its own copy, and the idea of sharing libcommon-cs is no longer applicable. Move it to libbsc where it belongs. Change-Id: I30e4b5dab9c6d761d20a3a0e5b9ec2d65e64ebc5
2018-02-14drop libcommon completely, move remaining files to libbscNeels Hofmeyr19-43/+3
Move gsm_data.c and handover_cfg.c to libbsc, where they belong. This leaves libcommon utterly empty, drop it. Change-Id: I6178061fa30c7e1a4c22c29d3c8f508b1033569f
2018-02-14libcommon: join gsm_data_shared.* into gsm_data.*Neels Hofmeyr10-1794/+1736
The separation of gsm_data_shared.* from gsm_data.* historically allowed compiling parts of it into osmo-bts, which we have dropped since (osmo-bts has its own copy now). Even though gsm_data.* now becomes rather large by it, remove the legacy separation to get rid of the "shared" naming, which is no longer meaningful. A future patch might separate into meaningful smaller bits, if we get the time. Change-Id: Ie247bc492efb331871d970c56700595ad3f7e201
2018-02-14ipaccess-proxy: don't redefine tall_bsc_ctxNeels Hofmeyr1-2/+1
Curiously, ipaccess-proxy.c is able to define its own tall_bsc_ctx even though the same should already be linked from gsm_data.c. Declare it extern instead and hence use the one from gsm_data.c. A linking error actually occured as soon as I joined gsm_data_shared.c into gsm_data.c, which is how my attention was drawn to this. I assume that linking didn't necessarily pull in gsm_data.c before and a separate tall_bsc_ctx was permitted. As soon as the next commit merges those files (Ie247bc492efb331871d970c56700595ad3f7e201), it becomes impossible to define a second tall_bsc_ctx in ipaccess-proxy.c. Change-Id: I9c9c8540419876696e65a690717144d497d60fb2
2018-02-14libcommon: eliminate talloc_ctx.cNeels Hofmeyr9-56/+24
Interestingly enough, talloc_ctx_init() actually had not a single caller. While dropping it, see to it that the few contexts that are actually used in osmo-bsc.git are indeed initialized in all the main scopes. Also initialize two void* ctxts as NULL explicitly, to allocate under the NULL context in case some main scope forgets to branch it off another root context. (That's bsc, fle and paging, all others actually come in from other libraries.) Change-Id: I344a3d07e146999e154824837ed95db2b9879356
2018-02-14libcommon: eliminate socket.cNeels Hofmeyr7-158/+37
Replace calls to make_sock() with osmo_sock_init_ofd(). Shame on me for not testing every single one in practice, I hope for peer review to confirm that this should be correct... Read closely please! The IPPROTO_GRE define seems to be unused (at least in osmo-bsc.git), drop it completely. Change-Id: Ia6e4e0e1eed3328fa25b3b90be376d532ad0e56b
2018-02-14libcommon: eliminate debug.cNeels Hofmeyr17-243/+373
Provide concise log categories for each main scope. Move the complete log categories 1:1 to osmo_bsc_main.c. In bsc_nat.c, omit obviously unused log categories. In tests, omit almost all log categories, except for those explicitly tested as the expected output. Note: should any logging occur for a log category that is omitted by accident, such will end up being logged as DLGLOBAL, so it will show up and we can fix it, and it will not get lost silently. Change-Id: Ib524e49ec211662e0dfde8161495a72aa8ad76cf
2018-02-14libcommon: eliminate common_vty.cNeels Hofmeyr5-157/+129
Move bsc_vty_go_parent() to osmo_bsc_main.c and bsc_nat.c, and drop those nodes that aren't used in the respective main scope. Change-Id: I22ebb76742e9c5ab9dd608ac089a5c558aceeb36
2018-02-14libcommon: eliminate bsc_version.cNeels Hofmeyr3-33/+7
Move the copyright string into osmo_bsc_main.c. No other users of it exist. Change-Id: I7f48924f484e4e4b98be4ca8bee253e6ea9c0576
2018-02-14osmo-bsc: Add talloc context introspection via VTYHarald Welte2-0/+2
This requires libosmocore with Change-Id I43fc42880b22294d83c565ae600ac65e4f38b30d or later. Change-Id: I1245b6818eb54f5c0c8adddcd3a01ecc7363fb42
2018-02-13Make RSL connection attempts time out.Stefan Sperling4-8/+44
If a BTS/TRX does not respond to the "IPA RSL Connect" command, pretend that the BTS has sent a NACK for the connection. To ensure that osmo_timer_del(&trx->rsl_connection_timeout) is not called before this timer is initialized with osmo_timer_setup(), the E1 layer now drops incoming RSL messages from a BTS/TRX in LOCKED administrative state. We cancel the timeout if we receive an RSL Connect ACK or NACK from the BTS, and if the underlying E1 link does down. While here, add a missing message buffer free() in bts_isdn_sign_link(). The callers do not free it. Change-Id: Ia72b65a0f15f47dcb8a6f944f6c3695a4a64b923 Related: OS#2716
2018-02-06abis_rsl: do not allow SACCH in MF mode on SAPI=0Philipp Maier1-0/+15
osmo-msc does currently not check if an MS attemts to establish an SACCH link on SAPI=0. This combination is not permitted and should be dropped. - Make sure that establish indication for SAPI=0 and channel-type=SACCH is not permitted See also TTCN3 Testcase: BSC_Tests.TC_rll_est_ind_inval_sacch Change-Id: I12962072c0ea689d2b9b5c2899ffb59be9310dcb Closes: OS#2750
2018-02-06abis_rsl: permit first EstablishInd only on SAPI=0Philipp Maier1-0/+14
When the first EstablishInd (DCCH) is made by the MS, then it must by definition happen on SAPI=0. The MS may establish multiple connection on other SAPIs afterwards. At the moment osmo-bsc does not check the SAPI when an EstablishInd message is received. - For SAPIs other than 0, check if SAPI=0 has already been established See also TTCN3 testcases: BSC_Tests.TC_rll_est_ind_inval_sapi1 BSC_Tests.TC_rll_est_ind_inval_sapi3 Change-Id: Ibf4fca0bd42dfe5bc63e8b08754ebbe0a5e252c3 Closes: OS#2750
2018-02-05Add stat items for the BTS's channel load average and T3122.Stefan Sperling4-2/+28
In addition to logging the current values of a BTS's channel load average and T3122 override, maintain stat items for these values. This allows for plotting these values over time, for instance. These values show up in the VTY under 'show stats' like this: base transceiver station: Channel load average.: 25 % T3122 IMMEDIATE ASSIGNMENT REJECT wait indicator.: 32 s Change-Id: Icace0176e8b1d23d7c7b4816f7c67c65312844fa Suggested-by: laforge
2018-02-05vty: print RTP IP of lchan if actually bound; print remote (mgw) IPHarald Welte1-5/+15
Change-Id: I87840aa0f5b9c04d7736bf5f649142219853711a
2018-02-05gsm_data_shared.h: Remove unused sacch_deact member fieldHarald Welte1-1/+0
Change-Id: I806b957b7f6fbbb1206d29ceeccd401c98c26990
2018-02-05Make "waiting indicator" of IMMEDIATE ASSIGN REJECT dynamic.Stefan Sperling11-30/+143
The IMMEDIATE ASSIGN REJECT message contains a wait indicator which tells an MS requesting a channel to wait for a specified amount of time before trying to request a channel again, i.e. the wait indicator controls the T3122 timeout value in the MS. Previously, the wait indicator was fixed to 10 seconds. This is not sufficient if there are a lot of MS requesting channels because the MS will retry too soon. Instead of using a fixed value, maintain a dynamic wait indicator value based on average channel load. The load (used vs. available channels on a BTS) is sampled once per second, and once 8 samples have been collected we update a BTS-specific T3122 wait indicator based on the measured load. While the wait indicator could go up to 255 seconds, this initial implementation keeps it in the range from 10 to 128 seconds. Further experimentation and testing will show whether higher wait indicator values are desirable, if the sampling rate needs to change, or if the function mapping the load measurement to a wait indicator value should change (currently we map the load average linearly into the range [10, 128] inclusive). Change-Id: I57e38f6d6ba3b23cc6e1f9520b90261dbb1f1cec Related: OS#2592
2018-01-28update.gitignore with 'tags' files and 'deps' directoryHarald Welte1-0/+2
Change-Id: I2b51bbb4dce9757be7f2430b83b7879e24aa0df1
2018-01-28remove obsolete gsm_subscriber_connection.bts memberHarald Welte4-11/+5
This member was merely a cache for conn->lchan->ts->trx->bts, so let's avoid having to keep copies of the same data (which needs to be kept up to date). Change-Id: Id3bff8b18425ef5d45eb460ac9eb620023013ba0
2018-01-28bssmap_handle_assignm_req(): Decode channel type as first stepHarald Welte1-17/+16
The decision on how to further process the request depends significantly on the contents of the only mandatory IE of the ASSIGNMENT REQ. Let's decode it first. Change-Id: I030b5ad6f4fc33da9155c8a6061fd982312b20fb
2018-01-28cosmetic: Remove data/len variables in bssmap_handle_assignm_req()Harald Welte1-11/+6
There's nothing wrong with passing the TLVP_VAL/TLVP_LEN directly into the gsm0808 decoder functions, let's avoid variables that get assigned and used only once - particularly with such generic names. Change-Id: I8557b4a9ff4de28b76c7d618631a4700176a8669
2018-01-26Improve an error message in page_lai_and_lac()Stefan Sperling2-2/+2
This error message is now contained in a loop, and the current iteration does not know whether paging will fail entirely or if later iterations will succeed. Update the error message accordingly. This also makes the error message consistent with the one in page_cgi(). Change-Id: I8ec229702343343dedcbb242b6d0ed170c858511
2018-01-25Remove an unused variable.Stefan Sperling1-7/+0
Local variable is_lu in rsl_rx_chan_rqd() has been unused since commit 6082a377f57901302816a2e57e75a3f262b7b7e6. Change-Id: I67594afcfa4104e91352d50f40ae9f2bc20f4cd1
2018-01-24cosmetic: log prim operation as textMax1-1/+2
When logging SCCP error, log failed primitive operation as text. Change-Id: I91f739cea9f518a24fff6870f7dceab8175c9646 Related: OS#2851
2018-01-22HO: Add a penalty timer list to the subscriber connection entityAndreas Eversberg2-0/+20
This penalty timer is used to temporarily block cells where handover or assignment failed or where handover is not allowed. This is usefull to prevent repeated handover attempts to broken cells or cells that have limited allowed distance. Change-Id: I95cb7e3211b2470b773965e7aa94d8eb6c8c1a3a
2018-01-21Fix: meas_rep.c will only use valid DL measurement reportsAndreas Eversberg1-4/+19
When averaging measurements, only the valid reports are used. If there is no valid report in the averaging window at all, an error is returned. Change-Id: I33056225ead788340755e98113d72e1cbf3ebce6
2018-01-19HO: add queue to cache DTAP messages during handover/assignmentAndreas Eversberg2-0/+77
Add ho_dtap_cache to gsm_subscriber_connection, a stock msgb queue to be used with msgb_enqueue() and msgb_dequeue(). Keep a counter of queue length, to enforce a sane maximum counter for cached messages. So far a hardcoded maximum of 23 messages will be cached. Have balanced ho_dtap_cache_add() and ho_dtap_cache_flush() functions. The original patch was by jolly, but I have basically completely replaced it with the simpler msgb queue pattern. Change-Id: I6e4d93628befb3d97e5cee0343cd9f8ba0b8620c
2018-01-19HO: make bts_by_arfcn_bsic() publicNeels Hofmeyr2-2/+4
Change-Id: Ie746f76433f6b46a71a91b7714cc034c4355d993
2018-01-19HO: rename gsm_bts_neighbor() to bts_by_arfcn_bsic()Neels Hofmeyr1-14/+13
The name sounds like it is looking up neighbors, instead it simply traverses the global list of BTSes. Rename to reflect what it does. Move FIXME comment to its logical place: at the invocation of bts_by_arfcn_bsic(). Change-Id: I063870d09f782a4f18e85d87c7cd17fe660cb3fd
2018-01-19HO: Count number of free timeslot on a given BTSAndreas Eversberg2-0/+67
This is needed for handover algorithm to balance free slots and to prevent congestion of one cell, while other cells still have free capacities. Change-Id: Ic8bee8a515ee8aa9a99af71756fe60b8dd8f868b
2018-01-19HO: Change debug category at handover decision: DHO -> DHODECAndreas Eversberg1-8/+8
Change-Id: I7322b790f647b681bfe3a9c95b04516fbc7cbf88
2018-01-19HO: add handover algo 2 parameters; skip HO 1 if HO 2 is configuredNeels Hofmeyr3-12/+336
Change-Id: I8811ee8a75be09048042b511ee4bd9bc1de63976