sysmobts: Improve the shutdown of the DSP on exit
Issue the RfDeactivate.REQ before sending the MphClose.REQ. Ideally we would issue MphClose.REQ after the RfDeactivate.CNF but this is not possible right now. The current approach makes the following warning of the DSP go away on shutdown. This was tested with my E71 and an active silent-call using a SDCCH. DSP Warning: [ERROR] : DeviceMng_ValidateL1Handle() => Invalid layer 1 handle
This commit is contained in:
parent
305d8314bc
commit
faba73a812
|
@ -33,6 +33,7 @@ int bts_model_rsl_deact_sacch(struct gsm_lchan *lchan);
|
|||
int bts_model_rsl_mode_modify(struct gsm_lchan *lchan);
|
||||
|
||||
int bts_model_trx_deact_rf(struct gsm_bts_trx *trx);
|
||||
int bts_model_trx_close(struct gsm_bts_trx *trx);
|
||||
|
||||
void bts_model_rtp_rx_cb(struct osmo_rtp_socket *rs, const uint8_t *rtp_pl,
|
||||
unsigned int rtp_pl_len);
|
||||
|
|
|
@ -126,8 +126,10 @@ void bts_shutdown(struct gsm_bts *bts, const char *reason)
|
|||
LOGP(DOML, LOGL_NOTICE, "Shutting down BTS %u, Reason %s\n",
|
||||
bts->nr, reason);
|
||||
|
||||
llist_for_each_entry(trx, &bts->trx_list, list)
|
||||
llist_for_each_entry(trx, &bts->trx_list, list) {
|
||||
bts_model_trx_deact_rf(trx);
|
||||
bts_model_trx_close(trx);
|
||||
}
|
||||
|
||||
/* shedule a timer to make sure select loop logic can run again
|
||||
* to dispatch any pending primitives */
|
||||
|
|
|
@ -310,7 +310,7 @@ static int trx_close_compl_cb(struct gsm_bts_trx *trx, struct msgb *l1_msg)
|
|||
return 0;
|
||||
}
|
||||
|
||||
static int trx_close(struct gsm_bts_trx *trx)
|
||||
int bts_model_trx_close(struct gsm_bts_trx *trx)
|
||||
{
|
||||
struct femtol1_hdl *fl1h = trx_femtol1_hdl(trx);
|
||||
struct msgb *msg;
|
||||
|
@ -1149,6 +1149,5 @@ int bts_model_trx_deact_rf(struct gsm_bts_trx *trx)
|
|||
{
|
||||
struct femtol1_hdl *fl1 = trx_femtol1_hdl(trx);
|
||||
|
||||
trx_close(trx);
|
||||
return l1if_activate_rf(fl1, 0);
|
||||
}
|
||||
|
|
|
@ -108,6 +108,8 @@ int bts_model_rsl_deact_sacch(struct gsm_lchan *lchan)
|
|||
|
||||
int bts_model_trx_deact_rf(struct gsm_bts_trx *trx)
|
||||
{ return 0; }
|
||||
int bts_model_trx_close(struct gsm_bts_trx *trx)
|
||||
{ return 0; }
|
||||
int bts_model_check_oml(struct gsm_bts *bts, uint8_t msg_type,
|
||||
struct tlv_parsed *old_attr, struct tlv_parsed *new_attr,
|
||||
void *obj)
|
||||
|
|
|
@ -142,6 +142,8 @@ int bts_model_rsl_deact_sacch(struct gsm_lchan *lchan)
|
|||
|
||||
int bts_model_trx_deact_rf(struct gsm_bts_trx *trx)
|
||||
{ return 0; }
|
||||
int bts_model_trx_close(struct gsm_bts_trx *trx)
|
||||
{ return 0; }
|
||||
int bts_model_check_oml(struct gsm_bts *bts, uint8_t msg_type,
|
||||
struct tlv_parsed *old_attr, struct tlv_parsed *new_attr,
|
||||
void *obj)
|
||||
|
|
Loading…
Reference in New Issue