Hi Ivan,
First nice idea and thanks for the picking up the ball.
Before a detail review,
The main thing I'm concerned about is that this print will be partially
supported,
I know that you covered this issue by printing unknown for unsupported
item/actions,
but this will mean that it is enough
This patchset adds flow table management support for
Thor network adapter and introduces enhancements to
the existing flow table management functionality.
Farah Smith (11):
net/bnxt: add base TRUFLOW support for Thor
net/bnxt: update TRUFLOW resources
net/bnxt: add action SRAM Translation
From: Jeffrey Huang
Before introducing more HCAPI components to DPDK, the CFA code needs
to be organized into a dedicated folder so it is separated from
other new HCAPI components
Signed-off-by: Jeffrey Huang
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Farah Smith
From: Farah Smith
Add infrastructure code to support TRUFLOW on Thor NICs.
Also update meson.build
Signed-off-by: Farah Smith
Signed-off-by: Peter Spreadborough
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Randy Schacher
---
drivers/net/bnxt/tf_core/cfa_resource
From: Jay Ding
- Allow tf_open to continue if no resource is allocated
for some table type.
- Close the session if binding fails for any table.
- Close the session if no resource is allocated for all tables.
Signed-off-by: Jay Ding
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
From: Peter Spreadborough
Add get mailbox dev op so that mailbox offset is based on device
rather than hard coded.
Signed-off-by: Peter Spreadborough
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Farah Smith
---
drivers/net/bnxt/tf_core/tf_device.c | 21 ++
d
From: Farah Smith
- Remove unused tables from tf_tbl_type
- Encode flow type into flow handle (internal or external)
- Clean up Whitney resource tables
- Clean up Truflow CLI open tables and update Thor resources
- Add Thor SRAM and external pool types to core API
- Remove unneeded Stingray table
From: Peter Spreadborough
Main TF changes to support EM insert with FKB.
Flexible Key builder is required to create Wild Card
and Exact Match keys for TCAM lookups.
Signed-off-by: Peter Spreadborough
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Farah Smith
---
dr
From: Jay Ding
1. Add set/get/free/alloc for WC TCAM
2. Rework the key size in slice management.
3. Add 3 FKB WC keys for WC TCAM set cli cmd
4. Add transform key function for WC TCAM FKB key
5. Add checking for key buffer length for get_tcam
Signed-off-by: Jay Ding
Signed-off-by: Randy Schache
From: Farah Smith
- Translate Truflow action types for Thor to HCAPI RM
resource defined SRAM banks.
- move module type enum definitions to tf_core API
- Switch to subtype concept for RM.
- alloc/free working for Thor SRAM table type for full AR.
Signed-off-by: Farah Smith
Signed-off-by: Rand
From: Jay Ding
- Implement TCAM get in host
- Add Thor support for TCAM set/free
Signed-off-by: Jay Ding
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Peter Spreadborough
Reviewed-by: Farah Smith
---
drivers/net/bnxt/tf_core/hwrm_tf.h | 1 +
drivers/net/bn
From: Jay Ding
RM databases are statically defined in each module. New static
database needs to be defined in the code when multiple sessions
are added. Add dynamic alloc database and associate it to each
session.
Signed-off-by: Jay Ding
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvv
From: Farah Smith
- Move Bulk get to a direct HWRM message
- Deprecate code based on HCAPI changes
Signed-off-by: Farah Smith
Signed-off-by: Peter Spreadborough
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Randy Schacher
---
drivers/net/bnxt/hcapi/cfa/hcapi_cfa.
From: Peter Spreadborough
- Move HCAPI hashing code to common file and add Thor support.
- Change DPDK EM insert for FKB to use limited size Type 3 key.
- Update FKB builder to be able to tell between EM and WC keys
during transform.
Signed-off-by: Peter Spreadborough
Signed-off-by: Randy Sch
From: Farah Smith
HCAPI RM now manages 64B records instead of 8B.
Truflow core RM will manage the same. The tf_tbl core
APIs now return 8B pointer addresses. These can
be used directly as SRAM pointers in Action Records.
When communicating with the firmware 8B addresses
will be used.
Signed-off-
From: Jay Ding
There are 2 types of sessions - shared and non-shared. For non-shared
all the allocated resources are owned and managed by a single
session instance. No other applications have access to the
resources owned by the non-shared session. For a shared session,
resources are shared bet
From: Peter Spreadborough
The dpool allocator supports variable size entries and
also supports defragmentation of the allocation space.
EM will by default use the fixed size stack allocator.
The dynamic allocator may be selected at build time.
The dpool allocator supports variable size entries an
From: Jay Ding
Allow an application to only reserve resources for one direction.
Signed-off-by: Jay Ding
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Ajit Kumar Khaparde
Reviewed-by: Randy Schacher
---
drivers/net/bnxt/tf_core/tf_core.c| 4 --
drivers/n
From: Jay Ding
- Distinguish the shared session on host side using PCI address
- One session could be shared by multiple interfaces.
Signed-off-by: Jay Ding
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Farah Smith
---
drivers/net/bnxt/tf_core/tf_core.c|
From: Farah Smith
Add profile_id to set_prof_tcam.
Signed-off-by: Farah Smith
Signed-off-by: Jay Ding
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Randy Schacher
Reviewed-by: Peter Spreadborough
---
drivers/net/bnxt/tf_core/tf_device.c | 90 ---
From: Farah Smith
Provide tf API to get the shared table increment value
for a given TF table type.
Signed-off-by: Farah Smith
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Randy Schacher
---
drivers/net/bnxt/tf_core/tf_core.c| 52 +++
From: Farah Smith
- clean up all allocated hi or lo pool tcam regions on close
- message cleanup
- remove unsupported multi-slice
- find next free entry should start from 0 first time
- update reserved resources so cli open session can come up
Signed-off-by: Farah Smith
Signed-off-by: Randy Sch
From: Farah Smith
- Add new API to move wc tcam regions from the hi pool
to the low pool.
- Enable shared tcam get/set functions on Thor.
Signed-off-by: Farah Smith
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Randy Schacher
---
drivers/net/bnxt/tf_core/tf_core
From: Farah Smith
- switch to single slice management on Wh+
- Support of shared session WC_TCAM_HIGH and WC_TCAM_LOW regions
- Enable/disable using TF_TCAM_SHARED flag in tf_core.h
- Fix empty session module DBs in the case that none are
allocated for a given module type
Signed-off-by: Farah
Currently, a devarg (host-based-truflow) is passed while launching
the app to enable truflow feature. However, this mechanism doesn't
give a seamless experience when flow offloads has to work with
firmware that doesn't/does support truflow feature. Also, it's
likely that customers may not want to u
From: Kishore Padmanabha
Added support for mapper flow database opcode to enable
shared resources like mirror action. This allows mapper
to conditionally populate flow database based on template content.
Signed-off-by: Kishore Padmanabha
Signed-off-by: Venkat Duvvuru
Reviewed-by: Michael Bauco
From: Jay Ding
- Close fw session if session open fails after fw session open.
- Additional WC TCAM debug info to help in future debug
- Reduce key/mask buffer sizes for performance
- When a 64b counter is freed, clear the entry
Signed-off-by: Jay Ding
Signed-off-by: Farah Smith
Signed-off-by:
From: Farah Smith
Provide TRUFLOW API to clear either the hi or the low region
for ungraceful exit cleanup.
Signed-off-by: Farah Smith
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duvvuru
Reviewed-by: Jay Ding
Reviewed-by: Peter Spreadborough
---
drivers/net/bnxt/tf_core/tf_core.c
From: Kishore Padmanabha
Added TCAM table specific opcode to process TCAM entry creation
and reuse. This change removes the TCAM cache mechanism and uses
the generic table mechanism for reuse of TCAM entries.
Signed-off-by: Kishore Padmanabha
Signed-off-by: Venkat Duvvuru
Reviewed-by: Michael
The driver is using physical port id as the index into
the tunnel inner flow table. However, this will not work in case
of multichannel mode where multiple physical functions are going
to share the same physical port id.
When tunnel inner flow offload request comes before tunnel
outer flow offload
From: Mike Baucom
Conditional execution and rejection processing added for templates and
tables. This allows the mapper to skip tables and reject templates
based on the content without having to hard code rules.
Signed-off-by: Mike Baucom
Signed-off-by: Venkat Duvvuru
Reviewed-by: Kishore Pad
From: Jay Ding
If the session shares WC TCAM entries with others,
specify it in the session name by attach "-wc_tcam".
Firmware will flush the shared WC TCAM entries if
the last shared session using them is closed.
Signed-off-by: Jay Ding
Signed-off-by: Randy Schacher
Signed-off-by: Venkat Duv
From: Kishore Padmanabha
The shared handle is set in the mapper params when generic resource
are created, this shall be used by application as a handle to the
shared resource like mirror handle.
Signed-off-by: Kishore Padmanabha
Signed-off-by: Venkat Duvvuru
Reviewed-by: Michael Baucom
---
d
From: Kishore Padmanabha
1. Added interface table specific opcode to process interface table
entry creation and reuse. This allows reuse of the interface table
entry for multiple flows. Changed the regfile apis to store
the data in big endian format.
2. The result blob creation being done in tca
From: Kishore Padmanabha
For support for wild card TCAM, some of the header fields have to
be partially written, hence this new opcode is added.
Signed-off-by: Kishore Padmanabha
Signed-off-by: Venkat Duvvuru
Reviewed-by: Ajit Kumar Khaparde
Reviewed-by: Michael Baucom
---
drivers/net/bnxt/
This patch does the following to support GRE flows:
1. RTE_FLOW_ITEM_TYPE_ANY & RTE_FLOW_ITEM_TYPE_GRE processing
2. Calculate the absolute function ID from the logical VF ID
passed as part of RTE_FLOW_ACTION_TYPE_VF action.
3. Move bnxt_get_bp API to bnxt_ethdev.c
Signed-off-by: Venkat Duvvuru
From: Kishore Padmanabha
1. The internal and external exact match table resource types
is combined since the resource handle contains the encoded
type whether it is internal or external exact match entry.
2. When a flow doesn't hit the offloaded rules, the default action is
to send it to the ker
From: Kishore Padmanabha
The computational field is enabled for wild card pattern support.
The template checks the computational field to add a flow as wild
card entry or exact match entry.
Signed-off-by: Kishore Padmanabha
Signed-off-by: Venkat Duvvuru
Reviewed-by: Venkat Duvvuru
Reviewed-by
From: Mike Baucom
Shared session permits cooperative sharing of prescribed resources
between applications.
- devargs added for app-id in order to enable sharing session
resources across applications
- shared session management added
- tf resource reservations are now app and device dependent
From: Mike Baucom
Shared sessions require both named and unnamed resources to be requested
during a tf_open_session. ULP uses named resources for global resources
that are pre-allocated and remain through the life of the application.
Unnamed resources are generally per flow resources and allocate
From: Mike Baucom
Added the ability for cooperative applications to share resources and
perform some high availability functions.
Signed-off-by: Mike Baucom
Signed-off-by: Venkat Duvvuru
Reviewed-by: Shahaji Bhosle
Reviewed-by: Randy Schacher
---
drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 5
From: Kishore Padmanabha
This patch adds support for parsing rte_flow items for icmp6 flows.
Signed-off-by: Kishore Padmanabha
Signed-off-by: Venkat Duvvuru
Reviewed-by: Randy Schacher
---
drivers/net/bnxt/tf_ulp/ulp_rte_handler_tbl.c | 4 +-
drivers/net/bnxt/tf_ulp/ulp_rte_parser.c |
From: Kishore Padmanabha
The alarm callback needs to have a valid context pointer when it is
invoked, the context could become invalid if the port goes down and
the callback is invoked before it is cancelled.
Signed-off-by: Kishore Padmanabha
Signed-off-by: Venkat Duvvuru
Reviewed-by: Randy Sc
Currently the flow dump provides few information about actions
- just the pointers. Add implementations to display details for
counter, modify_hdr and encap_decap actions.
For counter, the regular flow operation query is engaged and
the counter content information is provided, including hits
and b
On Sun, 30 May 2021 07:27:32 +
Ori Kam wrote:
> >
> > DPDK applications (for example, OvS) or tests which use RTE flow API need to
> > log created or rejected flow rules to help to recognise what goes right or
> > wrong. From this standpoint, testpmd-compliant format is nice for the
> > purp
In max10_staging_area_init(), variable "start" from fdt_get_reg() may
be invalid, it should be checked before assigning to member variable
"staging_area_base" of structure "intel_max10_device".
Coverity issue: 367480, 367482
Fixes: a05bd1b40bde ("raw/ifpga: add FPGA RSU APIs")
Cc: sta...@dpdk.org
Support AVF RSS and FDIR for GTPoGRE packet.
Lingyu Liu (4):
net/iavf: support flow pattern for GTPoGRE
common/iavf: add header types for GRE
net/iavf: support AVF FDIR for GTPoGRE tunnel packet
net/iavf: support AVF RSS for GTPoGRE packet
---
V2 change:
- append new protocol to the tail
Add GTPoGRE pattern support for AVF FDIR and RSS.
Signed-off-by: Lingyu Liu
---
drivers/net/iavf/iavf_generic_flow.c | 288 +++
drivers/net/iavf/iavf_generic_flow.h | 40
2 files changed, 328 insertions(+)
diff --git a/drivers/net/iavf/iavf_generic_flow.c
b/driver
Add a virtchnl protocol header type to support AVF FDIR and RSS for GRE.
Signed-off-by: Lingyu Liu
---
drivers/common/iavf/virtchnl.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/common/iavf/virtchnl.h b/drivers/common/iavf/virtchnl.h
index 3a60faff93..197edce8a1 100644
--- a/driv
Support AVF FDIR for inner header of GTPoGRE tunnel packet.
+---+---+
| Pattern |Input Set |
+---+---+
|eth/ipv4/gre/ipv4/udp/gtpu/
Support AVF RSS for inner most header of GTPoGRE packet. It supports
RSS based on inner most IP src + dst address and TCP/UDP src + dst
port.
Signed-off-by: Lingyu Liu
---
drivers/net/iavf/iavf_hash.c | 24
1 file changed, 24 insertions(+)
diff --git a/drivers/net/iavf/
From: Xuan Ding
For async copy, it is unsafe to directly use the physical address.
and current address translation from GPA to HPA via SW also takes
CPU cycles, these can all benefit from IOMMU.
Since the existing DMA engine supports to use platform IOMMU,
this patch enables IOMMU for async vhos
52 matches
Mail list logo
