[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <AM0PR05MB44032E09426E5E1DB5AD149ED25D0@AM0PR05MB4403.eurprd05.prod.outlook.com>
Date: Mon, 16 Jul 2018 21:46:15 +0000
From: Mark Bloch <markb@...lanox.com>
To: Or Gerlitz <gerlitz.or@...il.com>
CC: Doug Ledford <dledford@...hat.com>,
Jason Gunthorpe <jgg@...lanox.com>,
Leon Romanovsky <leonro@...lanox.com>,
RDMA mailing list <linux-rdma@...r.kernel.org>,
Saeed Mahameed <saeedm@...lanox.com>,
linux-netdev <netdev@...r.kernel.org>
Subject: RE: [RFC PATCH rdma-next 13/18] RDMA/mlx5: Enable decap and packet
reformat on flow tables
> -----Original Message-----
> From: Or Gerlitz [mailto:gerlitz.or@...il.com]
> Sent: Monday, July 16, 2018 2:24 PM
> To: Mark Bloch <markb@...lanox.com>
> Cc: Doug Ledford <dledford@...hat.com>; Jason Gunthorpe
> <jgg@...lanox.com>; Leon Romanovsky <leonro@...lanox.com>; RDMA
> mailing list <linux-rdma@...r.kernel.org>; Saeed Mahameed
> <saeedm@...lanox.com>; linux-netdev <netdev@...r.kernel.org>
> Subject: Re: [RFC PATCH rdma-next 13/18] RDMA/mlx5: Enable decap and
> packet reformat on flow tables
>
> On Mon, Jul 16, 2018 at 11:23 AM, Leon Romanovsky <leon@...nel.org>
> wrote:
> > From: Mark Bloch <markb@...lanox.com>
> >
> > If NIC RX flow tables support decap opertion, enable it on creation.
>
> opertion --> operation
>
> > If NIC TX flow tables support reformat opertion, enable it on creation.
>
> What is the trigger to use the decap flag on RX table or encap flag on
> TX table?
>
It has no performance penalty to always enable that, so that's what I do if supported.
> Please note that we have a short blanket w.r.t mutual usage by
FDB and NIC steering tables have different limitations, so encap/decap on NIC steering
have nothing to do with the limitations the FDB has with those operations.
> NIC vs e-Switch steering, did you consider to do that on demand?
The flow table needs to be created with those flags set if we want to attach
decap/packet reformat action to it. BTW, there is no modify action for those bits
so that's why I'm doing it on creation.
Mark
Powered by blists - more mailing lists