DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list
@ 2017-11-21  1:42 Ferruh Yigit
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD " Ferruh Yigit
                   ` (5 more replies)
  0 siblings, 6 replies; 26+ messages in thread
From: Ferruh Yigit @ 2017-11-21  1:42 UTC (permalink / raw)
  To: Thomas Monjalon, John McNamara; +Cc: dev, Ferruh Yigit

Some library modifications require extra checks or modifications in PMDs
but currently we don't have a good way to trace these modifications.

And number of these kind of updates increasing by time.

This is an effort to document and track library updates that requires
attention in PMDs.

Although this is under documentation main use case is for internal
development.

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 MAINTAINERS               |  1 +
 doc/guides/nics/index.rst |  1 +
 doc/guides/nics/todo.rst  | 43 +++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 45 insertions(+)
 create mode 100644 doc/guides/nics/todo.rst

diff --git a/MAINTAINERS b/MAINTAINERS
index f0baeb423..c9eaff774 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -315,6 +315,7 @@ Networking Drivers
 M: Ferruh Yigit <ferruh.yigit@intel.com>
 T: git://dpdk.org/next/dpdk-next-net
 F: doc/guides/nics/features/default.ini
+F: doc/guides/nics/todo.rst
 
 Link bonding
 M: Declan Doherty <declan.doherty@intel.com>
diff --git a/doc/guides/nics/index.rst b/doc/guides/nics/index.rst
index 23babe933..f257fbd87 100644
--- a/doc/guides/nics/index.rst
+++ b/doc/guides/nics/index.rst
@@ -37,6 +37,7 @@ Network Interface Controller Drivers
 
     overview
     features
+    todo
     build_and_test
     ark
     avp
diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
new file mode 100644
index 000000000..874811ad5
--- /dev/null
+++ b/doc/guides/nics/todo.rst
@@ -0,0 +1,43 @@
+..  BSD LICENSE
+    Copyright(c) 2017 Intel Corporation. All rights reserved.
+    All rights reserved.
+
+    Redistribution and use in source and binary forms, with or without
+    modification, are permitted provided that the following conditions
+    are met:
+
+    * Redistributions of source code must retain the above copyright
+    notice, this list of conditions and the following disclaimer.
+    * Redistributions in binary form must reproduce the above copyright
+    notice, this list of conditions and the following disclaimer in
+    the documentation and/or other materials provided with the
+    distribution.
+    * Neither the name of Intel Corporation nor the names of its
+    contributors may be used to endorse or promote products derived
+    from this software without specific prior written permission.
+
+    THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+    "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+    LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+    A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+    OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+    SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+    LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+    DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+    THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+    (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+    OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+
+PMDs internal TODO list
+=======================
+
+This is the list for tracking required PMD changes triggered by library modifications.
+
+.. table:: PMDs internal TODO list
+
+ +------+------+----------+----------------+------+
+ | TODO | PMDs | Deadline | Related Commit | Note |
+ +======+======+==========+================+======+
+ |      |      |          |                |      |
+ +------+------+----------+----------------+------+
-- 
2.13.6

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

* [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD todo list
  2017-11-21  1:42 [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list Ferruh Yigit
@ 2017-11-21  1:42 ` Ferruh Yigit
  2017-12-11 14:00   ` Marko Kovacevic
                     ` (3 more replies)
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 3/6] doc: add dynamic logging " Ferruh Yigit
                   ` (4 subsequent siblings)
  5 siblings, 4 replies; 26+ messages in thread
From: Ferruh Yigit @ 2017-11-21  1:42 UTC (permalink / raw)
  To: Thomas Monjalon, John McNamara; +Cc: dev, Ferruh Yigit

To track modification:
8f094a9ac5d7 ("mbuf: set mbuf fields while in pool")

Proposed deadline for PMDs is v18.02

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 doc/guides/nics/todo.rst | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
index 874811ad5..b408fc691 100644
--- a/doc/guides/nics/todo.rst
+++ b/doc/guides/nics/todo.rst
@@ -36,8 +36,17 @@ This is the list for tracking required PMD changes triggered by library modifica
 
 .. table:: PMDs internal TODO list
 
- +------+------+----------+----------------+------+
- | TODO | PMDs | Deadline | Related Commit | Note |
- +======+======+==========+================+======+
- |      |      |          |                |      |
- +------+------+----------+----------------+------+
+ +-------------------+--------------------------------+----------+----------------+-----------------------------------+
+ | TODO              | PMDs                           | Deadline | Related Commit | Note                              |
+ +===================+================================+==========+================+===================================+
+ | mbuf organisation | | af_packet, ark, avp, bnx2x,  | v18.02   | 8f094a9ac5d7   | Remove unnecessary mbuf field     |
+ |                   | | bnxt, bonding, cxgbe, dpaa,  |          |                | initialization.                   |
+ |                   | | dpaa2, e1000, ena,           |          |                |                                   |
+ |                   | | failsafe, fm10k,             |          |                |                                   |
+ |                   | | kni, mlx4,                   |          |                |                                   |
+ |                   | | mrvl, nfp, octeontx,         |          |                |                                   |
+ |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
+ |                   | | softnic, szedata2, tap,      |          |                |                                   |
+ |                   | | thunderx, vhost,             |          |                |                                   |
+ |                   | | vmxnet3                      |          |                |                                   |
+ +-------------------+--------------------------------+----------+----------------+-----------------------------------+
-- 
2.13.6

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

* [dpdk-dev] [PATCH 3/6] doc: add dynamic logging to PMD todo list
  2017-11-21  1:42 [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list Ferruh Yigit
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD " Ferruh Yigit
@ 2017-11-21  1:42 ` Ferruh Yigit
  2017-12-08  5:12   ` Hemant Agrawal
  2017-12-11 14:09   ` Mcnamara, John
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 4/6] doc: add descriptor status API " Ferruh Yigit
                   ` (3 subsequent siblings)
  5 siblings, 2 replies; 26+ messages in thread
From: Ferruh Yigit @ 2017-11-21  1:42 UTC (permalink / raw)
  To: Thomas Monjalon, John McNamara; +Cc: dev, Ferruh Yigit

To track modification:
c1b5fa94a46f ("eal: support dynamic log types")

Proposed deadline for PMDs is v18.08

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 doc/guides/nics/todo.rst | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
index b408fc691..973379a3d 100644
--- a/doc/guides/nics/todo.rst
+++ b/doc/guides/nics/todo.rst
@@ -50,3 +50,14 @@ This is the list for tracking required PMD changes triggered by library modifica
  |                   | | thunderx, vhost,             |          |                |                                   |
  |                   | | vmxnet3                      |          |                |                                   |
  +-------------------+--------------------------------+----------+----------------+-----------------------------------+
+ | dynamic logging   | | af_packet, ark, avp, bnx2x,  | v18.08   | c1b5fa94a46f   | Switch to dynamic logging         |
+ |                   | | bnxt, bonding, cxgbe, dpaa,  |          |                | functions, remove static debug    |
+ |                   | | dpaa2, e1000, ena, enic,     |          |                | config options.                   |
+ |                   | | failsafe, fm10k, ixgbe,      |          |                |                                   |
+ |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
+ |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
+ |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
+ |                   | | softnic, szedata2, tap,      |          |                |                                   |
+ |                   | | thunderx, vhost, virtio,     |          |                |                                   |
+ |                   | | vmxnet3                      |          |                |                                   |
+ +-------------------+--------------------------------+----------+----------------+-----------------------------------+
-- 
2.13.6

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

* [dpdk-dev] [PATCH 4/6] doc: add descriptor status API to PMD todo list
  2017-11-21  1:42 [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list Ferruh Yigit
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD " Ferruh Yigit
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 3/6] doc: add dynamic logging " Ferruh Yigit
@ 2017-11-21  1:42 ` Ferruh Yigit
  2017-12-11 14:11   ` Mcnamara, John
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 5/6] doc: add offload flag " Ferruh Yigit
                   ` (2 subsequent siblings)
  5 siblings, 1 reply; 26+ messages in thread
From: Ferruh Yigit @ 2017-11-21  1:42 UTC (permalink / raw)
  To: Thomas Monjalon, John McNamara; +Cc: dev, Ferruh Yigit

To track modification:
b1b700ce7d6f ("ethdev: add descriptor status API")

Proposed deadline for PMDs is v19.02

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 doc/guides/nics/todo.rst | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
index 973379a3d..9ace19716 100644
--- a/doc/guides/nics/todo.rst
+++ b/doc/guides/nics/todo.rst
@@ -61,3 +61,14 @@ This is the list for tracking required PMD changes triggered by library modifica
  |                   | | thunderx, vhost, virtio,     |          |                |                                   |
  |                   | | vmxnet3                      |          |                |                                   |
  +-------------------+--------------------------------+----------+----------------+-----------------------------------+
+ | | descriptor      | | af_packet, ark, avp, bnx2x,  | v19.02   | b1b700ce7d6f   | Replace descriptor_done API with  |
+ | | status API      | | bnxt, bonding, cxgbe, dpaa,  |          |                | rx_descriptor_status and          |
+ |                   | | dpaa2, ena, enic,            |          |                | tx_descriptor_status APIs.        |
+ |                   | | failsafe, fm10k,             |          |                |                                   |
+ |                   | | kni, liquidio, mlx4,         |          |                |                                   |
+ |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
+ |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
+ |                   | | softnic, szedata2, tap,      |          |                |                                   |
+ |                   | | thunderx, vhost, virtio,     |          |                |                                   |
+ |                   | | vmxnet3                      |          |                |                                   |
+ +-------------------+--------------------------------+----------+----------------+-----------------------------------+
-- 
2.13.6

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

* [dpdk-dev] [PATCH 5/6] doc: add offload flag to PMD todo list
  2017-11-21  1:42 [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list Ferruh Yigit
                   ` (2 preceding siblings ...)
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 4/6] doc: add descriptor status API " Ferruh Yigit
@ 2017-11-21  1:42 ` Ferruh Yigit
  2017-11-21  8:32   ` Shahaf Shuler
  2017-12-11 14:11   ` Mcnamara, John
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN " Ferruh Yigit
  2017-12-11 11:52 ` [dpdk-dev] [PATCH 1/6] doc: add empty PMDs " Mcnamara, John
  5 siblings, 2 replies; 26+ messages in thread
From: Ferruh Yigit @ 2017-11-21  1:42 UTC (permalink / raw)
  To: Thomas Monjalon, John McNamara; +Cc: dev, Ferruh Yigit

To track modification:
ce17eddefc20 ("ethdev: introduce Rx queue offloads API")
cba7f53b717d ("ethdev: introduce Tx queue offloads API")

Proposed deadline for PMDs is v18.05

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 doc/guides/nics/todo.rst | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
index 9ace19716..61dd9cbc9 100644
--- a/doc/guides/nics/todo.rst
+++ b/doc/guides/nics/todo.rst
@@ -72,3 +72,14 @@ This is the list for tracking required PMD changes triggered by library modifica
  |                   | | thunderx, vhost, virtio,     |          |                |                                   |
  |                   | | vmxnet3                      |          |                |                                   |
  +-------------------+--------------------------------+----------+----------------+-----------------------------------+
+ | new offload flags | | af_packet, ark, avp, bnx2x,  | v18.05   | ce17eddefc20   | Use new ethdev offloads filed     |
+ |                   | | bnxt, bonding, cxgbe, dpaa,  |          | cba7f53b717d   | to get requested offload list     |
+ |                   | | dpaa2, e1000, ena, enic,     |          |                | instead of bitfield values.       |
+ |                   | | failsafe, fm10k, i40e, ixgbe,|          |                |                                   |
+ |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
+ |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
+ |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
+ |                   | | softnic, szedata2, tap,      |          |                |                                   |
+ |                   | | thunderx, vhost, virtio,     |          |                |                                   |
+ |                   | | vmxnet3                      |          |                |                                   |
+ +-------------------+--------------------------------+----------+----------------+-----------------------------------+
-- 
2.13.6

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

* [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN flag to PMD todo list
  2017-11-21  1:42 [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list Ferruh Yigit
                   ` (3 preceding siblings ...)
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 5/6] doc: add offload flag " Ferruh Yigit
@ 2017-11-21  1:42 ` Ferruh Yigit
  2017-11-21  1:53   ` Ferruh Yigit
                     ` (4 more replies)
  2017-12-11 11:52 ` [dpdk-dev] [PATCH 1/6] doc: add empty PMDs " Mcnamara, John
  5 siblings, 5 replies; 26+ messages in thread
From: Ferruh Yigit @ 2017-11-21  1:42 UTC (permalink / raw)
  To: Thomas Monjalon, John McNamara; +Cc: dev, Ferruh Yigit

To track modification:
380a7aab1ae2 ("mbuf: rename deprecated VLAN flags")

Proposed deadline for PMDs is v18.02

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 doc/guides/nics/todo.rst | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
index 61dd9cbc9..493a49ff2 100644
--- a/doc/guides/nics/todo.rst
+++ b/doc/guides/nics/todo.rst
@@ -83,3 +83,14 @@ This is the list for tracking required PMD changes triggered by library modifica
  |                   | | thunderx, vhost, virtio,     |          |                |                                   |
  |                   | | vmxnet3                      |          |                |                                   |
  +-------------------+--------------------------------+----------+----------------+-----------------------------------+
