Re: [dpdk-dev] [PATCH v4 00/25] bnxt patches

[Ferruh Yigit <ferruh.yigit@intel.com>]

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)
>>
>>