From: Dhananjay Phadke <dhananjay@netxen.com>
To: David Miller <davem@davemloft.net>
Cc: "netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Re: [patch next 4/4] netxen: disable rss for GbE ports
Date: Wed, 6 May 2009 16:06:30 -0700 [thread overview]
Message-ID: <4A0217F6.7060005@netxen.com> (raw)
In-Reply-To: <20090506.153714.122287244.davem@davemloft.net>
The reason for RSS was to distribute the ingress traffic
to different CPUs. At 10Gbps, without RSS, one CPU was
saturating, acting as bottleneck. I can do 1Gbps with
only one CPU only 50% saturated, this to on a low end
server.
The fallout is more msix vectors being consumed for RSS,
when they don't really bring any performance gain.
Since NetXen has single driver, unlike others, it needs
to be dealt based on NIC board type.
-Dhananjay
David Miller wrote:
> From: Dhananjay Phadke <dhananjay@netxen.com>
> Date: Tue, 5 May 2009 22:05:09 -0700
>
>> There's no advantage of receive side scaling on GbE ports, so
>> restrict rss support to 10GbE ports.
>>
>> Signed-off-by: Dhananjay Phadke <dhananjay@netxen.com>
>
> This is your opinion. And even if it is more than opinion,
> you give zero technical arguments in your commit message so
> we can see why this should be done.
>
> Tell that opinion to the person with 10 netxen cards in their
> machine running at full tilt.
>
> Tell that opinion to the person with extremely complicated and
> cpu intensive netfilter and routing rules in use.
>
> I'm not applying this patch.
next prev parent reply other threads:[~2009-05-06 23:06 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-06 5:05 [patch next 0/4] netxen fixes Dhananjay Phadke
2009-05-06 5:05 ` [patch next 1/4] netxen: fix race in tx ring acccess Dhananjay Phadke
2009-05-06 22:35 ` David Miller
2009-05-06 5:05 ` [patch next 2/4] netxen: fix mac list management Dhananjay Phadke
2009-05-06 22:35 ` David Miller
2009-05-06 5:05 ` [patch next 3/4] netxen: fix bonding support Dhananjay Phadke
2009-05-06 22:35 ` David Miller
2009-10-23 5:57 ` Eric W. Biederman
2009-10-24 1:54 ` Dhananjay Phadke
2009-10-24 2:13 ` Eric W. Biederman
2009-10-24 17:29 ` Dhananjay Phadke
2009-10-27 5:50 ` Eric W. Biederman
2009-10-27 6:06 ` Dhananjay Phadke
2009-05-06 5:05 ` [patch next 4/4] netxen: disable rss for GbE ports Dhananjay Phadke
2009-05-06 22:37 ` David Miller
2009-05-06 23:06 ` Dhananjay Phadke [this message]
2009-05-06 23:31 ` David Miller
2009-05-07 0:08 ` Dhananjay Phadke
2009-05-07 0:28 ` David Miller
2009-05-07 0:31 ` Dhananjay Phadke
2009-05-07 0:40 ` David Miller
2009-05-07 0:42 ` Dhananjay Phadke
2009-05-07 6:15 ` Jesper Dangaard Brouer
2009-05-07 23:44 ` Dhananjay Phadke
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A0217F6.7060005@netxen.com \
--to=dhananjay@netxen.com \
--cc=davem@davemloft.net \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).