Attention is currently required from: pespin.

View Change

1 comment:

• File src/tbf.h:

  • Patch Set #1, Line 260: state_fi

    Because there are several fi on a TBF controling different stuff. […]

    But this is one is the main tbf_fsm, right? Both ul_ass_fsm and dl_ass_fsm are supplementary ones, AFAICS. IMO, it's even more confusing now: you have 'state_fi' and 'state_fsm'.

    And why the internal fsm data (i.e. tbf_ul_ass_fsm_ctx, tbf_ul_ass_fsm_ctx, and tbf_fsm_ctx) is available here? Shouldn't it be dynamically allocated and assigned to fi->priv?

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