<p style="white-space: pre-wrap; word-wrap: break-word;">IMHO, this amount of logging is only suitable for development stage. For a regular user/customer trying to understand what's going on it's too verbose.</p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888">View Change</a></p><p>9 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c">File src/common/l1sap.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@938">Patch Set #1, Line 938:</a> <code style="font-family:monospace,monospace">DL1C</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Pau was complaining about unreadable DL1C (https://osmocom.org/issues/5007), maybe DL1P?<br>The 'L1C' stands for "Layer 1 Control", 'DL1P' stands for "Layer 1 Primitives".</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@938">Patch Set #1, Line 938:</a> <code style="font-family:monospace,monospace">DL-FACCH repetition: repeating FACCH channel</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">How can you 'repeat FACCH channel'? You actually repeat a 'FACCH block', or more precisely 'retransmit' it.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@943">Patch Set #1, Line 943:</a> <code style="font-family:monospace,monospace">channel</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same here.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@994">Patch Set #1, Line 994:</a> <code style="font-family:monospace,monospace">               LOGPLCHAN(lchan, DL1C, LOGL_DEBUG, "DL-FACCH repetition: disabled by BSC\n");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This looks redundant to me, if it's disabled that I would not expect any logging.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@1044">Patch Set #1, Line 1044:</a> <code style="font-family:monospace,monospace"> if (rxqual >= upper) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">How about storing the current 'repeated_dl_facch_active' value and logging only if it changes from true to false and vice versa?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@1066">Patch Set #1, Line 1066:</a> <code style="font-family:monospace,monospace">            LOGPLCHAN(lchan, DL1C, LOGL_DEBUG, "DL-SACCH repetition: MS does not request DL-SACCH repetition (SRR=0)\n");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">This also looks redundant, and would probably make 'DL1C' unreadable.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@1083">Patch Set #1, Line 1083:</a> <code style="font-family:monospace,monospace">channel</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">block</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@1091">Patch Set #1, Line 1091:</a> <code style="font-family:monospace,monospace"></code></p><p style="white-space: pre-wrap; word-wrap: break-word;">ws, unrelated</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888/1/src/common/l1sap.c@1468">Patch Set #1, Line 1468:</a> <code style="font-family:monospace,monospace">          LOGPLCHAN(lchan, DL1C, LOGL_DEBUG, "UL-SACCH repetition: disabled by BSC\n");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Same here.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/22888">change 22888</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-bts/+/22888"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bts </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I59d11fd03be3d29fb8a4279d9945b03006764c0e </div>
<div style="display:none"> Gerrit-Change-Number: 22888 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 12 Feb 2021 22:39:42 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>