laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-e1d/+/28065 ) Change subject: e1oip: introduce #defines for BATCHING_FACTOR / PREFILL_COUNT ...................................................................... e1oip: introduce #defines for BATCHING_FACTOR / PREFILL_COUNT Change-Id: Ia0da9b038312b010bfdf88570733bf1bf004b7a8 --- M src/octoi/e1oip.c M src/octoi/e1oip.h 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/src/octoi/e1oip.c b/src/octoi/e1oip.c index 19eb3bf..a577a92 100644 --- a/src/octoi/e1oip.c +++ b/src/octoi/e1oip.c @@ -302,8 +302,8 @@ iline->stats = osmo_stat_item_group_alloc(iline, &iline_stats_desc, ctr_idx); e1oip_line_set_name(iline, peer->name); - iline->cfg.batching_factor = 32; - iline->cfg.prefill_frame_count = 200; /* 25ms */ + iline->cfg.batching_factor = DEFAULT_BATCHING_FACTOR; + iline->cfg.prefill_frame_count = DEFAULT_PREFILL_FRAME_COUNT; e1oip_line_reset(iline); diff --git a/src/octoi/e1oip.h b/src/octoi/e1oip.h index da4716b..2c88346 100644 --- a/src/octoi/e1oip.h +++ b/src/octoi/e1oip.h @@ -15,6 +15,9 @@ #define FRAMES_PER_SEC_THRESHOLD 7500 +#define DEFAULT_BATCHING_FACTOR 32 +#define DEFAULT_PREFILL_FRAME_COUNT 200 /* 25ms */ + enum e1oip_line_ctr { LINE_CTR_E1oIP_UNDERRUN, LINE_CTR_E1oIP_SUBSTITUTED, -- To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/28065 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-e1d Gerrit-Branch: master Gerrit-Change-Id: Ia0da9b038312b010bfdf88570733bf1bf004b7a8 Gerrit-Change-Number: 28065 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged