This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/osmocom-commitlog@lists.osmocom.org/.
gitosis at osmocom.org gitosis at osmocom.orgThis is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "UNNAMED PROJECT". The branch, jerlbeck/master has been updated discards 0c0ed3e98aecb331d67910dad28842e97ece7ec8 (commit) via 377250f3ada5701a104ccb5388bfe13640b53eb7 (commit) via e0c734dcfe6501198e56bd3a89ad03ce5cfd5129 (commit) via 07111668d4b4bfe44c02077b29ed7d3cb8415134 (commit) via 4944c195d4a71f4aa5d9a3274907fa0bfed0e6ce (commit) via 5cd496d2083bef03d9f2425cabec9ec51c8e9254 (commit) via 83426b20a30832cbacd8165f5b757910e98f5417 (commit) via 617c7127f434fc866ca370e8812f1ad01ad90d27 (commit) via 23f93a15ca759b55b4713148f06d9a2d152278ab (commit) via ec478756ccc8e8df264811741bfc9c362cb9233d (commit) via 9380f5d2181c63922627cbff6abe5e3cd05523e4 (commit) via ac89a555fa73b24a77bb861811774a5af885bc84 (commit) via 699b8dca49df30b7bf6db7090eb416d1ad4fd46d (commit) via ccc34e4d30b962a2963c004d003d3851801ffb3b (commit) via c6d4ceeda6e6b96cc5bbc07fb9489587ed181f60 (commit) via 9ec49e2411abb206c82af643d18e2ada82e865e2 (commit) via fa464bbce90130a7f429cdbafac7b64b1fdacd32 (commit) via e565564bc9d74e09ec1962f86aa5155e97377e1f (commit) via cb1b494c896243a92a1969bb4c0ee4baddad2423 (commit) This update added new revisions after undoing existing revisions. That is to say, the old revision is not a strict subset of the new revision. This situation occurs when you --force push a change and generate a repository containing something like this: * -- * -- B -- O -- O -- O (0c0ed3e98aecb331d67910dad28842e97ece7ec8) \ N -- N -- N (377250f3ada5701a104ccb5388bfe13640b53eb7) When this happens we assume that you've already had alert emails for all of the O revisions, and so we here report only the revisions in the N branch from the common base, B. Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://cgit.osmocom.org/osmo-pcu/commit/?id=377250f3ada5701a104ccb5388bfe13640b53eb7 commit 377250f3ada5701a104ccb5388bfe13640b53eb7 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Thu Jul 2 12:54:26 2015 +0200 tbf: Add Poll Timeout counters This commits adds three poll timeout counters - RLC Assign Timeout - RLC Ack Timeout - RLC Release Timeout to help diagnosing to cause for these events. There seems to be an increased rate of these when a PDCH is shared by multiple TBFs. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=e0c734dcfe6501198e56bd3a89ad03ce5cfd5129 commit e0c734dcfe6501198e56bd3a89ad03ce5cfd5129 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Fri Jul 3 14:03:33 2015 +0200 Revert "tbf: Add GprsMs* argument to update() and use it in reuse_tbf" This reverts commit 2272a83a13b57ea7e99fe96ac76e4ad892e19e90. The modification is no longer needed, since the call to update has been removed from reuse_tbf(). Conflicts: src/tbf_dl.cpp Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=07111668d4b4bfe44c02077b29ed7d3cb8415134 commit 07111668d4b4bfe44c02077b29ed7d3cb8415134 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Mon Jun 29 13:30:43 2015 +0200 tbf: Remove call to update() in reuse_tbf Since both TBF are based on the same reservation which means that they should be compatible with respect to the slot usage, and since the new TBF has not been forced to single slot usage, an update of the allocation is not necessary now. This commit removes the call to update() from within reuse_tbf(). Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=4944c195d4a71f4aa5d9a3274907fa0bfed0e6ce commit 4944c195d4a71f4aa5d9a3274907fa0bfed0e6ce Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Thu Jul 2 12:04:20 2015 +0200 tbf: Set ms in call to tbf_alloc_dl_tbf The call to tbf_alloc_dl_tbf misses the pointer to the GprsMs object which is already known in that case (tbf_reuse). This leads to a full reallocation of the PDCH slots, which is possibly incompatible with the old set of slots. This can result in hanging TCP connections and TCP connection failures. This commit replaces the old NULL value by the actual GprsMs object. Since the set_ms() is also done within the tbf_alloc_dl_tbf method, that call is removed. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=5cd496d2083bef03d9f2425cabec9ec51c8e9254 commit 5cd496d2083bef03d9f2425cabec9ec51c8e9254 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Tue Jun 30 10:24:37 2015 +0200 alloc: Base algo A on reserved PDCHs Currently algorithm A bases its time slots selection on the number of TBF actively using the PDCHs. This statistically prefers the first time slots, especially with short living TBFs. So when the first TBF is triggered by an uplink transfer (which generally results in a short-lived TBF) the potentially longer living DL TBF will be bound to the same slot. When another MS then requests an uplink TBF, it will get the same slot (no UL TBF currently active). This commit changes the algorithm to base its selection on reserved slots instead. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=83426b20a30832cbacd8165f5b757910e98f5417 commit 83426b20a30832cbacd8165f5b757910e98f5417 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Tue Jun 30 09:44:05 2015 +0200 alloc: Ignore slots with differing TSC if multiple slots are requested According to TS 45.002, 6.4.2 the training sequence (TSC) must be the same for all slots in a multi-slot set. This commit updates find_possible_pdchs() to only consider slots with the same TSC if more that 1 slot shall be assigned. Note that the first PDCH's TSC will be used as reference, so if two or more groups with a common TSC are configured, only the first will be used. This restriction does not apply to algorithm A, since it will not assign more than one slot and therefore sets the max_slots parameter to 1. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=617c7127f434fc866ca370e8812f1ad01ad90d27 commit 617c7127f434fc866ca370e8812f1ad01ad90d27 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Tue Jun 30 09:18:30 2015 +0200 ms: Get the set of slots currently active This commits adds methods to GprsMs and gprs_rlcmac_tbf to retrieve the slots that are actively used. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=23f93a15ca759b55b4713148f06d9a2d152278ab commit 23f93a15ca759b55b4713148f06d9a2d152278ab Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Tue Jun 30 08:52:54 2015 +0200 ms: Add support for slot reservation In contrast to the slots currently used by existing TBFs, the reserved slots refer to the time slots that can be used for newly allocated TBFs without causing conflicts (given that the first common TS does not change). They correspond to the potential use of the PDCHs and can be used to achieve a more uniform slot allocation. This commit adds bit set based methods to GprsMs and gprs_rlcmac_trx and a counter to gprs_rlcmac_pdch. The current TRX will also be stored in the MS object. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=ec478756ccc8e8df264811741bfc9c362cb9233d commit ec478756ccc8e8df264811741bfc9c362cb9233d Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Fri Jun 19 16:35:38 2015 +0200 alloc: Load balancing for algo A Currently only the first enabled PDCH will be used. Beside the throughput this will also limit the number of TBFs: - number of UL TBFs <= 7 - number of DL TBFs <= 32 This commit changes the allocation algorithm to use the PDCH with the least number of attached TBFs. This will improve the troughput in both directions and the UL limits: - number of UL TBFs <= min(32, N_PDCH * 7) UL TBFs Ticket: #1794 Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=9380f5d2181c63922627cbff6abe5e3cd05523e4 commit 9380f5d2181c63922627cbff6abe5e3cd05523e4 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Fri Jul 3 12:38:59 2015 +0200 tbf: Add GprsMs* argument to update() and use it in reuse_tbf Since set_ms() is caled on the new DL TBF, the old DL TBF loses the reference to the MS object. This will lead to a segfault, when update() is called in reuse_tbf(). This commit adds an optional GprsMs* parameter to update() and uses it for the slot allocation. This fixes a TbfTest crash that would otherwise occur after applying the next commit. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=ac89a555fa73b24a77bb861811774a5af885bc84 commit ac89a555fa73b24a77bb861811774a5af885bc84 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Mon Jun 29 14:18:46 2015 +0200 ms: Add tbf() method to get the TBF based on the direction To avoid the need for a switch or conditional statement when needing a TBF from an MS object in direction independant code, this method contains that case distinction. For additional flexibility, a reverse() function is added to get the opposite direction. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=699b8dca49df30b7bf6db7090eb416d1ad4fd46d commit 699b8dca49df30b7bf6db7090eb416d1ad4fd46d Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Mon Jun 29 14:05:55 2015 +0200 ms: Add first_common_ts method to GprsMs This method gets the index (0 based) of first common time slot used by the TBFs attach to it. It expects that all of them have the same notion of this. If no TBF is attached, -1 will be returned. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=ccc34e4d30b962a2963c004d003d3851801ffb3b commit ccc34e4d30b962a2963c004d003d3851801ffb3b Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Mon Jun 29 13:45:05 2015 +0200 tbf: Maintain the number of TBF per PDCH Currently the PDCH object do not know anything about the TBFs using them. To make the slot allocation load dependant, at least some numbers are required. This commit adds TBF counters (one per direction) and the related methods attach_tbf, detach_tbf, and num_tbfs to gprs_rlcmac_pdch. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=c6d4ceeda6e6b96cc5bbc07fb9489587ed181f60 commit c6d4ceeda6e6b96cc5bbc07fb9489587ed181f60 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Mon Jun 29 13:03:46 2015 +0200 tbf/test: Add assertions These assertions check for the TBF allocation result before the tbf object is dereferenced the first time. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=9ec49e2411abb206c82af643d18e2ada82e865e2 commit 9ec49e2411abb206c82af643d18e2ada82e865e2 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Mon Jun 29 13:00:20 2015 +0200 alloc/test: Use LOGL_DEBUG environment variable When this environment variable is set, the logging level is set to LOGL_DEBUG to help debugging without putting everything into AllocTest.err. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=fa464bbce90130a7f429cdbafac7b64b1fdacd32 commit fa464bbce90130a7f429cdbafac7b64b1fdacd32 Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Mon Jun 29 12:45:11 2015 +0200 alloc/test: Enhance test_alloc_a This commit adds: - an assertion to check that count is valid, - an assertion the verify that tbf_alloc did not fail - a slots parameter to specify the enabled slots - additional test invocations with more slots enabled Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=e565564bc9d74e09ec1962f86aa5155e97377e1f commit e565564bc9d74e09ec1962f86aa5155e97377e1f Author: Jacob Erlbeck <jerlbeck at sysmocom.de> Date: Mon Jun 29 12:19:52 2015 +0200 alloc/test: Add test for successive allocation This test allocates as many as possible UL/DL TBF pairs, shows their PDCH usage, and checks how many of them has been created successfully. Sponsored-by: On-Waves ehf ----------------------------------------------------------------------- Summary of changes: src/bts.cpp | 48 ++++++ src/bts.h | 37 +++++ src/gprs_ms.cpp | 83 +++++++++- src/gprs_ms.h | 29 ++++ src/gprs_rlcmac_ts_alloc.cpp | 153 +++++++++++++++++-- src/tbf.cpp | 59 ++++++- src/tbf.h | 7 + src/tbf_dl.cpp | 6 +- tests/alloc/AllocTest.cpp | 222 ++++++++++++++++++++++++++- tests/alloc/AllocTest.err | 19 ++- tests/alloc/AllocTest.ok | 355 +++++++++++++++++++++++++++++++++++++++++++ tests/ms/MsTest.cpp | 3 + tests/tbf/TbfTest.cpp | 14 ++ tests/tbf/TbfTest.err | 210 ++++++++++++++++++++++--- tests/tbf/TbfTest.ok | 2 + 15 files changed, 1188 insertions(+), 59 deletions(-) hooks/post-receive -- UNNAMED PROJECT