[PATCH v2] doc: add known uio_pci_generic issue for i40e

[PATCH v2] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

From: "Guo, Jia" <jia.guo@intel.com>

When bind with uio_pci_generic in X710/XL710/XXV710, the result
is failed. uio_pci_generic is not supported by X710/XL710/XXV710.

Signed-off-by: Guo, Jia <jia.guo@intel.com>
---
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..1751451 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,27 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+uio_pci_generic bind failed in X710/XL710/XXV710
+-------------------------------------------------------
+
+**Description**:
+   uio_pci_generic is not supported by XL710, since the errata of XL710 that the feature
+   about the Interrupt Status bit is not implemented.The errata is the item #71 from the
+   `xl710 controller spec  <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When bind uio_pci_generic, the uio_pci_generic probe device and check the Interrupt
+   Status bit related feature of NICs , since it is not supported by X710/XL710/XXV710,
+   so it return failed.
+
+**Resolution/Workaround**:
+   Do not bind to use uio_pci_generic in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

[PATCH v2] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

From: "Guo, Jia" <jia.guo@intel.com>

When bind with uio_pci_generic in X710/XL710/XXV710, the result
is failed. uio_pci_generic is not supported by X710/XL710/XXV710.

Signed-off-by: Guo, Jia <jia.guo@intel.com>
---
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..1751451 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,27 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+uio_pci_generic bind failed in X710/XL710/XXV710
+-------------------------------------------------------
+
+**Description**:
+   uio_pci_generic is not supported by XL710, since the errata of XL710 that the feature
+   about the Interrupt Status bit is not implemented.The errata is the item #71 from the
+   `xl710 controller spec  <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When bind uio_pci_generic, the uio_pci_generic probe device and check the Interrupt
+   Status bit related feature of NICs , since it is not supported by X710/XL710/XXV710,
+   so it return failed.
+
+**Resolution/Workaround**:
+   Do not bind to use uio_pci_generic in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

[PATCH] doc: add known issue for uio_pci_generic in XL710

[Jeff Guo]

When bind with uio_pci_generic in XL710, the result is failed.
uio_pci_generic is not supported by XL710.

Signed-off-by: Jeff Guo <jia.guo@intel.com>
---
 doc/guides/rel_notes/known_issues.rst | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..3abcb64 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,25 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+uio_pci_generic bind failed in XL710
+-------------------------------------------------------
+
+**Description**:
+   uio_pci_generic is not supported by XL710, since the errata of XL710 that the feature
+   about the Interrupt Status bit is not implemented.The errata is the item #71 from the
+   `xl710 controller spec  <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+
+**Implication**:
+   When bind uio_pci_generic, the uio_pci_generic probe device and check the Interrupt
+   Status bit related feature of NICs , since it is not supported by XL710 so it return failed.
+
+**Resolution/Workaround**:
+   Do not bind to use uio_pci_generic in XL710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
1.9.3

Re: [PATCH] doc: add known issue for uio_pci_generic in XL710

[Wu, Jingjing]

> -----Original Message-----
> From: Guo, Jia
> Sent: Friday, December 23, 2016 10:35 AM
> To: Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
> Cc: dev@dpdk.org; Guo, Jia <jia.guo@intel.com>
> Subject: [PATCH] doc: add known issue for uio_pci_generic in XL710
> 
> When bind with uio_pci_generic in XL710, the result is failed.
> uio_pci_generic is not supported by XL710.
> 
> Signed-off-by: Jeff Guo <jia.guo@intel.com>

Acked-by: Jingjing Wu <jingjing.wu@intel.com>

[PATCH v2] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

From: "Guo, Jia" <jia.guo@intel.com>

When bind with uio_pci_generic in X710/XL710/XXV710, the result
is failed. uio_pci_generic is not supported by X710/XL710/XXV710.

Signed-off-by: Guo, Jia <jia.guo@intel.com>
---
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..1751451 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,27 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+uio_pci_generic bind failed in X710/XL710/XXV710
+-------------------------------------------------------
+
+**Description**:
+   uio_pci_generic is not supported by XL710, since the errata of XL710 that the feature
+   about the Interrupt Status bit is not implemented.The errata is the item #71 from the
+   `xl710 controller spec  <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When bind uio_pci_generic, the uio_pci_generic probe device and check the Interrupt
+   Status bit related feature of NICs , since it is not supported by X710/XL710/XXV710,
+   so it return failed.
+
+**Resolution/Workaround**:
+   Do not bind to use uio_pci_generic in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

Re: [PATCH v2] doc: add known uio_pci_generic issue for i40e

[Ferruh Yigit]

On 2/12/2016 4:39 PM, Jeff Guo wrote:
> From: "Guo, Jia" <jia.guo@intel.com>
> 
> When bind with uio_pci_generic in X710/XL710/XXV710, the result
> is failed. uio_pci_generic is not supported by X710/XL710/XXV710.

Hi Jeff,

There is a Linux commit causing the issue, right. Commit done because of
the reasons mentioned in "Description". Do you think does it make sense
to note that Linux commit, and Linux kernel version that has been
introduced?

> 
> Signed-off-by: Guo, Jia <jia.guo@intel.com>

You are known as "Jeff Guo <jia.guo@intel.com>" to DPDK repo, based on
previous commits.
If this is correct, is there any problem to continue like this?

> ---
> v2: add X710/XXV710 limitation case

Also v1 Acked-by JingJing already, you can keep her ack in next version
of the patch.

> ---
>  doc/guides/rel_notes/known_issues.rst | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
> index 3cd4237..1751451 100644
> --- a/doc/guides/rel_notes/known_issues.rst
> +++ b/doc/guides/rel_notes/known_issues.rst
> @@ -640,3 +640,27 @@ I40e VF may not receive packets in the promiscuous mode
>  
>  **Driver/Module**:
>     Poll Mode Driver (PMD).
> +
> +
> +uio_pci_generic bind failed in X710/XL710/XXV710
> +-------------------------------------------------------
> +
> +**Description**:
> +   uio_pci_generic is not supported by XL710, since the errata of XL710 that the feature
> +   about the Interrupt Status bit is not implemented.The errata is the item #71 from the
> +   `xl710 controller spec  <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
> +   The hw limitation is the same as other X710/XXV710 NICs.
> +
> +**Implication**:
> +   When bind uio_pci_generic, the uio_pci_generic probe device and check the Interrupt
> +   Status bit related feature of NICs , since it is not supported by X710/XL710/XXV710,
> +   so it return failed.
> +
> +**Resolution/Workaround**:
> +   Do not bind to use uio_pci_generic in X710/XL710/XXV710 NICs.
> +
> +**Affected Environment/Platform**:
> +   All.
> +
> +**Driver/Module**:
> +   Poll Mode Driver (PMD).
> 

Re: [PATCH v2] doc: add known uio_pci_generic issue for i40e

[Wu, Jingjing]

> -----Original Message-----
> From: Guo, Jia
> Sent: Thursday, January 12, 2017 11:28 AM
> To: Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
> Cc: dev@dpdk.org; Guo, Jia <jia.guo@intel.com>
> Subject: [PATCH v2] doc: add known uio_pci_generic issue for i40e
> 
> From: "Guo, Jia" <jia.guo@intel.com>
> 
> When bind with uio_pci_generic in X710/XL710/XXV710, the result is failed.
> uio_pci_generic is not supported by X710/XL710/XXV710.
> 
> Signed-off-by: Guo, Jia <jia.guo@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>

Re: [PATCH v2] doc: add known uio_pci_generic issue for i40e

[Wu, Jingjing]

> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, January 17, 2017 4:31 AM
> To: Guo, Jia <jia.guo@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu,
> Jingjing <jingjing.wu@intel.com>
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] doc: add known uio_pci_generic issue
> for i40e
> 
> On 2/12/2016 4:39 PM, Jeff Guo wrote:
> > From: "Guo, Jia" <jia.guo@intel.com>
> >
> > When bind with uio_pci_generic in X710/XL710/XXV710, the result is
> > failed. uio_pci_generic is not supported by X710/XL710/XXV710.
> 
> Hi Jeff,
> 
> There is a Linux commit causing the issue, right. Commit done because of the
> reasons mentioned in "Description". Do you think does it make sense to note
> that Linux commit, and Linux kernel version that has been introduced?
> 
Hi, Ferruh
At beginning, we found this issue in latest kernel version, such as kernel 4.7.9 on fedora24.
But later, we observed that this issue also exist on REL7.3.
So, it may be a little difficult to us to figure out exactly which kernel version
doesn't support. And the root cause is the HW's errata which makes uio_pci_generic
cannot be used on XL710 series NICs.

Thanks
Jingjing

Re: [PATCH v2] doc: add known uio_pci_generic issue for i40e

[Ferruh Yigit]

On 1/17/2017 5:00 PM, Wu, Jingjing wrote:
> 
> 
>> -----Original Message-----
>> From: Yigit, Ferruh
>> Sent: Tuesday, January 17, 2017 4:31 AM
>> To: Guo, Jia <jia.guo@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu,
>> Jingjing <jingjing.wu@intel.com>
>> Cc: dev@dpdk.org
>> Subject: Re: [dpdk-dev] [PATCH v2] doc: add known uio_pci_generic issue
>> for i40e
>>
>> On 2/12/2016 4:39 PM, Jeff Guo wrote:
>>> From: "Guo, Jia" <jia.guo@intel.com>
>>>
>>> When bind with uio_pci_generic in X710/XL710/XXV710, the result is
>>> failed. uio_pci_generic is not supported by X710/XL710/XXV710.
>>
>> Hi Jeff,
>>
>> There is a Linux commit causing the issue, right. Commit done because of the
>> reasons mentioned in "Description". Do you think does it make sense to note
>> that Linux commit, and Linux kernel version that has been introduced?
>>
> Hi, Ferruh
> At beginning, we found this issue in latest kernel version, such as kernel 4.7.9 on fedora24.
> But later, we observed that this issue also exist on REL7.3.

It won't work custom kernels, but upstream kernel version. If you think
that will be confusion, that is OK.

What do you think just adding Linux kernel commit information, can it be
useful?

> So, it may be a little difficult to us to figure out exactly which kernel version
> doesn't support. And the root cause is the HW's errata which makes uio_pci_generic
> cannot be used on XL710 series NICs.
> 
> Thanks
> Jingjing
> 

Re: [PATCH v2] doc: add known uio_pci_generic issue for i40e

[Guo, Jia]

On 1/18/2017 1:36 AM, Ferruh Yigit wrote:
> On 1/17/2017 5:00 PM, Wu, Jingjing wrote:
>>
>>> -----Original Message-----
>>> From: Yigit, Ferruh
>>> Sent: Tuesday, January 17, 2017 4:31 AM
>>> To: Guo, Jia <jia.guo@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu,
>>> Jingjing <jingjing.wu@intel.com>
>>> Cc: dev@dpdk.org
>>> Subject: Re: [dpdk-dev] [PATCH v2] doc: add known uio_pci_generic issue
>>> for i40e
>>>
>>> On 2/12/2016 4:39 PM, Jeff Guo wrote:
>>>> From: "Guo, Jia" <jia.guo@intel.com>
>>>>
>>>> When bind with uio_pci_generic in X710/XL710/XXV710, the result is
>>>> failed. uio_pci_generic is not supported by X710/XL710/XXV710.
>>> Hi Jeff,
>>>
>>> There is a Linux commit causing the issue, right. Commit done because of the
>>> reasons mentioned in "Description". Do you think does it make sense to note
>>> that Linux commit, and Linux kernel version that has been introduced?
>>>
>> Hi, Ferruh
>> At beginning, we found this issue in latest kernel version, such as kernel 4.7.9 on fedora24.
>> But later, we observed that this issue also exist on REL7.3.
> It won't work custom kernels, but upstream kernel version. If you think
> that will be confusion, that is OK.
>
> What do you think just adding Linux kernel commit information, can it be
> useful?
yes, the root cause of the issue is the HW' errata, but not the linux 
kernel commit,  because even without the linux kernel commit and 
uio_pci_generic can even be bind and use,  but still will cause other 
interrupt status issue. fix this interrupt status issue that is the 
purpose of the linux kernel commit.
so i think add linux kernel commit information in dpdk can be somehow to 
useful for track issue, but if let the kernel issue be manage by kernel 
org must be better.
ferruh and jingjing, Do you all agree with me. beg on your feedback.
>> So, it may be a little difficult to us to figure out exactly which kernel version
>> doesn't support. And the root cause is the HW's errata which makes uio_pci_generic
>> cannot be used on XL710 series NICs.
>>
>> Thanks
>> Jingjing
>>

Re: [PATCH v2] doc: add known uio_pci_generic issue for i40e

[Ferruh Yigit]

On 1/18/2017 2:48 AM, Guo, Jia wrote:
> 
> 
> On 1/18/2017 1:36 AM, Ferruh Yigit wrote:
>> On 1/17/2017 5:00 PM, Wu, Jingjing wrote:
>>>
>>>> -----Original Message-----
>>>> From: Yigit, Ferruh
>>>> Sent: Tuesday, January 17, 2017 4:31 AM
>>>> To: Guo, Jia <jia.guo@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu,
>>>> Jingjing <jingjing.wu@intel.com>
>>>> Cc: dev@dpdk.org
>>>> Subject: Re: [dpdk-dev] [PATCH v2] doc: add known uio_pci_generic issue
>>>> for i40e
>>>>
>>>> On 2/12/2016 4:39 PM, Jeff Guo wrote:
>>>>> From: "Guo, Jia" <jia.guo@intel.com>
>>>>>
>>>>> When bind with uio_pci_generic in X710/XL710/XXV710, the result is
>>>>> failed. uio_pci_generic is not supported by X710/XL710/XXV710.
>>>> Hi Jeff,
>>>>
>>>> There is a Linux commit causing the issue, right. Commit done because of the
>>>> reasons mentioned in "Description". Do you think does it make sense to note
>>>> that Linux commit, and Linux kernel version that has been introduced?
>>>>
>>> Hi, Ferruh
>>> At beginning, we found this issue in latest kernel version, such as kernel 4.7.9 on fedora24.
>>> But later, we observed that this issue also exist on REL7.3.
>> It won't work custom kernels, but upstream kernel version. If you think
>> that will be confusion, that is OK.
>>
>> What do you think just adding Linux kernel commit information, can it be
>> useful?
> yes, the root cause of the issue is the HW' errata, but not the linux 
> kernel commit,  because even without the linux kernel commit and 
> uio_pci_generic can even be bind and use,  but still will cause other 
> interrupt status issue. fix this interrupt status issue that is the 
> purpose of the linux kernel commit.
> so i think add linux kernel commit information in dpdk can be somehow to 
> useful for track issue, 

That was my reasoning for asking this. It may help others to understand
the issue more.

But is seems both you and Jingjing think this cause more confusion than
benefit. So I am OK with your proposal, if nobody else have objection.

> but if let the kernel issue be manage by kernel 
> org must be better.
> ferruh and jingjing, Do you all agree with me. beg on your feedback.
>>> So, it may be a little difficult to us to figure out exactly which kernel version
>>> doesn't support. And the root cause is the HW's errata which makes uio_pci_generic
>>> cannot be used on XL710 series NICs.
>>>
>>> Thanks
>>> Jingjing
>>>
> 

[PATCH v3] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

From: "Guo, Jia" <jia.guo@intel.com>

When bind with the "uio_pci_generic" module in X710/XL710/XXV710, the result
is failed. The "uio_pci_generic" module is not supported by X710/XL710/XXV710.

Signed-off-by: Guo, Jia <jia.guo@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
---
v3: add related linux kernel commit info and fix some syntax issues
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..8b083a0 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,29 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+The uio_pci_generic module bind failed in X710/XL710/XXV710
+-----------------------------------------------------------
+
+**Description**:
+   The “uio_pci_generic” module is not supported by XL710, since the errata of XL710
+   states that the Interrupt Status bit is not implemented. The errata is the item #71
+   from the `xl710 controller spec
+   <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When bind the “uio_pci_generic” module, it probe device and check the Interrupt Status
+   bit. Since it is not supported by X710/XL710/XXV710, it return a *failed* value. The statement
+   that these products don’t support INTx masking, is indicated in the related `linux kernel commit
+   < https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc8650e3491aec>`_.
+
+**Resolution/Workaround**:
+   Do not bind to use the “uio_pci_generic” module in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

