Attention is currently required from: dexter.

Patch set 1:Code-Review -1

View Change

1 comment:

• Patchset:

  • Patch Set #1:

    This is something that needs to be done carefully: you're adding a new field, but the PCUIF version remains unchanged. Old osmo-{bts,bsc} speaking the same PCUIF version will not include this field, so the INFO.ind message will get dropped in `pcu_rx()`, see `CHECK_IF_MSG_SIZE`. I know PCUIF sucks when it comes to adding new fields, I wish at least the INFO.ind would employ TLV encoding rather than static structure...

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