osmo-pcu.git branch jerlbeck/master updated. 0.2-581-g0eac0b6

gitosis at osmocom.org gitosis at osmocom.org
Mon Aug 24 10:54:02 UTC 2015

This 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
       via  0eac0b6b43283fd7bcd6f8d7c7e55dc9ae97c516 (commit)
       via  c62216b4fc2a42ff9a8bea016565a772c585e2fb (commit)
       via  6835cead8c9e13fbbbb7b100a4c18a031f92421f (commit)
       via  6e013a136a9286e92cee0d9b5b4f0c7bcea5fd51 (commit)
       via  af387e2199de698fab9f180d68923c65b819be0c (commit)
       via  444bc82081be0fd371805017a4c97b01cb41c15d (commit)
       via  23c4b3f15899c0849fe63726fea404b9b215f103 (commit)
       via  af45473cd5b256e987515f0dc9d75ba604e5c5f1 (commit)
       via  cef20ae67a75b830cf024b5aa6dbc57ca9d07dbd (commit)
       via  ee31090b2e4204348f7b73d11abe2a07a723d6d8 (commit)
       via  64921d217badb9f6938453174da6c2172a1e4cc6 (commit)
       via  56f99d19c310b90632ac9b0b12898f4882f1020c (commit)
       via  e0b21f41c2b66d3f06a9aea5ac397857832671c8 (commit)
       via  257b630216f0dc702013ecc51ac680b5296ae898 (commit)
       via  e91bd3babd5c04a154f296607b401a5050dcba31 (commit)
       via  b139598b1c5bd271675386be77370899b5707d61 (commit)
       via  076f5c794d4af02914cd633671176437367b5af8 (commit)
       via  537b149828be91189f99acc8d9c655b8a75b95ee (commit)
       via  4a6fe534ce39b87f64a9b2013b654b92e6d7737d (commit)
       via  2b349b5d33050f6484c375e545ecb9f05a6be961 (commit)
      from  ebebad1c92e8bcbe28780b4d5ee2ac32091089ff (commit)

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 -----------------------------------------------------------------

commit 0eac0b6b43283fd7bcd6f8d7c7e55dc9ae97c516
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Aug 21 16:02:11 2015 +0200

    ms/vty: Show LLC queue octets and packets in both views
    Currently the per IMSI/TLLI view only shows the number of packets and
    the 'all' view does not show any LLC related information at all. This
    information was really missing while debugging on a running system.
    This commit adds the number of packets to the 'all' view and the
    number of octets to the IMSI/TLLI views.
    Sponsored-by: On-Waves ehf


commit c62216b4fc2a42ff9a8bea016565a772c585e2fb
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Aug 21 15:46:16 2015 +0200

    ms/vty: Show old TBFs
    This commit extends the 'show ms imsi|tlli' command to show the old
    TBFs, too.
    Sponsored-by: On-Waves ehf


commit 6835cead8c9e13fbbbb7b100a4c18a031f92421f
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Aug 21 15:24:02 2015 +0200

    ms: Store references to replaced TBFs in the MS object
    Currently when calling GprsMs::attach_tbf and a TBF of the same
    direction already exists, the old TBF gets detached from the MS
    Therefore that TBF object loses access to that MS object including
    for instance TLLI and IMSI.
    This leads to failing DL TBF reuses, since the downlink assigment
    cannot be sent on the PACCH later on because that must be sent on the
    old DL TBF which ms() is NULL and the new DL TBF cannot be retrieved.
    This commit fixes this bug by changing the GprsMs implementation to
    keep a list of replaced (old) TBFs. TBFs are only removed when they
    are being detached explicitely (see tbf_free and set_ms).
    tbf.cpp:741 We have a schedule for downlink assignment at uplink
    TBF(TFI=1 TLLI=0xf35a680e DIR=UL STATE=RELEASING), but there is no
    downlink TBF
    Sponsored-by: On-Waves ehf


commit 6e013a136a9286e92cee0d9b5b4f0c7bcea5fd51
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Aug 21 15:10:23 2015 +0200

    bssgp: Only call bssgp_tx_llc_discarded if the bctx exists
    While this does not happen in real use, and unset btcx can lead to
    segfaults in test cases. The other code outside of gprs_bssgp_pcu.cpp
    does not depend on bctx being non-NULL:
    Sponsored-by: On-Waves ehf


commit af387e2199de698fab9f180d68923c65b819be0c
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Aug 21 12:05:30 2015 +0200

    llist: Add missing const qualifier in llist cast method
    The missing const qualifier prevents the llist_empty() C++ wrapper
    function from being compiled successfully when it is used.
    Sponsored-by: On-Waves ehf