[PATCH v3] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

From: "Guo, Jia" <jia.guo@intel.com>

When bind with the "uio_pci_generic" module in X710/XL710/XXV710,
the result is failed. The "uio_pci_generic" module is not supported
by X710/XL710/XXV710.

Signed-off-by: Guo, Jia <jia.guo@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
---
v3: add related linux kernel commit info and fix some syntax issues
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..8b083a0 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,29 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+The uio_pci_generic module bind failed in X710/XL710/XXV710
+-----------------------------------------------------------
+
+**Description**:
+   The “uio_pci_generic” module is not supported by XL710, since the errata of XL710
+   states that the Interrupt Status bit is not implemented. The errata is the item #71
+   from the `xl710 controller spec
+   <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When bind the “uio_pci_generic” module, it probe device and check the Interrupt Status
+   bit. Since it is not supported by X710/XL710/XXV710, it return a *failed* value. The statement
+   that these products don’t support INTx masking, is indicated in the related `linux kernel commit
+   < https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc8650e3491aec>`_.
+
+**Resolution/Workaround**:
+   Do not bind to use the “uio_pci_generic” module in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

Re: [PATCH v3] doc: add known uio_pci_generic issue for i40e

[Mcnamara, John]

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jeff Guo
> Sent: Thursday, January 19, 2017 10:57 AM
> To: Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>
> Cc: dev@dpdk.org; Guo, Jia <jia.guo@intel.com>; Guo@dpdk.org
> Subject: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic issue for
> i40e
> 
> +
> +The uio_pci_generic module bind failed in X710/XL710/XXV710
> +-----------------------------------------------------------
> +
> +**Description**:
> +   The “uio_pci_generic” module is not supported by XL710, since the
> errata of XL710
> +   states that the Interrupt Status bit is not implemented. The errata is
> the item #71
> +   from the `xl710 controller spec
> +

