<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 3:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 3:</p></blockquote><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Couldn't we simply allocate teh msgb's once (from heap) at start and never free them at runtime, using msgb_reset or the like once they are recycled?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">any feedback here?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I see this as a structured approach to move away from msgbs for those static buffers without trying to fix everything at once, which already didn't work so well half a year ago. This, however, works.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The combined memory overhead of talloc and msgbs is already so large, it feels like embedded java, while contributing very little in this case, and talloc makes debugging things that should be plain old arrays or reasoning about wurst-käse memory allocations difficult - neither of which are things i'd like to see when doing embedded work.</p><p><a href="https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/21071">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/21071">change 21071</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-ccid-firmware/+/21071"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ccid-firmware </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I65e77c376aca9ed50e234a0b58a7450a8bbd4fe0 </div>
<div style="display:none"> Gerrit-Change-Number: 21071 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 17 Nov 2020 10:39:50 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>