Change in libosmocore[master]: frame_relay: Send "Fuil Status" ENQUIRY after link recovers

[laforge]

laforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/22573 )

Change subject: frame_relay: Send "Fuil Status" ENQUIRY after link recovers
......................................................................

frame_relay: Send "Fuil Status" ENQUIRY after link recovers

If we are the 'user' side of FR and a link has just recovered,
we should ensure the next STATUS is for "full status".  This way
we learn about the present DLCs as quickly as possible, saving up
to 10 seconds of further delay in link recovery.

Related: OS#4999
Change-Id: I6f905a18a7d130a3c02b4a3e7a2a2dc24afc0ea1
---
M src/gb/frame_relay.c
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  daniel: Looks good to me, but someone else must approve
  lynxis lazus: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/src/gb/frame_relay.c b/src/gb/frame_relay.c
index 3a51700..03b847a 100644
--- a/src/gb/frame_relay.c
+++ b/src/gb/frame_relay.c
@@ -430,8 +430,12 @@
 
 		LOGPFRL(link, LOGL_NOTICE, "Link recovered\n");
 		link->state = true;
-		if (link->role == FR_ROLE_USER_EQUIPMENT)
+		if (link->role == FR_ROLE_USER_EQUIPMENT) {
+			/* make sure the next STATUS ENQUIRY is for a full
+			 * status report to get the configred DLCs ASAP */
+			link->polling_count = 0;
 			return;
+		}
 
 		llist_for_each_entry(dlc, &link->dlc_list, list) {
 			if (!dlc->add && !dlc->del)

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I6f905a18a7d130a3c02b4a3e7a2a2dc24afc0ea1
Gerrit-Change-Number: 22573
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann at sysmocom.de>
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210201/24aa302a/attachment.htm>