> <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710
> -10-40-controller-spec-update.html>`_.
> +   The hw limitation is the same as other X710/XXV710 NICs.
> +
> +**Implication**:
> +   When bind the “uio_pci_generic” module, it probe device and check the
> Interrupt Status

Better as: When using ``bind uio_pci_generic``, it probes the device


> +   bit. Since it is not supported by X710/XL710/XXV710, it return a
> *failed* value. The statement
> +   that these products don’t support INTx masking, is indicated in the
> related `linux kernel commit
> +   < https://git.kernel.org/cgit/linux/kernel/git/stable/linux-
> stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc8650
> e3491aec>`_.
> +
> +**Resolution/Workaround**:
> +   Do not bind to use the “uio_pci_generic” module in X710/XL710/XXV710
> NICs.

Better as: Do not bind to ``uio_pci_generic`` in X710/XL710/XXV710 NICs.

Re: [PATCH v3] doc: add known uio_pci_generic issue for i40e

[Ferruh Yigit]

On 1/19/2017 10:56 AM, Jeff Guo wrote:
> From: "Guo, Jia" <jia.guo@intel.com>
> 
> When bind with the "uio_pci_generic" module in X710/XL710/XXV710,
> the result is failed. The "uio_pci_generic" module is not supported
> by X710/XL710/XXV710.
> 
> Signed-off-by: Guo, Jia <jia.guo@intel.com>

To be consistent on names, this should be "Jeff Guo <jia.guo@intel.com>"
both in sign-off and author field.

If you will send a new version of the patch, please take care of this
too, if there won't be a new version please confirm it is OK to fix
while applying.

Thanks,
ferruh

<...>

Re: [PATCH v3] doc: add known uio_pci_generic issue for i40e

[Guo, Jia]

Hi,john
      The "bind uio_pci_generic`` is not the definitely command, but very appreciate for your correcting the other syntax issue. I will involve them in v4, Thanks very much.

Best regards,
Jeff Guo


-----Original Message-----
From: Mcnamara, John 
Sent: Thursday, January 19, 2017 7:17 PM
To: Guo, Jia <jia.guo@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
Cc: dev@dpdk.org; Guo, Jia <jia.guo@intel.com>; Guo@dpdk.org
Subject: RE: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic issue for i40e



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jeff Guo
> Sent: Thursday, January 19, 2017 10:57 AM
> To: Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing 
> <jingjing.wu@intel.com>
> Cc: dev@dpdk.org; Guo, Jia <jia.guo@intel.com>; Guo@dpdk.org
> Subject: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic issue 
> for i40e
> 
> +
> +The uio_pci_generic module bind failed in X710/XL710/XXV710
> +-----------------------------------------------------------
> +
> +**Description**:
> +   The “uio_pci_generic” module is not supported by XL710, since the
> errata of XL710
> +   states that the Interrupt Status bit is not implemented. The 
> + errata is
> the item #71
> +   from the `xl710 controller spec
> +