+ | | check new mbuf  | | af_packet, ark, avp, bnx2x,  | v18.02   | 380a7aab1ae2   | mbuf flag PKT_RX_VLAN_PKT renamed |
+ | | VLAN flag       | | bnxt, bonding, cxgbe, dpaa,  |          |                | to PKT_RX_VLAN and its meaning    |
+ | | PKT_RX_VLAN     | | dpaa2, e1000, ena, enic,     |          |                | changed, confirm PMD uses new     |
+ |                   | | failsafe, fm10k, i40e, ixgbe,|          |                | flag correct.                     |
+ |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
+ |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
+ |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
+ |                   | | softnic, szedata2, tap,      |          |                |                                   |
+ |                   | | thunderx, vhost, virtio,     |          |                |                                   |
+ |                   | | vmxnet3                      |          |                |                                   |
+ +-------------------+--------------------------------+----------+----------------+-----------------------------------+
-- 
2.13.6

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

* Re: [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN flag to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN " Ferruh Yigit
@ 2017-11-21  1:53   ` Ferruh Yigit
  2017-12-06  9:24   ` Hemant Agrawal
                     ` (3 subsequent siblings)
  4 siblings, 0 replies; 26+ messages in thread
From: Ferruh Yigit @ 2017-11-21  1:53 UTC (permalink / raw)
  To: Thomas Monjalon, John McNamara; +Cc: dev, Olivier MATZ

On 11/20/2017 5:42 PM, Ferruh Yigit wrote:
> To track modification:
> 380a7aab1ae2 ("mbuf: rename deprecated VLAN flags")

I recognized that almost all are by Olivier, cc'ed :)

> Proposed deadline for PMDs is v18.02
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

<...>

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

* Re: [dpdk-dev] [PATCH 5/6] doc: add offload flag to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 5/6] doc: add offload flag " Ferruh Yigit
@ 2017-11-21  8:32   ` Shahaf Shuler
  2017-11-21 18:02     ` Ferruh Yigit
  2017-12-11 14:11   ` Mcnamara, John
  1 sibling, 1 reply; 26+ messages in thread
From: Shahaf Shuler @ 2017-11-21  8:32 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, John McNamara; +Cc: dev

Hi Ferruh,

Tuesday, November 21, 2017 3:43 AM, Ferruh Yigit:
> To track modification:
> ce17eddefc20 ("ethdev: introduce Rx queue offloads API") cba7f53b717d
> ("ethdev: introduce Tx queue offloads API")
> 
> Proposed deadline for PMDs is v18.05

The deprecation of the old offloads API is set to 18.05 [1]. my understanding was that we agreed to convert all PMDs on 18.02.
If the PMD deadline is going to be postponed I think the ethdev deprecation should be updated. 

[1] http://dpdk.org/doc/guides/rel_notes/deprecation.html

> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  doc/guides/nics/todo.rst | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst index
> 9ace19716..61dd9cbc9 100644
> --- a/doc/guides/nics/todo.rst
> +++ b/doc/guides/nics/todo.rst
> @@ -72,3 +72,14 @@ This is the list for tracking required PMD changes
> triggered by library modifica
>   |                   | | thunderx, vhost, virtio,     |          |                |                                   |
>   |                   | | vmxnet3                      |          |                |                                   |
>   +-------------------+--------------------------------+----------+----------------+-------
> ----------------------------+
> + | new offload flags | | af_packet, ark, avp, bnx2x,  | v18.05   |
> ce17eddefc20   | Use new ethdev offloads filed     |
> + |                   | | bnxt, bonding, cxgbe, dpaa,  |          | cba7f53b717d   | to get
> requested offload list     |
> + |                   | | dpaa2, e1000, ena, enic,     |          |                | instead of bitfield
> values.       |
> + |                   | | failsafe, fm10k, i40e, ixgbe,|          |                |
> |
> + |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
> + |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
> + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> + |                   | | thunderx, vhost, virtio,     |          |                |                                   |
> + |                   | | vmxnet3                      |          |                |                                   |
> + +-------------------+--------------------------------+----------+----------------+------
> -----------------------------+
> --
> 2.13.6

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

* Re: [dpdk-dev] [PATCH 5/6] doc: add offload flag to PMD todo list
  2017-11-21  8:32   ` Shahaf Shuler
@ 2017-11-21 18:02     ` Ferruh Yigit
  2017-11-22  8:43       ` Shahaf Shuler
  0 siblings, 1 reply; 26+ messages in thread
From: Ferruh Yigit @ 2017-11-21 18:02 UTC (permalink / raw)
  To: Shahaf Shuler, Thomas Monjalon, John McNamara; +Cc: dev

On 11/21/2017 12:32 AM, Shahaf Shuler wrote:
> Hi Ferruh,
> 
> Tuesday, November 21, 2017 3:43 AM, Ferruh Yigit:
>> To track modification:
>> ce17eddefc20 ("ethdev: introduce Rx queue offloads API") cba7f53b717d
>> ("ethdev: introduce Tx queue offloads API")
>>
>> Proposed deadline for PMDs is v18.05
> 
> The deprecation of the old offloads API is set to 18.05 [1]. my understanding was that we agreed to convert all PMDs on 18.02.
> If the PMD deadline is going to be postponed I think the ethdev deprecation should be updated. 

Hi Shahaf,

I put deadlines roughly and they need to be discussed here.

But 18.02 deadline is not far away.
I think it is not realistic to expect all relevant PMDs do this conversion on
time, what do you think?

My concern is if there can be some PMD maintainers just learned that an update
is required. I put these docs for that synchronization, but not sure how
efficient or noisy it will be.

Thanks,
ferruh

> 
> [1] http://dpdk.org/doc/guides/rel_notes/deprecation.html
> 
>>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>> ---
>>  doc/guides/nics/todo.rst | 11 +++++++++++
>>  1 file changed, 11 insertions(+)
>>
>> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst index
>> 9ace19716..61dd9cbc9 100644
>> --- a/doc/guides/nics/todo.rst
>> +++ b/doc/guides/nics/todo.rst
>> @@ -72,3 +72,14 @@ This is the list for tracking required PMD changes
>> triggered by library modifica
>>   |                   | | thunderx, vhost, virtio,     |          |                |                                   |
>>   |                   | | vmxnet3                      |          |                |                                   |
>>   +-------------------+--------------------------------+----------+----------------+-------
>> ----------------------------+
>> + | new offload flags | | af_packet, ark, avp, bnx2x,  | v18.05   |
>> ce17eddefc20   | Use new ethdev offloads filed     |
>> + |                   | | bnxt, bonding, cxgbe, dpaa,  |          | cba7f53b717d   | to get
>> requested offload list     |
>> + |                   | | dpaa2, e1000, ena, enic,     |          |                | instead of bitfield
>> values.       |
>> + |                   | | failsafe, fm10k, i40e, ixgbe,|          |                |
>> |
>> + |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
>> + |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
>> + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
>> + |                   | | softnic, szedata2, tap,      |          |                |                                   |
>> + |                   | | thunderx, vhost, virtio,     |          |                |                                   |
>> + |                   | | vmxnet3                      |          |                |                                   |
>> + +-------------------+--------------------------------+----------+----------------+------
>> -----------------------------+
>> --
>> 2.13.6
> 

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

* Re: [dpdk-dev] [PATCH 5/6] doc: add offload flag to PMD todo list
  2017-11-21 18:02     ` Ferruh Yigit
@ 2017-11-22  8:43       ` Shahaf Shuler
  2017-11-22  8:54         ` Thomas Monjalon
  0 siblings, 1 reply; 26+ messages in thread
From: Shahaf Shuler @ 2017-11-22  8:43 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, John McNamara; +Cc: dev

Tuesday, November 21, 2017 8:03 PM, Ferruh Yigit:
> On 11/21/2017 12:32 AM, Shahaf Shuler wrote:
> >
> > The deprecation of the old offloads API is set to 18.05 [1]. my
> understanding was that we agreed to convert all PMDs on 18.02.
> > If the PMD deadline is going to be postponed I think the ethdev
> deprecation should be updated.
> 
> Hi Shahaf,
> 
> I put deadlines roughly and they need to be discussed here.
> 
> But 18.02 deadline is not far away.
> I think it is not realistic to expect all relevant PMDs do this conversion on
> time, what do you think?

I cannot tell about other vendors plans. For mlx4 and mlx5 I plan to convert on 18.02.
From the discussion we had on 17.11 for this feature we agreed to split it into 2, and the PMD part will be on 18.02. no one objected. 

Maybe on the next Tech-board you can have a sync between some of the vendors to see the plans.
If not all can make it, we should postpone the deprecation of the old API.  

> 
> My concern is if there can be some PMD maintainers just learned that an
> update is required. I put these docs for that synchronization, but not sure
> how efficient or noisy it will be.
> 
> Thanks,
> ferruh
> 
> >
> > [1]
> >
> https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpd
> k
> >
> .org%2Fdoc%2Fguides%2Frel_notes%2Fdeprecation.html&data=02%7C01%7
> Cshah
> >
> afs%40mellanox.com%7Cd356ec7687fb4ef96d8608d5310a0c27%7Ca652971c7
> d2e4d
> >
> 9ba6a4d149256f461b%7C0%7C0%7C636468841603090882&sdata=cXeLzqmlly
> nIg7xS
> > zCIUJsHGFLHfA0LBdCWLv0%2B3Ko0%3D&reserved=0
> >
> >>
> >> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> >> ---
> >>  doc/guides/nics/todo.rst | 11 +++++++++++
> >>  1 file changed, 11 insertions(+)
> >>
> >> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> >> index
> >> 9ace19716..61dd9cbc9 100644
> >> --- a/doc/guides/nics/todo.rst
> >> +++ b/doc/guides/nics/todo.rst
> >> @@ -72,3 +72,14 @@ This is the list for tracking required PMD changes
> >> triggered by library modifica
> >>   |                   | | thunderx, vhost, virtio,     |          |                |
> |
> >>   |                   | | vmxnet3                      |          |                |                                   |
> >>
> >> +-------------------+--------------------------------+----------+----
> >> ------------+-------
> >> ----------------------------+
> >> + | new offload flags | | af_packet, ark, avp, bnx2x,  | v18.05   |
> >> ce17eddefc20   | Use new ethdev offloads filed     |
> >> + |                   | | bnxt, bonding, cxgbe, dpaa,  |          | cba7f53b717d   | to
> get
> >> requested offload list     |
> >> + |                   | | dpaa2, e1000, ena, enic,     |          |                | instead of
> bitfield
> >> values.       |
> >> + |                   | | failsafe, fm10k, i40e, ixgbe,|          |                |
> >> |
> >> + |                   | | kni, liquidio, mlx4, mlx5,   |          |                |
> |
> >> + |                   | | mrvl, nfp, null, octeontx,   |          |                |
> |
> >> + |                   | | pcap, qede, ring, sfc,       |          |                |
> |
> >> + |                   | | softnic, szedata2, tap,      |          |                |
> |
> >> + |                   | | thunderx, vhost, virtio,     |          |                |
> |
> >> + |                   | | vmxnet3                      |          |                |                                   |
> >> + +-------------------+--------------------------------+----------+----------------+---
> ---
> >> -----------------------------+
> >> --
> >> 2.13.6
> >


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

* Re: [dpdk-dev] [PATCH 5/6] doc: add offload flag to PMD todo list
  2017-11-22  8:43       ` Shahaf Shuler
@ 2017-11-22  8:54         ` Thomas Monjalon
  0 siblings, 0 replies; 26+ messages in thread
From: Thomas Monjalon @ 2017-11-22  8:54 UTC (permalink / raw)
  To: Shahaf Shuler, Ferruh Yigit; +Cc: John McNamara, dev

22/11/2017 09:43, Shahaf Shuler:
> Tuesday, November 21, 2017 8:03 PM, Ferruh Yigit:
> > On 11/21/2017 12:32 AM, Shahaf Shuler wrote:
> > >
> > > The deprecation of the old offloads API is set to 18.05 [1]. my
> > understanding was that we agreed to convert all PMDs on 18.02.
> > > If the PMD deadline is going to be postponed I think the ethdev
> > deprecation should be updated.
> > 
> > Hi Shahaf,
> > 
> > I put deadlines roughly and they need to be discussed here.
> > 
> > But 18.02 deadline is not far away.
> > I think it is not realistic to expect all relevant PMDs do this conversion on
> > time, what do you think?
> 
> I cannot tell about other vendors plans. For mlx4 and mlx5 I plan to convert on 18.02.
> From the discussion we had on 17.11 for this feature we agreed to split it into 2, and the PMD part will be on 18.02. no one objected. 
> 
> Maybe on the next Tech-board you can have a sync between some of the vendors to see the plans.
> If not all can make it, we should postpone the deprecation of the old API.  

The current plan looks good to me.
We try to migrate to the new API in 18.02.
If some drivers are not migrated in 18.02, we urge them to do so in 18.05.
And we remove the old API in 18.05.

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

* Re: [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN flag to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN " Ferruh Yigit
  2017-11-21  1:53   ` Ferruh Yigit
@ 2017-12-06  9:24   ` Hemant Agrawal
  2017-12-11 14:16   ` Mcnamara, John
                     ` (2 subsequent siblings)
  4 siblings, 0 replies; 26+ messages in thread
From: Hemant Agrawal @ 2017-12-06  9:24 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, John McNamara; +Cc: dev

On 11/21/2017 7:12 AM, Ferruh Yigit wrote:
> To track modification:
> 380a7aab1ae2 ("mbuf: rename deprecated VLAN flags")
>
> Proposed deadline for PMDs is v18.02
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  doc/guides/nics/todo.rst | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> index 61dd9cbc9..493a49ff2 100644
> --- a/doc/guides/nics/todo.rst
> +++ b/doc/guides/nics/todo.rst
> @@ -83,3 +83,14 @@ This is the list for tracking required PMD changes triggered by library modifica
>   |                   | | thunderx, vhost, virtio,     |          |                |                                   |
>   |                   | | vmxnet3                      |          |                |                                   |
>   +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> + | | check new mbuf  | | af_packet, ark, avp, bnx2x,  | v18.02   | 380a7aab1ae2   | mbuf flag PKT_RX_VLAN_PKT renamed |
> + | | VLAN flag       | | bnxt, bonding, cxgbe, dpaa,  |          |                | to PKT_RX_VLAN and its meaning    |
> + | | PKT_RX_VLAN     | | dpaa2, e1000, ena, enic,     |          |                | changed, confirm PMD uses new     |
> + |                   | | failsafe, fm10k, i40e, ixgbe,|          |                | flag correct.                     |
> + |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
> + |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
> + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> + |                   | | thunderx, vhost, virtio,     |          |                |                                   |
> + |                   | | vmxnet3                      |          |                |                                   |
> + +-------------------+--------------------------------+----------+----------------+-----------------------------------+
>
w.r.t dpaa and dpaa2
Reviewed-by:  Hemant Agrawal <hemant.agrawal@nxp.com>

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

* Re: [dpdk-dev] [PATCH 3/6] doc: add dynamic logging to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 3/6] doc: add dynamic logging " Ferruh Yigit
@ 2017-12-08  5:12   ` Hemant Agrawal
  2017-12-11 14:09   ` Mcnamara, John
  1 sibling, 0 replies; 26+ messages in thread
From: Hemant Agrawal @ 2017-12-08  5:12 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, John McNamara; +Cc: dev

On 11/21/2017 7:12 AM, Ferruh Yigit wrote:
> To track modification:
> c1b5fa94a46f ("eal: support dynamic log types")
>
> Proposed deadline for PMDs is v18.08
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  doc/guides/nics/todo.rst | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> index b408fc691..973379a3d 100644
> --- a/doc/guides/nics/todo.rst
> +++ b/doc/guides/nics/todo.rst
> @@ -50,3 +50,14 @@ This is the list for tracking required PMD changes triggered by library modifica
>   |                   | | thunderx, vhost,             |          |                |                                   |
>   |                   | | vmxnet3                      |          |                |                                   |
>   +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> + | dynamic logging   | | af_packet, ark, avp, bnx2x,  | v18.08   | c1b5fa94a46f   | Switch to dynamic logging         |
> + |                   | | bnxt, bonding, cxgbe, dpaa,  |          |                | functions, remove static debug    |
> + |                   | | dpaa2, e1000, ena, enic,     |          |                | config options.                   |
> + |                   | | failsafe, fm10k, ixgbe,      |          |                |                                   |
> + |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
> + |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
> + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> + |                   | | thunderx, vhost, virtio,     |          |                |                                   |
> + |                   | | vmxnet3                      |          |                |                                   |
> + +-------------------+--------------------------------+----------+----------------+-----------------------------------+
>

One small correction -  dpaa is already supporting dynamic logging.

It is a good idea to have todo item documented.

Series-acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

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

* Re: [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list
  2017-11-21  1:42 [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list Ferruh Yigit
                   ` (4 preceding siblings ...)
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN " Ferruh Yigit
@ 2017-12-11 11:52 ` Mcnamara, John
  5 siblings, 0 replies; 26+ messages in thread
From: Mcnamara, John @ 2017-12-11 11:52 UTC (permalink / raw)
  To: Yigit, Ferruh, Thomas Monjalon; +Cc: dev



> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, November 21, 2017 1:43 AM
> To: Thomas Monjalon <thomas@monjalon.net>; Mcnamara, John
> <john.mcnamara@intel.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH 1/6] doc: add empty PMDs todo list
> 
> Some library modifications require extra checks or modifications in PMDs
> but currently we don't have a good way to trace these modifications.
> 
> And number of these kind of updates increasing by time.
> 
> This is an effort to document and track library updates that requires
> attention in PMDs.
> 
> Although this is under documentation main use case is for internal
> development.
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

>...
>
> +PMDs internal TODO list
> +=======================
> +
> +This is the list for tracking required PMD changes triggered by library
> modifications.
> +
> +.. table:: PMDs internal TODO list
> +
> + +------+------+----------+----------------+------+
> + | TODO | PMDs | Deadline | Related Commit | Note |
> + +======+======+==========+================+======+
> + |      |      |          |                |      |
> + +------+------+----------+----------------+------+


The table should have a label, and it should be indented to the level of .. table
as shown in the guidelines: http://dpdk.org/doc/guides/contributing/documentation.html#rst-guidelines

Something like this:

.. _table_nic_todo:

.. table:: PMDs internal TODO list

   +------+------+----------+----------------+------+
   | TODO | PMDs | Deadline | Related Commit | Note |
   +======+======+==========+================+======+
   |      |      |          |                |      |
   +------+------+----------+----------------+------+

However, it isn't an error so:


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

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

* Re: [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD " Ferruh Yigit
@ 2017-12-11 14:00   ` Marko Kovacevic
  2017-12-11 14:09   ` Mcnamara, John
                     ` (2 subsequent siblings)
  3 siblings, 0 replies; 26+ messages in thread
From: Marko Kovacevic @ 2017-12-11 14:00 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, John McNamara; +Cc: dev



On 21/11/2017 01:42, Ferruh Yigit wrote:
> To track modification:
> 8f094a9ac5d7 ("mbuf: set mbuf fields while in pool")
>
> Proposed deadline for PMDs is v18.02
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>
Acked-by:Marko Kovacevic<marko.kovacevic@intel.com>

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

* Re: [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD " Ferruh Yigit
  2017-12-11 14:00   ` Marko Kovacevic
@ 2017-12-11 14:09   ` Mcnamara, John
  2017-12-29 11:58   ` Andrew Rybchenko
  2017-12-29 17:07   ` Stephen Hemminger
  3 siblings, 0 replies; 26+ messages in thread
From: Mcnamara, John @ 2017-12-11 14:09 UTC (permalink / raw)
  To: Yigit, Ferruh, Thomas Monjalon; +Cc: dev



> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, November 21, 2017 1:43 AM
> To: Thomas Monjalon <thomas@monjalon.net>; Mcnamara, John
> <john.mcnamara@intel.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH 2/6] doc: add mbuf reorg to PMD todo list
> 
> To track modification:
> 8f094a9ac5d7 ("mbuf: set mbuf fields while in pool")
> 
> Proposed deadline for PMDs is v18.02
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

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

* Re: [dpdk-dev] [PATCH 3/6] doc: add dynamic logging to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 3/6] doc: add dynamic logging " Ferruh Yigit
  2017-12-08  5:12   ` Hemant Agrawal
@ 2017-12-11 14:09   ` Mcnamara, John
  1 sibling, 0 replies; 26+ messages in thread
From: Mcnamara, John @ 2017-12-11 14:09 UTC (permalink / raw)
  To: Yigit, Ferruh, Thomas Monjalon; +Cc: dev



> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, November 21, 2017 1:43 AM
> To: Thomas Monjalon <thomas@monjalon.net>; Mcnamara, John
> <john.mcnamara@intel.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH 3/6] doc: add dynamic logging to PMD todo list
> 
> To track modification:
> c1b5fa94a46f ("eal: support dynamic log types")
> 
> Proposed deadline for PMDs is v18.08
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

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

* Re: [dpdk-dev] [PATCH 4/6] doc: add descriptor status API to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 4/6] doc: add descriptor status API " Ferruh Yigit
@ 2017-12-11 14:11   ` Mcnamara, John
  0 siblings, 0 replies; 26+ messages in thread
From: Mcnamara, John @ 2017-12-11 14:11 UTC (permalink / raw)
  To: Yigit, Ferruh, Thomas Monjalon; +Cc: dev



> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, November 21, 2017 1:43 AM
> To: Thomas Monjalon <thomas@monjalon.net>; Mcnamara, John
> <john.mcnamara@intel.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH 4/6] doc: add descriptor status API to PMD todo list
> 
> To track modification:
> b1b700ce7d6f ("ethdev: add descriptor status API")
> 
> Proposed deadline for PMDs is v19.02
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

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

* Re: [dpdk-dev] [PATCH 5/6] doc: add offload flag to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 5/6] doc: add offload flag " Ferruh Yigit
  2017-11-21  8:32   ` Shahaf Shuler
@ 2017-12-11 14:11   ` Mcnamara, John
  1 sibling, 0 replies; 26+ messages in thread
From: Mcnamara, John @ 2017-12-11 14:11 UTC (permalink / raw)
  To: Yigit, Ferruh, Thomas Monjalon; +Cc: dev



> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, November 21, 2017 1:43 AM
> To: Thomas Monjalon <thomas@monjalon.net>; Mcnamara, John
> <john.mcnamara@intel.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH 5/6] doc: add offload flag to PMD todo list
> 
> To track modification:
> ce17eddefc20 ("ethdev: introduce Rx queue offloads API") cba7f53b717d
> ("ethdev: introduce Tx queue offloads API")
> 
> Proposed deadline for PMDs is v18.05
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

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

