DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance
@ 2017-07-03  3:57 Qi Zhang
  2017-07-03 10:01 ` Mcnamara, John
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Qi Zhang @ 2017-07-03  3:57 UTC (permalink / raw)
  To: john.mcnamara, jingjing.wu; +Cc: helin.zhang, dev, Qi Zhang

VF performance is limited by the kernel PCI extended tag setting.
Update the document to explain the known issue and the workaround.

Signed-off-by: Qi Zhang <qi.z.zhang@intel.com>
---

v2:
- follow number list format.
- improve the comments.

 doc/guides/nics/i40e.rst | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/doc/guides/nics/i40e.rst b/doc/guides/nics/i40e.rst
index 4d3c7ca..2f2cf6d 100644
--- a/doc/guides/nics/i40e.rst
+++ b/doc/guides/nics/i40e.rst
@@ -447,3 +447,30 @@ It means if APP has set the max bandwidth for that TC, it comes to no
 effect.
 It's suggested to set the strict priority mode for a TC that is latency
 sensitive but no consuming much bandwidth.
+
+VF performance is impacted by PCI extended tag setting
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+To reach maximum NIC performance in the VF the PCI extended tag must be
+enabled. The DPDK I40E PF drvier will set this feature during initialization,
+but the kernel PF driver does not. So when running traffic on a VF which is
+managed by the kernel PF driver, a significent NIC performance downgrade has
+been observed (for 64 byte packets, there is about 25% linerate downgrade for
+a 25G device and about 35% for a 40G device).
+
+For kernel version >= 4.11, the kernel's PCI driver will enable the extended
+tag if it detects that the device supports it. So by default, this is not an
+issue. For kernels <= 4.11 or When the PCI extended tag is disabled it can be
+enabled using the steps below.
+
+#. Get the current value of the PCI configure register::
+
+      setpci -s <XX:XX.X> a8.w
+
+#. Set bit 8::
+
+      value = value | 0x100
+
+#. Set the PCI configure register with new value::
+
+      setpci -s <XX:XX.X> a8.w=<value>
-- 
2.9.3

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

* Re: [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance
  2017-07-03  3:57 [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance Qi Zhang
@ 2017-07-03 10:01 ` Mcnamara, John
  2017-07-03 10:26 ` Ferruh Yigit
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Mcnamara, John @ 2017-07-03 10:01 UTC (permalink / raw)
  To: Zhang, Qi Z, Wu, Jingjing; +Cc: Zhang, Helin, dev



> -----Original Message-----
> From: Zhang, Qi Z
> Sent: Monday, July 3, 2017 4:58 AM
> To: Mcnamara, John <john.mcnamara@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>
> Cc: Zhang, Helin <helin.zhang@intel.com>; dev@dpdk.org; Zhang, Qi Z
> <qi.z.zhang@intel.com>
> Subject: [PATCH v2] doc: add known issue for i40e VF performance
> 
> VF performance is limited by the kernel PCI extended tag setting.
> Update the document to explain the known issue and the workaround.
> 
> Signed-off-by: Qi Zhang <qi.z.zhang@intel.com>

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

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

* Re: [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance
  2017-07-03  3:57 [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance Qi Zhang
  2017-07-03 10:01 ` Mcnamara, John
