tg3: Assign rx ret producer indexes by vector

When RSS is enabled, the status block format changes slightly.  The
"rx_jumbo_consumer", "reserved", and "rx_mini_consumer" members get
mapped to the other three rx return ring producer indexes.  This patch
introduces a new per-interrupt member which identifies which location
in the status block a particular vector should look for return ring
updates.

Signed-off-by: Matt Carlson <mcarlson@broadcom.com>
Reviewed-by: Benjamin Li <benli@broadcom.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 27 insertions, 4 deletions

diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c
index 5782511c945..f71ea462ca4 100644
--- a/drivers/net/tg3.c
+++ b/drivers/net/tg3.c
@@ -667,7 +667,7 @@ static inline unsigned int tg3_has_work(struct tg3_napi *tnapi)
         }
         /* check for RX/TX work to do */
         if (sblk->idx[0].tx_consumer != tnapi->tx_cons ||
-            sblk->idx[0].rx_producer != tnapi->rx_rcb_ptr)
+            *(tnapi->rx_rcb_prod_idx) != tnapi->rx_rcb_ptr)
                 work_exists = 1;
 
         return work_exists;
@@ -4518,7 +4518,7 @@ static int tg3_rx(struct tg3_napi *tnapi, int budget)
         int received;
         struct tg3_rx_prodring_set *tpr = &tp->prodring[0];
 
-        hw_idx = tnapi->hw_status->idx[0].rx_producer;
+        hw_idx = *(tnapi->rx_rcb_prod_idx);
         /*
          * We need to order the read of hw_idx and the read of
          * the opaque cookie.
@@ -4649,7 +4649,7 @@ next_pkt_nopost:
 
                 /* Refresh hw_idx to see if there is new work */
                 if (sw_idx == hw_idx) {
-                        hw_idx = tnapi->hw_status->idx[0].rx_producer;
+                        hw_idx = *(tnapi->rx_rcb_prod_idx);
                         rmb();
                 }
         }
@@ -4711,7 +4711,7 @@ static int tg3_poll_work(struct tg3_napi *tnapi, int work_done, int budget)
          * All RX "locking" is done by ensuring outside
          * code synchronizes with tg3->napi.poll()
          */
-        if (sblk->idx[0].rx_producer != tnapi->rx_rcb_ptr)
+        if (*(tnapi->rx_rcb_prod_idx) != tnapi->rx_rcb_ptr)
                 work_done += tg3_rx(tnapi, budget - work_done);
 
         return work_done;
@@ -5896,6 +5896,7 @@ static int tg3_alloc_consistent(struct tg3 *tp)
 
         for (i = 0; i < tp->irq_cnt; i++) {
                 struct tg3_napi *tnapi = &tp->napi[i];
+                struct tg3_hw_status *sblk;
 
                 tnapi->hw_status = pci_alloc_consistent(tp->pdev,
                                                         TG3_HW_STATUS_SIZE,
@@ -5904,6 +5905,28 @@ static int tg3_alloc_consistent(struct tg3 *tp)
                         goto err_out;
 
                 memset(tnapi->hw_status, 0, TG3_HW_STATUS_SIZE);
+                sblk = tnapi->hw_status;
+
+                /*
+                 * When RSS is enabled, the status block format changes
+                 * slightly.  The "rx_jumbo_consumer", "reserved",
+                 * and "rx_mini_consumer" members get mapped to the
+                 * other three rx return ring producer indexes.
+                 */
+                switch (i) {
+                default:
+                        tnapi->rx_rcb_prod_idx = &sblk->idx[0].rx_producer;
+                        break;
+                case 2:
+                        tnapi->rx_rcb_prod_idx = &sblk->rx_jumbo_consumer;
+                        break;
+                case 3:
+                        tnapi->rx_rcb_prod_idx = &sblk->reserved;
+                        break;
+                case 4:
+                        tnapi->rx_rcb_prod_idx = &sblk->rx_mini_consumer;
+                        break;
+                }
 
                 /*
                  * If multivector RSS is enabled, vector 0 does not handle