aboutsummaryrefslogtreecommitdiffstats
path: root/openbsc
diff options
context:
space:
mode:
Diffstat (limited to 'openbsc')
-rw-r--r--openbsc/include/openbsc/gprs_sgsn.h1
-rw-r--r--openbsc/src/gprs/gprs_subscriber.c14
2 files changed, 14 insertions, 1 deletions
diff --git a/openbsc/include/openbsc/gprs_sgsn.h b/openbsc/include/openbsc/gprs_sgsn.h
index 2ed46ffbe..2b94096f6 100644
--- a/openbsc/include/openbsc/gprs_sgsn.h
+++ b/openbsc/include/openbsc/gprs_sgsn.h
@@ -277,6 +277,7 @@ struct sgsn_subscriber_data {
int auth_triplets_updated;
int error_cause;
struct osmo_timer_list timer;
+ int retries;
};
#define LOGGSUBSCRP(level, subscr, fmt, args...) \
diff --git a/openbsc/src/gprs/gprs_subscriber.c b/openbsc/src/gprs/gprs_subscriber.c
index 58203bab8..8399ea137 100644
--- a/openbsc/src/gprs/gprs_subscriber.c
+++ b/openbsc/src/gprs/gprs_subscriber.c
@@ -32,6 +32,9 @@
#include <netinet/in.h>
#include <arpa/inet.h>
+#define SGSN_SUBSCR_MAX_RETRIES 3
+#define SGSN_SUBSCR_RETRY_INTERVAL 10
+
extern void *tall_bsc_ctx;
static int gsup_read_cb(struct gprs_gsup_client *gsupc, struct msgb *msg);
@@ -92,6 +95,7 @@ void gprs_subscr_start_timer(struct gsm_subscriber *subscr, unsigned seconds)
if (!subscr->sgsn_data->timer.data) {
subscr->sgsn_data->timer.cb = sgsn_subscriber_timeout_cb;
subscr->sgsn_data->timer.data = subscr_get(subscr);
+ subscr->sgsn_data->retries = 0;
}
osmo_timer_schedule(&subscr->sgsn_data->timer, seconds, 0);
@@ -107,7 +111,8 @@ static void sgsn_subscriber_timeout_cb(void *subscr_)
subscr_get(subscr);
/* Check, whether to cleanup immediately */
- if (!(subscr->flags & GPRS_SUBSCRIBER_ENABLE_PURGE))
+ if (!(subscr->flags & GPRS_SUBSCRIBER_ENABLE_PURGE) ||
+ subscr->sgsn_data->retries >= SGSN_SUBSCR_MAX_RETRIES)
goto force_cleanup;
/* Send a 'purge MS' message to the HLR */
@@ -116,6 +121,13 @@ static void sgsn_subscriber_timeout_cb(void *subscr_)
/* Purge request has been sent */
+ /* Check, whether purge is still enabled */
+ if (!(subscr->flags & GPRS_SUBSCRIBER_ENABLE_PURGE))
+ goto force_cleanup;
+
+ /* Make sure this will be tried again if there is no response in time */
+ subscr->sgsn_data->retries += 1;
+ gprs_subscr_start_timer(subscr, SGSN_SUBSCR_RETRY_INTERVAL);
subscr_put(subscr);
return;