<div dir="ltr"><div><div><div>Hi everybody,<br><br></div>I've just sent the patches to libqmi. Waiting for feedbacks<br><br></div>Best regards,<br></div>Carlo<br></div><br><div class="gmail_quote"><div dir="ltr">On Wed, 15 Feb 2017 at 10:34 Daniele Palmas <<a href="mailto:dnlplm@gmail.com">dnlplm@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Bjørn and all,<br class="gmail_msg">
<br class="gmail_msg">
2017-01-17 16:12 GMT+01:00 Bjørn Mork <<a href="mailto:bjorn@mork.no" class="gmail_msg" target="_blank">bjorn@mork.no</a>>:<br class="gmail_msg">
> Daniele Palmas <<a href="mailto:dnlplm@gmail.com" class="gmail_msg" target="_blank">dnlplm@gmail.com</a>> writes:<br class="gmail_msg">
>> 2017-01-12 14:53 GMT+01:00 Bjørn Mork <<a href="mailto:bjorn@mork.no" class="gmail_msg" target="_blank">bjorn@mork.no</a>>:<br class="gmail_msg">
>>> Daniele Palmas <<a href="mailto:dnlplm@gmail.com" class="gmail_msg" target="_blank">dnlplm@gmail.com</a>> writes:<br class="gmail_msg">
>>><br class="gmail_msg">
>>>> 2017-01-11 21:21 GMT+01:00 Bjørn Mork <<a href="mailto:bjorn@mork.no" class="gmail_msg" target="_blank">bjorn@mork.no</a>>:<br class="gmail_msg">
>>>><br class="gmail_msg">
>>>> Thanks!! I'll try to take a look at this and, if possible, do some tests!<br class="gmail_msg">
>>><br class="gmail_msg">
>>> I know I wasn't going to spend more time on this, but the attached<br class="gmail_msg">
>>> "Doh!" fix showed up in the back of my head while I was sleeping.  It<br class="gmail_msg">
>>> saves you from testing how many netdes your system can handle before it<br class="gmail_msg">
>>> slowly stops ;)<br class="gmail_msg">
>>><br class="gmail_msg">
>>> Note that it still doesn't work. I'm doing something wrong when looking<br class="gmail_msg">
>>> up the interfaces. But the lookup code should probably be made much more<br class="gmail_msg">
>>> efficient than looping over a list anyway, since we'll do it for each<br class="gmail_msg">
>>> packet received.<br class="gmail_msg">
>><br class="gmail_msg">
>> I'm slowly taking a look and I'm not an expert, so please forgive me<br class="gmail_msg">
>> if I say something silly.<br class="gmail_msg">
>><br class="gmail_msg">
>> In qmimux_find_dev should netdev_for_each_lower_dev replaced by<br class="gmail_msg">
>> something like netdev_for_each_upper_dev_rcu ?<br class="gmail_msg">
>><br class="gmail_msg">
>> Similar fix also in the other parts (e.g. add_mux_show).<br class="gmail_msg">
><br class="gmail_msg">
> Yes, that is probably correct.  I have a vertigo issue with network<br class="gmail_msg">
> device stacking ;)<br class="gmail_msg">
><br class="gmail_msg">
<br class="gmail_msg">
attached my fixes to Bjørn's implementation to be applied on top of<br class="gmail_msg">
Bjørn's patches, not sure if this is how they should be done, but<br class="gmail_msg">
things seem to work properly since I'm now able to have two concurrent<br class="gmail_msg">
data connections.<br class="gmail_msg">
<br class="gmail_msg">
qmi_disconnect implementation is due to a problem when disconnecting<br class="gmail_msg">
the device (or removing the kernel module) if a qmimux network<br class="gmail_msg">
interface was still present.<br class="gmail_msg">
<br class="gmail_msg">
Carlo is patching libqmi to add support at the userspace level and<br class="gmail_msg">
will send the patches once available.<br class="gmail_msg">
<br class="gmail_msg">
Daniele<br class="gmail_msg">
<br class="gmail_msg">
><br class="gmail_msg">
> Bjørn<br class="gmail_msg">
</blockquote></div>