diff mbox series

doc: update release notes for bnxt PMD

Message ID 20211105165531.15368-1-ajit.khaparde@broadcom.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series doc: update release notes for bnxt PMD |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-testing warning apply patch failure
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS

Commit Message

Ajit Khaparde Nov. 5, 2021, 4:55 p.m. UTC 
  Update release notes for new functionality added to the bnxt PMD.

Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 doc/guides/rel_notes/release_21_11.rst | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

Comments

Ferruh Yigit Nov. 5, 2021, 6:14 p.m. UTC | #1 
On 11/5/2021 4:55 PM, Ajit Khaparde wrote:
> Update release notes for new functionality added to the bnxt PMD.
> 
> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> ---
>   doc/guides/rel_notes/release_21_11.rst | 9 ++++++++-
>   1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
> index 3367a15c85..d8b9531af1 100644
> --- a/doc/guides/rel_notes/release_21_11.rst
> +++ b/doc/guides/rel_notes/release_21_11.rst
> @@ -152,7 +152,14 @@ New Features
>   
>     * Added flow offload support for Thor.
>     * Implement support for tunnel offload.
> -  * Updated HWRM API to version 1.10.2.44
> +  * Updated HWRM API to version 1.10.2.68.
> +  * Updated support for RTE_FLOW_ACTION_TYPE_RSS.

I am not sure which commit this update corresponds to, can you please
share the commit that this update should go?
Ajit Khaparde Nov. 5, 2021, 6:22 p.m. UTC | #2 
On Fri, Nov 5, 2021 at 11:14 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> On 11/5/2021 4:55 PM, Ajit Khaparde wrote:
> > Update release notes for new functionality added to the bnxt PMD.
> >
> > Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> > ---
> >   doc/guides/rel_notes/release_21_11.rst | 9 ++++++++-
> >   1 file changed, 8 insertions(+), 1 deletion(-)
> >
> > diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
> > index 3367a15c85..d8b9531af1 100644
> > --- a/doc/guides/rel_notes/release_21_11.rst
> > +++ b/doc/guides/rel_notes/release_21_11.rst
> > @@ -152,7 +152,14 @@ New Features
> >
> >     * Added flow offload support for Thor.
> >     * Implement support for tunnel offload.
> > -  * Updated HWRM API to version 1.10.2.44
> > +  * Updated HWRM API to version 1.10.2.68.
> > +  * Updated support for RTE_FLOW_ACTION_TYPE_RSS.
>
> I am not sure which commit this update corresponds to, can you please
> share the commit that this update should go?
I think you are looking for this -
239695f754cb ("net/bnxt: enhance RSS action support")

Thanks
Ajit
Ferruh Yigit Nov. 5, 2021, 6:38 p.m. UTC | #3 
On 11/5/2021 6:22 PM, Ajit Khaparde wrote:
> On Fri, Nov 5, 2021 at 11:14 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>>
>> On 11/5/2021 4:55 PM, Ajit Khaparde wrote:
>>> Update release notes for new functionality added to the bnxt PMD.
>>>
>>> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
>>> ---
>>>    doc/guides/rel_notes/release_21_11.rst | 9 ++++++++-
>>>    1 file changed, 8 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
>>> index 3367a15c85..d8b9531af1 100644
>>> --- a/doc/guides/rel_notes/release_21_11.rst
>>> +++ b/doc/guides/rel_notes/release_21_11.rst
>>> @@ -152,7 +152,14 @@ New Features
>>>
>>>      * Added flow offload support for Thor.
>>>      * Implement support for tunnel offload.
>>> -  * Updated HWRM API to version 1.10.2.44
>>> +  * Updated HWRM API to version 1.10.2.68.
>>> +  * Updated support for RTE_FLOW_ACTION_TYPE_RSS.
>>
>> I am not sure which commit this update corresponds to, can you please
>> share the commit that this update should go?
> I think you are looking for this -
> 239695f754cb ("net/bnxt: enhance RSS action support")
> 

Ahh, that is pre -rc1, and not related to the latest based code series either.

Since original commit is part of -rc1, that can't be squashed of course, can
you please send a separate patch for it, with proper fixes tag etc?

Thanks,
ferruh
Ajit Khaparde Nov. 5, 2021, 6:41 p.m. UTC | #4 
On Fri, Nov 5, 2021 at 11:38 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> On 11/5/2021 6:22 PM, Ajit Khaparde wrote:
> > On Fri, Nov 5, 2021 at 11:14 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> >>
> >> On 11/5/2021 4:55 PM, Ajit Khaparde wrote:
> >>> Update release notes for new functionality added to the bnxt PMD.
> >>>
> >>> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> >>> ---
> >>>    doc/guides/rel_notes/release_21_11.rst | 9 ++++++++-
> >>>    1 file changed, 8 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
> >>> index 3367a15c85..d8b9531af1 100644
> >>> --- a/doc/guides/rel_notes/release_21_11.rst
> >>> +++ b/doc/guides/rel_notes/release_21_11.rst
> >>> @@ -152,7 +152,14 @@ New Features
> >>>
> >>>      * Added flow offload support for Thor.
> >>>      * Implement support for tunnel offload.
> >>> -  * Updated HWRM API to version 1.10.2.44
> >>> +  * Updated HWRM API to version 1.10.2.68.
> >>> +  * Updated support for RTE_FLOW_ACTION_TYPE_RSS.
> >>
> >> I am not sure which commit this update corresponds to, can you please
> >> share the commit that this update should go?
> > I think you are looking for this -
> > 239695f754cb ("net/bnxt: enhance RSS action support")
> >
>
> Ahh, that is pre -rc1, and not related to the latest based code series either.
>
> Since original commit is part of -rc1, that can't be squashed of course, can
> you please send a separate patch for it, with proper fixes tag etc?
Sure. I will send a separate patch for that.
Do you want me to respin a new patch for the remaining?
Or you can remove the line and squash?
>
> Thanks,
> ferruh
Ferruh Yigit Nov. 5, 2021, 6:42 p.m. UTC | #5 
On 11/5/2021 6:41 PM, Ajit Khaparde wrote:
> On Fri, Nov 5, 2021 at 11:38 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>>
>> On 11/5/2021 6:22 PM, Ajit Khaparde wrote:
>>> On Fri, Nov 5, 2021 at 11:14 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>>>>
>>>> On 11/5/2021 4:55 PM, Ajit Khaparde wrote:
>>>>> Update release notes for new functionality added to the bnxt PMD.
>>>>>
>>>>> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
>>>>> ---
>>>>>     doc/guides/rel_notes/release_21_11.rst | 9 ++++++++-
>>>>>     1 file changed, 8 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
>>>>> index 3367a15c85..d8b9531af1 100644
>>>>> --- a/doc/guides/rel_notes/release_21_11.rst
>>>>> +++ b/doc/guides/rel_notes/release_21_11.rst
>>>>> @@ -152,7 +152,14 @@ New Features
>>>>>
>>>>>       * Added flow offload support for Thor.
>>>>>       * Implement support for tunnel offload.
>>>>> -  * Updated HWRM API to version 1.10.2.44
>>>>> +  * Updated HWRM API to version 1.10.2.68.
>>>>> +  * Updated support for RTE_FLOW_ACTION_TYPE_RSS.
>>>>
>>>> I am not sure which commit this update corresponds to, can you please
>>>> share the commit that this update should go?
>>> I think you are looking for this -
>>> 239695f754cb ("net/bnxt: enhance RSS action support")
>>>
>>
>> Ahh, that is pre -rc1, and not related to the latest based code series either.
>>
>> Since original commit is part of -rc1, that can't be squashed of course, can
>> you please send a separate patch for it, with proper fixes tag etc?
> Sure. I will send a separate patch for that.
> Do you want me to respin a new patch for the remaining?
> Or you can remove the line and squash?

Except 'RTE_FLOW_ACTION_TYPE_RSS' update,
Squashed into relevant commits in next-net, thanks.
Ajit Khaparde Nov. 5, 2021, 6:48 p.m. UTC | #6 
On Fri, Nov 5, 2021 at 11:42 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> On 11/5/2021 6:41 PM, Ajit Khaparde wrote:
> > On Fri, Nov 5, 2021 at 11:38 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> >>
> >> On 11/5/2021 6:22 PM, Ajit Khaparde wrote:
> >>> On Fri, Nov 5, 2021 at 11:14 AM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> >>>>
> >>>> On 11/5/2021 4:55 PM, Ajit Khaparde wrote:
> >>>>> Update release notes for new functionality added to the bnxt PMD.
> >>>>>
> >>>>> Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> >>>>> ---
> >>>>>     doc/guides/rel_notes/release_21_11.rst | 9 ++++++++-
> >>>>>     1 file changed, 8 insertions(+), 1 deletion(-)
> >>>>>
> >>>>> diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
> >>>>> index 3367a15c85..d8b9531af1 100644
> >>>>> --- a/doc/guides/rel_notes/release_21_11.rst
> >>>>> +++ b/doc/guides/rel_notes/release_21_11.rst
> >>>>> @@ -152,7 +152,14 @@ New Features
> >>>>>
> >>>>>       * Added flow offload support for Thor.
> >>>>>       * Implement support for tunnel offload.
> >>>>> -  * Updated HWRM API to version 1.10.2.44
> >>>>> +  * Updated HWRM API to version 1.10.2.68.
> >>>>> +  * Updated support for RTE_FLOW_ACTION_TYPE_RSS.
> >>>>
> >>>> I am not sure which commit this update corresponds to, can you please
> >>>> share the commit that this update should go?
> >>> I think you are looking for this -
> >>> 239695f754cb ("net/bnxt: enhance RSS action support")
> >>>
> >>
> >> Ahh, that is pre -rc1, and not related to the latest based code series either.
> >>
> >> Since original commit is part of -rc1, that can't be squashed of course, can
> >> you please send a separate patch for it, with proper fixes tag etc?
> > Sure. I will send a separate patch for that.
> > Do you want me to respin a new patch for the remaining?
> > Or you can remove the line and squash?
>
> Except 'RTE_FLOW_ACTION_TYPE_RSS' update,
Sent a patch [1] for this. Added the entry at the end of the list.
Hopefully it will apply cleanly. Thanks

[1] https://patchwork.dpdk.org/project/dpdk/patch/20211105184634.20546-1-ajit.khaparde@broadcom.com/

> Squashed into relevant commits in next-net, thanks.
>
diff mbox series

Patch

diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
index 3367a15c85..d8b9531af1 100644
--- a/doc/guides/rel_notes/release_21_11.rst
+++ b/doc/guides/rel_notes/release_21_11.rst
@@ -152,7 +152,14 @@  New Features
 
   * Added flow offload support for Thor.
   * Implement support for tunnel offload.
-  * Updated HWRM API to version 1.10.2.44
+  * Updated HWRM API to version 1.10.2.68.
+  * Updated support for RTE_FLOW_ACTION_TYPE_RSS.
+  * Added support for socket redirection.
+  * Added support for inner IP header for GRE tunnel flows.
+  * Added NAT support for dest IP and port combination.
+  * Added TruFlow and AFM SRAM partitioning support.
+  * Added wildcard match support for ingress flows.
+  * Removed devargs for stats accumulation.
 
 * **Updated Cisco enic driver.**