Attention is currently required from: fixeria.

pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmo-gprs/+/35858?usp=email )

Change subject: rlcmac: uncomment RLC/MAC msg_type checking in DATA.cnf
......................................................................


Patch Set 3:

(1 comment)

File tests/rlcmac/rlcmac_prim_test.c:

https://gerrit.osmocom.org/c/libosmo-gprs/+/35858/comment/dcc9347f_302637f4
PS3, Line 640:  rlcmac_prim = 
osmo_gprs_rlcmac_prim_alloc_l1ctl_pdch_data_cnf(ts_nr, rts_fn, NULL, 0); // 
FIXME: pass a valid RLC/MAC block here
The easiest is probably that you memcpy() the buffer in line 495 when 
receiveign the DATA.req in the callback where this test emulates the lower 
layer.



--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/35858?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I4a0c553201d51453d9197d254f796f6574a2dab0
Gerrit-Change-Number: 35858
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pes...@sysmocom.de>
Gerrit-Attention: fixeria <vyanits...@sysmocom.de>
Gerrit-Comment-Date: Tue, 06 Feb 2024 10:50:24 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to