> <http://www.intel.com/content/www/us/en/embedded/products/networking/x
> l710 -10-40-controller-spec-update.html>`_.
> +   The hw limitation is the same as other X710/XXV710 NICs.
> +
> +**Implication**:
> +   When bind the “uio_pci_generic” module, it probe device and check 
> +the
> Interrupt Status

Better as: When using ``bind uio_pci_generic``, it probes the device


> +   bit. Since it is not supported by X710/XL710/XXV710, it return a
> *failed* value. The statement
> +   that these products don’t support INTx masking, is indicated in 
> + the
> related `linux kernel commit
> +   < https://git.kernel.org/cgit/linux/kernel/git/stable/linux-
> stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc
> 8650
> e3491aec>`_.
> +
> +**Resolution/Workaround**:
> +   Do not bind to use the “uio_pci_generic” module in 
> +X710/XL710/XXV710
> NICs.

Better as: Do not bind to ``uio_pci_generic`` in X710/XL710/XXV710 NICs.

Re: [PATCH v3] doc: add known uio_pci_generic issue for i40e

[Guo, Jia]

Yes, rerruh, I will recheck the gitconfig and consistent the sign-off name in v4. thanks for your suggestion. 

Best regards,
Jeff Guo


-----Original Message-----
From: Yigit, Ferruh 
Sent: Thursday, January 19, 2017 8:06 PM
To: Guo, Jia <jia.guo@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
Cc: dev@dpdk.org; Guo@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic issue for i40e

On 1/19/2017 10:56 AM, Jeff Guo wrote:
> From: "Guo, Jia" <jia.guo@intel.com>
> 
> When bind with the "uio_pci_generic" module in X710/XL710/XXV710, the 
> result is failed. The "uio_pci_generic" module is not supported by 
> X710/XL710/XXV710.
> 
> Signed-off-by: Guo, Jia <jia.guo@intel.com>

To be consistent on names, this should be "Jeff Guo <jia.guo@intel.com>"
both in sign-off and author field.

If you will send a new version of the patch, please take care of this too, if there won't be a new version please confirm it is OK to fix while applying.

Thanks,
ferruh

<...>

[PATCH v4] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

When bind the "uio_pci_generic" module in X710/XL710/XXV710,
the result is failed. The "uio_pci_generic" module is not
supported by X710/XL710/XXV710.

Signed-off-by: Jeff Guo <jia.guo@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
---
v4: modify the signed-off name to be consistent.
v3: add related linux kernel commit info and fix some syntax issues
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..c1e06fc 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,29 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+The uio_pci_generic module bind failed in X710/XL710/XXV710
+-----------------------------------------------------------
+
+**Description**:
+   The “uio_pci_generic” module is not supported by XL710, since the errata of XL710
+   states that the Interrupt Status bit is not implemented. The errata is the item #71
+   from the `xl710 controller spec
+   <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When bind the “uio_pci_generic” module, it probes device and check the Interrupt Status
+   bit. Since it is not supported by X710/XL710/XXV710, it return a *failed* value. The statement
+   that these products don’t support INTx masking, is indicated in the related `linux kernel commit
+   < https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc8650e3491aec>`_.
+
+**Resolution/Workaround**:
+   Do not bind the “uio_pci_generic” module in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

