On Sun, 2020-06-21 at 16:25 -0400, Jarod Wilson wrote: > On Thu, Jun 11, 2020 at 5:51 PM Saeed Mahameed <sae...@mellanox.com> > wrote: > > On Wed, 2020-06-10 at 14:59 -0400, Jarod Wilson wrote: > > > I've been unable to get my hands on suitable supported hardware > > > to > > > date, > > > but I believe this ought to be all that is needed to enable the > > > mlx5 > > > driver to also work with bonding active-backup crypto offload > > > passthru. > > > > > > CC: Boris Pismenny <bor...@mellanox.com> > > > CC: Saeed Mahameed <sae...@mellanox.com> > > > CC: Leon Romanovsky <l...@kernel.org> > > > CC: Jay Vosburgh <j.vosbu...@gmail.com> > > > CC: Veaceslav Falico <vfal...@gmail.com> > > > CC: Andy Gospodarek <a...@greyhouse.net> > > > CC: "David S. Miller" <da...@davemloft.net> > > > CC: Jeff Kirsher <jeffrey.t.kirs...@intel.com> > > > CC: Jakub Kicinski <k...@kernel.org> > > > CC: Steffen Klassert <steffen.klass...@secunet.com> > > > CC: Herbert Xu <herb...@gondor.apana.org.au> > > > CC: net...@vger.kernel.org > > > Signed-off-by: Jarod Wilson <ja...@redhat.com> > > > --- > > > drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c | 6 > > > ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > diff --git > > > a/drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c > > > b/drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c > > > index 92eb3bad4acd..72ad6664bd73 100644 > > > --- a/drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c > > > +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c > > > @@ -210,6 +210,9 @@ static inline int > > > mlx5e_xfrm_validate_state(struct xfrm_state *x) > > > struct net_device *netdev = x->xso.dev; > > > struct mlx5e_priv *priv; > > > > > > + if (x->xso.slave_dev) > > > + netdev = x->xso.slave_dev; > > > + > > > > Do we really need to repeat this per driver ? > > why not just setup xso.real_dev, in xfrm layer once and for all > > before > > calling device drivers ? > > > > Device drivers will use xso.real_dev blindly. > > > > Will be useful in the future when you add vlan support, etc.. > > Apologies, I didn't catch your reply until just recently. Yeah, that > sounds like a better approach, if I can work it out cleanly. We just > init xso.real_dev to the same thing as xso.dev, then overwrite it in > the upper layer drivers (bonding, vlan, etc), while device drivers > just always use xso.real_dev, if I'm understanding your suggestion. > I'll see what I can come up with. > >
Yes, exactly what i meant, Thanks !