* Re: [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN flag to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN " Ferruh Yigit
  2017-11-21  1:53   ` Ferruh Yigit
  2017-12-06  9:24   ` Hemant Agrawal
@ 2017-12-11 14:16   ` Mcnamara, John
  2017-12-29 11:50   ` Andrew Rybchenko
  2017-12-29 17:09   ` Stephen Hemminger
  4 siblings, 0 replies; 26+ messages in thread
From: Mcnamara, John @ 2017-12-11 14:16 UTC (permalink / raw)
  To: Yigit, Ferruh, Thomas Monjalon; +Cc: dev



> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, November 21, 2017 1:43 AM
> To: Thomas Monjalon <thomas@monjalon.net>; Mcnamara, John
> <john.mcnamara@intel.com>
> Cc: dev@dpdk.org; Yigit, Ferruh <ferruh.yigit@intel.com>
> Subject: [PATCH 6/6] doc: add mbuf VLAN flag to PMD todo list
> 
> To track modification:
> 380a7aab1ae2 ("mbuf: rename deprecated VLAN flags")
> 
> Proposed deadline for PMDs is v18.02
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>


It might be worth putting | before all the text in the first column,
even if it is just a single item, since it will justify the text in
the same way.

Also it might be worth adding it to the text in the last column so
that the column isn't too wide.

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

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

* Re: [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN flag to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN " Ferruh Yigit
                     ` (2 preceding siblings ...)
  2017-12-11 14:16   ` Mcnamara, John
@ 2017-12-29 11:50   ` Andrew Rybchenko
  2017-12-29 17:09   ` Stephen Hemminger
  4 siblings, 0 replies; 26+ messages in thread
From: Andrew Rybchenko @ 2017-12-29 11:50 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, John McNamara; +Cc: dev

On 11/21/2017 04:42 AM, Ferruh Yigit wrote:
> To track modification:
> 380a7aab1ae2 ("mbuf: rename deprecated VLAN flags")
>
> Proposed deadline for PMDs is v18.02
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>   doc/guides/nics/todo.rst | 11 +++++++++++
>   1 file changed, 11 insertions(+)
>
> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> index 61dd9cbc9..493a49ff2 100644
> --- a/doc/guides/nics/todo.rst
> +++ b/doc/guides/nics/todo.rst
> @@ -83,3 +83,14 @@ This is the list for tracking required PMD changes triggered by library modifica
>    |                   | | thunderx, vhost, virtio,     |          |                |                                   |
>    |                   | | vmxnet3                      |          |                |                                   |
>    +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> + | | check new mbuf  | | af_packet, ark, avp, bnx2x,  | v18.02   | 380a7aab1ae2   | mbuf flag PKT_RX_VLAN_PKT renamed |
> + | | VLAN flag       | | bnxt, bonding, cxgbe, dpaa,  |          |                | to PKT_RX_VLAN and its meaning    |
> + | | PKT_RX_VLAN     | | dpaa2, e1000, ena, enic,     |          |                | changed, confirm PMD uses new     |
> + |                   | | failsafe, fm10k, i40e, ixgbe,|          |                | flag correct.                     |
> + |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
> + |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
> + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> + |                   | | thunderx, vhost, virtio,     |          |                |                                   |
> + |                   | | vmxnet3                      |          |                |                                   |
> + +-------------------+--------------------------------+----------+----------------+-----------------------------------+

sfc may be excluded from the list since neither old nor new flag is used.
Of course, I can send a patch to remove it, if required.

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

* Re: [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD " Ferruh Yigit
  2017-12-11 14:00   ` Marko Kovacevic
  2017-12-11 14:09   ` Mcnamara, John
@ 2017-12-29 11:58   ` Andrew Rybchenko
  2017-12-29 17:07   ` Stephen Hemminger
  3 siblings, 0 replies; 26+ messages in thread
From: Andrew Rybchenko @ 2017-12-29 11:58 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, John McNamara; +Cc: dev

On 11/21/2017 04:42 AM, Ferruh Yigit wrote:
> To track modification:
> 8f094a9ac5d7 ("mbuf: set mbuf fields while in pool")
>
> Proposed deadline for PMDs is v18.02
>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>   doc/guides/nics/todo.rst | 19 ++++++++++++++-----
>   1 file changed, 14 insertions(+), 5 deletions(-)
>
> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> index 874811ad5..b408fc691 100644
> --- a/doc/guides/nics/todo.rst
> +++ b/doc/guides/nics/todo.rst
> @@ -36,8 +36,17 @@ This is the list for tracking required PMD changes triggered by library modifica
>   
>   .. table:: PMDs internal TODO list
>   
> - +------+------+----------+----------------+------+
> - | TODO | PMDs | Deadline | Related Commit | Note |
> - +======+======+==========+================+======+
> - |      |      |          |                |      |
> - +------+------+----------+----------------+------+
> + +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> + | TODO              | PMDs                           | Deadline | Related Commit | Note                              |
> + +===================+================================+==========+================+===================================+
> + | mbuf organisation | | af_packet, ark, avp, bnx2x,  | v18.02   | 8f094a9ac5d7   | Remove unnecessary mbuf field     |
> + |                   | | bnxt, bonding, cxgbe, dpaa,  |          |                | initialization.                   |
> + |                   | | dpaa2, e1000, ena,           |          |                |                                   |
> + |                   | | failsafe, fm10k,             |          |                |                                   |
> + |                   | | kni, mlx4,                   |          |                |                                   |
> + |                   | | mrvl, nfp, octeontx,         |          |                |                                   |
> + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> + |                   | | thunderx, vhost,             |          |                |                                   |
> + |                   | | vmxnet3                      |          |                |                                   |
> + +-------------------+--------------------------------+----------+----------------+-----------------------------------+

sfc may be removed from the list since it has already been done.
Reference counter and number of segments are still set as part
of 64-bit rearm data, but I think it is rather a question of separate
performance investigation and optimization.
Of course, we can send a patch to remove from todo list with
explanations why.

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

* Re: [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD " Ferruh Yigit
                     ` (2 preceding siblings ...)
  2017-12-29 11:58   ` Andrew Rybchenko
@ 2017-12-29 17:07   ` Stephen Hemminger
  2018-01-03 14:00     ` Olivier Matz
  3 siblings, 1 reply; 26+ messages in thread
From: Stephen Hemminger @ 2017-12-29 17:07 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Thomas Monjalon, John McNamara, dev

On Tue, 21 Nov 2017 01:42:37 +0000
Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> To track modification:
> 8f094a9ac5d7 ("mbuf: set mbuf fields while in pool")
> 
> Proposed deadline for PMDs is v18.02
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  doc/guides/nics/todo.rst | 19 ++++++++++++++-----
>  1 file changed, 14 insertions(+), 5 deletions(-)
> 
> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> index 874811ad5..b408fc691 100644
> --- a/doc/guides/nics/todo.rst
> +++ b/doc/guides/nics/todo.rst
> @@ -36,8 +36,17 @@ This is the list for tracking required PMD changes triggered by library modifica
>  
>  .. table:: PMDs internal TODO list
>  
> - +------+------+----------+----------------+------+
> - | TODO | PMDs | Deadline | Related Commit | Note |
> - +======+======+==========+================+======+
> - |      |      |          |                |      |
> - +------+------+----------+----------------+------+
> + +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> + | TODO              | PMDs                           | Deadline | Related Commit | Note                              |
> + +===================+================================+==========+================+===================================+
> + | mbuf organisation | | af_packet, ark, avp, bnx2x,  | v18.02   | 8f094a9ac5d7   | Remove unnecessary mbuf field     |
> + |                   | | bnxt, bonding, cxgbe, dpaa,  |          |                | initialization.                   |
> + |                   | | dpaa2, e1000, ena,           |          |                |                                   |
> + |                   | | failsafe, fm10k,             |          |                |                                   |
> + |                   | | kni, mlx4,                   |          |                |                                   |
> + |                   | | mrvl, nfp, octeontx,         |          |                |                                   |
> + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> + |                   | | thunderx, vhost,             |          |                |                                   |
> + |                   | | vmxnet3                      |          |                |                                   |
> + +-------------------+--------------------------------+----------+----------------+-----------------------------------+

NAK

This open source, if the fix is trivial please just fix all the drivers directly now.
These TODO lists never get done.

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

* Re: [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN flag to PMD todo list
  2017-11-21  1:42 ` [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN " Ferruh Yigit
                     ` (3 preceding siblings ...)
  2017-12-29 11:50   ` Andrew Rybchenko
@ 2017-12-29 17:09   ` Stephen Hemminger
  2018-01-03 14:14     ` Olivier Matz
  4 siblings, 1 reply; 26+ messages in thread
From: Stephen Hemminger @ 2017-12-29 17:09 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Thomas Monjalon, John McNamara, dev

On Tue, 21 Nov 2017 01:42:41 +0000
Ferruh Yigit <ferruh.yigit@intel.com> wrote:

> To track modification:
> 380a7aab1ae2 ("mbuf: rename deprecated VLAN flags")
> 
> Proposed deadline for PMDs is v18.02
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  doc/guides/nics/todo.rst | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> index 61dd9cbc9..493a49ff2 100644
> --- a/doc/guides/nics/todo.rst
> +++ b/doc/guides/nics/todo.rst
> @@ -83,3 +83,14 @@ This is the list for tracking required PMD changes triggered by library modifica
>   |                   | | thunderx, vhost, virtio,     |          |                |                                   |
>   |                   | | vmxnet3                      |          |                |                                   |
>   +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> + | | check new mbuf  | | af_packet, ark, avp, bnx2x,  | v18.02   | 380a7aab1ae2   | mbuf flag PKT_RX_VLAN_PKT renamed |
> + | | VLAN flag       | | bnxt, bonding, cxgbe, dpaa,  |          |                | to PKT_RX_VLAN and its meaning    |
> + | | PKT_RX_VLAN     | | dpaa2, e1000, ena, enic,     |          |                | changed, confirm PMD uses new     |
> + |                   | | failsafe, fm10k, i40e, ixgbe,|          |                | flag correct.                     |
> + |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
> + |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
> + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> + |                   | | thunderx, vhost, virtio,     |          |                |                                   |
> + |                   | | vmxnet3                      |          |                |                                   |
> + +-------------------+--------------------------------+----------+----------------+-----------------------------------+


NAK
Fix the other drivers.

This open source, if you introduce a new behavior it is the responsibility of the person adding
the new stuff to fix every other driver.

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

* Re: [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD todo list
  2017-12-29 17:07   ` Stephen Hemminger
@ 2018-01-03 14:00     ` Olivier Matz
  0 siblings, 0 replies; 26+ messages in thread
From: Olivier Matz @ 2018-01-03 14:00 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: Ferruh Yigit, Thomas Monjalon, John McNamara, dev

Hi,

On Fri, Dec 29, 2017 at 09:07:49AM -0800, Stephen Hemminger wrote:
> On Tue, 21 Nov 2017 01:42:37 +0000
> Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> 
> > To track modification:
> > 8f094a9ac5d7 ("mbuf: set mbuf fields while in pool")
> > 
> > Proposed deadline for PMDs is v18.02
> > 
> > Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> > ---
> >  doc/guides/nics/todo.rst | 19 ++++++++++++++-----
> >  1 file changed, 14 insertions(+), 5 deletions(-)
> > 
> > diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> > index 874811ad5..b408fc691 100644
> > --- a/doc/guides/nics/todo.rst
> > +++ b/doc/guides/nics/todo.rst
> > @@ -36,8 +36,17 @@ This is the list for tracking required PMD changes triggered by library modifica
> >  
> >  .. table:: PMDs internal TODO list
> >  
> > - +------+------+----------+----------------+------+
> > - | TODO | PMDs | Deadline | Related Commit | Note |
> > - +======+======+==========+================+======+
> > - |      |      |          |                |      |
> > - +------+------+----------+----------------+------+
> > + +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> > + | TODO              | PMDs                           | Deadline | Related Commit | Note                              |
> > + +===================+================================+==========+================+===================================+
> > + | mbuf organisation | | af_packet, ark, avp, bnx2x,  | v18.02   | 8f094a9ac5d7   | Remove unnecessary mbuf field     |
> > + |                   | | bnxt, bonding, cxgbe, dpaa,  |          |                | initialization.                   |
> > + |                   | | dpaa2, e1000, ena,           |          |                |                                   |
> > + |                   | | failsafe, fm10k,             |          |                |                                   |
> > + |                   | | kni, mlx4,                   |          |                |                                   |
> > + |                   | | mrvl, nfp, octeontx,         |          |                |                                   |
> > + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> > + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> > + |                   | | thunderx, vhost,             |          |                |                                   |
> > + |                   | | vmxnet3                      |          |                |                                   |
> > + +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> 
> NAK
> 
> This open source, if the fix is trivial please just fix all the drivers directly now.
> These TODO lists never get done.

The fix is not that trivial, especially for vector PMDs.
It can also impact performance.

It looks dangerous for me to blindly do a patch without any hardware to
do the test.

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

* Re: [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN flag to PMD todo list
  2017-12-29 17:09   ` Stephen Hemminger
@ 2018-01-03 14:14     ` Olivier Matz
  0 siblings, 0 replies; 26+ messages in thread
From: Olivier Matz @ 2018-01-03 14:14 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: Ferruh Yigit, Thomas Monjalon, John McNamara, dev

On Fri, Dec 29, 2017 at 09:09:25AM -0800, Stephen Hemminger wrote:
> On Tue, 21 Nov 2017 01:42:41 +0000
> Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> 
> > To track modification:
> > 380a7aab1ae2 ("mbuf: rename deprecated VLAN flags")
> > 
> > Proposed deadline for PMDs is v18.02
> > 
> > Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> > ---
> >  doc/guides/nics/todo.rst | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> > 
> > diff --git a/doc/guides/nics/todo.rst b/doc/guides/nics/todo.rst
> > index 61dd9cbc9..493a49ff2 100644
> > --- a/doc/guides/nics/todo.rst
> > +++ b/doc/guides/nics/todo.rst
> > @@ -83,3 +83,14 @@ This is the list for tracking required PMD changes triggered by library modifica
> >   |                   | | thunderx, vhost, virtio,     |          |                |                                   |
> >   |                   | | vmxnet3                      |          |                |                                   |
> >   +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> > + | | check new mbuf  | | af_packet, ark, avp, bnx2x,  | v18.02   | 380a7aab1ae2   | mbuf flag PKT_RX_VLAN_PKT renamed |
> > + | | VLAN flag       | | bnxt, bonding, cxgbe, dpaa,  |          |                | to PKT_RX_VLAN and its meaning    |
> > + | | PKT_RX_VLAN     | | dpaa2, e1000, ena, enic,     |          |                | changed, confirm PMD uses new     |
> > + |                   | | failsafe, fm10k, i40e, ixgbe,|          |                | flag correct.                     |
> > + |                   | | kni, liquidio, mlx4, mlx5,   |          |                |                                   |
> > + |                   | | mrvl, nfp, null, octeontx,   |          |                |                                   |
> > + |                   | | pcap, qede, ring, sfc,       |          |                |                                   |
> > + |                   | | softnic, szedata2, tap,      |          |                |                                   |
> > + |                   | | thunderx, vhost, virtio,     |          |                |                                   |
> > + |                   | | vmxnet3                      |          |                |                                   |
> > + +-------------------+--------------------------------+----------+----------------+-----------------------------------+
> 
> 
> NAK
> Fix the other drivers.
> 
> This open source, if you introduce a new behavior it is the responsibility of the person adding
> the new stuff to fix every other driver.

Actually, no new behavior was introduced. The meaning of the vlan flags
has been clarified. The purpose of this note is to ask the maintainers
to ensure that the behavior of the PMDs matches the API. This operation
is not easy to do for someone that does not know the hardware, and
that cannot test it.

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

end of thread, other threads:[~2018-01-03 14:14 UTC | newest]

Thread overview: 26+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-21  1:42 [dpdk-dev] [PATCH 1/6] doc: add empty PMDs todo list Ferruh Yigit
2017-11-21  1:42 ` [dpdk-dev] [PATCH 2/6] doc: add mbuf reorg to PMD " Ferruh Yigit
2017-12-11 14:00   ` Marko Kovacevic
2017-12-11 14:09   ` Mcnamara, John
2017-12-29 11:58   ` Andrew Rybchenko
2017-12-29 17:07   ` Stephen Hemminger
2018-01-03 14:00     ` Olivier Matz
2017-11-21  1:42 ` [dpdk-dev] [PATCH 3/6] doc: add dynamic logging " Ferruh Yigit
2017-12-08  5:12   ` Hemant Agrawal
2017-12-11 14:09   ` Mcnamara, John
2017-11-21  1:42 ` [dpdk-dev] [PATCH 4/6] doc: add descriptor status API " Ferruh Yigit
2017-12-11 14:11   ` Mcnamara, John
2017-11-21  1:42 ` [dpdk-dev] [PATCH 5/6] doc: add offload flag " Ferruh Yigit
2017-11-21  8:32   ` Shahaf Shuler
2017-11-21 18:02     ` Ferruh Yigit
2017-11-22  8:43       ` Shahaf Shuler
2017-11-22  8:54         ` Thomas Monjalon
2017-12-11 14:11   ` Mcnamara, John
2017-11-21  1:42 ` [dpdk-dev] [PATCH 6/6] doc: add mbuf VLAN " Ferruh Yigit
2017-11-21  1:53   ` Ferruh Yigit
2017-12-06  9:24   ` Hemant Agrawal
2017-12-11 14:16   ` Mcnamara, John
2017-12-29 11:50   ` Andrew Rybchenko
2017-12-29 17:09   ` Stephen Hemminger
2018-01-03 14:14     ` Olivier Matz
2017-12-11 11:52 ` [dpdk-dev] [PATCH 1/6] doc: add empty PMDs " Mcnamara, John

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