openbsc.git branch jerlbeck/wip/gprs-patching updated. 0.13.0-848-gb392f1b

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.org
Mon Jan 5 10:54:10 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  3552fc0ea816e0278737a9adf02bbebc0c9f3ea9 (commit)
  discards  b350f9b704e00b517b64374023c135fe447b2f74 (commit)
  discards  d472750bb78afdabc77e6b288ef31d1631f768ce (commit)
  discards  5ae6ad4fd7365012256202177d876bbacb1be4b9 (commit)
  discards  6ec5309642f5abd26571e8f4e4e40357f624382e (commit)
  discards  9d6c76aa03956bc0176cc343f7599be2c203a734 (commit)
  discards  5487c6829df406fcbadb0ad3f76dfa197f9b49ed (commit)
  discards  fb7c1d7d4539b3a1ed5de9d5fbe0ffd0ec99e9fb (commit)
  discards  4d04272041fd00483c60135e8af978bab2d7bf28 (commit)
  discards  b887882fe3cc8df8684c64c22ab34118716612ed (commit)
  discards  73b62d9f06461dcbf98159037337846d3d239093 (commit)
  discards  0d06eb3b833f04a59621b84687c60780a40f49b1 (commit)
  discards  57548870c02cb7408d1705e4af5fa54dcc64904a (commit)
  discards  7900fcaf62feca60030c0cfa4c722b28f674258b (commit)
  discards  a86fa318ad1e420d66fc1b4f7928b9c3edea40b6 (commit)
  discards  0e593b9acc8779dc4a88ef922ec3404f4eee76fd (commit)
  discards  48136c24be1e7e0e46bfa2f2bc7c41c3a0ce8d8e (commit)
  discards  143d59838a268fdbe733f98d91998f20b9d4dcec (commit)
  discards  4196c41274448a6dc6bff78dd6545058fd9a09bd (commit)
  discards  c24ff2bf62e7ecbd6a55d413a23329d7f8837b31 (commit)
  discards  6f45250cd43d17b5ca84fd35523d3afc785ee397 (commit)
  discards  d878b4d6aac174e9a4e8cda48ba64e8204fc79d6 (commit)
  discards  01d24b8e05a63a1df4e08de2982b274b811fb60e (commit)
       via  b392f1b1448b3eebf779457577fb439d533781c0 (commit)
       via  6888fef2394ec6e7cf9fa1667f6d990ea3323d6f (commit)
       via  4bddf37c8d4b54d94e2245250bdf44916f657ea2 (commit)
       via  af31f22589327f0c4a13587ead4b0413af95f845 (commit)
       via  618dce764997e543cbe7e8b5c9313f981122f576 (commit)
       via  1793b8df0dda07fcf29baf31b3b08c748103887f (commit)
       via  fa9af3fe29a549523a30e2af521f73b2f17819da (commit)
       via  00ae6ee4ed6677cfe3ac10aeea76c1b0ac125f8e (commit)
       via  50a3edc6c2cca52464c40dd24756365acf57680d (commit)
       via  4cb99aad4fa93be5e4381152ae3fb1583b429b75 (commit)
       via  d0d5a90281f54b0ddcfa54a715b01a30045ede62 (commit)
       via  a4b21b7f740defe76e70fcb459667436da8bc8fd (commit)
       via  bcb60cccc36aa2b48abeec55081decdce623b9a1 (commit)
       via  cde63d0a32b7610ed077c7831b960c009e024b77 (commit)
       via  95961dd1282789ddd55cbf1c32ea2ce62bec61f0 (commit)
       via  694aad91d4b1bf658cfac767871ddffdcceb45d3 (commit)
       via  4fe374421b460a0f023a4fd7fda048d675d793d1 (commit)
       via  84c734570291d7e240ee473a614ac8f3af14771a (commit)
       via  65f625f2283553497d7cc0d15a4c9ca8e6656442 (commit)
       via  0163dbcc30575ee6953222183f2cbbd0741486b5 (commit)
       via  e14a5903e5d1c9fce9d13f91e9a08eeade42a541 (commit)
       via  21aa2a90ea2f5cf10fb9db75c597b8fe572c51cd (commit)
       via  5ca5699e61b2395fd822027c425be5f26bd33ac2 (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 (3552fc0ea816e0278737a9adf02bbebc0c9f3ea9)
            \
             N -- N -- N (b392f1b1448b3eebf779457577fb439d533781c0)

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/openbsc/commit/?id=b392f1b1448b3eebf779457577fb439d533781c0

commit b392f1b1448b3eebf779457577fb439d533781c0
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Tue Dec 23 14:24:16 2014 +0100

    sgsn: Handle GSUP errors (WIP)
    
    Ticket: OW#????
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=6888fef2394ec6e7cf9fa1667f6d990ea3323d6f

commit 6888fef2394ec6e7cf9fa1667f6d990ea3323d6f
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Tue Sep 30 13:53:26 2014 +0200

    sgsn: Added TODOs (TODO)
    
    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
      - remove this commit
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=4bddf37c8d4b54d94e2245250bdf44916f657ea2

commit 4bddf37c8d4b54d94e2245250bdf44916f657ea2
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
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.
    
    TODO:
      - 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

http://cgit.osmocom.org/openbsc/commit/?id=af31f22589327f0c4a13587ead4b0413af95f845

commit af31f22589327f0c4a13587ead4b0413af95f845
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Wed Dec 17 14:03:35 2014 +0100

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

http://cgit.osmocom.org/openbsc/commit/?id=618dce764997e543cbe7e8b5c9313f981122f576

commit 618dce764997e543cbe7e8b5c9313f981122f576
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 19:11:20 2014 +0100

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

http://cgit.osmocom.org/openbsc/commit/?id=1793b8df0dda07fcf29baf31b3b08c748103887f

commit 1793b8df0dda07fcf29baf31b3b08c748103887f
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 19:19:46 2014 +0100

    sgsn: Show GSUP client info on 'show sgsn'
    
    This commit adds a line to the output of 'show sgsn' if the GSUP
    client has been initialized:
    
      - Remote authorization: [not] connected to HOST:PORT via GSUP
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=fa9af3fe29a549523a30e2af521f73b2f17819da

commit fa9af3fe29a549523a30e2af521f73b2f17819da
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 19:18:54 2014 +0100

    sgsn: Send PING and eventually reconnect
    
    Currently, the reconnect mechanism relies on gsup_client_updown_cb
    which in turn gets called based on the OS' view of connection state.
    
    This patch adds a timer based PING mechanism that regularly sends
    PING messages and forces a reconnect if a PONG message won't be
    received until the next PING message is scheduled. The current ping
    interval is 20s.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=00ae6ee4ed6677cfe3ac10aeea76c1b0ac125f8e

commit 00ae6ee4ed6677cfe3ac10aeea76c1b0ac125f8e
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 19:15:55 2014 +0100

    sgsn: Handle incoming IPA CCM message in gsup_client_read_cb
    
    Currently the IPA CCM messages are not handled by the GSUP client.
    This means, that the client doesn't answer to PING and ID_GET and
    logs notices when receiving PONG or ID_ACK. At least the PING
    functionality (remotely originated PING) shall be supported.
    
    This patch extends gsup_client_read_cb by a call to
    ipaccess_bts_handle_ccm. Only when the return code is 0, the message
    is processed further and checked for an OSMO/GSUP message. ID_GET
    messages are answered by a dummy description, where only the
    unit_name is set to 'SGSN'.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=50a3edc6c2cca52464c40dd24756365acf57680d

commit 50a3edc6c2cca52464c40dd24756365acf57680d
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 19:00:56 2014 +0100

    sgsn: Handle return code of ipa_client_conn_open correctly
    
    The ipa_client_conn_open function does not distinguish between a
    connection being already established or waiting for establishment.
    In either case, the application gets informed about the connection
    state via the updown_cb. The 'up' parameter is only set, if
    poll/select consider the socket as writable.
    
    This patch handles both cases equally and fully relies on the
    updown_cb to adjust the gsupc obejct state.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=4cb99aad4fa93be5e4381152ae3fb1583b429b75

commit 4cb99aad4fa93be5e4381152ae3fb1583b429b75
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 18:50:05 2014 +0100

    sgsn: Avoid sending stale GSUP requests after reconnect
    
    Currently, messages are added to the tx queue even if the connection
    is down for some reason and all of these messages are eventually sent
    after a re-connect.  The MS has probably sent several Attach Requests
    while the connection was down and will continue doing so. Therefore
    these stored messages could be dropped.
    
    This patch clears the queue before re-connecting and also extends
    gprs_gsup_client_send to return immediately, when the connection is
    not established instead of calling ipa_client_conn_send.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=d0d5a90281f54b0ddcfa54a715b01a30045ede62

commit d0d5a90281f54b0ddcfa54a715b01a30045ede62
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Thu Dec 18 15:00:29 2014 +0100

    sgsn: Add automatic re-connect if the GSUP connection is down
    
    Currently the GSUP connection to a server is not restarted if the
    connection cannot be established or is terminated during operation.
    
    This commit adds a timer based connection mechanism, basically
    consisting of a timer callback that calls gsup_client_connect. The
    timer is eventually triggered (up == 0) or cleared (up != 0) by
    gsup_client_updown_cb. It adds calls to osmo_timer_del() to
    gsup_client_connect and gprs_gsup_client_destroy. The latter is now
    called instead of talloc_free in gprs_gsup_client_create on error to
    be on the safe side.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=a4b21b7f740defe76e70fcb459667436da8bc8fd

commit a4b21b7f740defe76e70fcb459667436da8bc8fd
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 18:30:41 2014 +0100

    sgsn/test: Extend tests to simulate lost GSUP requests
    
    This patch extends test_gmm_attach to optionally resend Attach
    Requests until the SGSN responds to it instead of calling
    OSMO_ASSERT at a few places. The test_gmm_attach_subscr_gsup_auth
    test optionally uses this feature. It is called once in either mode.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=bcb60cccc36aa2b48abeec55081decdce623b9a1

commit bcb60cccc36aa2b48abeec55081decdce623b9a1
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 18:26:09 2014 +0100

    sgsn/test: Add GMM test for a GSUP based attach procedure
    
    This test calls test_gmm_attach() where the mocked subscriber
    functions insert GSUP messages instead of manipulating the
    subscriber structure directly.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=cde63d0a32b7610ed077c7831b960c009e024b77

commit cde63d0a32b7610ed077c7831b960c009e024b77
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 18:19:50 2014 +0100

    sgsn/test: Add wrapper for gprs_subscr_rx_gsup_message
    
    Currently, several lines of boiler plate are needed around a call to
    gprs_subscr_rx_gsup_message.
    
    This patch puts all of this (including the call to
    gprs_subscr_rx_gsup_message) into a separate function.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=95961dd1282789ddd55cbf1c32ea2ce62bec61f0

commit 95961dd1282789ddd55cbf1c32ea2ce62bec61f0
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 19 18:08:48 2014 +0100

    sgsn: Add global require_update_location flag
    
    This flag is used to determine, whether the Update Location procedure
    shall be invoked. This is currently only set, when the 'remote'
    authorization policy is set. When the flag is set, sgsn_auth_update
    will not never be called directly by sgsn_auth_request, if an Attach
    Request procedure is pending, even if the remote connection fails for
    some reason.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=694aad91d4b1bf658cfac767871ddffdcceb45d3

commit 694aad91d4b1bf658cfac767871ddffdcceb45d3
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Wed Dec 17 13:20:08 2014 +0100

    sgsn: Replace subscr.authenticate by global require_authentication flag
    
    Currently the flag 'authenticate' is managed per subscriber.
    
    This patch replaces that flag by a global cfg.require_authentication
    flag that enables/disables the use of the Auth & Ciph procedure for
    every subscriber. The flag is set by the VTY, if and only if the
    authorization policy is 'remote'.
    
    The VTY command
    
      - update-subscriber imsi IMSI insert authenticate <0-1>
    
    is removed.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=4fe374421b460a0f023a4fd7fda048d675d793d1

commit 4fe374421b460a0f023a4fd7fda048d675d793d1
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Thu Dec 18 12:46:47 2014 +0100

    sgsn: Integrate the GSUP client into the SGSN
    
    This commit adds GSUP client configuration (via VTY), connection set
    up, and real message sending.
    
    The following configuration commands are added:
    
     - gsup remote-ip A.B.C.D            set server IP address
     - gsup remote-port PORT             set server TCP port
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=84c734570291d7e240ee473a614ac8f3af14771a

commit 84c734570291d7e240ee473a614ac8f3af14771a
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Thu Dec 18 12:28:21 2014 +0100

    sgsn: Add GSUP client
    
    This commit adds the client code to get subscriber information from a
    remote server. It provides an IPA over TCP connection to transmit and
    receive GSUP messages.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=65f625f2283553497d7cc0d15a4c9ca8e6656442

commit 65f625f2283553497d7cc0d15a4c9ca8e6656442
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Mon Dec 15 10:16:02 2014 +0100

    sgsn/doc: Add IPA default parameters
    
    This commit adds the default protocol identifiers that should be used
    for GSUP by the IPA protocol.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=0163dbcc30575ee6953222183f2cbbd0741486b5

commit 0163dbcc30575ee6953222183f2cbbd0741486b5
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Fri Dec 12 15:01:37 2014 +0100

    gprs: Add subscriber functions to create/handle GSUP messages
    
    This patch extends gprs_subscr_query_auth_info and
    gprs_subscr_location_update to create GSUP messages with the help of
    a static gprs_subscr_tx_gsup_message function. A corresponding
    gprs_subscr_rx_gsup_message is added which takes a messages, gets the
    subscr, and updates it accordingly.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=e14a5903e5d1c9fce9d13f91e9a08eeade42a541

commit e14a5903e5d1c9fce9d13f91e9a08eeade42a541
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Thu Dec 11 16:54:14 2014 +0100

    gprs: Add encoder/decoder for the Subscriber Update Protocol
    
    This patch adds functions to encode and decode GSUP messages. This
    does not include the layer 1 framing (IPA). The messages so far
    supported are: send_auth_info_*, update_location_*,
    location_cancellation_*.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=21aa2a90ea2f5cf10fb9db75c597b8fe572c51cd

commit 21aa2a90ea2f5cf10fb9db75c597b8fe572c51cd
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Thu Dec 11 15:14:14 2014 +0100

    sgsn/doc: Add protocol specification for remote subscriber update
    
    This is the first version of the specification for the GPRS
    Subscriber Update Protocol.
    
    Sponsored-by: On-Waves ehf

http://cgit.osmocom.org/openbsc/commit/?id=5ca5699e61b2395fd822027c425be5f26bd33ac2

commit 5ca5699e61b2395fd822027c425be5f26bd33ac2
Author: Jacob Erlbeck <jerlbeck at sysmocom.de>
Date:   Mon Jan 5 09:43:51 2015 +0100

    sgsn: Fix VTY command error handling (Coverity)
    
    Currently the result of the osmo_hexparse function in
    update_subscr_insert_auth_triplet is not handled correctly. There is
    a misplaced leading exclamation mark in a few conditional
    expressions. This effectively disables the error checks, as it is
    noticed by Coverity ("Missing parentheses" followed by "Logically
    dead code").
    
    This patch removes the exclamation marks.
    
    Fixes: Coverity CID 1260435 and CID 1260434
    Sponsored-by: On-Waves ehf

-----------------------------------------------------------------------

Summary of changes:


hooks/post-receive
-- 
The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)



More information about the osmocom-commitlog mailing list