Attention is currently required from: arehbein, pespin.

Patch set 12:Code-Review -1

View Change

1 comment:

• Patchset:

  • Patch Set #12:

    Hmm, the task this patch has is actually quite difficult, for these reasons:

    • the osmo_tdefs API was implemented with T1234 and X1234 in mind, but not N1234. Looks like we won't get around adding another namespace for N, in the libosmocore tdef API itself. Mainly because T3101 and N3101 collide. (If it's only this one maybe we want an ugly workaround instead? T23101 is N3101?).
    • Adding the 'N' category is a separate discussion, Vadim and I discussed it a bit in PM and let's say there's potential for diverse opinions.
    • the osmo_tdefs API was implemented with a single global timer config in mind, not individual per-object timers. The tdefs API is modular enough to be helpful in implementing this, so no problem here, just many choices to be made.
    • confusion: we already have a global timer 'net' / 'T3101'; now we are also adding a per-BTS T3101 as well as N3101. So per-BTS for GPRS, but only global T for CS. And these seemingly identical T3101 items do not interact...? ("are you sure they are separate?" / "did you really use the correct T3101?"...)

    I'll not dive into per-line comments now, but I'd like to already say that we need to still work on this patch (and it's not arehbein's fault)

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