mbox series

[0/2] net/mlx5: copy the item flags from prefix flow

Message ID 1582122664-54731-1-git-send-email-suanmingm@mellanox.com (mailing list archive)
Headers
Series net/mlx5: copy the item flags from prefix flow |

Message

Suanming Mou Feb. 19, 2020, 2:31 p.m. UTC 
  For flow split to several subflows, the match items maybe in the prefix
flow, and the actions are split to the suffix flow.

In this case, for the actions need the user defined match item will not
create correctly.

Copy the item layers flags to the suffix flow from prefix flow to fix
the issue.

This patch series should be applied after:
https://patches.dpdk.org/project/dpdk/list/?series=8613

Suanming Mou (2):
  net/mlx5: fix layer flags missing in metadata
  net/mlx5: fix lack of match information in meter

 drivers/net/mlx5/mlx5_flow.c    | 41 +++++++++++++++++++-------
 drivers/net/mlx5/mlx5_flow_dv.c | 64 +++++++++++++++++++++++++++++++----------
 2 files changed, 79 insertions(+), 26 deletions(-)

Comments

Raslan Darawsheh Feb. 20, 2020, 12:13 p.m. UTC | #1 
Hi,

> -----Original Message-----
> From: Suanming Mou <suanmingm@mellanox.com>
> Sent: Wednesday, February 19, 2020 4:31 PM
> To: Slava Ovsiienko <viacheslavo@mellanox.com>; Matan Azrad
> <matan@mellanox.com>
> Cc: dev@dpdk.org; Raslan Darawsheh <rasland@mellanox.com>
> Subject: [PATCH 0/2] net/mlx5: copy the item flags from prefix flow
> 
> For flow split to several subflows, the match items maybe in the prefix
> flow, and the actions are split to the suffix flow.
> 
> In this case, for the actions need the user defined match item will not
> create correctly.
> 
> Copy the item layers flags to the suffix flow from prefix flow to fix
> the issue.
> 
> This patch series should be applied after:
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatch
> es.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D8613&amp;data=0
> 2%7C01%7Crasland%40mellanox.com%7C18e60c79e62d40313b6008d7b5485c
> df%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C6371771947191293
> 09&amp;sdata=QE%2FeFbc3rp1m5KgmtmR%2Bfu5%2B2v%2BeBANO1u80R2
> PdS9A%3D&amp;reserved=0
> 
> Suanming Mou (2):
>   net/mlx5: fix layer flags missing in metadata
>   net/mlx5: fix lack of match information in meter
> 
>  drivers/net/mlx5/mlx5_flow.c    | 41 +++++++++++++++++++-------
>  drivers/net/mlx5/mlx5_flow_dv.c | 64
> +++++++++++++++++++++++++++++++----------
>  2 files changed, 79 insertions(+), 26 deletions(-)
> 
> --
> 1.8.3.1


Series applied to next-net-mlx,

Kindest regards,
Raslan Darawsheh
Raslan Darawsheh Feb. 20, 2020, 1:46 p.m. UTC | #2 
Hi,

Taking this series out of next-net-mlx,
Waiting for a v2, we are seeing a Seg. Fault due to the first patch of this series.


Kindest regards,
Raslan Darawsheh

> -----Original Message-----
> From: Raslan Darawsheh
> Sent: Thursday, February 20, 2020 2:13 PM
> To: Suanming Mou <suanmingm@mellanox.com>; Slava Ovsiienko
> <viacheslavo@mellanox.com>; Matan Azrad <matan@mellanox.com>
> Cc: dev@dpdk.org
> Subject: RE: [PATCH 0/2] net/mlx5: copy the item flags from prefix flow
> 
> Hi,
> 
> > -----Original Message-----
> > From: Suanming Mou <suanmingm@mellanox.com>
> > Sent: Wednesday, February 19, 2020 4:31 PM
> > To: Slava Ovsiienko <viacheslavo@mellanox.com>; Matan Azrad
> > <matan@mellanox.com>
> > Cc: dev@dpdk.org; Raslan Darawsheh <rasland@mellanox.com>
> > Subject: [PATCH 0/2] net/mlx5: copy the item flags from prefix flow
> >
> > For flow split to several subflows, the match items maybe in the prefix
> > flow, and the actions are split to the suffix flow.
> >
> > In this case, for the actions need the user defined match item will not
> > create correctly.
> >
> > Copy the item layers flags to the suffix flow from prefix flow to fix
> > the issue.
> >
> > This patch series should be applied after:
> >
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatch
> >
> es.dpdk.org%2Fproject%2Fdpdk%2Flist%2F%3Fseries%3D8613&amp;data=0
> >
> 2%7C01%7Crasland%40mellanox.com%7C18e60c79e62d40313b6008d7b5485c
> >
> df%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C6371771947191293
> >
> 09&amp;sdata=QE%2FeFbc3rp1m5KgmtmR%2Bfu5%2B2v%2BeBANO1u80R2
> > PdS9A%3D&amp;reserved=0
> >
> > Suanming Mou (2):
> >   net/mlx5: fix layer flags missing in metadata
> >   net/mlx5: fix lack of match information in meter
> >
> >  drivers/net/mlx5/mlx5_flow.c    | 41 +++++++++++++++++++-------
> >  drivers/net/mlx5/mlx5_flow_dv.c | 64
> > +++++++++++++++++++++++++++++++----------
> >  2 files changed, 79 insertions(+), 26 deletions(-)
> >
> > --
> > 1.8.3.1
> 
> 
> Series applied to next-net-mlx,
> 
> Kindest regards,
> Raslan Darawsheh