Re: [PATCH v3] doc: add known uio_pci_generic issue for i40e

[Mcnamara, John]

> -----Original Message-----
> From: Guo, Jia
> Sent: Thursday, January 19, 2017 1:50 PM
> To: Mcnamara, John <john.mcnamara@intel.com>; Zhang, Helin
> <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
> Cc: dev@dpdk.org; Guo@dpdk.org
> Subject: RE: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic issue
> for i40e
> 
> Hi,john
>       The "bind uio_pci_generic`` is not the definitely command, but very
> appreciate for your correcting the other syntax issue. I will involve them
> in v4, Thanks very much.

Hi Jeff, 

Variables, functions, program names, file names and any other literal should 
be in backtick quotes, as outlined in the DPDK Documentation: 

http://dpdk.org/doc/guides/contributing/documentation.html#code-and-literal-block-sections

So uio_pci_generic should be quoted with backticks. If you want to add bind then do:
``--bind= uio_pci_generic`` or similar.

Either way it shouldn't be quoted using the "smart" Unicode or non-Unicode quotes as in the
patch. All quotes should be straight ASCII double quotes (""). The Html/PDF renderers
will convert to the correct quotes as required.



> > related `linux kernel commit
> > +   < https://git.kernel.org/cgit/linux/kernel/git/stable/linux-
> > stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc
> > 8650
> > e3491aec>`_.

