cellular-infrastructure
/
osmo-bsc
mirror of https://gerrit.osmocom.org/osmo-bsc
9
0
Fork 0
Code Issues Releases Wiki Activity

Wrap channel state assignment in macro 

Previously we've used function so debug print always pointed to the same
place which is not very useful. Wrap it with macro so proper
file:line is printed. Also, make sure that we always change state only
through this wrapper and log only when the state has changed.

Change-Id: I21789f8021290965b61a54a2b23177ccbbfe8321
This commit is contained in:
Max 2017-09-06 18:20:36 +02:00
parent 53daffbd99
commit dc10ce90b2
3 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
include/osmocom/bsc/abis_rsl.h
View File

@ -34,6 +34,9 @@ struct gsm_bts_trx_ts;
#define GSM48_LEN2PLEN(a) (((a) << 2) | 1)
#define rsl_lchan_set_state(lch_, st_) \
rsl_lchan_set_state_with_log(lch_, st_, __BASE_FILE__, __LINE__)
int rsl_bcch_info(const struct gsm_bts_trx *trx, enum osmo_sysinfo_type si_type, const uint8_t *data, int len);
int rsl_sacch_filling(struct gsm_bts_trx *trx, uint8_t type,
const uint8_t *data, int len);
@ -74,7 +77,7 @@ uint64_t str_to_imsi(const char *imsi_str);
int rsl_release_request(struct gsm_lchan *lchan, uint8_t link_id,
enum rsl_rel_mode release_mode);
int rsl_lchan_set_state(struct gsm_lchan *lchan, int);
int rsl_lchan_set_state_with_log(struct gsm_lchan *lchan, enum gsm_lchan_state state, const char *file, unsigned line);
int rsl_lchan_mark_broken(struct gsm_lchan *lchan, const char *broken);
/* to be provided by external code */

9
src/libbsc/abis_rsl.c
View File

@ -1225,11 +1225,12 @@ int rsl_lchan_mark_broken(struct gsm_lchan *lchan, const char *reason)
return 0;
}
int rsl_lchan_set_state(struct gsm_lchan *lchan, int state)
int rsl_lchan_set_state_with_log(struct gsm_lchan *lchan, enum gsm_lchan_state state, const char *file, unsigned line)
{
DEBUGP(DRSL, "%s state %s -> %s\n",
gsm_lchan_name(lchan), gsm_lchans_name(lchan->state),
gsm_lchans_name(state));
if (lchan->state != state)
LOGPSRC(DRSL, LOGL_DEBUG, file, line, "%s state %s -> %s\n",
gsm_lchan_name(lchan), gsm_lchans_name(lchan->state), gsm_lchans_name(state));
lchan->state = state;
return 0;
}

2
src/libbsc/chan_alloc.c
View File

@ -448,7 +448,7 @@ void lchan_reset(struct gsm_lchan *lchan)
osmo_timer_del(&lchan->error_timer);
lchan->type = GSM_LCHAN_NONE;
lchan->state = LCHAN_S_NONE;
rsl_lchan_set_state(lchan, LCHAN_S_NONE);
if (lchan->abis_ip.rtp_socket) {
rtp_socket_free(lchan->abis_ip.rtp_socket);