nfp: ccm: increase message limits
authorDirk van der Merwe <dirk.vandermerwe@netronome.com>
Tue, 9 Jul 2019 02:53:11 +0000 (19:53 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 9 Jul 2019 03:21:09 +0000 (20:21 -0700)
Increase the batch limit to consume small message bursts more
effectively. Practically, the effect on the 'add' messages is not
significant since the mailbox is sized such that the 'add' messages are
still limited to the same order of magnitude that it was originally set
for.

Furthermore, increase the queue size limit to 1024 entries. This further
improves the handling of bursts of small control messages.

Signed-off-by: Dirk van der Merwe <dirk.vandermerwe@netronome.com>
Reviewed-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/netronome/nfp/ccm_mbox.c

index d160ac794d9872f452ad36a03727251c8ead49cd..f0783aa9e66e08f16718b8eff3f0557e8d82e3b3 100644 (file)
@@ -13,7 +13,7 @@
  * form a batch.  Threads come in with CMSG formed in an skb, then
  * enqueue that skb onto the request queue.  If threads skb is first
  * in queue this thread will handle the mailbox operation.  It copies
- * up to 16 messages into the mailbox (making sure that both requests
+ * up to 64 messages into the mailbox (making sure that both requests
  * and replies will fit.  After FW is done processing the batch it
  * copies the data out and wakes waiting threads.
  * If a thread is waiting it either gets its the message completed
@@ -23,9 +23,9 @@
  * to limit potential cache line bounces.
  */
 
-#define NFP_CCM_MBOX_BATCH_LIMIT       16
+#define NFP_CCM_MBOX_BATCH_LIMIT       64
 #define NFP_CCM_TIMEOUT                        (NFP_NET_POLL_TIMEOUT * 1000)
-#define NFP_CCM_MAX_QLEN               256
+#define NFP_CCM_MAX_QLEN               1024
 
 enum nfp_net_mbox_cmsg_state {
        NFP_NET_MBOX_CMSG_STATE_QUEUED,