mbox series

[v4,00/25] bnxt patches

Message ID 20200627100050.19688-1-ajit.khaparde@broadcom.com (mailing list archive)
Headers
Series bnxt patches |

Message

Ajit Khaparde June 27, 2020, 10 a.m. UTC
  This patchset consists of enhancements for the host based flow table
management mechanism that was introduced in 20.05. Also has some
feature support for offloading vxlan encap/decap and on-chip exact match
flows.

v3->v4:
- Squashed patches and reduced number of patches from 36 to 25
- Updated commit logs
- Fixed Signed-off-by and Reviewed-by tags for some contrubuters

Kishore Padmanabha (22):
  net/bnxt: changes to makefile
  net/bnxt: remove fields from bitmap and mapper table
  net/bnxt: support more resource functions in flow database
  net/bnxt: add computed header field in result opcode
  net/bnxt: update compute field list and access macros
  net/bnxt: change default identifier to global resource
  net/bnxt: add resource sub type to class and action tables
  net/bnxt: remove cache tbl id from mapper class table
  net/bnxt: add support for action bitmap opcode
  net/bnxt: process action templates
  net/bnxt: use vport in the phy port act handler
  net/bnxt: add enum to the critical resource
  net/bnxt: refactor and rename some fields and enums
  net/bnxt: add support for vxlan encap and decap templates
  net/bnxt: flow db API to get VF rep action record
  net/bnxt: remove the implicit bitset update for vnic action
  net/bnxt: divide ulp template db file to smaller modules
  net/bnxt: add support for internal exact match flows
  net/bnxt: add vfr flag to the mark manager
  net/bnxt: support for mark action for LFID rules
  net/bnxt: rename fields in the device params structure
  net/bnxt: update ulp template database for new opcodes

Mike Baucom (1):
  net/bnxt: refactor the mapper opcodes

Shuanglin Wang (1):
  net/bnxt: add a devarg to set max flow count

Venkat Duvvuru (1):
  net/bnxt: modify IPV6 vtc flow field parsing

 drivers/net/bnxt/Makefile                     |   29 +-
 drivers/net/bnxt/bnxt.h                       |    3 +
 drivers/net/bnxt/bnxt_ethdev.c                |   62 +-
 drivers/net/bnxt/bnxt_rxr.c                   |   38 +-
 drivers/net/bnxt/meson.build                  |    4 +-
 drivers/net/bnxt/tf_core/Makefile             |   16 +
 drivers/net/bnxt/tf_ulp/Makefile              |   18 +
 drivers/net/bnxt/tf_ulp/bnxt_ulp.c            |   85 +-
 drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c       |    7 +-
 drivers/net/bnxt/tf_ulp/ulp_flow_db.c         |  231 +-
 drivers/net/bnxt/tf_ulp/ulp_flow_db.h         |   51 +-
 drivers/net/bnxt/tf_ulp/ulp_mapper.c          |  882 +++--
 drivers/net/bnxt/tf_ulp/ulp_mapper.h          |   25 +-
 drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c        |   13 +-
 drivers/net/bnxt/tf_ulp/ulp_mark_mgr.h        |    4 +
 drivers/net/bnxt/tf_ulp/ulp_matcher.c         |   25 +-
 drivers/net/bnxt/tf_ulp/ulp_matcher.h         |    2 +-
 drivers/net/bnxt/tf_ulp/ulp_rte_parser.c      |  167 +-
 drivers/net/bnxt/tf_ulp/ulp_rte_parser.h      |   13 +-
 drivers/net/bnxt/tf_ulp/ulp_template_db.c     | 3406 +++++++++++++++--
 drivers/net/bnxt/tf_ulp/ulp_template_db.h     |  332 +-
 drivers/net/bnxt/tf_ulp/ulp_template_db_act.c |  545 +++
 .../net/bnxt/tf_ulp/ulp_template_db_class.c   | 2497 ++++++++++++
 .../net/bnxt/tf_ulp/ulp_template_db_enum.h    |  591 +++
 .../net/bnxt/tf_ulp/ulp_template_db_field.h   |  225 ++
 drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c |  548 +++
 .../net/bnxt/tf_ulp/ulp_template_field_db.h   |  163 +-
 drivers/net/bnxt/tf_ulp/ulp_template_struct.h |   77 +-
 drivers/net/bnxt/tf_ulp/ulp_utils.c           |   67 +-
 drivers/net/bnxt/tf_ulp/ulp_utils.h           |   40 +-
 30 files changed, 9201 insertions(+), 965 deletions(-)
 create mode 100644 drivers/net/bnxt/tf_core/Makefile
 create mode 100644 drivers/net/bnxt/tf_ulp/Makefile
 create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_act.c
 create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_class.c
 create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_enum.h
 create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_field.h
 create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c
  

Comments

Ajit Khaparde June 27, 2020, 3:47 p.m. UTC | #1
On Sat, Jun 27, 2020 at 3:01 AM Ajit Khaparde <ajit.khaparde@broadcom.com>
wrote:

> This patchset consists of enhancements for the host based flow table
> management mechanism that was introduced in 20.05. Also has some
> feature support for offloading vxlan encap/decap and on-chip exact match
> flows.
>

Patchset applied to dpdk-next-net-brcm. Thanks

>
> v3->v4:
> - Squashed patches and reduced number of patches from 36 to 25
> - Updated commit logs
> - Fixed Signed-off-by and Reviewed-by tags for some contrubuters
>
> Kishore Padmanabha (22):
>   net/bnxt: changes to makefile
>   net/bnxt: remove fields from bitmap and mapper table
>   net/bnxt: support more resource functions in flow database
>   net/bnxt: add computed header field in result opcode
>   net/bnxt: update compute field list and access macros
>   net/bnxt: change default identifier to global resource
>   net/bnxt: add resource sub type to class and action tables
>   net/bnxt: remove cache tbl id from mapper class table
>   net/bnxt: add support for action bitmap opcode
>   net/bnxt: process action templates
>   net/bnxt: use vport in the phy port act handler
>   net/bnxt: add enum to the critical resource
>   net/bnxt: refactor and rename some fields and enums
>   net/bnxt: add support for vxlan encap and decap templates
>   net/bnxt: flow db API to get VF rep action record
>   net/bnxt: remove the implicit bitset update for vnic action
>   net/bnxt: divide ulp template db file to smaller modules
>   net/bnxt: add support for internal exact match flows
>   net/bnxt: add vfr flag to the mark manager
>   net/bnxt: support for mark action for LFID rules
>   net/bnxt: rename fields in the device params structure
>   net/bnxt: update ulp template database for new opcodes
>
> Mike Baucom (1):
>   net/bnxt: refactor the mapper opcodes
>
> Shuanglin Wang (1):
>   net/bnxt: add a devarg to set max flow count
>
> Venkat Duvvuru (1):
>   net/bnxt: modify IPV6 vtc flow field parsing
>
>  drivers/net/bnxt/Makefile                     |   29 +-
>  drivers/net/bnxt/bnxt.h                       |    3 +
>  drivers/net/bnxt/bnxt_ethdev.c                |   62 +-
>  drivers/net/bnxt/bnxt_rxr.c                   |   38 +-
>  drivers/net/bnxt/meson.build                  |    4 +-
>  drivers/net/bnxt/tf_core/Makefile             |   16 +
>  drivers/net/bnxt/tf_ulp/Makefile              |   18 +
>  drivers/net/bnxt/tf_ulp/bnxt_ulp.c            |   85 +-
>  drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c       |    7 +-
>  drivers/net/bnxt/tf_ulp/ulp_flow_db.c         |  231 +-
>  drivers/net/bnxt/tf_ulp/ulp_flow_db.h         |   51 +-
>  drivers/net/bnxt/tf_ulp/ulp_mapper.c          |  882 +++--
>  drivers/net/bnxt/tf_ulp/ulp_mapper.h          |   25 +-
>  drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c        |   13 +-
>  drivers/net/bnxt/tf_ulp/ulp_mark_mgr.h        |    4 +
>  drivers/net/bnxt/tf_ulp/ulp_matcher.c         |   25 +-
>  drivers/net/bnxt/tf_ulp/ulp_matcher.h         |    2 +-
>  drivers/net/bnxt/tf_ulp/ulp_rte_parser.c      |  167 +-
>  drivers/net/bnxt/tf_ulp/ulp_rte_parser.h      |   13 +-
>  drivers/net/bnxt/tf_ulp/ulp_template_db.c     | 3406 +++++++++++++++--
>  drivers/net/bnxt/tf_ulp/ulp_template_db.h     |  332 +-
>  drivers/net/bnxt/tf_ulp/ulp_template_db_act.c |  545 +++
>  .../net/bnxt/tf_ulp/ulp_template_db_class.c   | 2497 ++++++++++++
>  .../net/bnxt/tf_ulp/ulp_template_db_enum.h    |  591 +++
>  .../net/bnxt/tf_ulp/ulp_template_db_field.h   |  225 ++
>  drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c |  548 +++
>  .../net/bnxt/tf_ulp/ulp_template_field_db.h   |  163 +-
>  drivers/net/bnxt/tf_ulp/ulp_template_struct.h |   77 +-
>  drivers/net/bnxt/tf_ulp/ulp_utils.c           |   67 +-
>  drivers/net/bnxt/tf_ulp/ulp_utils.h           |   40 +-
>  30 files changed, 9201 insertions(+), 965 deletions(-)
>  create mode 100644 drivers/net/bnxt/tf_core/Makefile
>  create mode 100644 drivers/net/bnxt/tf_ulp/Makefile
>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_act.c
>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_class.c
>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_enum.h
>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_field.h
>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c
>
> --
> 2.21.1 (Apple Git-122.3)
>
>
  
