r8152: limit the RX buffer size of RTL8153A for USB 2.0
authorHayes Wang <hayeswang@realtek.com>
Fri, 19 Mar 2021 07:37:21 +0000 (15:37 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 19 Mar 2021 18:55:09 +0000 (11:55 -0700)
commitf91a50d8b51b5c8ef1cfb08115a005bba4250507
tree9a16d7dc2c195ec434e618e25a34752a43152cb7
parent8a2dc6af67a0c9f65a22ea40fc79974ee8f368c7
r8152: limit the RX buffer size of RTL8153A for USB 2.0

If the USB host controller is EHCI, the throughput is reduced from
300Mb/s to 60Mb/s, when the rx buffer size is modified from 16K to
32K.

According to the EHCI spec, the maximum size of the qTD is 20K.
Therefore, when the driver uses more than 20K buffer, the latency
time of EHCI would be increased. And, it let the RTL8153A get worse
throughput.

However, the driver uses alloc_pages() for rx buffer, so I limit
the rx buffer to 16K rather than 20K.

BugLink: https://bugzilla.kernel.org/show_bug.cgi?id=205923
Fixes: ec5791c202ac ("r8152: separate the rx buffer size")
Reported-by: Robert Davies <robdavies1977@gmail.com>
Signed-off-by: Hayes Wang <hayeswang@realtek.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/usb/r8152.c