Re: [PATCH net-next 1/1] net: usb: qmi_wwan: add default rx_urb_size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Daniele,

On Thu, Nov 12, 2020 at 7:29 PM Daniele Palmas <dnlplm@xxxxxxxxx> wrote:
> thanks for testing. Still thinking it could be better to differentiate
> between raw-ip and qmap, but not yet able to find the time to perform
> some tests on my own.

I agree that separating between qmap and non-qmap would be nice.
However, with my modules I have not noticed any issues when using 32KB
as the URB size. Still, the results show that there is no gain in
increasing the aggregation size from 16 to 32KB. Capturing traffic
from the modem reveals that the hardware still only generates 16KB
URBs (even in high-speed networks). I also see that for example the
r8152 driver uses a static URB size of 16384.

> Is the dongle driver based on usbnet? Besides the aggregated datagram
> size, did you also try different datagram max numbers?

The dongle driver is not based on usbnet, it is r8152. I tried to
increase the maximum datagrams from 32 to 64 (as well as some other
values), but it had no effect on the perfrormance.

> The only advice I can give you is to check if other drivers are
> performing better, e.g. did you try the MBIM composition? not sure it
> will make much difference, since it's based on usbnet, but could be
> worth trying.

I tried to use MBIM, but the performance was the same as with QMI. I
will take a look at r8152 and experiment with implementing some of the
differences in usbnet/qmi_wwan. I see for example that r8152 uses
NAPI, which while not a perfect fit for USB could be worth a try.
Based on some discussions I found on the mailing list from 2011,
implementing NAPI in usbnet could be worthwhile.

Thanks for the reply!

BR,
Kristian



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux