<div dir="ltr">On Sun, Jun 22, 2014 at 7:34 AM, Aleksander Morgado <span dir="ltr"><<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Greg,<br>
<div class=""><br>
On Fri, Jun 13, 2014 at 3:51 PM, Aleksander Morgado<br>
<<a href="mailto:aleksander@aleksander.es">aleksander@aleksander.es</a>> wrote:<br>
> On Mon, Jun 9, 2014 at 9:00 PM, Greg Suarez <<a href="mailto:gpsuarez2512@gmail.com">gpsuarez2512@gmail.com</a>> wrote:<br>
>> Previously mbim-proxy tracked and filtered indications of non-standard<br>
>> services based on whether a client has sent a message to the device with<br>
>> a matching UUID. This patch removes that tracking and filtering and<br>
>> instead tracks and filters indications based on the client sending<br>
>> the MBIM_CID_BASIC_CONNECT_DEVICE_SERVICE_SUBSCRIBE_LIST command to<br>
>> the device.<br>
>><br>
>> The client will receive all indications if it never sends the<br>
>> MBIM_CID_BASIC_CONNECT_DEVICE_SERVICE_SUBSCRIBE_LIST command.<br>
>> Once the client sends the<br>
>> MBIM_CID_BASIC_CONNECT_DEVICE_SERVICE_SUBSCRIBE_LIST command, the<br>
>> client will only receive indications specified in the command.<br>
><br>
> Pushed to the greg/proxy branch for now; I'll try to review it this weekend.<br>
><br>
<br>
</div>Reviewed it now; and also added some more commits o the greg/proxy<br>
branch. Please re-create your local branch based on the new greg/proxy<br>
branch because I rebased it.<br></blockquote><div><br></div><div>Thanks<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
There's one key thing missing still in the behavior we're supposed to<br>
see w.r.t. indications. In the current code with your latest patches,<br>
there is no difference between standard and non-standard services. If<br>
I'm not mistaken, by default all indications of "standard" services<br>
should be treated as enabled; while all indications of non-standard<br>
services shouldn't. <br></blockquote><div><br></div><div>Just so I understand what you're saying...<br></div><div>So Client-A enables indications for non-standard services and then Client-B<br></div><div>connects to the proxy and expects the non-standard indications disabled.<br>
</div><div>So the proxy shouldn't forward the non-standard indications to Client-B until<br>Client-B enables the indications, correct?<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 Is that something easy to add?<br></blockquote><div><br></div><div>Yes, I think so.  I should be able to get to it this week.<br><br></div><div>Regards,<br><br>Greg <br></div></div></div></div>