Attention is currently required from: laforge, pespin.

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

Change subject: {trxcon,virt_phy}: shared GPRS L1 (MAC) implementation ......................................................................

Patch Set 11:

(1 comment)

File src/shared/l1gprs.c:

https://gerrit.osmocom.org/c/osmocom-bb/+/30743/comment/6e1c14c0_a931897c PS10, Line 83: static struct l1gprs_tbf *l1gprs_find_tbf(struct l1gprs_state *gprs,

Iterating over the interesting subset instead of iterating over the entire set and skipping half. […]

Yes, but I don't think this is critical... unless you want to reconfigure hundreds of TBFs per second. And in general I don't expect this llist to grow a lot, most of the time you have one UL and one DL TBF anyway.