DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev
@ 2017-07-12  7:58 Zhiyong Yang
  2017-08-04  1:12 ` Tan, Jianfeng
                   ` (2 more replies)
  0 siblings, 3 replies; 19+ messages in thread
From: Zhiyong Yang @ 2017-07-12  7:58 UTC (permalink / raw)
  To: dev, zhihong.wang; +Cc: john.mcnamara, oilvier.matz, thomas, Zhiyong Yang

  This is an API/ABI change notice for DPDK 17.11 on redefinition of
port_id. port_id is defined as uint8_t by now, which is just ranged
from 0 to 255. For more and more scenerioes, more than 256 devices are
needed to support for vdev scalability.

  It is necessary for redefinition of port_id to extend from 1 bytes
to 2 bytes. All ethdev APIs and use cases related to port_id will be
changed at the same time.

Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
---
 doc/guides/rel_notes/deprecation.rst | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 257dcba..f265980 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -54,6 +54,10 @@ Deprecation Notices
   Target release for removal of the legacy API will be defined once most
   PMDs have switched to rte_flow.
 
+* ABI/API changes are planned for 17.11 in the "rte_eth_dev_data" structure.
+  Change the definition of port_id from 8bits to 16bits in order to support
+  more than 256 devices in DPDK.
+
 * librte_table: The ``key_mask`` parameter will be added to all the hash tables
   that currently do not have it, as well as to the hash compute function prototype.
   The non-"do-sig" versions of the hash tables will be removed
-- 
2.9.3

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

