This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
laforge gerrit-no-reply at lists.osmocom.orglaforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/25556 ) Change subject: ns2: fsm: add comment don't answer on a STATUS with a STATUS ...................................................................... Patch Set 1: Code-Review+2 (1 comment) https://gerrit.osmocom.org/c/libosmocore/+/25556/1/src/gb/gprs_ns2_vc_fsm.c File src/gb/gprs_ns2_vc_fsm.c: https://gerrit.osmocom.org/c/libosmocore/+/25556/1/src/gb/gprs_ns2_vc_fsm.c@864 PS1, Line 864: /* don't answer on a STATUS with a STATUS */ > what? can you add more words there? I would guess it's obvious: If receiving an error mesage (status) will generate that same error message (status), the peer will in turn generate another one and you will have an endless flood of packets ping-poinging between both implementations. That's why any protocol spec I've seen usually strictly forbids such behavior. -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/25556 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: Ib8f700f9193a96a7bada3b0293dcecf6a05d6efc Gerrit-Change-Number: 25556 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis at fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel <dwillmann at sysmocom.de> Gerrit-Reviewer: laforge <laforge at osmocom.org> Gerrit-CC: pespin <pespin at sysmocom.de> Gerrit-Comment-Date: Thu, 23 Sep 2021 13:04:56 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: pespin <pespin at sysmocom.de> Gerrit-MessageType: comment -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210923/04b4f03b/attachment.htm>