Attention is currently required from: pespin.

View Change

1 comment:

• File src/osmo-bts-trx/sched_lchan_tchh.c:

  • Patch Set #1, Line 414: if (!sched_tchh_dl_facch_map[br->fn % 26])

    I'm a bit lost here. […]

    "Why first try to send dummy stuff" - where? This is not the case. Currently (without this patch) if tch_dl_dequeue() yields nothing we're scheduling dummy FACCH *regardless* if it's permitted to start at the given TDMA FN or not. This may result in misaligned FACCH transmission, so the MS will not able to decode anything and will report BER>0.

    With this patch applied we a) schedule FACCH if it's allowed to start at the current TDMA FN; b) send whatever was in the burst buffer (garbage) if FACCH is not permitted. This is the best we can do without introducing additional complexity. Ideally the upper layers should guarantee that the queue is never empty, but currently this is not the case.

To view, visit change 27836. To unsubscribe, or for help writing mail filters, visit settings.