* Re: [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev
  2017-07-12  7:58 [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev Zhiyong Yang
@ 2017-08-04  1:12 ` Tan, Jianfeng
  2017-08-04  4:07 ` Yuanhan Liu
  2017-08-04  5:27 ` [dpdk-dev] [PATCH v2] " Zhiyong Yang
  2 siblings, 0 replies; 19+ messages in thread
From: Tan, Jianfeng @ 2017-08-04  1:12 UTC (permalink / raw)
  To: Zhiyong Yang, dev, zhihong.wang; +Cc: john.mcnamara, oilvier.matz, thomas



On 7/12/2017 12:58 AM, Zhiyong Yang wrote:
>    This is an API/ABI change notice for DPDK 17.11 on redefinition of

A nit: we usually don't indent at the commit message paragraph.

> port_id. port_id is defined as uint8_t by now, which is just ranged
> from 0 to 255. For more and more scenerioes, more than 256 devices are
> needed to support for vdev scalability.
>
>    It is necessary for redefinition of port_id to extend from 1 bytes
> to 2 bytes. All ethdev APIs and use cases related to port_id will be
> changed at the same time.
>
> Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>

Acked-by: Jianfeng Tan <jianfeng.tan@intel.com>

Thanks,
Jianfeng

> ---
>   doc/guides/rel_notes/deprecation.rst | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 257dcba..f265980 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -54,6 +54,10 @@ Deprecation Notices
>     Target release for removal of the legacy API will be defined once most
>     PMDs have switched to rte_flow.
>   
> +* ABI/API changes are planned for 17.11 in the "rte_eth_dev_data" structure.
> +  Change the definition of port_id from 8bits to 16bits in order to support
> +  more than 256 devices in DPDK.
> +
>   * librte_table: The ``key_mask`` parameter will be added to all the hash tables
>     that currently do not have it, as well as to the hash compute function prototype.
>     The non-"do-sig" versions of the hash tables will be removed

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

* Re: [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev
  2017-07-12  7:58 [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev Zhiyong Yang
  2017-08-04  1:12 ` Tan, Jianfeng
@ 2017-08-04  4:07 ` Yuanhan Liu
  2017-08-04  4:13   ` Jerin Jacob
  2017-08-04  5:36   ` Yang, Zhiyong
  2017-08-04  5:27 ` [dpdk-dev] [PATCH v2] " Zhiyong Yang
  2 siblings, 2 replies; 19+ messages in thread
From: Yuanhan Liu @ 2017-08-04  4:07 UTC (permalink / raw)
  To: Zhiyong Yang; +Cc: dev, zhihong.wang, john.mcnamara, oilvier.matz, thomas

On Wed, Jul 12, 2017 at 03:58:46PM +0800, Zhiyong Yang wrote:
>   This is an API/ABI change notice for DPDK 17.11 on redefinition of

Don't put 2 whitespace char indentation here.

> port_id. port_id is defined as uint8_t by now, which is just ranged
> from 0 to 255. For more and more scenerioes, more than 256 devices are
> needed to support for vdev scalability.
> 
>   It is necessary for redefinition of port_id to extend from 1 bytes
> to 2 bytes. All ethdev APIs and use cases related to port_id will be
> changed at the same time.
> 
> Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 257dcba..f265980 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -54,6 +54,10 @@ Deprecation Notices
>    Target release for removal of the legacy API will be defined once most
>    PMDs have switched to rte_flow.
>  
> +* ABI/API changes are planned for 17.11 in the "rte_eth_dev_data" structure.

That's for ABI only and you missed the API part. Meaning, you need also
mention that all APIs have port_id parameter will be changed too.

Also, rte_eth_dev_data is not the only structure has ABI changes. There
are few more (for instance, rte_port_ethdev.h). As said before, you don't
have to list all of them, but at least you should not list one example
only as it seems that's the only structure has ABI change.

> +  Change the definition of port_id from 8bits to 16bits in order to support
> +  more than 256 devices in DPDK.

8bits -> 8 bits. You might want to reword it further. Besides that, this
change is indeed needed. Thus,

Acked-by: Yuanhan Liu <yliu@fridaylinux.org>

	--yliu
> +
>  * librte_table: The ``key_mask`` parameter will be added to all the hash tables
>    that currently do not have it, as well as to the hash compute function prototype.
>    The non-"do-sig" versions of the hash tables will be removed
> -- 
> 2.9.3

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

* Re: [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev
  2017-08-04  4:07 ` Yuanhan Liu
@ 2017-08-04  4:13   ` Jerin Jacob
  2017-08-04  5:36   ` Yang, Zhiyong
  1 sibling, 0 replies; 19+ messages in thread
From: Jerin Jacob @ 2017-08-04  4:13 UTC (permalink / raw)
  To: Yuanhan Liu
  Cc: Zhiyong Yang, dev, zhihong.wang, john.mcnamara, oilvier.matz, thomas

-----Original Message-----
> Date: Fri, 4 Aug 2017 12:07:28 +0800
> From: Yuanhan Liu <yliu@fridaylinux.org>
> To: Zhiyong Yang <zhiyong.yang@intel.com>
> Cc: dev@dpdk.org, zhihong.wang@intel.com, john.mcnamara@intel.com,
>  oilvier.matz@6wind.com, thomas@monjalon.net
> Subject: Re: [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev
> User-Agent: Mutt/1.5.24 (2015-08-30)
> 
> On Wed, Jul 12, 2017 at 03:58:46PM +0800, Zhiyong Yang wrote:
> >   This is an API/ABI change notice for DPDK 17.11 on redefinition of
> 
> Don't put 2 whitespace char indentation here.
> 
> > port_id. port_id is defined as uint8_t by now, which is just ranged
> > from 0 to 255. For more and more scenerioes, more than 256 devices are
> > needed to support for vdev scalability.
> > 
> >   It is necessary for redefinition of port_id to extend from 1 bytes
> > to 2 bytes. All ethdev APIs and use cases related to port_id will be
> > changed at the same time.
> > 
> > Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> > index 257dcba..f265980 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -54,6 +54,10 @@ Deprecation Notices
> >    Target release for removal of the legacy API will be defined once most
> >    PMDs have switched to rte_flow.
> >  
> > +* ABI/API changes are planned for 17.11 in the "rte_eth_dev_data" structure.
> 
> That's for ABI only and you missed the API part. Meaning, you need also
> mention that all APIs have port_id parameter will be changed too.
> 
> Also, rte_eth_dev_data is not the only structure has ABI changes. There
> are few more (for instance, rte_port_ethdev.h). As said before, you don't
> have to list all of them, but at least you should not list one example
> only as it seems that's the only structure has ABI change.
> 
> > +  Change the definition of port_id from 8bits to 16bits in order to support
> > +  more than 256 devices in DPDK.
> 
> 8bits -> 8 bits. You might want to reword it further. Besides that, this
> change is indeed needed. Thus,
> 
> Acked-by: Yuanhan Liu <yliu@fridaylinux.org>

Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>


> 
> 	--yliu
> > +
> >  * librte_table: The ``key_mask`` parameter will be added to all the hash tables
> >    that currently do not have it, as well as to the hash compute function prototype.
> >    The non-"do-sig" versions of the hash tables will be removed
> > -- 
> > 2.9.3

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

* [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-07-12  7:58 [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev Zhiyong Yang
  2017-08-04  1:12 ` Tan, Jianfeng
  2017-08-04  4:07 ` Yuanhan Liu
@ 2017-08-04  5:27 ` Zhiyong Yang
  2017-08-04  6:56   ` Remy Horton
  2017-08-07 12:42   ` [dpdk-dev] [PATCH v3] " Zhiyong Yang
  2 siblings, 2 replies; 19+ messages in thread
From: Zhiyong Yang @ 2017-08-04  5:27 UTC (permalink / raw)
  To: dev; +Cc: jianfeng.tan, yliu, jerin.jacob, thomas, Zhiyong Yang

This is an API/ABI change notice for DPDK 17.11 on redefinition of
port_id. port_id is defined as uint8_t by now, which is just ranged
from 0 to 255. For more and more scenerioes, more than 256 ports are
needed to support for vdev scalability.

It is necessary for redefinition of port_id to extend from 1 bytes
to 2 bytes. All ethdev APIs and use cases related to port_id will be
changed at the same time.

cc: jianfeng.tan@intel.com
cc: yliu@fridaylinux.org
cc: jerin.jacob@caviumnetworks.com
cc: thomas@monjalon.net

Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
Acked-by: Jianfeng Tan <jianfeng.tan@intel.com>
Acked-by: Yuanhan Liu <yliu@fridaylinux.org>
Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
---

Change in V2.
1. remove indent of paragraph in commit log.
2. add more description about API/ABI according to yuanhan's comments.

 doc/guides/rel_notes/deprecation.rst | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 72aa40495..4edaf92fb 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -49,6 +49,12 @@ Deprecation Notices
   Target release for removal of the legacy API will be defined once most
   PMDs have switched to rte_flow.
 
+* ABI/API changes are planned for 17.11 in all structures which include port_id
+  definition such as "rte_eth_dev_data", "rte_port_ethdev_reader_params",
+  "rte_port_ethdev_writer_params", and so on. The definition of port_id will be
+  changed from 8 bits to 16 bits in order to support more than 256 ports in
+  DPDK. All APIs which have port_id parameter will be changed at the same time.
+
 * librte_table: The ``key_mask`` parameter will be added to all the hash tables
   that currently do not have it, as well as to the hash compute function prototype.
   The non-"do-sig" versions of the hash tables will be removed
-- 
2.13.3

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

* Re: [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev
  2017-08-04  4:07 ` Yuanhan Liu
  2017-08-04  4:13   ` Jerin Jacob
@ 2017-08-04  5:36   ` Yang, Zhiyong
  1 sibling, 0 replies; 19+ messages in thread
From: Yang, Zhiyong @ 2017-08-04  5:36 UTC (permalink / raw)
  To: Yuanhan Liu; +Cc: dev, Wang, Zhihong, Mcnamara, John, thomas, Olivier Matz

Hi, yuanhan:

> -----Original Message-----
> From: Yuanhan Liu [mailto:yliu@fridaylinux.org]
> Sent: Friday, August 4, 2017 12:07 PM
> To: Yang, Zhiyong <zhiyong.yang@intel.com>
> Cc: dev@dpdk.org; Wang, Zhihong <zhihong.wang@intel.com>; Mcnamara,
> John <john.mcnamara@intel.com>; oilvier.matz@6wind.com;
> thomas@monjalon.net
> Subject: Re: [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev
> 
> On Wed, Jul 12, 2017 at 03:58:46PM +0800, Zhiyong Yang wrote:
> >   This is an API/ABI change notice for DPDK 17.11 on redefinition of
> 
> Don't put 2 whitespace char indentation here.
> 
> > port_id. port_id is defined as uint8_t by now, which is just ranged
> > from 0 to 255. For more and more scenerioes, more than 256 devices are
> > needed to support for vdev scalability.
> >
> >   It is necessary for redefinition of port_id to extend from 1 bytes
> > to 2 bytes. All ethdev APIs and use cases related to port_id will be
> > changed at the same time.
> >
> > Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> > ---
> >  doc/guides/rel_notes/deprecation.rst | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/doc/guides/rel_notes/deprecation.rst
> > b/doc/guides/rel_notes/deprecation.rst
> > index 257dcba..f265980 100644
> > --- a/doc/guides/rel_notes/deprecation.rst
> > +++ b/doc/guides/rel_notes/deprecation.rst
> > @@ -54,6 +54,10 @@ Deprecation Notices
> >    Target release for removal of the legacy API will be defined once most
> >    PMDs have switched to rte_flow.
> >
> > +* ABI/API changes are planned for 17.11 in the "rte_eth_dev_data" structure.
> 
> That's for ABI only and you missed the API part. Meaning, you need also mention
> that all APIs have port_id parameter will be changed too.
> 
> Also, rte_eth_dev_data is not the only structure has ABI changes. There are few
> more (for instance, rte_port_ethdev.h). As said before, you don't have to list all
> of them, but at least you should not list one example only as it seems that's the
> only structure has ABI change.
> 
> > +  Change the definition of port_id from 8bits to 16bits in order to
> > + support  more than 256 devices in DPDK.
> 
> 8bits -> 8 bits. You might want to reword it further. Besides that, this change is
> indeed needed. Thus,
> 
> Acked-by: Yuanhan Liu <yliu@fridaylinux.org>
> 
> 	--yliu

Thank you, yuanhan. I have reworded and sent v2  according to your comments.

Zhiyong Yang

> > +
> >  * librte_table: The ``key_mask`` parameter will be added to all the hash tables
> >    that currently do not have it, as well as to the hash compute function
> prototype.
> >    The non-"do-sig" versions of the hash tables will be removed
> > --
> > 2.9.3

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-08-04  5:27 ` [dpdk-dev] [PATCH v2] " Zhiyong Yang
@ 2017-08-04  6:56   ` Remy Horton
  2017-08-05 10:34     ` Yang, Zhiyong
  2017-08-07 15:20     ` Shahaf Shuler
  2017-08-07 12:42   ` [dpdk-dev] [PATCH v3] " Zhiyong Yang
  1 sibling, 2 replies; 19+ messages in thread
From: Remy Horton @ 2017-08-04  6:56 UTC (permalink / raw)
  To: Zhiyong Yang, dev; +Cc: jianfeng.tan, yliu, jerin.jacob, thomas


On 04/08/2017 06:27, Zhiyong Yang wrote:
> This is an API/ABI change notice for DPDK 17.11 on redefinition of
> port_id. port_id is defined as uint8_t by now, which is just ranged
> from 0 to 255. For more and more scenerioes, more than 256 ports are
> needed to support for vdev scalability.
>
> It is necessary for redefinition of port_id to extend from 1 bytes
> to 2 bytes. All ethdev APIs and use cases related to port_id will be
> changed at the same time.

I think this reads a little better:

This is an API/ABI change notice for DPDK 17.11 announcing the 
redefinition of port_id. port_id is currently defined as uint8_t, which 
is limited to the range 0 to 255. A larger range is required for vdev 
scalability.

It is necessary for a redefinition of port_id to extend it from 1 bytes 
to 2 bytes. All ethdev APIs and usages related to port_id will be 
changed at the same time.

>  doc/guides/rel_notes/deprecation.rst | 6 ++++++
>  1 file changed, 6 insertions(+)

Acked-by: Remy Horton <remy.horton@intel.com>

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-08-04  6:56   ` Remy Horton
@ 2017-08-05 10:34     ` Yang, Zhiyong
  2017-08-07  3:07       ` Yang, Zhiyong
  2017-08-07 15:20     ` Shahaf Shuler
  1 sibling, 1 reply; 19+ messages in thread
From: Yang, Zhiyong @ 2017-08-05 10:34 UTC (permalink / raw)
  To: Horton, Remy, dev; +Cc: Tan, Jianfeng, yliu, jerin.jacob, thomas

Hi, Remy:

> -----Original Message-----
> From: Horton, Remy
> Sent: Friday, August 4, 2017 2:56 PM
> To: Yang, Zhiyong <zhiyong.yang@intel.com>; dev@dpdk.org
> Cc: Tan, Jianfeng <jianfeng.tan@intel.com>; yliu@fridaylinux.org;
> jerin.jacob@caviumnetworks.com; thomas@monjalon.net
> Subject: Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for
> ethdev
> 
> 
> On 04/08/2017 06:27, Zhiyong Yang wrote:
> > This is an API/ABI change notice for DPDK 17.11 on redefinition of
> > port_id. port_id is defined as uint8_t by now, which is just ranged
> > from 0 to 255. For more and more scenerioes, more than 256 ports are
> > needed to support for vdev scalability.
> >
> > It is necessary for redefinition of port_id to extend from 1 bytes to
> > 2 bytes. All ethdev APIs and use cases related to port_id will be
> > changed at the same time.
> 
> I think this reads a little better:
> 
> This is an API/ABI change notice for DPDK 17.11 announcing the redefinition of
> port_id. port_id is currently defined as uint8_t, which is limited to the range 0 to
> 255. A larger range is required for vdev scalability.
> 
> It is necessary for a redefinition of port_id to extend it from 1 bytes to 2 bytes.
> All ethdev APIs and usages related to port_id will be changed at the same time.

Thanks Remy, of course. Your statement is better. I have a lot of things  to learn
when speaking in English. I hope I can describe things like you, a native speaker. :)
Thanks again for your ack.
 
Zhiyong

> 
> >  doc/guides/rel_notes/deprecation.rst | 6 ++++++
> >  1 file changed, 6 insertions(+)
> 
> Acked-by: Remy Horton <remy.horton@intel.com>

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-08-05 10:34     ` Yang, Zhiyong
@ 2017-08-07  3:07       ` Yang, Zhiyong
  2017-08-07 11:04         ` Thomas Monjalon
  0 siblings, 1 reply; 19+ messages in thread
From: Yang, Zhiyong @ 2017-08-07  3:07 UTC (permalink / raw)
  To: Yang, Zhiyong, Horton, Remy, dev; +Cc: Tan, Jianfeng, yliu, jerin.jacob, thomas

Hi, Thomas:
	Need I sent another V3 patch to update commit log according to Remy's suggestion?
Thanks
Zhiyong

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Yang, Zhiyong
> Sent: Saturday, August 5, 2017 6:35 PM
> To: Horton, Remy <remy.horton@intel.com>; dev@dpdk.org
> Cc: Tan, Jianfeng <jianfeng.tan@intel.com>; yliu@fridaylinux.org;
> jerin.jacob@caviumnetworks.com; thomas@monjalon.net
> Subject: Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for
> ethdev
> 
> Hi, Remy:
> 
> > -----Original Message-----
> > From: Horton, Remy
> > Sent: Friday, August 4, 2017 2:56 PM
> > To: Yang, Zhiyong <zhiyong.yang@intel.com>; dev@dpdk.org
> > Cc: Tan, Jianfeng <jianfeng.tan@intel.com>; yliu@fridaylinux.org;
> > jerin.jacob@caviumnetworks.com; thomas@monjalon.net
> > Subject: Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change
> > for ethdev
> >
> >
> > On 04/08/2017 06:27, Zhiyong Yang wrote:
> > > This is an API/ABI change notice for DPDK 17.11 on redefinition of
> > > port_id. port_id is defined as uint8_t by now, which is just ranged
> > > from 0 to 255. For more and more scenerioes, more than 256 ports are
> > > needed to support for vdev scalability.
> > >
> > > It is necessary for redefinition of port_id to extend from 1 bytes
> > > to
> > > 2 bytes. All ethdev APIs and use cases related to port_id will be
> > > changed at the same time.
> >
> > I think this reads a little better:
> >
> > This is an API/ABI change notice for DPDK 17.11 announcing the
> > redefinition of port_id. port_id is currently defined as uint8_t,
> > which is limited to the range 0 to 255. A larger range is required for vdev
> scalability.
> >
> > It is necessary for a redefinition of port_id to extend it from 1 bytes to 2 bytes.
> > All ethdev APIs and usages related to port_id will be changed at the same time.
> 
> Thanks Remy, of course. Your statement is better. I have a lot of things  to learn
> when speaking in English. I hope I can describe things like you, a native
> speaker. :) Thanks again for your ack.
> 
> Zhiyong
> 
> >
> > >  doc/guides/rel_notes/deprecation.rst | 6 ++++++
> > >  1 file changed, 6 insertions(+)
> >
> > Acked-by: Remy Horton <remy.horton@intel.com>

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-08-07  3:07       ` Yang, Zhiyong
@ 2017-08-07 11:04         ` Thomas Monjalon
  0 siblings, 0 replies; 19+ messages in thread
From: Thomas Monjalon @ 2017-08-07 11:04 UTC (permalink / raw)
  To: Yang, Zhiyong; +Cc: Horton, Remy, dev, Tan, Jianfeng, yliu, jerin.jacob

07/08/2017 05:07, Yang, Zhiyong:
> Hi, Thomas:
> 	Need I sent another V3 patch to update commit log according to Remy's suggestion?

Yes it would be better if you can do it please

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

* [dpdk-dev] [PATCH v3] doc: announce API and ABI change for ethdev
  2017-08-04  5:27 ` [dpdk-dev] [PATCH v2] " Zhiyong Yang
  2017-08-04  6:56   ` Remy Horton
@ 2017-08-07 12:42   ` Zhiyong Yang
  2017-08-08  4:02     ` Yang, Zhiyong
  1 sibling, 1 reply; 19+ messages in thread
From: Zhiyong Yang @ 2017-08-07 12:42 UTC (permalink / raw)
  To: dev; +Cc: jianfeng.tan, yliu, jerin.jacob, thomas, remy.horton, Zhiyong Yang

This is an API/ABI change notice for DPDK 17.11 announcing the redefinition
of port_id. port_id is currently defined as uint8_t, which is limited to
the range 0 to 255. A larger range is required for vdev scalability.

It is necessary for a redefinition of port_id to extend it from 1 bytes to
2 bytes. All ethdev APIs and usages related to port_id will be changed at
the same time.

cc: jianfeng.tan@intel.com
cc: yliu@fridaylinux.org
cc: jerin.jacob@caviumnetworks.com
cc: thomas@monjalon.net
cc: remy.horton@intel.com

Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
Acked-by: Jianfeng Tan <jianfeng.tan@intel.com>
Acked-by: Yuanhan Liu <yliu@fridaylinux.org>
Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
Acked-by: Remy Horton <remy.horton@intel.com>
---

Change in V3.
update the commit log. Thanks for Remy's comment.

Change in V2.
1. remove indent of paragraph in commit log.
2. add more description about API/ABI according to yuanhan's comments.

 doc/guides/rel_notes/deprecation.rst | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 72aa40495..4edaf92fb 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -49,6 +49,12 @@ Deprecation Notices
   Target release for removal of the legacy API will be defined once most
   PMDs have switched to rte_flow.
 
+* ABI/API changes are planned for 17.11 in all structures which include port_id
+  definition such as "rte_eth_dev_data", "rte_port_ethdev_reader_params",
+  "rte_port_ethdev_writer_params", and so on. The definition of port_id will be
+  changed from 8 bits to 16 bits in order to support more than 256 ports in
+  DPDK. All APIs which have port_id parameter will be changed at the same time.
+
 * librte_table: The ``key_mask`` parameter will be added to all the hash tables
   that currently do not have it, as well as to the hash compute function prototype.
   The non-"do-sig" versions of the hash tables will be removed
-- 
2.13.3

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-08-04  6:56   ` Remy Horton
  2017-08-05 10:34     ` Yang, Zhiyong
@ 2017-08-07 15:20     ` Shahaf Shuler
  1 sibling, 0 replies; 19+ messages in thread
From: Shahaf Shuler @ 2017-08-07 15:20 UTC (permalink / raw)
  To: Remy Horton, Zhiyong Yang, dev
  Cc: jianfeng.tan, yliu, jerin.jacob, Thomas Monjalon

Friday, August 4, 2017 9:56 AM, Remy Horton:
> On 04/08/2017 06:27, Zhiyong Yang wrote:
> > This is an API/ABI change notice for DPDK 17.11 on redefinition of
> > port_id. port_id is defined as uint8_t by now, which is just ranged
> > from 0 to 255. For more and more scenerioes, more than 256 ports are
> > needed to support for vdev scalability.
> >
> > It is necessary for redefinition of port_id to extend from 1 bytes to
> > 2 bytes. All ethdev APIs and use cases related to port_id will be
> > changed at the same time.
> 
> I think this reads a little better:
> 
> This is an API/ABI change notice for DPDK 17.11 announcing the redefinition
> of port_id. port_id is currently defined as uint8_t, which is limited to the
> range 0 to 255. A larger range is required for vdev scalability.
> 
> It is necessary for a redefinition of port_id to extend it from 1 bytes to 2
> bytes. All ethdev APIs and usages related to port_id will be changed at the
> same time.
> 
> >  doc/guides/rel_notes/deprecation.rst | 6 ++++++
> >  1 file changed, 6 insertions(+)
> 
> Acked-by: Remy Horton <remy.horton@intel.com>

Acked-by: Shahaf Shuler <shahafs@mellanox.com>

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

* Re: [dpdk-dev] [PATCH v3] doc: announce API and ABI change for ethdev
  2017-08-07 12:42   ` [dpdk-dev] [PATCH v3] " Zhiyong Yang
@ 2017-08-08  4:02     ` Yang, Zhiyong
  2017-08-08  9:35       ` Thomas Monjalon
  0 siblings, 1 reply; 19+ messages in thread
From: Yang, Zhiyong @ 2017-08-08  4:02 UTC (permalink / raw)
  To: dev; +Cc: Tan, Jianfeng, yliu, jerin.jacob, thomas, Horton, Remy, shahafs


> -----Original Message-----
> From: Yang, Zhiyong
> Sent: Monday, August 7, 2017 8:43 PM
> To: dev@dpdk.org
> Cc: Tan, Jianfeng <jianfeng.tan@intel.com>; yliu@fridaylinux.org;
> jerin.jacob@caviumnetworks.com; thomas@monjalon.net; Horton, Remy
> <remy.horton@intel.com>; Yang, Zhiyong <zhiyong.yang@intel.com>
> Subject: [PATCH v3] doc: announce API and ABI change for ethdev
> 
> This is an API/ABI change notice for DPDK 17.11 announcing the redefinition of
> port_id. port_id is currently defined as uint8_t, which is limited to the range 0 to
> 255. A larger range is required for vdev scalability.
> 
> It is necessary for a redefinition of port_id to extend it from 1 bytes to
> 2 bytes. All ethdev APIs and usages related to port_id will be changed at the
> same time.
> 
> cc: jianfeng.tan@intel.com
> cc: yliu@fridaylinux.org
> cc: jerin.jacob@caviumnetworks.com
> cc: thomas@monjalon.net
> cc: remy.horton@intel.com
> 
> Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> Acked-by: Jianfeng Tan <jianfeng.tan@intel.com>
> Acked-by: Yuanhan Liu <yliu@fridaylinux.org>
> Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> Acked-by: Remy Horton <remy.horton@intel.com>
> ---

This ack in v2 is added in v3. Thanks, Shahaf.

Acked-by: Shahaf Shuler <shahafs@mellanox.com>

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

* Re: [dpdk-dev] [PATCH v3] doc: announce API and ABI change for ethdev
  2017-08-08  4:02     ` Yang, Zhiyong
@ 2017-08-08  9:35       ` Thomas Monjalon
  0 siblings, 0 replies; 19+ messages in thread
From: Thomas Monjalon @ 2017-08-08  9:35 UTC (permalink / raw)
  To: Yang, Zhiyong
  Cc: dev, Tan, Jianfeng, yliu, jerin.jacob, Horton, Remy, shahafs

> > This is an API/ABI change notice for DPDK 17.11 announcing the redefinition of
> > port_id. port_id is currently defined as uint8_t, which is limited to the range 0 to
> > 255. A larger range is required for vdev scalability.
> > 
> > It is necessary for a redefinition of port_id to extend it from 1 bytes to
> > 2 bytes. All ethdev APIs and usages related to port_id will be changed at the
> > same time.
> > 
> > cc: jianfeng.tan@intel.com
> > cc: yliu@fridaylinux.org
> > cc: jerin.jacob@caviumnetworks.com
> > cc: thomas@monjalon.net
> > cc: remy.horton@intel.com
> > 
> > Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> > Acked-by: Jianfeng Tan <jianfeng.tan@intel.com>
> > Acked-by: Yuanhan Liu <yliu@fridaylinux.org>
> > Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
> > Acked-by: Remy Horton <remy.horton@intel.com>
> 
> This ack in v2 is added in v3. Thanks, Shahaf.
> 
> Acked-by: Shahaf Shuler <shahafs@mellanox.com>

Applied, thanks

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-01-05 15:25 ` [dpdk-dev] [PATCH v2] " Bernard Iremonger
  2017-02-13 17:57   ` Thomas Monjalon
@ 2017-02-14 19:37   ` Thomas Monjalon
  1 sibling, 0 replies; 19+ messages in thread
From: Thomas Monjalon @ 2017-02-14 19:37 UTC (permalink / raw)
  To: Bernard Iremonger; +Cc: dev

Applied

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-02-14  3:17     ` Jerin Jacob
@ 2017-02-14 10:33       ` Iremonger, Bernard
  0 siblings, 0 replies; 19+ messages in thread
From: Iremonger, Bernard @ 2017-02-14 10:33 UTC (permalink / raw)
  To: Jerin Jacob, Thomas Monjalon; +Cc: dev, Mcnamara, John



> -----Original Message-----
> From: Jerin Jacob [mailto:jerin.jacob@caviumnetworks.com]
> Sent: Tuesday, February 14, 2017 3:17 AM
> To: Thomas Monjalon <thomas.monjalon@6wind.com>
> Cc: Iremonger, Bernard <bernard.iremonger@intel.com>; dev@dpdk.org;
> Mcnamara, John <john.mcnamara@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for
> ethdev
> 
> On Mon, Feb 13, 2017 at 06:57:20PM +0100, Thomas Monjalon wrote:
> > 2017-01-05 15:25, Bernard Iremonger:
> > > In 17.05 nine rte_eth_dev_* functions will be removed from
> > > librte_ether, renamed and moved to the ixgbe PMD.
> > >
> > > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> >
> > "ixgbe bypass" should be in the title and the description.
> > I'll reword to:
> >
> > doc: announce move of ethdev bypass function to ixgbe API
> >
> > In 17.05, nine rte_eth_dev_* functions for bypass control, and
> > implemented only in ixgbe, will be removed from ethdev, renamed and
> > moved to the ixgbe PMD-specific API.
> >
> > Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> 
> Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>

Acked-by: Bernard Iremonger <bernard.iremonger@intel.com>

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-02-13 17:57   ` Thomas Monjalon
@ 2017-02-14  3:17     ` Jerin Jacob
  2017-02-14 10:33       ` Iremonger, Bernard
  0 siblings, 1 reply; 19+ messages in thread
From: Jerin Jacob @ 2017-02-14  3:17 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: Bernard Iremonger, dev, john.mcnamara

On Mon, Feb 13, 2017 at 06:57:20PM +0100, Thomas Monjalon wrote:
> 2017-01-05 15:25, Bernard Iremonger:
> > In 17.05 nine rte_eth_dev_* functions will be removed from
> > librte_ether, renamed and moved to the ixgbe PMD.
> > 
> > Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
> 
> "ixgbe bypass" should be in the title and the description.
> I'll reword to:
> 
> doc: announce move of ethdev bypass function to ixgbe API
> 
> In 17.05, nine rte_eth_dev_* functions for bypass control,
> and implemented only in ixgbe, will be removed from ethdev,
> renamed and moved to the ixgbe PMD-specific API.
> 
> Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>

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

* Re: [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-01-05 15:25 ` [dpdk-dev] [PATCH v2] " Bernard Iremonger
@ 2017-02-13 17:57   ` Thomas Monjalon
  2017-02-14  3:17     ` Jerin Jacob
  2017-02-14 19:37   ` Thomas Monjalon
  1 sibling, 1 reply; 19+ messages in thread
From: Thomas Monjalon @ 2017-02-13 17:57 UTC (permalink / raw)
  To: Bernard Iremonger; +Cc: dev, john.mcnamara

2017-01-05 15:25, Bernard Iremonger:
> In 17.05 nine rte_eth_dev_* functions will be removed from
> librte_ether, renamed and moved to the ixgbe PMD.
> 
> Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>

"ixgbe bypass" should be in the title and the description.
I'll reword to:

doc: announce move of ethdev bypass function to ixgbe API

In 17.05, nine rte_eth_dev_* functions for bypass control,
and implemented only in ixgbe, will be removed from ethdev,
renamed and moved to the ixgbe PMD-specific API.

Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>

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

* [dpdk-dev] [PATCH v2] doc: announce API and ABI change for ethdev
  2017-01-05 10:44 [dpdk-dev] [PATCH v1] " Bernard Iremonger
@ 2017-01-05 15:25 ` Bernard Iremonger
  2017-02-13 17:57   ` Thomas Monjalon
  2017-02-14 19:37   ` Thomas Monjalon
  0 siblings, 2 replies; 19+ messages in thread
From: Bernard Iremonger @ 2017-01-05 15:25 UTC (permalink / raw)
  To: dev, john.mcnamara; +Cc: Bernard Iremonger

In 17.05 nine rte_eth_dev_* functions will be removed from
librte_ether, renamed and moved to the ixgbe PMD.

Signed-off-by: Bernard Iremonger <bernard.iremonger@intel.com>
---
v2:
Used comma's to shorten lists.

 doc/guides/rel_notes/deprecation.rst | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 1438c77..985cda8 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -79,3 +79,22 @@ Deprecation Notices
   PMDs that implement the latter.
   Target release for removal of the legacy API will be defined once most
   PMDs have switched to rte_flow.
+
+* ethdev: for 17.05 it is planned to deprecate the following nine rte_eth_dev_* functions
+  and move them into the ixgbe PMD:
+
+  ``rte_eth_dev_bypass_init``, ``rte_eth_dev_bypass_state_set``, ``rte_eth_dev_bypass_state_show``,
+  ``rte_eth_dev_bypass_event_store``, ``rte_eth_dev_bypass_event_show``, ``rte_eth_dev_wd_timeout_store``,
+  ``rte_eth_dev_bypass_wd_timeout_show``, ``rte_eth_dev_bypass_ver_show``, ``rte_eth_dev_bypass_wd_reset``.
+
+  The following fields will be removed from ``struct eth_dev_ops``:
+
+  ``bypass_init_t``, ``bypass_state_set_t``, ``bypass_state_show_t``, ``bypass_event_set_t``,
+  ``bypass_event_show_t``, ``bypass_wd_timeout_set_t``, ``bypass_wd_timeout_show_t``,
+  ``bypass_ver_show_t``, ``bypass_wd_reset_t``.
+
+  The functions will be renamed to the following, and moved to the ``ixgbe`` PMD:
+
+  ``rte_pmd_ixgbe_bypass_init``, ``rte_pmd_ixgbe_bypass_state_set``, ``rte_pmd_ixgbe_bypass_state_show``,
+  ``rte_pmd_ixgbe_bypass_event_set``, ``rte_pmd_ixgbe_bypass_event_show``, ``rte_pmd_ixgbe_bypass_wd_timeout_set``,
+  ``rte_pmd_ixgbe_bypass_wd_timeout_show``, ``rte_pmd_ixgbe_bypass_ver_show``, ``rte_pmd_ixgbe_bypass_wd_reset``.
-- 
2.10.1

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

end of thread, other threads:[~2017-08-08 10:02 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-12  7:58 [dpdk-dev] [PATCH] doc: announce API and ABI change for ethdev Zhiyong Yang
2017-08-04  1:12 ` Tan, Jianfeng
2017-08-04  4:07 ` Yuanhan Liu
2017-08-04  4:13   ` Jerin Jacob
2017-08-04  5:36   ` Yang, Zhiyong
2017-08-04  5:27 ` [dpdk-dev] [PATCH v2] " Zhiyong Yang
2017-08-04  6:56   ` Remy Horton
2017-08-05 10:34     ` Yang, Zhiyong
2017-08-07  3:07       ` Yang, Zhiyong
2017-08-07 11:04         ` Thomas Monjalon
2017-08-07 15:20     ` Shahaf Shuler
2017-08-07 12:42   ` [dpdk-dev] [PATCH v3] " Zhiyong Yang
2017-08-08  4:02     ` Yang, Zhiyong
2017-08-08  9:35       ` Thomas Monjalon
  -- strict thread matches above, loose matches on Subject: below --
2017-01-05 10:44 [dpdk-dev] [PATCH v1] " Bernard Iremonger
2017-01-05 15:25 ` [dpdk-dev] [PATCH v2] " Bernard Iremonger
2017-02-13 17:57   ` Thomas Monjalon
2017-02-14  3:17     ` Jerin Jacob
2017-02-14 10:33       ` Iremonger, Bernard
2017-02-14 19:37   ` Thomas Monjalon

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