<p style="white-space: pre-wrap; word-wrap: break-word;">keeping the VTY context a separate context *not* underneath the applications global talloc context is actually a feature that was introduced intentionally.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The point is that VTY is a heavy user of talloc, and it generates quite lengthy reports.  By keeping the VTY alloctaions separate, we ensure that the normal talloc_report of the application context is not full of "vty spam" that we're not really interested.  OpenBSC (and everything that we created as derivatife of openbsc.git later on) use different signals (SIGUSR1 vs SIGUSR2) to get the report either with or without VTY.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not saying this is set in stone, but there is some logic to it and we should be careful about changing this.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I would rather want to move the SIGUSR* handling into libosmocore to avoid replicating this all over the place.  This way we can ensure consistent behavior with regard to the VTY...</p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/13973">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13973">change 13973</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/13973"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I65d71f8fc943e103b53b784f690ed56a6e373e3f </div>
<div style="display:none"> Gerrit-Change-Number: 13973 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 12 May 2019 10:22:09 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>