Ferruh Yigit June 30, 2020, 7:49 a.m. UTC | #2
On 6/27/2020 4:47 PM, Ajit Khaparde wrote:
> On Sat, Jun 27, 2020 at 3:01 AM Ajit Khaparde <ajit.khaparde@broadcom.com>
> wrote:
> 
>> This patchset consists of enhancements for the host based flow table
>> management mechanism that was introduced in 20.05. Also has some
>> feature support for offloading vxlan encap/decap and on-chip exact match
>> flows.
>>
> 
> Patchset applied to dpdk-next-net-brcm. Thanks

Hi Ajit,

I have moved some code to fix patch by patch build (some function/fields were
used before they are introduced) while pulling to next-net, final code should be
same but please double check the commits in the next-net.

Meanwhile I am not sure about the logical seperation of the patches, it looks to
me there are unrelated changes in some of them, although it is not easy for me
to tell this always. Can we please target more clean seperation next time?

Thanks,
ferruh

> 
>>
>> v3->v4:
>> - Squashed patches and reduced number of patches from 36 to 25
>> - Updated commit logs
>> - Fixed Signed-off-by and Reviewed-by tags for some contrubuters
>>
>> Kishore Padmanabha (22):
>>   net/bnxt: changes to makefile
>>   net/bnxt: remove fields from bitmap and mapper table
>>   net/bnxt: support more resource functions in flow database
>>   net/bnxt: add computed header field in result opcode
>>   net/bnxt: update compute field list and access macros
>>   net/bnxt: change default identifier to global resource
>>   net/bnxt: add resource sub type to class and action tables
>>   net/bnxt: remove cache tbl id from mapper class table
>>   net/bnxt: add support for action bitmap opcode
>>   net/bnxt: process action templates
>>   net/bnxt: use vport in the phy port act handler
>>   net/bnxt: add enum to the critical resource
>>   net/bnxt: refactor and rename some fields and enums
>>   net/bnxt: add support for vxlan encap and decap templates
>>   net/bnxt: flow db API to get VF rep action record
>>   net/bnxt: remove the implicit bitset update for vnic action
>>   net/bnxt: divide ulp template db file to smaller modules
>>   net/bnxt: add support for internal exact match flows
>>   net/bnxt: add vfr flag to the mark manager
>>   net/bnxt: support for mark action for LFID rules
>>   net/bnxt: rename fields in the device params structure
>>   net/bnxt: update ulp template database for new opcodes
>>
>> Mike Baucom (1):
>>   net/bnxt: refactor the mapper opcodes
>>
>> Shuanglin Wang (1):
>>   net/bnxt: add a devarg to set max flow count
>>
>> Venkat Duvvuru (1):
>>   net/bnxt: modify IPV6 vtc flow field parsing
>>
>>  drivers/net/bnxt/Makefile                     |   29 +-
>>  drivers/net/bnxt/bnxt.h                       |    3 +
>>  drivers/net/bnxt/bnxt_ethdev.c                |   62 +-
>>  drivers/net/bnxt/bnxt_rxr.c                   |   38 +-
>>  drivers/net/bnxt/meson.build                  |    4 +-
>>  drivers/net/bnxt/tf_core/Makefile             |   16 +
>>  drivers/net/bnxt/tf_ulp/Makefile              |   18 +
>>  drivers/net/bnxt/tf_ulp/bnxt_ulp.c            |   85 +-
>>  drivers/net/bnxt/tf_ulp/bnxt_ulp_flow.c       |    7 +-
>>  drivers/net/bnxt/tf_ulp/ulp_flow_db.c         |  231 +-
>>  drivers/net/bnxt/tf_ulp/ulp_flow_db.h         |   51 +-
>>  drivers/net/bnxt/tf_ulp/ulp_mapper.c          |  882 +++--
>>  drivers/net/bnxt/tf_ulp/ulp_mapper.h          |   25 +-
>>  drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c        |   13 +-
>>  drivers/net/bnxt/tf_ulp/ulp_mark_mgr.h        |    4 +
>>  drivers/net/bnxt/tf_ulp/ulp_matcher.c         |   25 +-
>>  drivers/net/bnxt/tf_ulp/ulp_matcher.h         |    2 +-
>>  drivers/net/bnxt/tf_ulp/ulp_rte_parser.c      |  167 +-
>>  drivers/net/bnxt/tf_ulp/ulp_rte_parser.h      |   13 +-
>>  drivers/net/bnxt/tf_ulp/ulp_template_db.c     | 3406 +++++++++++++++--
>>  drivers/net/bnxt/tf_ulp/ulp_template_db.h     |  332 +-
>>  drivers/net/bnxt/tf_ulp/ulp_template_db_act.c |  545 +++
>>  .../net/bnxt/tf_ulp/ulp_template_db_class.c   | 2497 ++++++++++++
>>  .../net/bnxt/tf_ulp/ulp_template_db_enum.h    |  591 +++
>>  .../net/bnxt/tf_ulp/ulp_template_db_field.h   |  225 ++
>>  drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c |  548 +++
>>  .../net/bnxt/tf_ulp/ulp_template_field_db.h   |  163 +-
>>  drivers/net/bnxt/tf_ulp/ulp_template_struct.h |   77 +-
>>  drivers/net/bnxt/tf_ulp/ulp_utils.c           |   67 +-
>>  drivers/net/bnxt/tf_ulp/ulp_utils.h           |   40 +-
>>  30 files changed, 9201 insertions(+), 965 deletions(-)
>>  create mode 100644 drivers/net/bnxt/tf_core/Makefile
>>  create mode 100644 drivers/net/bnxt/tf_ulp/Makefile
>>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_act.c
>>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_class.c
>>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_enum.h
>>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_field.h
>>  create mode 100644 drivers/net/bnxt/tf_ulp/ulp_template_db_tbl.c
>>
>> --
>> 2.21.1 (Apple Git-122.3)
>>
>>