be2net: get rid of TX budget
authorSathya Perla <sathya.perla@emulex.com>
Tue, 2 Sep 2014 04:26:55 +0000 (09:56 +0530)
committerDavid S. Miller <davem@davemloft.net>
Tue, 2 Sep 2014 19:46:00 +0000 (12:46 -0700)
Enforcing a budget on the TX completion processing in NAPI doesn't
benefit performance in anyway. Just get rid of it.

Signed-off-by: Sathya Perla <sathya.perla@emulex.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/emulex/benet/be.h
drivers/net/ethernet/emulex/benet/be_main.c

index 5ccfce7d616dbf86048a455d3841da8cc1f60ca6..d952ca2c0678c30c5c972c390852298f423d715c 100644 (file)
@@ -114,7 +114,6 @@ static inline char *nic_name(struct pci_dev *pdev)
 #define MAX_ROCE_EQS           5
 #define MAX_MSIX_VECTORS       32
 #define MIN_MSIX_VECTORS       1
-#define BE_TX_BUDGET           256
 #define BE_NAPI_WEIGHT         64
 #define MAX_RX_POST            BE_NAPI_WEIGHT /* Frags posted at a time */
 #define RX_FRAGS_REFILL_WM     (RX_Q_LEN - MAX_RX_POST)
@@ -200,7 +199,6 @@ struct be_eq_obj {
 
        u8 idx;                 /* array index */
        u8 msix_idx;
-       u16 tx_budget;
        u16 spurious_intr;
        struct napi_struct napi;
        struct be_adapter *adapter;
index fe007ea4a01c3947919211640c6ebe44dd95f526..b31b0f0789dbb6389e545104c2a52b01ec0f2b6c 100644 (file)
@@ -2143,7 +2143,6 @@ static int be_evt_queues_create(struct be_adapter *adapter)
                napi_hash_add(&eqo->napi);
                aic = &adapter->aic_obj[i];
                eqo->adapter = adapter;
-               eqo->tx_budget = BE_TX_BUDGET;
                eqo->idx = i;
                aic->max_eqd = BE_MAX_EQD;
                aic->enable = true;
@@ -2459,20 +2458,19 @@ static inline void lancer_update_tx_err(struct be_tx_obj *txo, u32 status)
        }
 }
 
-static bool be_process_tx(struct be_adapter *adapter, struct be_tx_obj *txo,
-                         int budget, int idx)
+static void be_process_tx(struct be_adapter *adapter, struct be_tx_obj *txo,
+                         int idx)
 {
        struct be_eth_tx_compl *txcp;
-       int num_wrbs = 0, work_done;
+       int num_wrbs = 0, work_done = 0;
        u32 compl_status;
+       u16 last_idx;
+
+       while ((txcp = be_tx_compl_get(&txo->cq))) {
+               last_idx = GET_TX_COMPL_BITS(wrb_index, txcp);
+               num_wrbs += be_tx_compl_process(adapter, txo, last_idx);
+               work_done++;
 
-       for (work_done = 0; work_done < budget; work_done++) {
-               txcp = be_tx_compl_get(&txo->cq);
-               if (!txcp)
-                       break;
-               num_wrbs += be_tx_compl_process(adapter, txo,
-                                               GET_TX_COMPL_BITS(wrb_index,
-                                                                 txcp));
                compl_status = GET_TX_COMPL_BITS(status, txcp);
                if (compl_status) {
                        if (lancer_chip(adapter))
@@ -2497,7 +2495,6 @@ static bool be_process_tx(struct be_adapter *adapter, struct be_tx_obj *txo,
                tx_stats(txo)->tx_compl += work_done;
                u64_stats_update_end(&tx_stats(txo)->sync_compl);
        }
-       return (work_done < budget); /* Done */
 }
 
 int be_poll(struct napi_struct *napi, int budget)
@@ -2506,17 +2503,12 @@ int be_poll(struct napi_struct *napi, int budget)
        struct be_adapter *adapter = eqo->adapter;
        int max_work = 0, work, i, num_evts;
        struct be_rx_obj *rxo;
-       bool tx_done;
 
        num_evts = events_get(eqo);
 
        /* Process all TXQs serviced by this EQ */
-       for (i = eqo->idx; i < adapter->num_tx_qs; i += adapter->num_evt_qs) {
-               tx_done = be_process_tx(adapter, &adapter->tx_obj[i],
-                                       eqo->tx_budget, i);
-               if (!tx_done)
-                       max_work = budget;
-       }
+       for (i = eqo->idx; i < adapter->num_tx_qs; i += adapter->num_evt_qs)
+               be_process_tx(adapter, &adapter->tx_obj[i], i);
 
        if (be_lock_napi(eqo)) {
                /* This loop will iterate twice for EQ0 in which