DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: add limitation of VF VLAN strip function
@ 2017-11-03  3:40 Beilei Xing
  2017-11-03 15:36 ` Mcnamara, John
  0 siblings, 1 reply; 3+ messages in thread
From: Beilei Xing @ 2017-11-03  3:40 UTC (permalink / raw)
  To: jingjing.wu; +Cc: dev

This patch adds limitation of VF VLAN strip function.

Signed-off-by: Beilei Xing <beilei.xing@intel.com>
---
 doc/guides/nics/i40e.rst | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/doc/guides/nics/i40e.rst b/doc/guides/nics/i40e.rst
index 05c1875..214fada 100644
--- a/doc/guides/nics/i40e.rst
+++ b/doc/guides/nics/i40e.rst
@@ -478,6 +478,11 @@ enabled using the steps below.
 
       setpci -s <XX:XX.X> a8.w=<value>
 
+Vlan strip of VF
+~~~~~~~~~~~~~~~~
+
+VF vlan strip function is supported from i40e kernel driver 2.1.26.
+
 High Performance of Small Packets on 40G NIC
 --------------------------------------------
 
-- 
2.5.5

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: add limitation of VF VLAN strip function
  2017-11-03  3:40 [dpdk-dev] [PATCH] doc: add limitation of VF VLAN strip function Beilei Xing
@ 2017-11-03 15:36 ` Mcnamara, John
  2017-11-03 22:58   ` Ferruh Yigit
  0 siblings, 1 reply; 3+ messages in thread
From: Mcnamara, John @ 2017-11-03 15:36 UTC (permalink / raw)
  To: Xing, Beilei, Wu, Jingjing; +Cc: dev



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Beilei Xing
> Sent: Friday, November 3, 2017 3:41 AM
> To: Wu, Jingjing <jingjing.wu@intel.com>
> Cc: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH] doc: add limitation of VF VLAN strip function
> 
> This patch adds limitation of VF VLAN strip function.
> 
> Signed-off-by: Beilei Xing <beilei.xing@intel.com>
> ---
>  doc/guides/nics/i40e.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/nics/i40e.rst b/doc/guides/nics/i40e.rst index
> 05c1875..214fada 100644
> --- a/doc/guides/nics/i40e.rst
> +++ b/doc/guides/nics/i40e.rst
> @@ -478,6 +478,11 @@ enabled using the steps below.
> 
>        setpci -s <XX:XX.X> a8.w=<value>
> 
> +Vlan strip of VF
> +~~~~~~~~~~~~~~~~
> +
> +VF vlan strip function is supported from i40e kernel driver 2.1.26.
> +
>  High Performance of Small Packets on 40G NIC
>  --------------------------------------------

Probably would be slightly better as:

The VF vlan strip function is only supported in the i40e kernel driver >= 2.1.26.

However, the existing text is probably okay so:

Acked-by: John McNamara <john.mcnamara@intel.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: add limitation of VF VLAN strip function
  2017-11-03 15:36 ` Mcnamara, John
@ 2017-11-03 22:58   ` Ferruh Yigit
  0 siblings, 0 replies; 3+ messages in thread
From: Ferruh Yigit @ 2017-11-03 22:58 UTC (permalink / raw)
  To: Mcnamara, John, Xing, Beilei, Wu, Jingjing; +Cc: dev

On 11/3/2017 8:36 AM, Mcnamara, John wrote:
> 
> 
>> -----Original Message-----
>> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Beilei Xing
>> Sent: Friday, November 3, 2017 3:41 AM
>> To: Wu, Jingjing <jingjing.wu@intel.com>
>> Cc: dev@dpdk.org
>> Subject: [dpdk-dev] [PATCH] doc: add limitation of VF VLAN strip function
>>
>> This patch adds limitation of VF VLAN strip function.
>>
>> Signed-off-by: Beilei Xing <beilei.xing@intel.com>

> Acked-by: John McNamara <john.mcnamara@intel.com>

Applied to dpdk-next-net/master, thanks.

(used suggested text, thanks.)

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-11-03 22:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-03  3:40 [dpdk-dev] [PATCH] doc: add limitation of VF VLAN strip function Beilei Xing
2017-11-03 15:36 ` Mcnamara, John
2017-11-03 22:58   ` Ferruh Yigit

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).