<div dir="ltr">Hello.<div><br></div><div>"Bearer control mode" is added in Release 7 as I am aware of. It's not vendor specific.</div><div>You can check here: <a href="http://www.tech-invite.com/3m29/tinv-3gpp-29-060.html">http://www.tech-invite.com/3m29/tinv-3gpp-29-060.html</a></div><div><br></div><div>"If Bearer Control Mode is provided by the GGSN in the PCO, the Bearer Control Mode IE shall be included in order to
inform the SGSN about the bearer control mode and shall indicate the same bearer control mode as indicated to the MS
in the PCO."</div><div><br></div><div>Wireshark (I'm using latest version) decodes this information element with no problem.</div><div><br></div><div>There's probably a method to select Release used by GGSN but the GGSN tests were done on is in a live mobile network and doing configuration changes on it is not an option.</div><div><br></div><div><br></div><div>I will make those changes in these files in a proper way so there's no confusion what is added and what is deleted (actually nothing is deleted).</div><div>I will use <span style="font-size:12.8px">git send-email/</span><span style="font-size:12.8px">git format-patch. I suppose the e-mail to send is the same (<a href="mailto:osmocom-net-gprs@lists.osmocom.org">osmocom-net-gprs@lists.osmocom.org</a>)?</span></div><div><br></div><div><br></div><div>Greetings.</div><div class="gmail_extra"><br><div class="gmail_quote">2016-09-14 0:43 GMT+02:00 Neels Hofmeyr <span dir="ltr"><<a href="mailto:nhofmeyr@sysmocom.de" target="_blank">nhofmeyr@sysmocom.de</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for your patch!<br>
It looks good in principle, but let's clarify some details.<br>
<br>
I can't find the 'Bearer Control Mode' in the GTP spec I have here (ETSI TS 129<br>
060 V6.9.0 (2005-06) / 3GPP TS 29.060 version 6.9.0 Release 6), so it looks<br>
like a vendor specific extension. You don't happen to have a spec on it?<br>
<br>
Your patch includes some unwanted whitespace changes, it would be great to get<br>
rid of those. I'm marking them below:<br>
<span class=""><br>
On Tue, Sep 13, 2016 at 10:00:37AM +0200, BJovke . wrote:<br>
>  - if (j < GTPIE_SIZE) {<br>
>  + case GTPIE_BCM:<br>
>  +      if (j < GTPIE_SIZE) {<br>
<br>
</span>The 'if (j...' line shouldn't change.<br>
<span class=""><br>
>  - iesize = 2;<br>
>  + case GTPIE_BCM:<br>
>  +        iesize = 2;<br>
<br>
</span>The iesize line shouldn't change. (x2)<br>
<br>
In fact, those whitespace changes look weird, because in the upstream file<br>
version, the indenting you're apparently adding is already present.<br>
<br>
It is easiest for us if you submit your patch using git send-email, or at least<br>
git format-patch, which also ensures that we get a commit log message and<br>
author information, and that the patch is based on our master HEAD.<br>
<br>
It would be great if you could resolve these issues. Thanks again!<br>
<span class="HOEnZb"><font color="#888888"><br>
~Neels<br>
<br>
--<br>
- Neels Hofmeyr <<a href="mailto:nhofmeyr@sysmocom.de">nhofmeyr@sysmocom.de</a>>          <a href="http://www.sysmocom.de/" rel="noreferrer" target="_blank">http://www.sysmocom.de/</a><br>
==============================<wbr>==============================<wbr>===========<br>
* sysmocom - systems for mobile communications GmbH<br>
* Alt-Moabit 93<br>
* 10559 Berlin, Germany<br>
* Sitz / Registered office: Berlin, HRB 134158 B<br>
* Geschäftsführer / Managing Directors: Harald Welte<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div><br></div><div>Jovan Bunjevački.<br></div></div>
</div></div>