Attention is currently required from: fixeria.

pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/32628 )

Change subject: trxcon/l1sched: allocate primitives of fixed size (64 + 64)
......................................................................


Patch Set 1:

(1 comment)

File src/host/trxcon/src/sched_prim.c:

https://gerrit.osmocom.org/c/osmocom-bb/+/32628/comment/4a9b78c3_606b2624
PS1, Line 69:   msg = msgb_alloc_headroom(L1SCHED_PRIM_HEADROOM + 
L1SCHED_PRIM_TAILROOM,
shouldn't this L1SCHED_PRIM_TAILROOM actually be sizeof(l1sched_prim) or alike?



--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/32628
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Ica87b147e11744a69dcd7c056376dcf6b98f9ca6
Gerrit-Change-Number: 32628
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Comment-Date: Fri, 05 May 2023 10:50:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to