Also, this link generates a doc build error. I think because of the space
between the < and the https:.

John

Re: [PATCH v3] doc: add known uio_pci_generic issue for i40e

[Guo, Jia]

Ok, that make sense, I will re-correct it then. Thanks.

Best regards,
Jeff Guo

-----Original Message-----
From: Mcnamara, John 
Sent: Thursday, January 19, 2017 10:54 PM
To: Guo, Jia <jia.guo@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
Cc: dev@dpdk.org; Guo@dpdk.org
Subject: RE: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic issue for i40e

> -----Original Message-----
> From: Guo, Jia
> Sent: Thursday, January 19, 2017 1:50 PM
> To: Mcnamara, John <john.mcnamara@intel.com>; Zhang, Helin 
> <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com>
> Cc: dev@dpdk.org; Guo@dpdk.org
> Subject: RE: [dpdk-dev] [PATCH v3] doc: add known uio_pci_generic 
> issue for i40e
> 
> Hi,john
>       The "bind uio_pci_generic`` is not the definitely command, but 
> very appreciate for your correcting the other syntax issue. I will 
> involve them in v4, Thanks very much.

Hi Jeff, 

Variables, functions, program names, file names and any other literal should be in backtick quotes, as outlined in the DPDK Documentation: 

http://dpdk.org/doc/guides/contributing/documentation.html#code-and-literal-block-sections

So uio_pci_generic should be quoted with backticks. If you want to add bind then do:
``--bind= uio_pci_generic`` or similar.

Either way it shouldn't be quoted using the "smart" Unicode or non-Unicode quotes as in the patch. All quotes should be straight ASCII double quotes (""). The Html/PDF renderers will convert to the correct quotes as required.



> > related `linux kernel commit
> > +   < https://git.kernel.org/cgit/linux/kernel/git/stable/linux-
> > stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132
> > bc
> > 8650
> > e3491aec>`_.

Also, this link generates a doc build error. I think because of the space between the < and the https:.

John

[PATCH v4] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

When bind the "uio_pci_generic" module in X710/XL710/XXV710,
the result is failed. The "uio_pci_generic" module is not
supported by X710/XL710/XXV710.

Signed-off-by: Jeff Guo <jia.guo@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
---
v4: modify the signed-off name to be consistent and fix make doc issue.
v3: add related linux kernel commit info and fix some syntax issues
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..b1e581b 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,29 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+The uio_pci_generic module bind failed in X710/XL710/XXV710
+-----------------------------------------------------------
+
+**Description**:
+   The “uio_pci_generic” module is not supported by XL710, since the errata of XL710
+   states that the Interrupt Status bit is not implemented. The errata is the item #71
+   from the `xl710 controller spec
+   <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When use "--bind=uio_pci_generic”, the "uio_pci_generic" module probes device and check the Interrupt
+   Status bit. Since it is not supported by X710/XL710/XXV710, it return a *failed* value. The statement
+   that these products don’t support INTx masking, is indicated in the related `linux kernel commit
+   <https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc8650e3491aec>`_.
+
+**Resolution/Workaround**:
+   Do not bind the “uio_pci_generic” module in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

[PATCH v4] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

When bind the "uio_pci_generic" module in X710/XL710/XXV710,
the result is failed. The "uio_pci_generic" module is not
supported by X710/XL710/XXV710.

Signed-off-by: Jeff Guo <jia.guo@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
---
v4: modify the signed-off name to be consistent, fix quotes and make doc issue.
v3: add related linux kernel commit info and fix some syntax issues
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..ef43d01 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,29 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+The uio_pci_generic module bind failed in X710/XL710/XXV710
+-----------------------------------------------------------
+
+**Description**:
+   The "uio_pci_generic" module is not supported by XL710, since the errata of XL710
+   states that the Interrupt Status bit is not implemented. The errata is the item #71
+   from the `xl710 controller spec
+   <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When use "--bind=uio_pci_generic", the "uio_pci_generic" module probes device and check the Interrupt
+   Status bit. Since it is not supported by X710/XL710/XXV710, it return a *failed* value. The statement
+   that these products don’t support INTx masking, is indicated in the related `linux kernel commit
+   <https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc8650e3491aec>`_.
+
+**Resolution/Workaround**:
+   Do not bind the "uio_pci_generic" module in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