commit 444bc82081be0fd371805017a4c97b01cb41c15d
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Thu Aug 20 17:49:03 2015 +0200

    tbf: Use C++/talloc magic to support TBF constructors/destructors
    The TBF object are currently created by using talloc_zero/talloc_free
    directly from plain functions. Therefore C++ constructors and destructors
    are not called. So the only initialisation that is done is setting
    every member to 0. Non POD members do not have their constructors
    called either, which makes it impossible to use the current LListHead
    class for real members when the LListHead::m_back member has to be set.
    This commit changes the TBF allocation functions to call the
    corresponding C++ constructor after the call to talloc_zero and to
    register the C++ destructor with the talloc context, so that is is
    called before talloc_free actually frees the memory.
    With this change, non-POD members and custom
    constructors/desctructors can be used with gprs_rlcmac_tbf,
    gprs_rlcmac_dl_tbf, and gprs_rlcmac_ul_tbf.
    Note that this change is only a single step of the plan to turn the
    TBF classes into real C++ classes.
    Sponsored-by: On-Waves ehf


commit 23c4b3f15899c0849fe63726fea404b9b215f103
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Aug 21 15:04:39 2015 +0200

    tbf/test: Add test_tbf_dl_reuse
    This tests the usage of an existing TBF that is no longer in FLOW
    state to request a new DL TBF via the old TBF's PACCH.
    The test triggers a bug that breaks the association between both TBF
    objects, resulting in packet loss and transmission stalling.
    Sponsored-by: On-Waves ehf


commit af45473cd5b256e987515f0dc9d75ba604e5c5f1
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Aug 21 15:03:23 2015 +0200

    tbf/test: Do RLC based ack instead of just faking
    Currently the assignment is completed by manipulating the state of
    the TBF objects directly by setting the state fields to fixed values.
    This way, the PCU's code that is responsible to update the state
    accordingly is not tested.
    This commit changes this to simulate RLC Control Acknowledgement
    messages instead.
    Sponsored-by: On-Waves ehf


commit cef20ae67a75b830cf024b5aa6dbc57ca9d07dbd
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Mon Aug 24 12:00:33 2015 +0200

    tbf/test: Rename send_rlc_block to request_dl_rlc_block
    This function basically request the generation of the next downlink
    RLC block. Since this will no really send somthing to the PCU, the
    current name can be misleading.
    This commit just renames the function.
    Sponsored-by: On-Waves ehf


commit ee31090b2e4204348f7b73d11abe2a07a723d6d8
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Mon Aug 24 11:55:17 2015 +0200

    tbf/test: Simplify RLC block number handling
    The block number can always be deduced from the frame number. The
    current test code handles the block number explicitely, which makes
    the code more complex and has also led to block number errors cause
    by not wrapping the numbers (valid block numbers range from 0 to 11).
    This commit changes send_rlc_block to always compute the block number
    based on the frame number. It also turns the block_nr into an
    optionaly output-only parameter.
    Sponsored-by: On-Waves ehf


commit 64921d217badb9f6938453174da6c2172a1e4cc6
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Mon Aug 24 11:34:47 2015 +0200

    tbf/test: Add send_rlc_block function with a TBF as parameter
    The current implementation takes a lot of parameters (bts, trx_no,
    ...) that can also be taken from a TBF object.
    This commit adds an alternative variant with just takes a TBF, the fn
    (in/out), and the block number (in/out).
    Sponsored-by: On-Waves ehf


Summary of changes:
 src/bts.cpp               |   15 +-
 src/bts.h                 |    6 +
 src/cxx_linuxlist.h       |    2 +-
 src/gprs_ms.cpp           |   48 +-
 src/gprs_ms.h             |   12 +-
 src/llc.cpp               |   53 ++
 src/llc.h                 |    1 +
 src/pcu_vty_functions.cpp |   15 +-
 src/tbf.cpp               |   50 +-
 src/tbf.h                 |   11 +
 src/tbf_dl.cpp            |    5 +-
 tests/llc/LlcTest.cpp     |   51 ++
 tests/llc/LlcTest.err     |    5 +
 tests/llc/LlcTest.ok      |    2 +
 tests/llist/LListTest.cpp |    4 +
 tests/ms/MsTest.cpp       |   28 +-
 tests/ms/MsTest.err       |    2 +-
 tests/tbf/TbfTest.cpp     |  584 +++++++++++++++---
 tests/tbf/TbfTest.err     | 1456 +++++++++++++++++++++++++++++++++++++++++++--
 tests/tbf/TbfTest.ok      |    8 +
 20 files changed, 2194 insertions(+), 164 deletions(-)


More information about the osmocom-commitlog mailing list