Skip to content

L
linux
Commit 69bfac96 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by David S. Miller
Browse files
Options

net: switchdev: add a context void pointer to struct switchdev_notifier_info 

In the case where the driver asks for a replay of a certain type of
event (port object or attribute) for a bridge port that is a LAG, it may
do so because this port has just joined the LAG.

But there might already be other switchdev ports in that LAG, and it is
preferable that those preexisting switchdev ports do not act upon the
replayed event.

The solution is to add a context to switchdev events, which is NULL most
of the time (when the bridge layer initiates the call) but which can be
set to a value controlled by the switchdev driver when a replay is
requested. The driver can then check the context to figure out if all
ports within the LAG should act upon the switchdev event, or just the
ones that match the context.

We have to modify all switchdev_handle_* helper functions as well as the
prototypes in the drivers that use these helpers too, because these
helpers hide the underlying struct switchdev_notifier_info from us and
there is no way to retrieve the context otherwise.

The context structure will be populated and used in later patches.
Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 97558e88
Hide whitespace changes
Inline Side-by-side
Showing with 44 additions and 37 deletions
drivers/net/ethernet/freescale/dpaa2/dpaa2-switch.c
View file @ 69bfac96
...@@ -1625,7 +1625,7 @@ static int dpaa2_switch_port_bridge_flags(struct net_device *netdev, ...@@ -1625,7 +1625,7 @@ static int dpaa2_switch_port_bridge_flags(struct net_device *netdev,
return 0; return 0;
} }
static int dpaa2_switch_port_attr_set(struct net_device *netdev, static int dpaa2_switch_port_attr_set(struct net_device *netdev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
......
drivers/net/ethernet/marvell/prestera/prestera_switchdev.c
View file @ 69bfac96
...@@ -708,7 +708,7 @@ static int prestera_port_attr_stp_state_set(struct prestera_port *port, ...@@ -708,7 +708,7 @@ static int prestera_port_attr_stp_state_set(struct prestera_port *port,
return err; return err;
} }
static int prestera_port_obj_attr_set(struct net_device *dev, static int prestera_port_obj_attr_set(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -1040,7 +1040,7 @@ static int prestera_port_vlans_add(struct prestera_port *port, ...@@ -1040,7 +1040,7 @@ static int prestera_port_vlans_add(struct prestera_port *port,
flag_pvid, extack); flag_pvid, extack);
} }
static int prestera_port_obj_add(struct net_device *dev, static int prestera_port_obj_add(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -1078,7 +1078,7 @@ static int prestera_port_vlans_del(struct prestera_port *port, ...@@ -1078,7 +1078,7 @@ static int prestera_port_vlans_del(struct prestera_port *port,
return 0; return 0;
} }
static int prestera_port_obj_del(struct net_device *dev, static int prestera_port_obj_del(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj) const struct switchdev_obj *obj)
{ {
struct prestera_port *port = netdev_priv(dev); struct prestera_port *port = netdev_priv(dev);
......
drivers/net/ethernet/mellanox/mlx5/core/en/rep/bridge.c
View file @ 69bfac96
...@@ -76,6 +76,7 @@ static int mlx5_esw_bridge_switchdev_port_event(struct notifier_block *nb, ...@@ -76,6 +76,7 @@ static int mlx5_esw_bridge_switchdev_port_event(struct notifier_block *nb,
} }
static int mlx5_esw_bridge_port_obj_add(struct net_device *dev, static int mlx5_esw_bridge_port_obj_add(struct net_device *dev,
const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -107,6 +108,7 @@ static int mlx5_esw_bridge_port_obj_add(struct net_device *dev, ...@@ -107,6 +108,7 @@ static int mlx5_esw_bridge_port_obj_add(struct net_device *dev,
} }
static int mlx5_esw_bridge_port_obj_del(struct net_device *dev, static int mlx5_esw_bridge_port_obj_del(struct net_device *dev,
const void *ctx,
const struct switchdev_obj *obj) const struct switchdev_obj *obj)
{ {
const struct switchdev_obj_port_vlan *vlan; const struct switchdev_obj_port_vlan *vlan;
...@@ -136,6 +138,7 @@ static int mlx5_esw_bridge_port_obj_del(struct net_device *dev, ...@@ -136,6 +138,7 @@ static int mlx5_esw_bridge_port_obj_del(struct net_device *dev,
} }
static int mlx5_esw_bridge_port_obj_attr_set(struct net_device *dev, static int mlx5_esw_bridge_port_obj_attr_set(struct net_device *dev,
const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
......
drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c
View file @ 69bfac96
...@@ -898,7 +898,7 @@ mlxsw_sp_port_attr_br_mrouter_set(struct mlxsw_sp_port *mlxsw_sp_port, ...@@ -898,7 +898,7 @@ mlxsw_sp_port_attr_br_mrouter_set(struct mlxsw_sp_port *mlxsw_sp_port,
return 0; return 0;
} }
static int mlxsw_sp_port_attr_set(struct net_device *dev, static int mlxsw_sp_port_attr_set(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -1766,7 +1766,7 @@ mlxsw_sp_port_mrouter_update_mdb(struct mlxsw_sp_port *mlxsw_sp_port, ...@@ -1766,7 +1766,7 @@ mlxsw_sp_port_mrouter_update_mdb(struct mlxsw_sp_port *mlxsw_sp_port,
} }
} }
static int mlxsw_sp_port_obj_add(struct net_device *dev, static int mlxsw_sp_port_obj_add(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -1916,7 +1916,7 @@ mlxsw_sp_bridge_port_mdb_flush(struct mlxsw_sp_port *mlxsw_sp_port, ...@@ -1916,7 +1916,7 @@ mlxsw_sp_bridge_port_mdb_flush(struct mlxsw_sp_port *mlxsw_sp_port,
} }
} }
static int mlxsw_sp_port_obj_del(struct net_device *dev, static int mlxsw_sp_port_obj_del(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj) const struct switchdev_obj *obj)
{ {
struct mlxsw_sp_port *mlxsw_sp_port = netdev_priv(dev); struct mlxsw_sp_port *mlxsw_sp_port = netdev_priv(dev);
......
drivers/net/ethernet/microchip/sparx5/sparx5_switchdev.c
View file @ 69bfac96
...@@ -65,7 +65,7 @@ static void sparx5_port_attr_ageing_set(struct sparx5_port *port, ...@@ -65,7 +65,7 @@ static void sparx5_port_attr_ageing_set(struct sparx5_port *port,
sparx5_set_ageing(port->sparx5, ageing_time); sparx5_set_ageing(port->sparx5, ageing_time);
} }
static int sparx5_port_attr_set(struct net_device *dev, static int sparx5_port_attr_set(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
......
drivers/net/ethernet/mscc/ocelot_net.c
View file @ 69bfac96
...@@ -939,7 +939,7 @@ static void ocelot_port_attr_mc_set(struct ocelot *ocelot, int port, bool mc) ...@@ -939,7 +939,7 @@ static void ocelot_port_attr_mc_set(struct ocelot *ocelot, int port, bool mc)
ANA_PORT_CPU_FWD_CFG, port); ANA_PORT_CPU_FWD_CFG, port);
} }
static int ocelot_port_attr_set(struct net_device *dev, static int ocelot_port_attr_set(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -1058,7 +1058,7 @@ ocelot_port_obj_mrp_del_ring_role(struct net_device *dev, ...@@ -1058,7 +1058,7 @@ ocelot_port_obj_mrp_del_ring_role(struct net_device *dev,
return ocelot_mrp_del_ring_role(ocelot, port, mrp); return ocelot_mrp_del_ring_role(ocelot, port, mrp);
} }
static int ocelot_port_obj_add(struct net_device *dev, static int ocelot_port_obj_add(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -1086,7 +1086,7 @@ static int ocelot_port_obj_add(struct net_device *dev, ...@@ -1086,7 +1086,7 @@ static int ocelot_port_obj_add(struct net_device *dev,
return ret; return ret;
} }
static int ocelot_port_obj_del(struct net_device *dev, static int ocelot_port_obj_del(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj) const struct switchdev_obj *obj)
{ {
int ret = 0; int ret = 0;
......
drivers/net/ethernet/ti/am65-cpsw-switchdev.c
View file @ 69bfac96
...@@ -84,7 +84,7 @@ static int am65_cpsw_port_attr_br_flags_pre_set(struct net_device *netdev, ...@@ -84,7 +84,7 @@ static int am65_cpsw_port_attr_br_flags_pre_set(struct net_device *netdev,
return 0; return 0;
} }
static int am65_cpsw_port_attr_set(struct net_device *ndev, static int am65_cpsw_port_attr_set(struct net_device *ndev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -302,7 +302,7 @@ static int am65_cpsw_port_mdb_del(struct am65_cpsw_port *port, ...@@ -302,7 +302,7 @@ static int am65_cpsw_port_mdb_del(struct am65_cpsw_port *port,
return 0; return 0;
} }
static int am65_cpsw_port_obj_add(struct net_device *ndev, static int am65_cpsw_port_obj_add(struct net_device *ndev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -329,7 +329,7 @@ static int am65_cpsw_port_obj_add(struct net_device *ndev, ...@@ -329,7 +329,7 @@ static int am65_cpsw_port_obj_add(struct net_device *ndev,
return err; return err;
} }
static int am65_cpsw_port_obj_del(struct net_device *ndev, static int am65_cpsw_port_obj_del(struct net_device *ndev, const void *ctx,
const struct switchdev_obj *obj) const struct switchdev_obj *obj)
{ {
struct switchdev_obj_port_vlan *vlan = SWITCHDEV_OBJ_PORT_VLAN(obj); struct switchdev_obj_port_vlan *vlan = SWITCHDEV_OBJ_PORT_VLAN(obj);
......
drivers/net/ethernet/ti/cpsw_switchdev.c
View file @ 69bfac96
...@@ -86,7 +86,7 @@ static int cpsw_port_attr_br_flags_pre_set(struct net_device *netdev, ...@@ -86,7 +86,7 @@ static int cpsw_port_attr_br_flags_pre_set(struct net_device *netdev,
return 0; return 0;
} }
static int cpsw_port_attr_set(struct net_device *ndev, static int cpsw_port_attr_set(struct net_device *ndev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -310,7 +310,7 @@ static int cpsw_port_mdb_del(struct cpsw_priv *priv, ...@@ -310,7 +310,7 @@ static int cpsw_port_mdb_del(struct cpsw_priv *priv,
return err; return err;
} }
static int cpsw_port_obj_add(struct net_device *ndev, static int cpsw_port_obj_add(struct net_device *ndev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -338,7 +338,7 @@ static int cpsw_port_obj_add(struct net_device *ndev, ...@@ -338,7 +338,7 @@ static int cpsw_port_obj_add(struct net_device *ndev,
return err; return err;
} }
static int cpsw_port_obj_del(struct net_device *ndev, static int cpsw_port_obj_del(struct net_device *ndev, const void *ctx,
const struct switchdev_obj *obj) const struct switchdev_obj *obj)
{ {
struct switchdev_obj_port_vlan *vlan = SWITCHDEV_OBJ_PORT_VLAN(obj); struct switchdev_obj_port_vlan *vlan = SWITCHDEV_OBJ_PORT_VLAN(obj);
......
include/net/switchdev.h
View file @ 69bfac96
...@@ -202,6 +202,7 @@ enum switchdev_notifier_type { ...@@ -202,6 +202,7 @@ enum switchdev_notifier_type {
struct switchdev_notifier_info { struct switchdev_notifier_info {
struct net_device *dev; struct net_device *dev;
struct netlink_ext_ack *extack; struct netlink_ext_ack *extack;
const void *ctx;
}; };
struct switchdev_notifier_fdb_info { struct switchdev_notifier_fdb_info {
...@@ -268,19 +269,19 @@ void switchdev_port_fwd_mark_set(struct net_device *dev, ...@@ -268,19 +269,19 @@ void switchdev_port_fwd_mark_set(struct net_device *dev,
int switchdev_handle_port_obj_add(struct net_device *dev, int switchdev_handle_port_obj_add(struct net_device *dev,
struct switchdev_notifier_port_obj_info *port_obj_info, struct switchdev_notifier_port_obj_info *port_obj_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*add_cb)(struct net_device *dev, int (*add_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack)); struct netlink_ext_ack *extack));
int switchdev_handle_port_obj_del(struct net_device *dev, int switchdev_handle_port_obj_del(struct net_device *dev,
struct switchdev_notifier_port_obj_info *port_obj_info, struct switchdev_notifier_port_obj_info *port_obj_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*del_cb)(struct net_device *dev, int (*del_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj)); const struct switchdev_obj *obj));
int switchdev_handle_port_attr_set(struct net_device *dev, int switchdev_handle_port_attr_set(struct net_device *dev,
struct switchdev_notifier_port_attr_info *port_attr_info, struct switchdev_notifier_port_attr_info *port_attr_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*set_cb)(struct net_device *dev, int (*set_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack)); struct netlink_ext_ack *extack));
#else #else
...@@ -352,7 +353,7 @@ static inline int ...@@ -352,7 +353,7 @@ static inline int
switchdev_handle_port_obj_add(struct net_device *dev, switchdev_handle_port_obj_add(struct net_device *dev,
struct switchdev_notifier_port_obj_info *port_obj_info, struct switchdev_notifier_port_obj_info *port_obj_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*add_cb)(struct net_device *dev, int (*add_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack)) struct netlink_ext_ack *extack))
{ {
...@@ -363,7 +364,7 @@ static inline int ...@@ -363,7 +364,7 @@ static inline int
switchdev_handle_port_obj_del(struct net_device *dev, switchdev_handle_port_obj_del(struct net_device *dev,
struct switchdev_notifier_port_obj_info *port_obj_info, struct switchdev_notifier_port_obj_info *port_obj_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*del_cb)(struct net_device *dev, int (*del_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj)) const struct switchdev_obj *obj))
{ {
return 0; return 0;
...@@ -373,7 +374,7 @@ static inline int ...@@ -373,7 +374,7 @@ static inline int
switchdev_handle_port_attr_set(struct net_device *dev, switchdev_handle_port_attr_set(struct net_device *dev,
struct switchdev_notifier_port_attr_info *port_attr_info, struct switchdev_notifier_port_attr_info *port_attr_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*set_cb)(struct net_device *dev, int (*set_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack)) struct netlink_ext_ack *extack))
{ {
......
net/dsa/slave.c
View file @ 69bfac96
...@@ -271,7 +271,7 @@ static int dsa_slave_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd) ...@@ -271,7 +271,7 @@ static int dsa_slave_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
return phylink_mii_ioctl(p->dp->pl, ifr, cmd); return phylink_mii_ioctl(p->dp->pl, ifr, cmd);
} }
static int dsa_slave_port_attr_set(struct net_device *dev, static int dsa_slave_port_attr_set(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -394,7 +394,7 @@ static int dsa_slave_vlan_add(struct net_device *dev, ...@@ -394,7 +394,7 @@ static int dsa_slave_vlan_add(struct net_device *dev,
return vlan_vid_add(master, htons(ETH_P_8021Q), vlan.vid); return vlan_vid_add(master, htons(ETH_P_8021Q), vlan.vid);
} }
static int dsa_slave_port_obj_add(struct net_device *dev, static int dsa_slave_port_obj_add(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack) struct netlink_ext_ack *extack)
{ {
...@@ -469,7 +469,7 @@ static int dsa_slave_vlan_del(struct net_device *dev, ...@@ -469,7 +469,7 @@ static int dsa_slave_vlan_del(struct net_device *dev,
return 0; return 0;
} }
static int dsa_slave_port_obj_del(struct net_device *dev, static int dsa_slave_port_obj_del(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj) const struct switchdev_obj *obj)
{ {
struct dsa_port *dp = dsa_slave_to_port(dev); struct dsa_port *dp = dsa_slave_to_port(dev);
......
net/switchdev/switchdev.c
View file @ 69bfac96
...@@ -381,19 +381,20 @@ EXPORT_SYMBOL_GPL(call_switchdev_blocking_notifiers); ...@@ -381,19 +381,20 @@ EXPORT_SYMBOL_GPL(call_switchdev_blocking_notifiers);
static int __switchdev_handle_port_obj_add(struct net_device *dev, static int __switchdev_handle_port_obj_add(struct net_device *dev,
struct switchdev_notifier_port_obj_info *port_obj_info, struct switchdev_notifier_port_obj_info *port_obj_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*add_cb)(struct net_device *dev, int (*add_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack)) struct netlink_ext_ack *extack))
{ {
struct switchdev_notifier_info *info = &port_obj_info->info;
struct netlink_ext_ack *extack; struct netlink_ext_ack *extack;
struct net_device *lower_dev; struct net_device *lower_dev;
struct list_head *iter; struct list_head *iter;
int err = -EOPNOTSUPP; int err = -EOPNOTSUPP;
extack = switchdev_notifier_info_to_extack(&port_obj_info->info); extack = switchdev_notifier_info_to_extack(info);
if (check_cb(dev)) { if (check_cb(dev)) {
err = add_cb(dev, port_obj_info->obj, extack); err = add_cb(dev, info->ctx, port_obj_info->obj, extack);
if (err != -EOPNOTSUPP) if (err != -EOPNOTSUPP)
port_obj_info->handled = true; port_obj_info->handled = true;
return err; return err;
...@@ -422,7 +423,7 @@ static int __switchdev_handle_port_obj_add(struct net_device *dev, ...@@ -422,7 +423,7 @@ static int __switchdev_handle_port_obj_add(struct net_device *dev,
int switchdev_handle_port_obj_add(struct net_device *dev, int switchdev_handle_port_obj_add(struct net_device *dev,
struct switchdev_notifier_port_obj_info *port_obj_info, struct switchdev_notifier_port_obj_info *port_obj_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*add_cb)(struct net_device *dev, int (*add_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj, const struct switchdev_obj *obj,
struct netlink_ext_ack *extack)) struct netlink_ext_ack *extack))
{ {
...@@ -439,15 +440,16 @@ EXPORT_SYMBOL_GPL(switchdev_handle_port_obj_add); ...@@ -439,15 +440,16 @@ EXPORT_SYMBOL_GPL(switchdev_handle_port_obj_add);
static int __switchdev_handle_port_obj_del(struct net_device *dev, static int __switchdev_handle_port_obj_del(struct net_device *dev,
struct switchdev_notifier_port_obj_info *port_obj_info, struct switchdev_notifier_port_obj_info *port_obj_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*del_cb)(struct net_device *dev, int (*del_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj)) const struct switchdev_obj *obj))
{ {
struct switchdev_notifier_info *info = &port_obj_info->info;
struct net_device *lower_dev; struct net_device *lower_dev;
struct list_head *iter; struct list_head *iter;
int err = -EOPNOTSUPP; int err = -EOPNOTSUPP;
if (check_cb(dev)) { if (check_cb(dev)) {
err = del_cb(dev, port_obj_info->obj); err = del_cb(dev, info->ctx, port_obj_info->obj);
if (err != -EOPNOTSUPP) if (err != -EOPNOTSUPP)
port_obj_info->handled = true; port_obj_info->handled = true;
return err; return err;
...@@ -476,7 +478,7 @@ static int __switchdev_handle_port_obj_del(struct net_device *dev, ...@@ -476,7 +478,7 @@ static int __switchdev_handle_port_obj_del(struct net_device *dev,
int switchdev_handle_port_obj_del(struct net_device *dev, int switchdev_handle_port_obj_del(struct net_device *dev,
struct switchdev_notifier_port_obj_info *port_obj_info, struct switchdev_notifier_port_obj_info *port_obj_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*del_cb)(struct net_device *dev, int (*del_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_obj *obj)) const struct switchdev_obj *obj))
{ {
int err; int err;
...@@ -492,19 +494,20 @@ EXPORT_SYMBOL_GPL(switchdev_handle_port_obj_del); ...@@ -492,19 +494,20 @@ EXPORT_SYMBOL_GPL(switchdev_handle_port_obj_del);
static int __switchdev_handle_port_attr_set(struct net_device *dev, static int __switchdev_handle_port_attr_set(struct net_device *dev,
struct switchdev_notifier_port_attr_info *port_attr_info, struct switchdev_notifier_port_attr_info *port_attr_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*set_cb)(struct net_device *dev, int (*set_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack)) struct netlink_ext_ack *extack))
{ {
struct switchdev_notifier_info *info = &port_attr_info->info;
struct netlink_ext_ack *extack; struct netlink_ext_ack *extack;
struct net_device *lower_dev; struct net_device *lower_dev;
struct list_head *iter; struct list_head *iter;
int err = -EOPNOTSUPP; int err = -EOPNOTSUPP;
extack = switchdev_notifier_info_to_extack(&port_attr_info->info); extack = switchdev_notifier_info_to_extack(info);
if (check_cb(dev)) { if (check_cb(dev)) {
err = set_cb(dev, port_attr_info->attr, extack); err = set_cb(dev, info->ctx, port_attr_info->attr, extack);
if (err != -EOPNOTSUPP) if (err != -EOPNOTSUPP)
port_attr_info->handled = true; port_attr_info->handled = true;
return err; return err;
...@@ -533,7 +536,7 @@ static int __switchdev_handle_port_attr_set(struct net_device *dev, ...@@ -533,7 +536,7 @@ static int __switchdev_handle_port_attr_set(struct net_device *dev,
int switchdev_handle_port_attr_set(struct net_device *dev, int switchdev_handle_port_attr_set(struct net_device *dev,
struct switchdev_notifier_port_attr_info *port_attr_info, struct switchdev_notifier_port_attr_info *port_attr_info,
bool (*check_cb)(const struct net_device *dev), bool (*check_cb)(const struct net_device *dev),
int (*set_cb)(struct net_device *dev, int (*set_cb)(struct net_device *dev, const void *ctx,
const struct switchdev_attr *attr, const struct switchdev_attr *attr,
struct netlink_ext_ack *extack)) struct netlink_ext_ack *extack))
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7