[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20081116.233006.128063706.davem@davemloft.net>
Date: Sun, 16 Nov 2008 23:30:06 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: robert@...ur.slu.se
Cc: jeffrey.t.kirsher@...el.com, netdev@...r.kernel.org,
jeff@...zik.org, jesse.brandeburg@...el.com,
robert.olsson@....uu.se
Subject: Re: [PATCH] pktgen: fix multiple queue warning
From: Robert Olsson <robert@...ur.slu.se>
Date: Sat, 8 Nov 2008 12:29:32 +0100
> Jeff Kirsher writes:
>
> > - if (ntxq <= num_online_cpus() && (pkt_dev->flags & F_QUEUE_MAP_CPU)) {
> > + if (ntxq > num_online_cpus() && (pkt_dev->flags & F_QUEUE_MAP_CPU)) {
> > printk(KERN_WARNING "pktgen: WARNING: QUEUE_MAP_CPU "
> > - "disabled because CPU count (%d) exceeds number ",
> > - num_online_cpus());
> > - printk(KERN_WARNING "pktgen: WARNING: of tx queues "
> > - "(%d) on %s \n", ntxq, pkt_dev->odev->name);
>
> This triggers with the niu driver which has 12 TX queues on our 8 CPU-core
> system. Might be best to make sure that pkt_dev->cur_queue_map never exceeds
> ntxq.
>
> pkt_dev->cur_queue_map = pkt_dev->cur_queue_map % pkt_dev->odev->real_num_tx_queues;
Can someone cook up a patch to fix this?
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists