Skip to content

Commit 25ac138

Browse files
Cosmin Ratiuklassert
authored andcommitted
xfrm: Use xdo.dev instead of xdo.real_dev
The policy offload struct was reused from the state offload and real_dev was copied from dev, but it was never set to anything else. Simplify the code by always using xdo.dev for policies. Signed-off-by: Cosmin Ratiu <[email protected]> Reviewed-by: Leon Romanovsky <[email protected]> Reviewed-by: Nikolay Aleksandrov <[email protected]> Signed-off-by: Steffen Klassert <[email protected]>
1 parent d79444e commit 25ac138

File tree

3 files changed

+1
-5
lines changed

3 files changed

+1
-5
lines changed

drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1164,7 +1164,7 @@ mlx5e_ipsec_build_accel_pol_attrs(struct mlx5e_ipsec_pol_entry *pol_entry,
11641164
static int mlx5e_xfrm_add_policy(struct xfrm_policy *x,
11651165
struct netlink_ext_ack *extack)
11661166
{
1167-
struct net_device *netdev = x->xdo.real_dev;
1167+
struct net_device *netdev = x->xdo.dev;
11681168
struct mlx5e_ipsec_pol_entry *pol_entry;
11691169
struct mlx5e_priv *priv;
11701170
int err;

net/xfrm/xfrm_device.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -378,7 +378,6 @@ int xfrm_dev_policy_add(struct net *net, struct xfrm_policy *xp,
378378

379379
xdo->dev = dev;
380380
netdev_tracker_alloc(dev, &xdo->dev_tracker, GFP_ATOMIC);
381-
xdo->real_dev = dev;
382381
xdo->type = XFRM_DEV_OFFLOAD_PACKET;
383382
switch (dir) {
384383
case XFRM_POLICY_IN:
@@ -400,7 +399,6 @@ int xfrm_dev_policy_add(struct net *net, struct xfrm_policy *xp,
400399
err = dev->xfrmdev_ops->xdo_dev_policy_add(xp, extack);
401400
if (err) {
402401
xdo->dev = NULL;
403-
xdo->real_dev = NULL;
404402
xdo->type = XFRM_DEV_OFFLOAD_UNSPECIFIED;
405403
xdo->dir = 0;
406404
netdev_put(dev, &xdo->dev_tracker);

net/xfrm/xfrm_state.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1555,15 +1555,13 @@ xfrm_state_find(const xfrm_address_t *daddr, const xfrm_address_t *saddr,
15551555
xso->type = XFRM_DEV_OFFLOAD_PACKET;
15561556
xso->dir = xdo->dir;
15571557
xso->dev = xdo->dev;
1558-
xso->real_dev = xdo->real_dev;
15591558
xso->flags = XFRM_DEV_OFFLOAD_FLAG_ACQ;
15601559
netdev_hold(xso->dev, &xso->dev_tracker, GFP_ATOMIC);
15611560
error = xso->dev->xfrmdev_ops->xdo_dev_state_add(x, NULL);
15621561
if (error) {
15631562
xso->dir = 0;
15641563
netdev_put(xso->dev, &xso->dev_tracker);
15651564
xso->dev = NULL;
1566-
xso->real_dev = NULL;
15671565
xso->type = XFRM_DEV_OFFLOAD_UNSPECIFIED;
15681566
x->km.state = XFRM_STATE_DEAD;
15691567
to_put = x;

0 commit comments

Comments
 (0)