osmo-bts[master]: rsl: do not allow MODE MODIFY request with unsupp. codec/rate

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

dexter gerrit-no-reply at lists.osmocom.org
Fri Feb 2 15:47:28 UTC 2018


Patch Set 1:

(4 comments)

> (1 comment)

https://gerrit.osmocom.org/#/c/6097/1/include/osmo-bts/gsm_data.h
File include/osmo-bts/gsm_data.h:

Line 97: 		const struct bts_cm *cm; /* Table with supp. ch rate/mode combinations */
> if this is alway an array that needs to be as large as MAX_PCHAN_T, then wh
This pointer is not for a dynamically allocated struct. Its set by bts_model_init() on startup to a hardcoded table. (e.g. see l1_if.c in in osmo-bts-octphy) The problem I wanted to solve here is that I wanted to have an easy to fill out, static table inside the hardware specific part. The pointer is set once on initialization to give access to the data. If we want to have this as an array, we would have to copy the table on startup. If you think that makes it better understandable I can do this of course. Any ideas?


https://gerrit.osmocom.org/#/c/6097/1/src/common/bts.c
File src/common/bts.c:

Line 679: int bts_supports_cm(struct gsm_bts_role_bts *bts,
> Why not to use bool here?
I wanted it bo be uniform with bts_supports_cipher() (a few lines above) there they also used int.


Line 693: 	for (i = 0;; i++) {
> Oops, I was mistaken. As the size of bts->support.cm
Yes, I will do this when we finally decided to turn support.cm into an array. At the moment it is a pointer to an array.


Line 702: 		if (supported[i].pchan == _GSM_PCHAN_MAX)
> This could be moved to the 'for' condition...
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id9b222b7ab19ece90591718bc562b3a8c5e02023
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: dexter <pmaier at sysmocom.de>
Gerrit-HasComments: Yes



More information about the gerrit-log mailing list