<p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/24707">View Change</a></p><p>3 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/+/24707/1/src/common/main.c">File src/common/main.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/+/24707/1/src/common/main.c@95">Patch Set #1, Line 95:</a> <code style="font-family:monospace,monospace">                  LOGP(DLGLOBAL, LOGL_FATAL, "%s: Unknown VTY reference generation "</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You shouldn't use libosmocore logging system here, because you are still configuring it from cmd line arguments in the loop under handle_options, so the program may end up doing unexpected stuff.</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/+/24707/1/src/common/main.c@306">Patch Set #1, Line 306:</a> <code style="font-family:monospace,monospace">             LOGP(DLGLOBAL, LOGL_FATAL, "Failed to create BTS structure\n");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure whether it's a 100% good idea to merge this patch, specially since we introduced support for async writes in the logging subsystem. That means the writes may not happen until the main loop is run, which in all these changes you do here is not, since we exit beforehand. As a result, messages could potentially be held in write queues.<br>On the other side, we win potentially writing those to eg. file targets, since it should be rare delaying writes for those messages since they are not that big.</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/+/24707/1/src/common/main.c@420">Patch Set #1, Line 420:</a> <code style="font-family:monospace,monospace">                  perror("Error during daemonize");</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">If all lines are changed, this perror() line should too.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/24707">change 24707</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/+/24707"/><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: I2ae4acf6a92137236e1b62c2d0aab79a34134f45 </div>
<div style="display:none"> Gerrit-Change-Number: 24707 </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-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 17 Jun 2021 11:28:04 +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>