<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/22873">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/22873/1//COMMIT_MSG">Commit Message:</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/libosmocore/+/22873/1//COMMIT_MSG@5">Patch Set #1, Line 5:</a> <code style="font-family:monospace,monospace">CommitDate: 2021-02-12 03:33:30 +0100</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think we should somehow specify what it means specifically.  Does this track the maximum BSSGP payload size, or does it include the NS header?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I would think it makes sense for the BSSGP layer to direclty only care with the size of its own messages, and not have to know how much overhead NS or any other layers below add.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So maye consider renaming it to something != MTU (as that always has that "ethernet" notion of being the physical transport size MTU) and have something like max_bssgp_size or the like.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In any case, ther are many options. The importance of my message is that the code should contain a clear definition (in comment / api doc / ...) what exact size including/excluding which headers at what layer is specified.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/22873">change 22873</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/libosmocore/+/22873"/><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-Change-Id: I9bb82ead27366b7370c9ff968e03ca2113ec11f0 </div>
<div style="display:none"> Gerrit-Change-Number: 22873 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 12 Feb 2021 09:06:23 +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>