From: David Miller <davem@davemloft.net>
To: hariprasad@chelsio.com
Cc: netdev@vger.kernel.org, leedom@chelsio.com, nirranjan@chelsio.com
Subject: Re: [PATCH net-next 1/3] cxgb4: Add support for loopback between VI of same port
Date: Sun, 17 May 2015 23:46:13 -0400 (EDT) [thread overview]
Message-ID: <20150517.234613.463972132331185537.davem@davemloft.net> (raw)
In-Reply-To: <1431859523-7423-2-git-send-email-hariprasad@chelsio.com>
From: Hariprasad Shenai <hariprasad@chelsio.com>
Date: Sun, 17 May 2015 16:15:21 +0530
> We now have a new cxgb4 module parameter "tx_vf" that when set
> to a non-zero value, causes cxgb4 to use the "Virtual Machine" version
> of the firmware Ethernet TX Packet Work Request (FW_ETH_TX_PKT_VM_WR)
> instead of the "normal" default non-VM Work Request (FW_ETH_TX_PKT_WR).
> This allows TX Packets sent by the cxgb4 PF Driver to be subject to the
> firmware's MPS TCAM Lookup and therefore elligable for loop back to other
> Virtual interfaces on the same port. This is useful for communicating with
> Virtual machines running the VF Driver (cxgb4vf) and also for interesting
> layered service applications. Enabling it by default lowers the
> performance, so maybe module parameter is the right way to this.
>
> Based on original work by Casey Leedom <leedom@chelsio.com>
>
> Signed-off-by: Hariprasad Shenai <hariprasad@chelsio.com>
Sorry, module parameters are veboten.
Especially for settings like this which are guaranteed to be
interesting for other NIC drivers, not just your's.
I'm really tired of explaining this to driver authors. Just
don't even try to push a module parameter past me.
Thanks.
next prev parent reply other threads:[~2015-05-18 3:46 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-17 10:45 [PATCH net-next 0/3] cxgb4/cxgb4vf: Adds VI loopback, FL starvation support and cleanup Hariprasad Shenai
2015-05-17 10:45 ` [PATCH net-next 1/3] cxgb4: Add support for loopback between VI of same port Hariprasad Shenai
2015-05-18 3:46 ` David Miller [this message]
2015-05-21 16:36 ` Casey Leedom
2015-05-21 19:30 ` David Miller
2015-05-22 16:49 ` Casey Leedom
2015-05-22 18:01 ` David Miller
2015-05-22 23:46 ` Casey Leedom
2015-05-17 10:45 ` [PATCH net-next 2/3] cxgb4: Add debugfs facility to inject FL starvation Hariprasad Shenai
2015-05-17 10:45 ` [PATCH net-next 3/3] cxgb4/cxgb4vf: function and argument name cleanup Hariprasad Shenai
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=20150517.234613.463972132331185537.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=hariprasad@chelsio.com \
--cc=leedom@chelsio.com \
--cc=netdev@vger.kernel.org \
--cc=nirranjan@chelsio.com \
/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).