https://osmocom.org/https://osmocom.org/favicon.ico?16647414092019-06-28T13:02:26ZOpen Source Mobile CommunicationsOsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=150312019-06-28T13:02:26Zpespin
<ul></ul><p><a class="user active" href="https://osmocom.org/users/7">laforge</a> I assigned it to you since you wrote the initial patches. Feel free to re-assign it to me if you want me to spend time on it.</p> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=150332019-06-28T20:25:47Zlaforge
<ul></ul><p>Not sure if he has time, but <a class="user active" href="https://osmocom.org/users/67">fixeria</a> is probably best qualified as he's just re-designing/re-writing the TRX protocol.</p> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155292019-08-04T20:38:11Zlaforge
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li><li><strong>Assignee</strong> changed from <i>laforge</i> to <i>fixeria</i></li></ul> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155322019-08-05T13:56:26Zfixeria
<ul><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' disabled> <i>OsmoTRX: support filtering TRXC and TRXD protocols (osmo_trx.data and osmo_trx.ctrl)</i> added</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' disabled> <i>OsmoTRXD: support filtering by direction (Rx or Tx bursts)</i> added</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' disabled> <i>OsmoTRXD: fix handling of NOPE indications</i> added</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' disabled> <i>OsmoTRXD: verify the burst length (using modulation type)</i> added</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' disabled> <i>OsmoTRXD: handle optional padding for Rx bursts</i> added</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' disabled> <i>OsmoTRXC: introduce a 'string_string' array for 'CMD', 'RSP' and 'IND'</i> added</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' disabled> <i>OsmoTRXC: enrich the protocol column with message type and verb</i> added</li><li><strong>File</strong> <a href="/attachments/3812">osmo_trx_capture.pcapng.gz</a> <a class="icon-only icon-download" title="Download" href="/attachments/download/3812/osmo_trx_capture.pcapng.gz">osmo_trx_capture.pcapng.gz</a> added</li><li><strong>Category</strong> set to <i>TRX Interface</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>80</i></li></ul><p>I just uploaded a new patch. This is not the final version, but much closer now. The dissector for OsmoTRXD (Data protocol, Rx / Tx bursts) is more or less complete. One missing thing is the ability to filter bursts by direction. Also (just discovered), handling of NOPE indications is broken. The dissector for OsmoTRXC (Clock & Control) still needs some improvements, but in general it works.</p>
<p><a class="external" href="https://code.wireshark.org/review/#/c/26796/">https://code.wireshark.org/review/#/c/26796/</a></p>
<p>Attached a capture file that I am using for testing.</p> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155362019-08-05T19:42:38Zfixeria
<ul><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' disabled> <i>OsmoTRX: make port numbers configurable in the protocol preferences</i> added</li></ul> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155382019-08-05T21:40:01Zfixeria
<ul><li><b>Checklist item</b> changed from <i>OsmoTRX: support filtering TRXC and TRXD protocols (osmo_trx.data and osmo_trx.ctrl)</i> to <i>OsmoTRX: support filtering TRXC and TRXD protocols (osmo_trxd and osmo_trxc)</i></li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' checked disabled> <i>OsmoTRX: support filtering TRXC and TRXD protocols (osmo_trxd and osmo_trxc)</i> set to Done</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' checked disabled> <i>OsmoTRXD: support filtering by direction (Rx or Tx bursts)</i> set to Done</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' checked disabled> <i>OsmoTRXD: fix handling of NOPE indications</i> set to Done</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' checked disabled> <i>OsmoTRXD: handle optional padding for Rx bursts</i> set to Done</li><li><strong>% Done</strong> changed from <i>80</i> to <i>90</i></li></ul><p>A new version has been uploaded.</p> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155402019-08-05T22:03:41Zfixeria
<ul><li><strong>Related to</strong> <i><a class="issue tracker-2 status-7 priority-2 priority-default" href="/issues/4006">Feature #4006</a>: TRX protocol: wind of change</i> added</li></ul> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155412019-08-06T10:22:50Zfixeria
<ul><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' checked disabled> <i>OsmoTRXC: introduce a 'string_string' array for 'CMD', 'RSP' and 'IND'</i> set to Done</li><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' checked disabled> <i>OsmoTRXC: enrich the protocol column with message type and verb</i> set to Done</li></ul> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155422019-08-06T10:25:44Zfixeria
<ul><li><b>Checklist item</b> <input type='checkbox' class='checklist-checkbox' checked disabled> <i>OsmoTRX: make port numbers configurable in the protocol preferences</i> set to Done</li></ul><blockquote>
<p>OsmoTRX: make port numbers configurable in the protocol preferences</p>
</blockquote>
<p>As it turns out, Wireshark adds some protocol preferences automatically.</p> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155432019-08-06T10:54:50Zfixeria
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Feedback</i></li></ul><p>I think I am done. <a class="user active" href="https://osmocom.org/users/7">laforge</a>, <a class="user active" href="https://osmocom.org/users/30187">pespin</a>, please see the latest patch version.</p>
<p><a class="external" href="https://code.wireshark.org/review/#/c/26796/16/">https://code.wireshark.org/review/#/c/26796/16/</a></p>
<blockquote>
<p>OsmoTRXD: verify the burst length (using modulation type)</p>
</blockquote>
<p>This check can be implemented later if needed.</p> OsmoTRX - Feature #4081: Add dissector for OsmoTRX protocolhttps://osmocom.org/issues/4081?journal_id=155982019-08-14T10:12:00Zfixeria
<ul><li><strong>Status</strong> changed from <i>Feedback</i> to <i>Resolved</i></li></ul><p>The patch has finally landed to Wireshark, enjoy!</p>
<p><a class="user active" href="https://osmocom.org/users/7">laforge</a> as you're the reporter of <a class="external" href="https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14814">https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=14814</a>, feel free to close it now.</p>