Hello Harald Welte, Jenkins Builder,

I'd like you to reexamine a change.  Please visit


to look at the new patch set (#4).

HO: clearly mark conn penalty timer member for hodec2

The conn's penalty timers will be used only for handover decision 2, make it
clear by a sub-struct.

hodec2 will also initialize the penalty timer list on demand only, so no need
to always initialize.

The 'failures' member will also be used by handover decision 2. It was added
in this patch inadvertently, but I will not bother to move that now...

Change-Id: Ie6c2bc2b10dc424dfd94a9e11a9a62f51c48aa10
M include/osmocom/bsc/gsm_data.h
M src/libbsc/bsc_api.c
2 files changed, 5 insertions(+), 3 deletions(-)

  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/95/6495/4

diff --git a/include/osmocom/bsc/gsm_data.h b/include/osmocom/bsc/gsm_data.h
index 65fb32b..3099348 100644
--- a/include/osmocom/bsc/gsm_data.h
+++ b/include/osmocom/bsc/gsm_data.h
@@ -115,7 +115,10 @@
        struct llist_head ho_dtap_cache;
        unsigned int ho_dtap_cache_len;
-       struct penalty_timers *ho_penalty_timers;
+       struct {
+               int failures;
+               struct penalty_timers *penalty_timers;
+       } hodec2;
        /* "Codec List (MSC Preferred)" as received by the BSSAP Assignment 
Request. 3GPP 48.008
         * says:
diff --git a/src/libbsc/bsc_api.c b/src/libbsc/bsc_api.c
index 5f45218..2dc7b9a 100644
--- a/src/libbsc/bsc_api.c
+++ b/src/libbsc/bsc_api.c
@@ -278,7 +278,6 @@
        conn->lchan = lchan;
        lchan->conn = conn;
-       conn->ho_penalty_timers = penalty_timers_init(conn);
        conn->sccp.conn_id = -1;
        llist_add_tail(&conn->entry, &net->subscr_conns);
        return conn;
@@ -353,7 +352,7 @@
        /* drop pending messages */
        ho_dtap_cache_flush(conn, 0);
-       penalty_timers_free(&conn->ho_penalty_timers);
+       penalty_timers_free(&conn->hodec2.penalty_timers);

To view, visit https://gerrit.osmocom.org/6495
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie6c2bc2b10dc424dfd94a9e11a9a62f51c48aa10
Gerrit-PatchSet: 4
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofm...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder

Reply via email to