Attention is currently required from: fixeria.
wbokslag has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-tetra/+/29387 )
Change subject: added support for fill bits and basic link defragmentation (also thanks to SQ5BPF)
......................................................................
Patch Set 3:
(1 comment)
File src/tetra_llc_pdu.h:
https://gerrit.osmocom.org/c/osmo-tetra/+/29387/comment/090b7175_8e946ef0
PS1, Line 80:
> I meant to say "unrelated WhiteSpace change", sorry for not being clear. […]
Ah I see, same tab issue. Amended.
--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/29387
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I41c9438b0b12c2fac9dff1b226eec5b33f30fbb4
Gerrit-Change-Number: 29387
Gerrit-PatchSet: 3
Gerrit-Owner: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Sun, 18 Sep 2022 16:03:16 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: wbokslag <w.bokslag(a)midnightblue.nl>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: wbokslag.
Hello Jenkins Builder, fixeria,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-tetra/+/29387
to look at the new patch set (#3).
Change subject: added support for fill bits and basic link defragmentation (also thanks to SQ5BPF)
......................................................................
added support for fill bits and basic link defragmentation (also thanks to SQ5BPF)
The upper mac now maintains a defragmentation buffer for each timeslot. Resources with length -1 (fragmentation start) are added to the buf for that slot, further mac/frag frames are appended. When a mac/end is encountered, the reconstructed l2 message is passed to rx_tm_sdu. The tetra_llc_pdu struct now uses a uint32_t for tl_sdu_len in order to account for the possibly longer sdus. Fill bits processing was required in order to reliably determine the end of a MAC PDU.
Change-Id: I41c9438b0b12c2fac9dff1b226eec5b33f30fbb4
---
M src/tetra-rx.c
M src/tetra_llc_pdu.h
M src/tetra_mac_pdu.c
M src/tetra_mac_pdu.h
M src/tetra_upper_mac.c
M src/tetra_upper_mac.h
6 files changed, 272 insertions(+), 38 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-tetra refs/changes/87/29387/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/29387
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I41c9438b0b12c2fac9dff1b226eec5b33f30fbb4
Gerrit-Change-Number: 29387
Gerrit-PatchSet: 3
Gerrit-Owner: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-MessageType: newpatchset
Attention is currently required from: wbokslag.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-tetra/+/29407
to look at the new patch set (#2).
Change subject: Reformatting to prevent linter warnings
......................................................................
Reformatting to prevent linter warnings
Change-Id: I0dc35e7d536900b4beed19042183838857b7dac7
---
M src/crc_test.c
M src/testpdu.c
M src/tetra_cmce_pdu.c
M src/tetra_common.c
M src/tetra_common.h
M src/tetra_mac_pdu.c
M src/tetra_mm_pdu.c
M src/tetra_mm_pdu.h
8 files changed, 67 insertions(+), 68 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-tetra refs/changes/07/29407/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/29407
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I0dc35e7d536900b4beed19042183838857b7dac7
Gerrit-Change-Number: 29407
Gerrit-PatchSet: 2
Gerrit-Owner: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-MessageType: newpatchset
Attention is currently required from: wbokslag.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-tetra/+/29389 )
Change subject: only compute filter when opt_afc is enabled
......................................................................
Patch Set 2: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/29389
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: Icc42a70b07600267ccfad3969b738b3b4a606a61
Gerrit-Change-Number: 29389
Gerrit-PatchSet: 2
Gerrit-Owner: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Comment-Date: Sun, 18 Sep 2022 15:53:50 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: wbokslag.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-tetra/+/29387 )
Change subject: added support for fill bits and basic link defragmentation (also thanks to SQ5BPF)
......................................................................
Patch Set 2:
(1 comment)
File src/tetra_mac_pdu.h:
https://gerrit.osmocom.org/c/osmo-tetra/+/29387/comment/cdf164ad_7f9f522c
PS1, Line 160:
> Hmm missed this one in my second patch set. I was trying to keep the linter from complaining. […]
Fine with me, thank you!
--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/29387
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I41c9438b0b12c2fac9dff1b226eec5b33f30fbb4
Gerrit-Change-Number: 29387
Gerrit-PatchSet: 2
Gerrit-Owner: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Comment-Date: Sun, 18 Sep 2022 15:50:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: wbokslag <w.bokslag(a)midnightblue.nl>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: wbokslag.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-tetra/+/29387 )
Change subject: added support for fill bits and basic link defragmentation (also thanks to SQ5BPF)
......................................................................
Patch Set 2: Code-Review+1
(2 comments)
Patchset:
PS2:
Not really critical, looks good in general.
File src/tetra_llc_pdu.h:
https://gerrit.osmocom.org/c/osmo-tetra/+/29387/comment/a75dc52c_27c6ef83
PS1, Line 80:
> tl_sdu_len needs to be changed from a uint8_t to uint32_t because, due to defragmentation, pdus may […]
I meant to say "unrelated WhiteSpace change", sorry for not being clear. What I wanted to point out is that you're adding two tabs here for no reason. You're changing the line below, this one should remain unchanged.
--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/29387
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I41c9438b0b12c2fac9dff1b226eec5b33f30fbb4
Gerrit-Change-Number: 29387
Gerrit-PatchSet: 2
Gerrit-Owner: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: wbokslag <w.bokslag(a)midnightblue.nl>
Gerrit-Comment-Date: Sun, 18 Sep 2022 15:49:43 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: wbokslag <w.bokslag(a)midnightblue.nl>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment