<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 5:</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 5: Code-Review-2</p><p style="white-space: pre-wrap; word-wrap: break-word;">why that? I thought the point was to _allow_ IP fragmentation in large TRXD PDUs? You're doing the exact opposite here?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I don't remember all the details of our previous conversation regarding the IP fragmentation, sorry. My understanding at the time of writing this patch was that we don't want the fragmentation because of the associated performance loss. It seems I understood you wrong.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">a) the protocol is mostly spoken over looback, where MTU is 64k<br>b) on other interfaces, yes, we should ideally stay belwo usual MTUs such as e.g. 1500 bytes</p><p style="white-space: pre-wrap; word-wrap: break-word;">In any case, this patch doesn't solve anything.  You cannot fix this potential problem with a socket option.  You need to make sure in the application code, that you don't aggregate more TRXD message to exceed the "reasonable minimum MTU".</p><p style="white-space: pre-wrap; word-wrap: break-word;">If you enable the DF bit, you are actively breaking existing situations that are working for the user (with fragmentation overhead).</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I did some tests with the current master (no explicit IP_MTU_DISCOVER setting), and as far as I can see, the IP fragmentation is employed by default if a TRXD PDU exceeds the MTU. So AFAIU, we want to make sure that it can be employed regardless of the system's default policy by setting IP_MTU_DISCOVER explicitly, right? If so, shall I use IP_PMTUDISC_DONT or IP_PMTUDISC_WANT?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">fragmentation is always enabled by default, AFAICT.  So nothing needs to be done to enable it.<br></p><p><a href="https://gerrit.osmocom.org/c/osmo-bts/+/23814">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-bts/+/23814">change 23814</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/+/23814"/><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: I25415b76b69ccf43f83ddfb0e7dd1e330fb23aa7 </div>
<div style="display:none"> Gerrit-Change-Number: 23814 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 26 Oct 2021 15:23:34 +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>