lkml.org 
[lkml]   [2016]   [Jul]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v2 1/5] ethernet: add sun8i-emac driver
Date
On Wednesday, July 20, 2016 10:03:16 AM CEST LABBE Corentin wrote:
> +
> + /* Benched on OPIPC with 100M, setting more than 256 does not give any
> + * perf boost
> + */
> + priv->nbdesc_rx = 128;
> + priv->nbdesc_tx = 256;
> +
>

256 tx descriptors can introduce a significant latency. Can you add
support for BQL (netdev_sent_queue/netdev_completed_queue) to limit
the queue size to the minimum?

I also noticed that your tx_lock() prevents you from concurrently
running sun8i_emac_complete_xmit() and sun8i_emac_xmit(). Is that
necessary? I'd think that you can find a way to make them work
concurrently.

Arnd

\
 
 \ /
  Last update: 2016-07-20 12:41    [W:0.234 / U:0.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site