openbsc.git branch jerlbeck/wip/gprs-patching updated. 0.13.0-882-g47c7031

gitosis at gitosis at
Fri Jan 16 08:44:24 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 "The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)".

The branch, jerlbeck/wip/gprs-patching has been updated
  discards  e0eb96d6a712a0278ecd00b7a1a9ba9685c306bf (commit)
  discards  30550cd100ef2ed127c6ac2c29b4d745fed49e35 (commit)
  discards  ea3cea021b59b8a6f9341faaf1081f031d04206c (commit)
  discards  3bcbbebcc04a1f8a6e0557bd5701b97af7015c24 (commit)
  discards  89be7c6d55347411aea23405236284f6e785eb57 (commit)
  discards  30997b9dd3040058dea7dd7c427d2c3d63db7de0 (commit)
  discards  d41ed3277136f2a35a88a8e9f04c4a52f34e47d2 (commit)
  discards  c014e1ad93e620be528c6ff0cc131be08d34a8f3 (commit)
  discards  427ecd1ebafe39fb95b0948b0f4b868d62186abb (commit)
       via  47c7031e8374c7f62fee4302ea4542db337dd805 (commit)
       via  a6f76044db12fad60f40df8711d5405956c9d121 (commit)
       via  be80c25828ed77525f4c16c8d4355aa082b549c7 (commit)
       via  de0a00f169dea08e6a9382d8c0b0954ee4d4dfbc (commit)
       via  de07f985b57ac534764d9020bac4d2e9550f3cb1 (commit)
       via  bb23b837207f293b8e375f44fe5c24a1172036f5 (commit)
       via  209b4e36e3e642b0986bbacf3f41c496e4a27a43 (commit)
       via  4a1139a07134956d1c68472381b6bebc5d29bb93 (commit)
       via  435ac559683d4813c6f4ae9896deae60dc6d7f09 (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 (e0eb96d6a712a0278ecd00b7a1a9ba9685c306bf)
             N -- N -- N (47c7031e8374c7f62fee4302ea4542db337dd805)

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

commit 47c7031e8374c7f62fee4302ea4542db337dd805
Author: Jacob Erlbeck <jerlbeck at>
Date:   Tue Sep 30 13:53:26 2014 +0200

    sgsn: Added TODOs (TODO)
      - fix and remove TODOs
      - add test cases for Detach(reattach)
      - add test cases for PDP context deletion (OW#1311)
      - add test cases for Cancel pending timer in sgsn_mm_ctx_free
      - implement age based MM context expiry
      - remove this commit
    Sponsored-by: On-Waves ehf

commit a6f76044db12fad60f40df8711d5405956c9d121
Author: Jacob Erlbeck <jerlbeck at>
Date:   Fri Oct 17 10:08:02 2014 +0200

    sgsn: Do not die in _bssgp_tx_dl_ud on TLLI mismatch (TODO)
    Currently an OSMO_ASSERT fails, if the message's and the mmctx's
    TLLIs do not match.
    This commit turns the assertion into an ERROR log message and uses
    the default values for IMSI, DRX, and RA CAP instead of the MM
    context values in this case.
      - find the real source for the problem
      - add test cases derived from what is being described in the ticket
    Ticket: OW#1322
    Sponsored-by: On-Waves ehf

commit be80c25828ed77525f4c16c8d4355aa082b549c7
Author: Jacob Erlbeck <jerlbeck at>
Date:   Wed Dec 17 14:03:35 2014 +0100

    sgsn: Add PDP info to subscriber data (TODO)
      - commit message
      - raw binary representation ???
      - tests
    Sponsored-by: On-Waves ehf

commit de0a00f169dea08e6a9382d8c0b0954ee4d4dfbc
Author: Jacob Erlbeck <jerlbeck at>
Date:   Fri Dec 19 19:11:20 2014 +0100

    sgsn: Add logging to connect_timer_cb (TODO)
      - LOGLevel ?
    Sponsored-by: On-Waves ehf

commit de07f985b57ac534764d9020bac4d2e9550f3cb1
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 15 18:51:31 2015 +0100

    sgsn: Fix P-TMSI generator's distance of equal values
    Currently sgsn_alloc_ptmsi uses rand() to get a new P-TMSI and then
    sets to upper 2 MSB. Therefore there is no lower limit of the
    distance between 2 identical P-TMSI.
    This patch changes the implementation to discard any random value
    above 2^30 and to generate a new random number in that case until a
    fitting number is found (or a repetition limit is reached). This way,
    all number below 2^30 within the PRNG's period are used.
    Ticket: OW#1362
    Sponsored-by: On-Waves ehf

commit bb23b837207f293b8e375f44fe5c24a1172036f5
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 8 15:29:01 2015 +0100

    gprs: Support the full cancellation procedure
    Currently no GSUP LocationCancellationResult message is sent back to
    the peer (HLR), if the procedure succeeded at the SGSN's side.
    This patch adds the missing message and put the whole request
    handling of this procedure into a separate function.
    Ticket: OW#1338
    Sponsored-by: On-Waves ehf

commit 209b4e36e3e642b0986bbacf3f41c496e4a27a43
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 15 17:50:16 2015 +0100

    gprs: Don't create a subscr entry on InsertSubscriberData
    Currently gprs_subscr_rx_gsup_message creates a subscriber entry if
    such an entry doesn't exist for the IMSI within an
    InsertSubscriberData GSUP message. This behaviour is not compliant to
    GSM 09.02, (Subscriber data management/SGSN) where it is
    defined, that an error ("Unidentified subscriber") shall be returned.
    This patch removes the case distinction, so that an existing
    subscriber entry is required for all incoming GSUP messages.
    Sponsored-by: On-Waves ehf

commit 4a1139a07134956d1c68472381b6bebc5d29bb93
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 15 17:08:30 2015 +0100

    gprs: Add replies for all GSUP requests
    Currently, an incoming GSUP request message isn't answered at all if
    it is not handled due to an error or missing implementation.
    This patch adds GSUP error replies for these requests (and only for
    requests). It also adds tests for these cases.
    Note that several of these tests check for
    GMM_CAUSE_MSGT_NOTEXIST_NOTIMPL, which will have to be changed, when
    the features are implemented.
    Sponsored-by: On-Waves ehf

commit 435ac559683d4813c6f4ae9896deae60dc6d7f09
Author: Jacob Erlbeck <jerlbeck at>
Date:   Thu Jan 8 16:23:25 2015 +0100

    gprs: Block other GSUP procedures during PURGE_MS
    GSM 09.02, mandates that no other MAP procedures shall be
    started until the PURGE_MS procedure has been completed.
    This patch implements this by adding corresponding state and checks
    to gprs_subscr_purge, gprs_subscr_location_update, and
    gprs_subscr_update_auth_info. If an Update Location or a Send Auth
    Info Req procedure is not started because of blocking, the retry
    mechanism is aborted to shorten the blocking time. The outstanding
    Purge MS procedure itself is not aborted.
    Sponsored-by: On-Waves ehf


Summary of changes:
 openbsc/include/openbsc/gprs_sgsn.h |  4 ++
 openbsc/src/gprs/gprs_subscriber.c  | 54 ++++++++++++----------
 openbsc/tests/sgsn/sgsn_test.c      | 92 +++++++++++++++++++++++++++++++++++++
 openbsc/tests/sgsn/sgsn_test.ok     |  1 +
 4 files changed, 126 insertions(+), 25 deletions(-)

The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)

More information about the osmocom-commitlog mailing list