daniel has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/21960 )
Change subject: gbproxy: Implment TLLI cache and use it for SUSPEND/RESUME ...................................................................... Patch Set 1: (2 comments) This change is ready for review. https://gerrit.osmocom.org/c/osmo-sgsn/+/21960/1/include/osmocom/sgsn/gb_proxy.h File include/osmocom/sgsn/gb_proxy.h: https://gerrit.osmocom.org/c/osmo-sgsn/+/21960/1/include/osmocom/sgsn/gb_proxy.h@177 PS1, Line 177: struct > a per-second granularity counter would probably be sufficient, but then I'm > likely over-optimizing. I'm only considering the second anyway. Changed https://gerrit.osmocom.org/c/osmo-sgsn/+/21960/1/src/gbproxy/gb_proxy_peer.c File src/gbproxy/gb_proxy_peer.c: https://gerrit.osmocom.org/c/osmo-sgsn/+/21960/1/src/gbproxy/gb_proxy_peer.c@215 PS1, Line 215: } > missing return before the end of the block? Ack -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/21960 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: I42adf70f560d2bb358a9e1c7614281e8d2967568 Gerrit-Change-Number: 21960 Gerrit-PatchSet: 1 Gerrit-Owner: daniel <dwillm...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <lafo...@osmocom.org> Gerrit-Comment-Date: Fri, 08 Jan 2021 01:23:24 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: laforge <lafo...@osmocom.org> Gerrit-MessageType: comment