Attention is currently required from: manawyrm, roox, tnt.
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-e1d/+/27841 )

Change subject: e1oip: Add rate_ctr for IP->E1 RIFO overflows
......................................................................


Patch Set 4:

(1 comment)

File src/octoi/e1oip.c:

https://gerrit.osmocom.org/c/osmo-e1d/+/27841/comment/496a4884_0cea5320
PS2, Line 255:          rc = frame_rifo_in(&iline->e1t.rifo, frame_buf, fn32+i);
> That's not really correct. […]
so what kind of naming would you prefer? e1oip:e1t_out_of_ragge? out_of_window? 
 I somehow like the orthogonality of e1t_overflow with e1o_overflow that we 
already have.  But not sure if that is worth introducing misnamed counters?



--
To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/27841
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-e1d
Gerrit-Branch: master
Gerrit-Change-Id: Ie0e8bb2f5d2ae4256952f6bf69e514d5c2627a76
Gerrit-Change-Number: 27841
Gerrit-PatchSet: 4
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: manawyrm <[email protected]>
Gerrit-Reviewer: roox <[email protected]>
Gerrit-Reviewer: tnt <[email protected]>
Gerrit-Attention: manawyrm <[email protected]>
Gerrit-Attention: roox <[email protected]>
Gerrit-Attention: tnt <[email protected]>
Gerrit-Comment-Date: Wed, 20 Apr 2022 12:58:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: tnt <[email protected]>
Gerrit-MessageType: comment

Reply via email to