[PATCH v4] doc: add known uio_pci_generic issue for i40e

[Jeff Guo]

When bind the ``uio_pci_generic`` module in X710/XL710/XXV710,
the result is failed. The ``uio_pci_generic`` module is not
supported by X710/XL710/XXV710.

Signed-off-by: Jeff Guo <jia.guo@intel.com>
Acked-by: Jingjing Wu <jingjing.wu@intel.com>
---
v4: modify the signed-off name to be consistent and fix make doc issue.
v3: add related linux kernel commit info and fix some syntax issues
v2: add X710/XXV710 limitation case
---
 doc/guides/rel_notes/known_issues.rst | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/doc/guides/rel_notes/known_issues.rst b/doc/guides/rel_notes/known_issues.rst
index 3cd4237..018e999 100644
--- a/doc/guides/rel_notes/known_issues.rst
+++ b/doc/guides/rel_notes/known_issues.rst
@@ -640,3 +640,29 @@ I40e VF may not receive packets in the promiscuous mode
 
 **Driver/Module**:
    Poll Mode Driver (PMD).
+
+
+uio pci generic module bind failed in X710/XL710/XXV710
+-------------------------------------------------------
+
+**Description**:
+   The ``uio_pci_generic`` module is not supported by XL710, since the errata of XL710
+   states that the Interrupt Status bit is not implemented. The errata is the item #71
+   from the `xl710 controller spec
+   <http://www.intel.com/content/www/us/en/embedded/products/networking/xl710-10-40-controller-spec-update.html>`_.
+   The hw limitation is the same as other X710/XXV710 NICs.
+
+**Implication**:
+   When use ``--bind=uio_pci_generic``, the ``uio_pci_generic`` module probes device and check the Interrupt
+   Status bit. Since it is not supported by X710/XL710/XXV710, it return a *failed* value. The statement
+   that these products don’t support INTx masking, is indicated in the related `linux kernel commit
+   <https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/commit/drivers/pci/quirks.c?id=8bcf4525c5d43306c5fd07e132bc8650e3491aec>`_.
+
+**Resolution/Workaround**:
+   Do not bind the ``uio_pci_generic`` module in X710/XL710/XXV710 NICs.
+
+**Affected Environment/Platform**:
+   All.
+
+**Driver/Module**:
+   Poll Mode Driver (PMD).
-- 
2.7.4

Re: [PATCH v4] doc: add known uio_pci_generic issue for i40e

[Mcnamara, John]

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jeff Guo
> Sent: Sunday, January 22, 2017 2:04 AM
> To: Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>
> Cc: dev@dpdk.org; Guo, Jia <jia.guo@intel.com>
> Subject: [dpdk-dev] [PATCH v4] doc: add known uio_pci_generic issue for
> i40e
> 
> When bind the ``uio_pci_generic`` module in X710/XL710/XXV710, the result
> is failed. The ``uio_pci_generic`` module is not supported by
> X710/XL710/XXV710.
> 
> Signed-off-by: Jeff Guo <jia.guo@intel.com>


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

Re: [PATCH v4] doc: add known uio_pci_generic issue for i40e

[Ferruh Yigit]

On 1/23/2017 8:04 AM, Mcnamara, John wrote:
> 
> 
>> -----Original Message-----
>> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Jeff Guo
>> Sent: Sunday, January 22, 2017 2:04 AM
>> To: Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing
>> <jingjing.wu@intel.com>
>> Cc: dev@dpdk.org; Guo, Jia <jia.guo@intel.com>
>> Subject: [dpdk-dev] [PATCH v4] doc: add known uio_pci_generic issue for
>> i40e
>>
>> When bind the ``uio_pci_generic`` module in X710/XL710/XXV710, the result
>> is failed. The ``uio_pci_generic`` module is not supported by
>> X710/XL710/XXV710.
>>
>> Signed-off-by: Jeff Guo <jia.guo@intel.com>
> 
> Acked-by: John McNamara <john.mcnamara@intel.com>

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