<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/22343">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/22343/9/src/gb/gprs_ns2_fr.c">File src/gb/gprs_ns2_fr.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/libosmocore/+/22343/9/src/gb/gprs_ns2_fr.c@804">Patch Set #9, Line 804:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">/* 48.016 § 6.1.4.2 default maximum information field size of 1600 octets<br>      * FR has 2 byte header */<br>    bind->mtu = 1600 - 2;<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think the "information field" is what comes after the header, so (without checking the specs again) my assumption was that on the wire it is 1600 bytes plus FR header plus CRC plus flag octets.  And hence here mtu = 1600.  <br>I just checked FRF.1.2 and Q.922 and I think it supports my position here.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Nevertheless, the actual device MTU should be read from the net-device once we actually bind to it.  I guess you're doing this in a follow-up patch in this series (haven't looked throuhg it yet).</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/22343/9/src/gb/gprs_ns2_frgre.c">File src/gb/gprs_ns2_frgre.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/libosmocore/+/22343/9/src/gb/gprs_ns2_frgre.c@577">Patch Set #9, Line 577:</a> <code style="font-family:monospace,monospace">    bind->mtu = 1600 - 2;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">same here. Also, probably worth adding a #define in the frame_relay.h file for this?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/22343">change 22343</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/+/22343"/><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: I5016b295db6185ec131d83089cf6c806e34ef1b6 </div>
<div style="display:none"> Gerrit-Change-Number: 22343 </div>
<div style="display:none"> Gerrit-PatchSet: 9 </div>
<div style="display:none"> Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 12 Feb 2021 08:54:33 +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>