VTY option codec-support is not used
osmo-bts has per-bts codec support flags, which are stored in struct bts_codec_conf *codec
The vty option codec-support sets those flags:
But the flags seem not to be used anywhere. In osmo_bsc_bssap.c osmo-bts decides for a codec depending on what the MS and the MSC offer. We should include those flags here as well.
#1 Updated by dexter over 2 years ago
struct gsm_bts->struct bts_codec_conf codec appears in the following locations:
src/osmo-bsc/bsc_vty.c (Sets configuration)
src/osmo-bsc/handover_decision_2.c (used by find_alternative_lchan())
tests/handover/handover_test.c (Sets some test values)
So we use those settings when we make the handover decision, but not when we do the assignment. Technically this would mean that we use gsm_bts->struct bts_codec_conf codec to restrict the values we have set up globally for the BSC in struct bsc_msc_data->struct gsm_audio_support audio_support even further.
#2 Updated by dexter over 2 years ago
- Status changed from New to In Progress
- % Done changed from 0 to 90
I have extended the unit-test so that they test the codec selection function that does all the intersection stuff. Then I integrated the codec-support flags into the decision. I will push the patch set later today.