@ 2017-07-03 10:26 ` Ferruh Yigit
  2017-07-09 22:49 ` Thomas Monjalon
  2017-07-18  9:52 ` [dpdk-dev] [PATCH v3] " Qi Zhang
  3 siblings, 0 replies; 6+ messages in thread
From: Ferruh Yigit @ 2017-07-03 10:26 UTC (permalink / raw)
  To: Qi Zhang, john.mcnamara, jingjing.wu; +Cc: helin.zhang, dev

On 7/3/2017 4:57 AM, Qi Zhang wrote:
> VF performance is limited by the kernel PCI extended tag setting.
> Update the document to explain the known issue and the workaround.
> 
> Signed-off-by: Qi Zhang <qi.z.zhang@intel.com>

Hi Qi, John,

Not related to the content, but related to location, where a "known
issue" should go, release notes or component specific document?

Thanks,
ferruh

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

* Re: [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance
  2017-07-03  3:57 [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance Qi Zhang
  2017-07-03 10:01 ` Mcnamara, John
  2017-07-03 10:26 ` Ferruh Yigit
@ 2017-07-09 22:49 ` Thomas Monjalon
  2017-07-18  9:52 ` [dpdk-dev] [PATCH v3] " Qi Zhang
  3 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2017-07-09 22:49 UTC (permalink / raw)
  To: Qi Zhang; +Cc: dev, john.mcnamara, jingjing.wu, helin.zhang

Hi,

Few typos spotted

03/07/2017 05:57, Qi Zhang:
> --- a/doc/guides/nics/i40e.rst
> +++ b/doc/guides/nics/i40e.rst
> @@ -447,3 +447,30 @@ It means if APP has set the max bandwidth for that TC, it comes to no
>  effect.
>  It's suggested to set the strict priority mode for a TC that is latency
>  sensitive but no consuming much bandwidth.
> +
> +VF performance is impacted by PCI extended tag setting
> +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> +
> +To reach maximum NIC performance in the VF the PCI extended tag must be
> +enabled. The DPDK I40E PF drvier will set this feature during initialization,

drvier -> driver

> +but the kernel PF driver does not. So when running traffic on a VF which is
> +managed by the kernel PF driver, a significent NIC performance downgrade has

significent -> significant

> +been observed (for 64 byte packets, there is about 25% linerate downgrade for
> +a 25G device and about 35% for a 40G device).
> +
> +For kernel version >= 4.11, the kernel's PCI driver will enable the extended
> +tag if it detects that the device supports it. So by default, this is not an
> +issue. For kernels <= 4.11 or When the PCI extended tag is disabled it can be

When -> when

> +enabled using the steps below.
> +
> +#. Get the current value of the PCI configure register::
> +
> +      setpci -s <XX:XX.X> a8.w
> +
> +#. Set bit 8::
> +
> +      value = value | 0x100
> +
> +#. Set the PCI configure register with new value::
> +
> +      setpci -s <XX:XX.X> a8.w=<value>

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

* [dpdk-dev] [PATCH v3] doc: add known issue for i40e VF performance
  2017-07-03  3:57 [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance Qi Zhang
                   ` (2 preceding siblings ...)
  2017-07-09 22:49 ` Thomas Monjalon
@ 2017-07-18  9:52 ` Qi Zhang
  2017-07-18 14:59   ` Ferruh Yigit
  3 siblings, 1 reply; 6+ messages in thread
From: Qi Zhang @ 2017-07-18  9:52 UTC (permalink / raw)
  To: thomas.monjalon, ferruh.yigit; +Cc: john.mcnamara, dev, Qi Zhang

VF performance is limited by the kernel PCI extended tag setting.
Update the document to explain the known issue and the workaround.

Signed-off-by: Qi Zhang <qi.z.zhang@intel.com>
---

v3:
- fix typos errors.

v2:
- follow number list format.
- improve the comments.

 doc/guides/nics/i40e.rst | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/doc/guides/nics/i40e.rst b/doc/guides/nics/i40e.rst
index 4d3c7ca..2f2cf6d 100644
--- a/doc/guides/nics/i40e.rst
+++ b/doc/guides/nics/i40e.rst
@@ -447,3 +447,30 @@ It means if APP has set the max bandwidth for that TC, it comes to no
 effect.
 It's suggested to set the strict priority mode for a TC that is latency
 sensitive but no consuming much bandwidth.
+
+VF performance is impacted by PCI extended tag setting
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+To reach maximum NIC performance in the VF the PCI extended tag must be
+enabled. The DPDK I40E PF driver will set this feature during initialization,
+but the kernel PF driver does not. So when running traffic on a VF which is
+managed by the kernel PF driver, a significant NIC performance downgrade has
+been observed (for 64 byte packets, there is about 25% linerate downgrade for
+a 25G device and about 35% for a 40G device).
+
+For kernel version >= 4.11, the kernel's PCI driver will enable the extended
+tag if it detects that the device supports it. So by default, this is not an
+issue. For kernels <= 4.11 or when the PCI extended tag is disabled it can be
+enabled using the steps below.
+
+#. Get the current value of the PCI configure register::
+
+      setpci -s <XX:XX.X> a8.w
+
+#. Set bit 8::
+
+      value = value | 0x100
+
+#. Set the PCI configure register with new value::
+
+      setpci -s <XX:XX.X> a8.w=<value>
-- 
2.9.3

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

* Re: [dpdk-dev] [PATCH v3] doc: add known issue for i40e VF performance
  2017-07-18  9:52 ` [dpdk-dev] [PATCH v3] " Qi Zhang
@ 2017-07-18 14:59   ` Ferruh Yigit
  0 siblings, 0 replies; 6+ messages in thread
From: Ferruh Yigit @ 2017-07-18 14:59 UTC (permalink / raw)
  To: Qi Zhang, thomas.monjalon; +Cc: john.mcnamara, dev

On 7/18/2017 10:52 AM, Qi Zhang wrote:
> VF performance is limited by the kernel PCI extended tag setting.
> Update the document to explain the known issue and the workaround.
> 
> Signed-off-by: Qi Zhang <qi.z.zhang@intel.com>

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

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

end of thread, other threads:[~2017-07-18 14:59 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-03  3:57 [dpdk-dev] [PATCH v2] doc: add known issue for i40e VF performance Qi Zhang
2017-07-03 10:01 ` Mcnamara, John
2017-07-03 10:26 ` Ferruh Yigit
2017-07-09 22:49 ` Thomas Monjalon
2017-07-18  9:52 ` [dpdk-dev] [PATCH v3] " Qi Zhang
2017-07-18 14:59   ` 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).