doc: add Vhost and Virtio updates to release note

Message ID 20201001103650.208049-1-maxime.coquelin@redhat.com (mailing list archive)
State Superseded, archived
Delegated to: Maxime Coquelin
Headers
Series doc: add Vhost and Virtio updates to release note |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation success Compilation OK
ci/travis-robot success Travis build: passed

Commit Message

Maxime Coquelin Oct. 1, 2020, 10:36 a.m. UTC
  Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
---
 doc/guides/rel_notes/release_20_11.rst | 10 ++++++++++
 1 file changed, 10 insertions(+)
  

Comments

Thomas Monjalon Oct. 1, 2020, 11:42 a.m. UTC | #1
01/10/2020 12:36, Maxime Coquelin:
> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> ---
>  doc/guides/rel_notes/release_20_11.rst | 10 ++++++++++
>  1 file changed, 10 insertions(+)

Those changes should have been part of the original commits.
Please can you add the "Fixes:" lines so we can have an easy
reference to the code change?
  
Maxime Coquelin Oct. 1, 2020, 11:45 a.m. UTC | #2
On 10/1/20 1:42 PM, Thomas Monjalon wrote:
> 01/10/2020 12:36, Maxime Coquelin:
>> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
>> ---
>>  doc/guides/rel_notes/release_20_11.rst | 10 ++++++++++
>>  1 file changed, 10 insertions(+)
> 
> Those changes should have been part of the original commits.
> Please can you add the "Fixes:" lines so we can have an easy
> reference to the code change?

I'm not convinced it is a good idea, as it creates avoidable conflicts
when someone wants to backport features.

But if you insist I can do it.
  
Thomas Monjalon Oct. 1, 2020, 12:53 p.m. UTC | #3
01/10/2020 13:45, Maxime Coquelin:
> 
> On 10/1/20 1:42 PM, Thomas Monjalon wrote:
> > 01/10/2020 12:36, Maxime Coquelin:
> >> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> >> ---
> >>  doc/guides/rel_notes/release_20_11.rst | 10 ++++++++++
> >>  1 file changed, 10 insertions(+)
> > 
> > Those changes should have been part of the original commits.
> > Please can you add the "Fixes:" lines so we can have an easy
> > reference to the code change?
> 
> I'm not convinced it is a good idea, as it creates avoidable conflicts
> when someone wants to backport features.

Not sure I understand the conflict you think of.

> But if you insist I can do it.
  
Thomas Monjalon Oct. 1, 2020, 9:33 p.m. UTC | #4
01/10/2020 14:53, Thomas Monjalon:
> 01/10/2020 13:45, Maxime Coquelin:
> > 
> > On 10/1/20 1:42 PM, Thomas Monjalon wrote:
> > > 01/10/2020 12:36, Maxime Coquelin:
> > >> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> > >> ---
> > >>  doc/guides/rel_notes/release_20_11.rst | 10 ++++++++++
> > >>  1 file changed, 10 insertions(+)
> > > 
> > > Those changes should have been part of the original commits.
> > > Please can you add the "Fixes:" lines so we can have an easy
> > > reference to the code change?
> > 
> > I'm not convinced it is a good idea, as it creates avoidable conflicts
> > when someone wants to backport features.
> 
> Not sure I understand the conflict you think of.
> 
> > But if you insist I can do it.

I did the release notes updates in the relevant patches
while pulling next-net.
I've removed the deprecation notice as well.
  
Maxime Coquelin Oct. 1, 2020, 9:41 p.m. UTC | #5
On 10/1/20 11:33 PM, Thomas Monjalon wrote:
> 01/10/2020 14:53, Thomas Monjalon:
>> 01/10/2020 13:45, Maxime Coquelin:
>>>
>>> On 10/1/20 1:42 PM, Thomas Monjalon wrote:
>>>> 01/10/2020 12:36, Maxime Coquelin:
>>>>> Signed-off-by: Maxime Coquelin <maxime.coquelin@redhat.com>
>>>>> ---
>>>>>  doc/guides/rel_notes/release_20_11.rst | 10 ++++++++++
>>>>>  1 file changed, 10 insertions(+)
>>>>
>>>> Those changes should have been part of the original commits.
>>>> Please can you add the "Fixes:" lines so we can have an easy
>>>> reference to the code change?
>>>
>>> I'm not convinced it is a good idea, as it creates avoidable conflicts
>>> when someone wants to backport features.
>>
>> Not sure I understand the conflict you think of.

I meant having both the code and release note parts in the same patch.
Sorry if I was not clear.

>>> But if you insist I can do it.
> 
> I did the release notes updates in the relevant patches
> while pulling next-net.
> I've removed the deprecation notice as well.

OK, that's fine anyway.

Thanks,
Maxime
  

Patch

diff --git a/doc/guides/rel_notes/release_20_11.rst b/doc/guides/rel_notes/release_20_11.rst
index 4eb3224a76..2a0f1605fe 100644
--- a/doc/guides/rel_notes/release_20_11.rst
+++ b/doc/guides/rel_notes/release_20_11.rst
@@ -78,6 +78,12 @@  New Features
     ``--portmask=N``
     where N represents the hexadecimal bitmask of ports used.
 
+* ** Updated Virtio driver.**
+
+  * Added support for Vhost-vDPA backend to Virtio-user PMD.
+  * Changed Virtio device default link speed to unknown and added support for
+    200G link speed.
+
 
 Removed Items
 -------------
@@ -91,6 +97,8 @@  Removed Items
    Also, make sure to start the actual text at the margin.
    =======================================================
 
+* vhost: Dequeue zero-copy support has been removed.
+
 
 API Changes
 -----------
@@ -172,6 +180,8 @@  API Changes
 
 * bpf: ``RTE_BPF_XTYPE_NUM`` has been dropped from ``rte_bpf_xtype``.
 
+* vhost: moved vDPA APIs from experimental to stable.
+
 
 ABI Changes
 -----------