osmocom-net-gprs

osmocom-net-gprs@lists.osmocom.org

483 discussions

bug in wireshark decoder of osmo-pcu
by jolly 11 Oct '12

11 Oct '12

hi, i get wrong decoding of RLCMAC control block. the decoder of osmo-pcu is decoding following sequence: 0x40,0x16,0x76,0x67,0x74,0x02,0x26,0x64,0xe8,0x65,0x64,0x69,0x00,0x3e,0x4c,0x00,0x2b,0x2b,0x2b,0x2b,0x2b,0x2b,0x2 this is the result: PayloadType = 1 | spare = 0 | R = 0 | MESSAGE_TYPE = 5 | Exist_ACCESS_TYPE = 1 | ACCESS_TYPE = 0 | : ID | Choice PacketResourceRequestID = 1 | u.TLLI = 0xd99dd008 | : End ID | Exist_MS_Radio_Access_capability = 1 | : MS_Radio_Access_capability | MS_RA_capability_value[0] { | Choice MS_RA_capability_value_Choice = 3 | u.Content length = 25 ... at this point, the length of the content is 25 bits: | RF_Power_Capability = 1 | Exist_A5_bits = 1 | A5_bits = 80 | ES_IND = 1 | PS = 1 | VGCS = 0 | VBS = 0 | Exist_Multislot_capability = 1 | : Multislot_capability | Exist_HSCSD_multislot_class = 0 | Exist_GPRS_multislot_class = 1 | GPRS_multislot_class = 12 | GPRS_Extended_Dynamic_Allocation_Capability = 1 | Exist_SM = 0 ... at this point all 25 bits are decoded, so the decoder must abort decoding of content of Multislot_capability_t (see gsm_rlcmac.cpp). instead, it continues with the data found after these 25 bits: (all crap from now on) | Exist_ECSD_multislot_class = 0 | Exist_EGPRS_multislot_class = 0 | Exist_DTM_GPRS_multislot_class = 1 | DTM_GPRS_multislot_class = 2 | Single_Slot_DTM = 1 | : DTM_EGPRS_Params | Exist_DTM_EGPRS_multislot_class = 0 | : End DTM_EGPRS_Params | : End Multislot_capability | Exist_Eight_PSK_Power_Capability = 0 | COMPACT_Interference_Measurement_Capability = 1 | Revision_Level_Indicator = 0 | UMTS_FDD_Radio_Access_Technology_Capability = 0 | UMTS_384_TDD_Radio_Access_Technology_Capability = 0 | CDMA2000_Radio_Access_Technology_Capability = 0 | UMTS_128_TDD_Radio_Access_Technology_Capability = 0 | GERAN_Feature_Package_1 = 0 | Exist_Extended_DTM_multislot_class = 0 | Modulation_based_multislot_class_support = 0 | Exist_HighMultislotCapability = 0 | Exist_GERAN_lu_ModeCapability = 0 | GMSK_MultislotPowerProfile = 3 | EightPSK_MultislotProfile = 3 | MultipleTBF_Capability = 1 | DownlinkAdvancedReceiverPerformance = 0 | ExtendedRLC_MAC_ControlMessageSegmentionsCapability = 1 | DTM_EnhancementsCapability = 0 | Exist_DTM_GPRS_HighMultislotClass = 0 | PS_HandoverCapability = 1 | MS_RA_capability_value[0] } | MS_RA_capability_value[0] { | Choice MS_RA_capability_value_Choice = 0 | u.Content length = 0 | RF_Power_Capability = 2 | Exist_A5_bits = 1 | A5_bits = 50 | ES_IND = 1 | PS = 0 | VGCS = 1 | VBS = 1 | Exist_Multislot_capability = 0 | Exist_Eight_PSK_Power_Capability = 0 | COMPACT_Interference_Measurement_Capability = 1 | Revision_Level_Indicator = 0 | UMTS_FDD_Radio_Access_Technology_Capability = 1 | UMTS_384_TDD_Radio_Access_Technology_Capability = 0 | CDMA2000_Radio_Access_Technology_Capability = 1 | UMTS_128_TDD_Radio_Access_Technology_Capability = 1 | GERAN_Feature_Package_1 = 0 | Exist_Extended_DTM_multislot_class = 0 | Modulation_based_multislot_class_support = 1 | Exist_HighMultislotCapability = 0 | Exist_GERAN_lu_ModeCapability = 1 | GERAN_lu_ModeCapability = 6 | GMSK_MultislotPowerProfile = 1 | EightPSK_MultislotProfile = 1 | MultipleTBF_Capability = 0 | DownlinkAdvancedReceiverPerformance = 3 | ExtendedRLC_MAC_ControlMessageSegmentionsCapability = 0 | DTM_EnhancementsCapability = 0 | Exist_DTM_GPRS_HighMultislotClass = 1 | DTM_GPRS_HighMultislotClass = 2 | : DTM_EGPRS_HighMultislotClass | Exist_DTM_EGPRS_HighMultislotClass = 1 | : End DTM_EGPRS_HighMultislotClass | : End MS_Radio_Access_capability | there are two problems with the decoder: - it does not check if the length has been exceeded while decoding Multislot_capability_t content. if the length is lower than all elements in Multislot_capabilit_t, the decoder must abort decoding the content. this is no bug. (the definition used at that point should be M_NEXT_EXIST_OR_NULL instead of M_NEXT_EXIST, see gsm_rlcmac.cpp) - even if the correct definition is used, the csn1 decoder will not use the length given at "u.Content length" to abort. instead it checks for reaching total length of coded data. i played a bit with the code, but could not fix it without breaking other things. but decoding with wireshark works. would it be possible to port latest wireshark code? regards, andreas

3 3

gprs PACCH downlink assignment problem
by Vladimir Rolbin 09 Oct '12

09 Oct '12

Hi, My set is openbts-p2.8-gprs-exp/osmo-pcu-jolly. I encounter the problem with PACCH downlink assignment: If the difference between FNs of DL_PACKET_DOWNLINK_ASSIGNMENT and corresponding UL_PACKET_CONTROL_ACKNOWLEDGEMENT is expected (RRBP = 0) 13 everything is OK (OpenBTS log): 1347379490.099008 3031505776: [ PCU -> BTS ] PACCH FN 2536893 FN52 21 CNTRL1 raw=(4f0800000c03400a0800802b2b2b2b2b2b2b2b2b2b2b2b) DL_PACKET_DOWNLINK_ASSIGNMENT 1347379490.118500 3063704432: [ BTS -> PCU ] PhDataInd: FN 2536885 FN52 13 DATA raw=(100101cc741f3203c005650000014500003c00010000ff) 1347379490.136108 3063704432: [ BTS -> PCU ] PhDataInd: FN 2536889 FN52 17 DATA raw=(080105cc741f3228e2a8c8aa0100000100000000000003) 1347379490.154624 3063704432: [ BTS -> PCU ] PhDataInd: FN 2536893 FN52 21 DATA raw=(040107cc741f32777777066b616e6e656c036f72670000) 1347379490.215582 3063704432: [ BTS -> PCU ] PhDataInd: FN 2536906 FN52 34 CNTRL1 raw=(400731d07ccb2b2b2b2b2b2b2b2b2b2b2b2b2b2b2b2b2b)UL_PACKET_CONTROL_ACKNOWLEDGEMENT But it's not always the case 1349788219.582282 3043367792: [ PCU -> BTS ] PACCH FN 2189728 FN52 8 CNTRL1 raw=(4f0800000c02800a0d00802b2b2b2b2b2b2b2b2b2b2b2b) DL_PACKET_DOWNLINK_ASSIGNMENT 1349788219.601978 3064179568: [ BTS -> PCU ] PhDataInd: FN 2189720 FN52 0 DATA raw=(040103ae3558373751f40103e800101773022a85464200) 1349788219.620378 3064179568: [ BTS -> PCU ] PhDataInd: FN 2189724 FN52 4 DATA raw=(0001042dae355837026a4e0432100000258ccc2b2b2b2b) 1349788219.638881 3064179568: [ BTS -> PCU ] PhDataInd: FN 2189728 FN52 8 DATA raw=(040103ae3558373751f40103e800101773022a85464200) 1349788219.680973 3064179568: [ BTS -> PCU ] PhDataInd: FN 2189737 FN52 17 CNTRL1 raw=(4006b8d560df2b2b2b2b2b2b2b2b2b2b2b2b2b2b2b2b2b) UL_PACKET_CONTROL_ACKNOWLEDGEMENT diff FN = 9, that causes the following behaviour in pcu: <0002> gprs_rlcmac_data.cpp:1753 TBF: START TFI: 0 TLLI: 0xae355837 Packet Downlink Assignment (PACCH) <0004> gprs_rlcmac_data.cpp:1442 Complete DL frame for TBF=0: len=9 <0002> gprs_rlcmac_data.cpp:281 PACKET CONTROL ACK with unknown FN=2163607 TLL=0xae355837 (TRX 0 TS 7) <0002> gprs_rlcmac_data.cpp:100 Poll timeout for UL TBF=0 <0002> gprs_rlcmac_data.cpp:107 - Timeout for polling PACKET CONTROL ACK for PACKET UPLINK ACK Any ideas? Is it the known issue? Regards, Vladimir Rolbin

1 0

packet paging procedure is gone.
by jolly 06 Oct '12

06 Oct '12

hi, just experiences some problems with my jolly branch (osmo-pcu): commit e6228b34a75efcb6b0700ac29672d62539860fbf Author: Andreas Eversberg <jolly(a)eversberg.eu> Date: Tue Jul 3 13:36:03 2012 +0200 TBF acknowledged mode finished for both link directions commit c7e7f6868b6f24346424dee904f4e76d3f216ff4 Author: Ivan Kluchnikov <kluchnikovi(a)gmail.com> Date: Fri Jun 29 22:53:15 2012 +0400 Implemented Paging procedure on CCCH. Added functions: - gprs_bssgp_pcu_rx_paging_ps() for handling paging message from BSSGP; - write_paging_request() for writing paging request message; - gprs_rlcmac_paging_request() and pcu_l1if_tx_pch() for sending paging request message to BTS. the lower commit adds paging procedure, but if i diff between these commits, i see that my upper commit removes the paging procedure. but i don't see it, if i "git show e6228b...". any ideas? regards, andreas

1 0

Jolly branch build is broken, please check the jenkins installation for details
by Holger Hans Peter Freyther 24 Sep '12

24 Sep '12

Hi, please check jenkins.osmocom.org for details. In general it is a good idea to subscribe to the RSS log of build failures. holger -- - Holger Freyther <hfreyther(a)sysmocom.de> http://www.sysmocom.de/ ======================================================================= * sysmocom - systems for mobile communications GmbH * Schivelbeiner Str. 5 * 10439 Berlin, Germany * Sitz / Registered office: Berlin, HRB 134158 B * Geschaeftsfuehrer / Managing Directors: Holger Freyther, Harald Welte

2 1

Multislot finished and well tested
by jolly 26 Jul '12

26 Jul '12

hi, i like to announce that multislot supports has been finished and is working well. it supports all classes, but i could not test other classes than semi-duplex class 12. the algorithm allocates as many downlink slots as allowed by class and as avaiable. it will only allocate a single uplink slot. there is no dynamic dl/up allocation depending on traffic. the code is located at osmo-pcu: jolly branch: git log 4b470ffe regards, andreas

4 16

talloc, malloc and memory leaks
by Holger Hans Peter Freyther 25 Jul '12

25 Jul '12

Hi, this is not meant as an intensive review on memory allocations in the PCU code. While browsing the code of the ready-to-send method to select the next payload I noticed the code below: RlcMacDownlink_t * mac_control_block = (RlcMacDownlink_t *)malloc(sizeof(RlcMacDownlink_t)); LOGP(DRLCMAC, LOGL_DEBUG, "+++++++++++++++++++++++++ TX : Packet Paging Request +++++++++++++++++++++++++\n"); decode_gsm_rlcmac_downlink(pag_vec, mac_control_block); LOGPC(DCSN1, LOGL_NOTICE, "\n"); LOGP(DRLCMAC, LOGL_DEBUG, "------------------------- TX : Packet Paging Request -------------------------\n"); bitvec_free(pag_vec); This and other all other places really should use talloc to make finding memory leaks more easy (e.g. by printing the leak report), the second part is that we appear to leak this on every paging message sent by the PCU and the third is actually a question. Should we use a GCC extension that helps dealing with local variables? GCC can call a cleanup function on exit of the scope, an example can be seen here[1] nad this[2] is the description of the feature. holger [1] https://mail.gnome.org/archives/gtk-devel-list/2011-November/msg00050.html [2] http://gcc.gnu.org/onlinedocs/gcc/Variable-Attributes.html#Variable-Attribu… -- - Holger Freyther <hfreyther(a)sysmocom.de> http://www.sysmocom.de/ ======================================================================= * sysmocom - systems for mobile communications GmbH * Schivelbeiner Str. 5 * 10439 Berlin, Germany * Sitz / Registered office: Berlin, HRB 134158 B * Geschaeftsfuehrer / Managing Directors: Holger Freyther, Harald Welte

2 2

Compiling with clang --analyze
by Holger Hans Peter Freyther 25 Jul '12

25 Jul '12

Hi, one more email for tonight. Using clang/smatch from time to time can highlight certain issues. The easiest way to invoke it is this: $ make CC="clang --analyze" CXX="clang++ --analyze" 1.) gprs_bssgp_pcu.cpp:241:6: warning: Access to field 'state' results in a dereference of a null pointer (loaded from variable 'bctx') if (bctx->state & BVC_S_BLOCKED && pdu_type != BSSGP_PDUT_STATUS) ^~~~ the handling of bctx is a bit weird, in theory it can be NULL but I am not sure if we are likely to receive the messages that would make the PCU crash though. gprs_bssgp_pcu_rcvmsg can call the above function/line with a NULL bctx. 2.) gprs_rlcmac.cpp:728:25: warning: Assigned value is garbage or undefined tbf->dir.ul.usf[ts] = usf[ts]; ^ ~~~~~~~ Probably true, alloc_algorithm_b is really too big to be readable to verify that this is not a false positive. cheers holger -- - Holger Freyther <hfreyther(a)sysmocom.de> http://www.sysmocom.de/ ======================================================================= * sysmocom - systems for mobile communications GmbH * Schivelbeiner Str. 5 * 10439 Berlin, Germany * Sitz / Registered office: Berlin, HRB 134158 B * Geschaeftsfuehrer / Managing Directors: Holger Freyther, Harald Welte

