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