DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: clarify PMD order in release notes
@ 2021-04-09 15:21 Ferruh Yigit
  2021-04-09 15:34 ` Thomas Monjalon
  2021-05-12 16:04 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
  0 siblings, 2 replies; 6+ messages in thread
From: Ferruh Yigit @ 2021-04-09 15:21 UTC (permalink / raw)
  To: dev; +Cc: Ferruh Yigit, John McNamara, Thomas Monjalon

PMD updates are expected in alphabetical order based on their vendor
name. Clarify this expectation in the section comment.

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
Cc: Thomas Monjalon <thomas@monjalon.net>
---
 doc/guides/rel_notes/release_21_05.rst | 1 +
 1 file changed, 1 insertion(+)

diff --git a/doc/guides/rel_notes/release_21_05.rst b/doc/guides/rel_notes/release_21_05.rst
index e739153b509f..20cca03cbb9d 100644
--- a/doc/guides/rel_notes/release_21_05.rst
+++ b/doc/guides/rel_notes/release_21_05.rst
@@ -45,6 +45,7 @@ New Features
      * Core libs (EAL, mempool, ring, mbuf, buses)
      * Device abstraction libs and PMDs
        - ethdev (lib, PMDs)
+         * PMDs are ordered alphabetically on vendor name
        - cryptodev (lib, PMDs)
        - eventdev (lib, PMDs)
        - etc
-- 
2.30.2


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

* Re: [dpdk-dev] [PATCH] doc: clarify PMD order in release notes
  2021-04-09 15:21 [dpdk-dev] [PATCH] doc: clarify PMD order in release notes Ferruh Yigit
@ 2021-04-09 15:34 ` Thomas Monjalon
  2021-05-12 16:04 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
  1 sibling, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2021-04-09 15:34 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: dev, John McNamara

09/04/2021 17:21, Ferruh Yigit:
> PMD updates are expected in alphabetical order based on their vendor
> name. Clarify this expectation in the section comment.
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
> Cc: Thomas Monjalon <thomas@monjalon.net>
> ---
> --- a/doc/guides/rel_notes/release_21_05.rst
> +++ b/doc/guides/rel_notes/release_21_05.rst
> @@ -45,6 +45,7 @@ New Features
>       * Core libs (EAL, mempool, ring, mbuf, buses)
>       * Device abstraction libs and PMDs
>         - ethdev (lib, PMDs)
> +         * PMDs are ordered alphabetically on vendor name
>         - cryptodev (lib, PMDs)
>         - eventdev (lib, PMDs)

It is not specific to ethdev.
It could be a general comment.




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

* [dpdk-dev] [PATCH v2] doc: clarify PMD order in release notes
  2021-04-09 15:21 [dpdk-dev] [PATCH] doc: clarify PMD order in release notes Ferruh Yigit
  2021-04-09 15:34 ` Thomas Monjalon
@ 2021-05-12 16:04 ` Ferruh Yigit
  2021-05-19 20:42   ` Thomas Monjalon
  1 sibling, 1 reply; 6+ messages in thread
From: Ferruh Yigit @ 2021-05-12 16:04 UTC (permalink / raw)
  To: dev; +Cc: Ferruh Yigit, Thomas Monjalon

PMD updates are expected in alphabetical order based on their vendor
name. Clarify this expectation in the section comment.

Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
v2:
* Do not restrict clarification for ethdev
---
 doc/guides/rel_notes/release_21_05.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/guides/rel_notes/release_21_05.rst b/doc/guides/rel_notes/release_21_05.rst
index b144ae9e9dbc..b56d66d2a245 100644
--- a/doc/guides/rel_notes/release_21_05.rst
+++ b/doc/guides/rel_notes/release_21_05.rst
@@ -43,7 +43,7 @@ New Features
 
      Suggested order in release notes items:
      * Core libs (EAL, mempool, ring, mbuf, buses)
-     * Device abstraction libs and PMDs
+     * Device abstraction libs and PMDs (PMDs ordered alphabetically by vendor name)
        - ethdev (lib, PMDs)
        - cryptodev (lib, PMDs)
        - eventdev (lib, PMDs)
-- 
2.31.1


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

* Re: [dpdk-dev] [PATCH v2] doc: clarify PMD order in release notes
  2021-05-12 16:04 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
@ 2021-05-19 20:42   ` Thomas Monjalon
  2021-05-19 23:06     ` Ferruh Yigit
  0 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2021-05-19 20:42 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: dev

12/05/2021 18:04, Ferruh Yigit:
> PMD updates are expected in alphabetical order based on their vendor
> name. Clarify this expectation in the section comment.
> 
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
> -     * Device abstraction libs and PMDs
> +     * Device abstraction libs and PMDs (PMDs ordered alphabetically by vendor name)

I would avoid the repetition of "PMDs".
May I apply with this modification?




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

* Re: [dpdk-dev] [PATCH v2] doc: clarify PMD order in release notes
  2021-05-19 20:42   ` Thomas Monjalon
@ 2021-05-19 23:06     ` Ferruh Yigit
  2021-05-20  7:34       ` Thomas Monjalon
  0 siblings, 1 reply; 6+ messages in thread
From: Ferruh Yigit @ 2021-05-19 23:06 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev

On 5/19/2021 9:42 PM, Thomas Monjalon wrote:
> 12/05/2021 18:04, Ferruh Yigit:
>> PMD updates are expected in alphabetical order based on their vendor
>> name. Clarify this expectation in the section comment.
>>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>> ---
>> -     * Device abstraction libs and PMDs
>> +     * Device abstraction libs and PMDs (PMDs ordered alphabetically by vendor name)
> 
> I would avoid the repetition of "PMDs".
> May I apply with this modification?
> 

Ack, if it is not too late.

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

* Re: [dpdk-dev] [PATCH v2] doc: clarify PMD order in release notes
  2021-05-19 23:06     ` Ferruh Yigit
@ 2021-05-20  7:34       ` Thomas Monjalon
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2021-05-20  7:34 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: dev

20/05/2021 01:06, Ferruh Yigit:
> On 5/19/2021 9:42 PM, Thomas Monjalon wrote:
> > 12/05/2021 18:04, Ferruh Yigit:
> >> PMD updates are expected in alphabetical order based on their vendor
> >> name. Clarify this expectation in the section comment.
> >>
> >> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
> >> ---
> >> -     * Device abstraction libs and PMDs
> >> +     * Device abstraction libs and PMDs (PMDs ordered alphabetically by vendor name)
> > 
> > I would avoid the repetition of "PMDs".
> > May I apply with this modification?
> 
> Ack, if it is not too late.

Applied, thanks




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

end of thread, other threads:[~2021-05-20  7:34 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-09 15:21 [dpdk-dev] [PATCH] doc: clarify PMD order in release notes Ferruh Yigit
2021-04-09 15:34 ` Thomas Monjalon
2021-05-12 16:04 ` [dpdk-dev] [PATCH v2] " Ferruh Yigit
2021-05-19 20:42   ` Thomas Monjalon
2021-05-19 23:06     ` Ferruh Yigit
2021-05-20  7:34       ` Thomas Monjalon

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git