2 1

Re: Multislot finished and well tested
by Andreas Eversberg 24 Jul '12

24 Jul '12

Holger Hans Peter Freyther wrote: > in the above it is possible that !poll_tbf and usf == 0x7, is this the > right behavior in case no TBF could be found? hi holger, if poll_tbf is set, we must use an unassigned usf, because only the polled mobile may transmit. (usf 7 is never assigned.) if poll_tbf is not set and if no uplink ressource is required, i use usf 7 (any other usf would also be ok, since we don't need any mobile to transmit on uplink, but i use 7, because it is never assigend to any mobile). regards, andreas

1 0

[PATCH] misc: Add the libosmocore CFLAGS to the CPPFLAGS
by Holger Hans Peter Freyther 23 Jul '12

23 Jul '12

From: Holger Hans Peter Freyther <holger(a)freyther.de> libosmocore might not be in the standard include path, add the CFLAGS to the preprocessor flags. This is fixing the build on the Osmocom Jenkins. --- src/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Makefile.am b/src/Makefile.am index 041831f..98574fa 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -18,7 +18,7 @@ # along with this program. If not, see <http://www.gnu.org/licenses/>. # -AM_CPPFLAGS = $(STD_DEFINES_AND_INCLUDES) +AM_CPPFLAGS = $(STD_DEFINES_AND_INCLUDES) $(LIBOSMOCORE_CFLAGS) $(LIBOSMOGB_CFLAGS) $(LIBOSMOGSM_CFLAGS) AM_CXXFLAGS = -Wall -ldl -pthread noinst_LTLIBRARIES = libgprs.la -- 1.7.10.4

2 1

[PATCH] misc: Fix typos in the comments
by Holger Hans Peter Freyther 23 Jul '12

23 Jul '12

From: Holger Hans Peter Freyther <holger(a)freyther.de> ressource -> resorce shure -> sure --- src/gprs_bssgp_pcu.cpp | 4 ++-- src/gprs_rlcmac.cpp | 14 +++++++------- src/gprs_rlcmac.h | 2 +- src/gprs_rlcmac_data.cpp | 10 +++++----- src/gprs_rlcmac_sched.cpp | 20 ++++++++++---------- src/tbf.txt | 2 +- 6 files changed, 26 insertions(+), 26 deletions(-) diff --git a/src/gprs_bssgp_pcu.cpp b/src/gprs_bssgp_pcu.cpp index 217acdb..dc8e454 100644 --- a/src/gprs_bssgp_pcu.cpp +++ b/src/gprs_bssgp_pcu.cpp @@ -190,7 +190,7 @@ int gprs_bssgp_pcu_rx_dl_ud(struct msgb *msg, struct tlv_parsed *tp) // Create new TBF (any TRX) tfi = tfi_alloc(GPRS_RLCMAC_DL_TBF, &trx, &ts, use_trx, first_ts); if (tfi < 0) { - LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH ressource\n"); + LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH resource\n"); /* FIXME: send reject */ return -EBUSY; } @@ -198,7 +198,7 @@ int gprs_bssgp_pcu_rx_dl_ud(struct msgb *msg, struct tlv_parsed *tp) tbf = tbf_alloc(tbf, GPRS_RLCMAC_DL_TBF, tfi, trx, ts, ms_class, ss); if (!tbf) { - LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH ressource\n"); + LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH resource\n"); /* FIXME: send reject */ return -EBUSY; } diff --git a/src/gprs_rlcmac.cpp b/src/gprs_rlcmac.cpp index d1f0b54..5284260 100644 --- a/src/gprs_rlcmac.cpp +++ b/src/gprs_rlcmac.cpp @@ -80,7 +80,7 @@ llist_head *gprs_rlcmac_tbfs_lists[] = { }; void *rlcmac_tall_ctx; -/* FIXME: spread ressources over multiple TRX. Also add option to use same +/* FIXME: spread resources over multiple TRX. Also add option to use same * TRX in case of existing TBF for TLLI in the other direction. */ /* search for free TFI and return TFI, TRX and first TS */ int tfi_alloc(enum gprs_rlcmac_tbf_direction dir, uint8_t *_trx, uint8_t *_ts, @@ -264,7 +264,7 @@ struct gprs_rlcmac_tbf *tbf_alloc(struct gprs_rlcmac_tbf *old_tbf, else rc = bts->alloc_algorithm(old_tbf, tbf, bts->alloc_algorithm_curst); - /* if no ressource */ + /* if no resource */ if (rc < 0) { talloc_free(tbf); return NULL; @@ -272,7 +272,7 @@ struct gprs_rlcmac_tbf *tbf_alloc(struct gprs_rlcmac_tbf *old_tbf, /* assign control ts */ tbf->control_ts = 0xff; rc = tbf_assign_control_ts(tbf); - /* if no ressource */ + /* if no resource */ if (rc < 0) { talloc_free(tbf); return NULL; @@ -767,12 +767,12 @@ void tbf_free(struct gprs_rlcmac_tbf *tbf) LOGP(DRLCMAC, LOGL_ERROR, "Software error: Pending uplink " "assignment. This may not happen, because the " "assignment message never gets transmitted. Please " - "be shure not to free in this state. PLEASE FIX!\n"); + "be sure not to free in this state. PLEASE FIX!\n"); if (tbf->dl_ass_state != GPRS_RLCMAC_DL_ASS_NONE) LOGP(DRLCMAC, LOGL_ERROR, "Software error: Pending downlink " "assignment. This may not happen, because the " "assignment message never gets transmitted. Please " - "be shure not to free in this state. PLEASE FIX!\n"); + "be sure not to free in this state. PLEASE FIX!\n"); tbf_timer_stop(tbf); while ((msg = msgb_dequeue(&tbf->llc_queue))) msgb_free(msg); @@ -802,9 +802,9 @@ int tbf_update(struct gprs_rlcmac_tbf *tbf) tbf_unlink_pdch(tbf); rc = bts->alloc_algorithm(ul_tbf, tbf, bts->alloc_algorithm_curst); - /* if no ressource */ + /* if no resource */ if (rc < 0) { - LOGP(DRLCMAC, LOGL_ERROR, "No ressource after update???\n"); + LOGP(DRLCMAC, LOGL_ERROR, "No resource after update???\n"); return -rc; } diff --git a/src/gprs_rlcmac.h b/src/gprs_rlcmac.h index 71ff608..fbae677 100644 --- a/src/gprs_rlcmac.h +++ b/src/gprs_rlcmac.h @@ -99,7 +99,7 @@ extern struct gprs_rlcmac_bts *gprs_rlcmac_bts; enum gprs_rlcmac_tbf_state { GPRS_RLCMAC_NULL = 0, /* new created TBF */ GPRS_RLCMAC_ASSIGN, /* wait for downlink assignment */ - GPRS_RLCMAC_FLOW, /* RLC/MAC flow, ressource needed */ + GPRS_RLCMAC_FLOW, /* RLC/MAC flow, resource needed */ GPRS_RLCMAC_FINISHED, /* flow finished, wait for release */ GPRS_RLCMAC_WAIT_RELEASE,/* wait for release or restart of DL TBF */ GPRS_RLCMAC_RELEASING, /* releasing, wait to free TBI/USF */ diff --git a/src/gprs_rlcmac_data.cpp b/src/gprs_rlcmac_data.cpp index 2f67bd0..b56d63d 100644 --- a/src/gprs_rlcmac_data.cpp +++ b/src/gprs_rlcmac_data.cpp @@ -220,7 +220,7 @@ uplink_request: /* create new TBF, use sme TRX as DL TBF */ tfi = tfi_alloc(GPRS_RLCMAC_UL_TBF, &trx, &ts, tbf->trx, tbf->first_ts); if (tfi < 0) { - LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH ressource\n"); + LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH resource\n"); /* FIXME: send reject */ break; } @@ -228,7 +228,7 @@ uplink_request: ul_tbf = tbf_alloc(tbf, GPRS_RLCMAC_UL_TBF, tfi, trx, ts, tbf->ms_class, 0); if (!ul_tbf) { - LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH ressource\n"); + LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH resource\n"); /* FIXME: send reject */ break; } @@ -269,7 +269,7 @@ uplink_request: } tlli = tbf->tlli; } - LOGP(DRLCMAC, LOGL_ERROR, "RX: [PCU <- BTS] %s TFI: %u TLLI: 0x%08x FIXME: Packet ressource request\n", (tbf->direction == GPRS_RLCMAC_UL_TBF) ? "UL" : "DL", tbf->tfi, tbf->tlli); + LOGP(DRLCMAC, LOGL_ERROR, "RX: [PCU <- BTS] %s TFI: %u TLLI: 0x%08x FIXME: Packet resource request\n", (tbf->direction == GPRS_RLCMAC_UL_TBF) ? "UL" : "DL", tbf->tfi, tbf->tlli); break; default: LOGP(DRLCMAC, LOGL_NOTICE, "RX: [PCU <- BTS] unknown control block received\n"); @@ -838,14 +838,14 @@ int gprs_rlcmac_rcv_rach(uint8_t ra, uint32_t Fn, int16_t qta) // Create new TBF tfi = tfi_alloc(GPRS_RLCMAC_UL_TBF, &trx, &ts, -1, -1); if (tfi < 0) { - LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH ressource\n"); + LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH resource\n"); /* FIXME: send reject */ return -EBUSY; } /* set class to 0, since we don't know the multislot class yet */ tbf = tbf_alloc(NULL, GPRS_RLCMAC_UL_TBF, tfi, trx, ts, 0, 1); if (!tbf) { - LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH ressource\n"); + LOGP(DRLCMAC, LOGL_NOTICE, "No PDCH resource\n"); /* FIXME: send reject */ return -EBUSY; } diff --git a/src/gprs_rlcmac_sched.cpp b/src/gprs_rlcmac_sched.cpp index f6af9b7..83a89ac 100644 --- a/src/gprs_rlcmac_sched.cpp +++ b/src/gprs_rlcmac_sched.cpp @@ -90,7 +90,7 @@ int gprs_rlcmac_rcv_rts_block(uint8_t trx, uint8_t ts, uint16_t arfcn, ul_ass_tbf = tbf; } - /* check uplink ressource for polling */ + /* check uplink resource for polling */ if (poll_tbf) { LOGP(DRLCMACSCHED, LOGL_DEBUG, "Received RTS for PDCH: TRX=%d " "TS=%d FN=%d block_nr=%d scheduling free USF for " @@ -99,17 +99,17 @@ int gprs_rlcmac_rcv_rts_block(uint8_t trx, uint8_t ts, uint16_t arfcn, (tbf->direction == GPRS_RLCMAC_UL_TBF) ? "UL" : "DL", poll_tbf->tfi); /* use free USF */ - /* else, we search for uplink ressource */ + /* else, we search for uplink resource */ } else { - /* select uplink ressource */ + /* select uplink resource */ for (i = 0, tfi = pdch->next_ul_tfi; i < 32; i++, tfi = (tfi + 1) & 31) { tbf = pdch->ul_tbf[tfi]; /* no TBF for this tfi, go next */ if (!tbf) continue; - /* no UL ressources needed, go next */ - /* we don't need to give ressources in FINISHED state, + /* no UL resources needed, go next */ + /* we don't need to give resources in FINISHED state, * because we have received all blocks and only poll * for packet control ack. */ if (tbf->state != GPRS_RLCMAC_FLOW) @@ -119,9 +119,9 @@ int gprs_rlcmac_rcv_rts_block(uint8_t trx, uint8_t ts, uint16_t arfcn, usf = tbf->dir.ul.usf[ts]; LOGP(DRLCMACSCHED, LOGL_DEBUG, "Received RTS for PDCH: " "TRX=%d TS=%d FN=%d block_nr=%d scheduling " - "USF=%d for required uplink ressource of " + "USF=%d for required uplink resource of " "UL TBF=%d\n", trx, ts, fn, block_nr, usf, tfi); - /* next TBF to handle ressource is the next one */ + /* next TBF to handle resource is the next one */ pdch->next_ul_tfi = (tfi + 1) & 31; break; } @@ -160,7 +160,7 @@ int gprs_rlcmac_rcv_rts_block(uint8_t trx, uint8_t ts, uint16_t arfcn, /* Prio 2: select data message for downlink */ if (!msg) { - /* select downlink ressource */ + /* select downlink resource */ for (i = 0, tfi = pdch->next_dl_tfi; i < 32; i++, tfi = (tfi + 1) & 31) { tbf = pdch->dl_tbf[tfi]; @@ -170,7 +170,7 @@ int gprs_rlcmac_rcv_rts_block(uint8_t trx, uint8_t ts, uint16_t arfcn, /* no DL TBF, go next */ if (tbf->direction != GPRS_RLCMAC_DL_TBF) continue; - /* no DL ressources needed, go next */ + /* no DL resources needed, go next */ if (tbf->state != GPRS_RLCMAC_FLOW && tbf->state != GPRS_RLCMAC_FINISHED) continue; @@ -178,7 +178,7 @@ int gprs_rlcmac_rcv_rts_block(uint8_t trx, uint8_t ts, uint16_t arfcn, LOGP(DRLCMACSCHED, LOGL_DEBUG, "Scheduling data " "message at RTS for DL TBF=%d (TRX=%d, " "TS=%d)\n", tfi, trx, ts); - /* next TBF to handle ressource is the next one */ + /* next TBF to handle resource is the next one */ pdch->next_dl_tfi = (tfi + 1) & 31; /* generate DL data block */ msg = gprs_rlcmac_send_data_block_acknowledged(tbf, fn, diff --git a/src/tbf.txt b/src/tbf.txt index eab58ba..025e56c 100644 --- a/src/tbf.txt +++ b/src/tbf.txt @@ -118,7 +118,7 @@ Control TS: Polling: In order to poll uplink control block from MS, a special poll state and frame number is stored at TBF. The scheduler reads that value and will not - assign uplink ressource for other TBFs at that frame number. + assign uplink resource for other TBFs at that frame number. When there is no uplink transmission received on the block, a timeout is indicated by layer 1 interface. There are two ways of checking timeout: -- 1.7.10.4

2 1

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

osmocom-net-gprs