Message ID | 20101006181458.GA6817@mcarlson.broadcom.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Le mercredi 06 octobre 2010 à 11:14 -0700, Matt Carlson a écrit : > Yes. We were missing a call to this function in the legacy case. > > > [PATCH net-next] tg3: Set real_num_rx_queues for non-multiq devs > > Commit 2ddaad397c47de012dfb956b0c05540da1a0dde5 entitled "tg3: Use > netif_set_real_num_{rx,tx}_queues()" added a new call to > netif_set_real_num_rx_queues in tg3_enable_msix(). This call also needs > to be added to the legacy path to correctly reflect the actual number of > rx queues. > > Signed-off-by: Matt Carlson <mcarlson@broadcom.com> > --- > drivers/net/tg3.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c > index 16e1a95..e5b9ec5 100644 > --- a/drivers/net/tg3.c > +++ b/drivers/net/tg3.c > @@ -8906,6 +8906,7 @@ defcfg: > tp->irq_cnt = 1; > tp->napi[0].irq_vec = tp->pdev->irq; > netif_set_real_num_tx_queues(tp->dev, 1); > + netif_set_real_num_rx_queues(tp->dev, 1); > } > } > Thanks, this fixed the thing for me, once device is UP # insmod drivers/net/tg3.ko # grep . /sys/class/net/eth3/queues/*/rps_cpus /sys/class/net/eth3/queues/rx-0/rps_cpus:00000000 /sys/class/net/eth3/queues/rx-1/rps_cpus:00000000 /sys/class/net/eth3/queues/rx-2/rps_cpus:00000000 /sys/class/net/eth3/queues/rx-3/rps_cpus:00000000 /sys/class/net/eth3/queues/rx-4/rps_cpus:00000000 # echo ff >/sys/class/net/eth3/queues/rx-2/rps_cpus # grep . /sys/class/net/eth3/queues/*/rps_cpus /sys/class/net/eth3/queues/rx-0/rps_cpus:00000000 /sys/class/net/eth3/queues/rx-1/rps_cpus:00000000 /sys/class/net/eth3/queues/rx-2/rps_cpus:000000ff /sys/class/net/eth3/queues/rx-3/rps_cpus:00000000 /sys/class/net/eth3/queues/rx-4/rps_cpus:00000000 # ifconfig eth3 up # grep . /sys/class/net/eth3/queues/*/rps_cpus 00000000 Acked-by: Eric Dumazet <eric.dumazet@gmail.com> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
From: Eric Dumazet <eric.dumazet@gmail.com> Date: Wed, 06 Oct 2010 20:49:05 +0200 > Le mercredi 06 octobre 2010 à 11:14 -0700, Matt Carlson a écrit : > >> Yes. We were missing a call to this function in the legacy case. >> >> >> [PATCH net-next] tg3: Set real_num_rx_queues for non-multiq devs >> >> Commit 2ddaad397c47de012dfb956b0c05540da1a0dde5 entitled "tg3: Use >> netif_set_real_num_{rx,tx}_queues()" added a new call to >> netif_set_real_num_rx_queues in tg3_enable_msix(). This call also needs >> to be added to the legacy path to correctly reflect the actual number of >> rx queues. >> >> Signed-off-by: Matt Carlson <mcarlson@broadcom.com> ... > Acked-by: Eric Dumazet <eric.dumazet@gmail.com> Applied, thanks everyone. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c index 16e1a95..e5b9ec5 100644 --- a/drivers/net/tg3.c +++ b/drivers/net/tg3.c @@ -8906,6 +8906,7 @@ defcfg: tp->irq_cnt = 1; tp->napi[0].irq_vec = tp->pdev->irq; netif_set_real_num_tx_queues(tp->dev, 1); + netif_set_real_num_rx_queues(tp->dev, 1); } }