diff mbox series

[v2,4/5] doc: update release notes for hns3

Message ID 1611837805-42429-5-git-send-email-oulijun@huawei.com (mailing list archive)
State Changes Requested, archived
Delegated to: Ferruh Yigit
Headers
Series update doc for hns3 |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Lijun Ou Jan. 28, 2021, 12:43 p.m. UTC 
  Add TM(Traffic Management) feature with hns3 in release notes.
It includes:
1. configure port's peak rate with PF
2. configure TC's peak rate with PF

Fixes: c09c7847d892 ("net/hns3: support traffic management")
Cc: stable@dpdk.org

Signed-off-by: Lijun Ou <oulijun@huawei.com>
---
 doc/guides/rel_notes/release_21_02.rst | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Ferruh Yigit Jan. 28, 2021, 2:46 p.m. UTC | #1 
On 1/28/2021 12:43 PM, Lijun Ou wrote:
> Add TM(Traffic Management) feature with hns3 in release notes.
> It includes:
> 1. configure port's peak rate with PF
> 2. configure TC's peak rate with PF
> 
> Fixes: c09c7847d892 ("net/hns3: support traffic management")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Lijun Ou <oulijun@huawei.com>
> ---
>   doc/guides/rel_notes/release_21_02.rst | 5 +++++
>   1 file changed, 5 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/release_21_02.rst b/doc/guides/rel_notes/release_21_02.rst
> index 23dcb37..08b73de 100644
> --- a/doc/guides/rel_notes/release_21_02.rst
> +++ b/doc/guides/rel_notes/release_21_02.rst
> @@ -82,6 +82,11 @@ New Features
>   
>     * Added support for 64B completion queue entries
>   
> +* **Updated hns3 driver.**
> +
> +  * Added support for traffic management
> +    Support for configuring port's peak rate and TC's peak rate with PF
> +

Hi Lijun,

Are these two different items, or second sentences is continuation of the first one?

>   * **Updated Intel ice driver.**
>   
>     Updated the Intel ice driver with new features and improvements, including:
>
Lijun Ou Jan. 29, 2021, 1:28 a.m. UTC | #2 
在 2021/1/28 22:46, Ferruh Yigit 写道:
> On 1/28/2021 12:43 PM, Lijun Ou wrote:
>> Add TM(Traffic Management) feature with hns3 in release notes.
>> It includes:
>> 1. configure port's peak rate with PF
>> 2. configure TC's peak rate with PF
>>
>> Fixes: c09c7847d892 ("net/hns3: support traffic management")
>> Cc: stable@dpdk.org
>>
>> Signed-off-by: Lijun Ou <oulijun@huawei.com>
>> ---
>>   doc/guides/rel_notes/release_21_02.rst | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/doc/guides/rel_notes/release_21_02.rst 
>> b/doc/guides/rel_notes/release_21_02.rst
>> index 23dcb37..08b73de 100644
>> --- a/doc/guides/rel_notes/release_21_02.rst
>> +++ b/doc/guides/rel_notes/release_21_02.rst
>> @@ -82,6 +82,11 @@ New Features
>>     * Added support for 64B completion queue entries
>> +* **Updated hns3 driver.**
>> +
>> +  * Added support for traffic management
>> +    Support for configuring port's peak rate and TC's peak rate with PF
>> +
> 
> Hi Lijun,
> 
> Are these two different items, or second sentences is continuation of 
> the first one?
Both the port peak rate and TC peak rate can be configured, or they can 
be configured separately.
> 
>>   * **Updated Intel ice driver.**
>>     Updated the Intel ice driver with new features and improvements, 
>> including:
>>
> 
> .
>
Ferruh Yigit Jan. 29, 2021, 8:48 a.m. UTC | #3 
On 1/29/2021 1:28 AM, oulijun wrote:
> 
> 
> 在 2021/1/28 22:46, Ferruh Yigit 写道:
>> On 1/28/2021 12:43 PM, Lijun Ou wrote:
>>> Add TM(Traffic Management) feature with hns3 in release notes.
>>> It includes:
>>> 1. configure port's peak rate with PF
>>> 2. configure TC's peak rate with PF
>>>
>>> Fixes: c09c7847d892 ("net/hns3: support traffic management")
>>> Cc: stable@dpdk.org
>>>
>>> Signed-off-by: Lijun Ou <oulijun@huawei.com>
>>> ---
>>>   doc/guides/rel_notes/release_21_02.rst | 5 +++++
>>>   1 file changed, 5 insertions(+)
>>>
>>> diff --git a/doc/guides/rel_notes/release_21_02.rst 
>>> b/doc/guides/rel_notes/release_21_02.rst
>>> index 23dcb37..08b73de 100644
>>> --- a/doc/guides/rel_notes/release_21_02.rst
>>> +++ b/doc/guides/rel_notes/release_21_02.rst
>>> @@ -82,6 +82,11 @@ New Features
>>>     * Added support for 64B completion queue entries
>>> +* **Updated hns3 driver.**
>>> +
>>> +  * Added support for traffic management
>>> +    Support for configuring port's peak rate and TC's peak rate with PF
>>> +
>>
>> Hi Lijun,
>>
>> Are these two different items, or second sentences is continuation of the 
>> first one?
> Both the port peak rate and TC peak rate can be configured, or they can be 
> configured separately.

I asked from formatting perspective. In the output html these two lines looks 
like a long single line, if the intention is to have them as they are seen in 
the .rst file additional formatting is required.

>>
>>>   * **Updated Intel ice driver.**
>>>     Updated the Intel ice driver with new features and improvements, including:
>>>
>>
>> .
>>
diff mbox series

Patch

diff --git a/doc/guides/rel_notes/release_21_02.rst b/doc/guides/rel_notes/release_21_02.rst
index 23dcb37..08b73de 100644
--- a/doc/guides/rel_notes/release_21_02.rst
+++ b/doc/guides/rel_notes/release_21_02.rst
@@ -82,6 +82,11 @@  New Features
 
   * Added support for 64B completion queue entries
 
+* **Updated hns3 driver.**
+
+  * Added support for traffic management
+    Support for configuring port's peak rate and TC's peak rate with PF
+
 * **Updated Intel ice driver.**
 
   Updated the Intel ice driver with new features and improvements, including: