[PATCH] osmo-bts[master]: log: sysmo, lc15: tweak log about sapi_cmds queue

[Neels Hofmeyr]

Review at  https://gerrit.osmocom.org/755

log: sysmo,lc15: tweak log about sapi_cmds queue

The osmo-bts log used to say this a lot:

DL1C <0006> oml.c:758 (bts=0,trx=0,ts=0,ss=1) End of queue encountered. Now empty? 1

- Move this to DEBUG level instead of NOTICE.

- Tweak wording and logic so it says one of:
  [...] End of SAPI cmd queue encountered. Queue is now empty.
  [...] End of SAPI cmd queue encountered. More pending.

Change-Id: I5a46c90d016cee9b50f32db2af568765d3cb74cc
---
M src/osmo-bts-litecell15/oml.c
M src/osmo-bts-sysmo/oml.c
2 files changed, 12 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/55/755/1

diff --git a/src/osmo-bts-litecell15/oml.c b/src/osmo-bts-litecell15/oml.c
index 5ce510a..c6787dd 100644
--- a/src/osmo-bts-litecell15/oml.c
+++ b/src/osmo-bts-litecell15/oml.c
@@ -744,9 +744,12 @@
 	talloc_free(cmd);
 
 	if (end || llist_empty(&lchan->sapi_cmds)) {
-		LOGP(DL1C, LOGL_NOTICE,
-			"%s End of queue encountered. Now empty? %d\n",
-			gsm_lchan_name(lchan), llist_empty(&lchan->sapi_cmds));
+		LOGP(DL1C, LOGL_DEBUG,
+			"%s End of SAPI cmd queue encountered.%s\n",
+			gsm_lchan_name(lchan),
+			llist_empty(&lchan->sapi_cmds)
+				? " Queue is now empty."
+				: " More pending.");
 		return;
 	}
 
diff --git a/src/osmo-bts-sysmo/oml.c b/src/osmo-bts-sysmo/oml.c
index 6f0629d..c4519ec 100644
--- a/src/osmo-bts-sysmo/oml.c
+++ b/src/osmo-bts-sysmo/oml.c
@@ -753,9 +753,12 @@
 	talloc_free(cmd);
 
 	if (end || llist_empty(&lchan->sapi_cmds)) {
-		LOGP(DL1C, LOGL_NOTICE,
-			"%s End of queue encountered. Now empty? %d\n",
-			gsm_lchan_name(lchan), llist_empty(&lchan->sapi_cmds));
+		LOGP(DL1C, LOGL_DEBUG,
+			"%s End of SAPI cmd queue encountered.%s\n",
+			gsm_lchan_name(lchan),
+			llist_empty(&lchan->sapi_cmds)
+				? " Queue is now empty."
+				: " More pending.");
 		return;
 	}
 

-- 
To view, visit https://gerrit.osmocom.org/755
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5a46c90d016cee9b50f32db2af568765d3cb74cc
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>