mlxsw: spectrum_switchdev: Pass extack to mlxsw_sp_br_ban_rif_pvid_change()

Currently the reason for rejection of PVID manipulation is dumped to
syslog, and a generic -EBUSY is returned to the userspace. But
switchdev_handle_port_obj_add(), through which we get to
mlxsw_sp_port_vlans_add(), handles extack just fine, and we can pass the
message this way.

This improves visibility into reasons why the request to change PVID
was rejected. Before the change:

 # bridge vlan add dev br vid 2 self pvid untagged
 RTNETLINK answers: Device or resource busy
 (plus a syslog line)

After the change:

 # bridge vlan add dev br vid 2 self pvid untagged
 Error: mlxsw_spectrum: Can't change PVID, it's used by router interface.

Note that this particular error message is going away in the following
patches. However the ability to pass error messages through extack will be
useful more broadly for communicating in particular reasons why a RIF
failed to be created.

Signed-off-by: Petr Machata <petrm@nvidia.com>
Reviewed-by: Danielle Ratson <danieller@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c
index d88e62b..a3365f7 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_switchdev.c
@@ -1481,7 +1481,8 @@ err_port_vlan_set:
 static int
 mlxsw_sp_br_ban_rif_pvid_change(struct mlxsw_sp *mlxsw_sp,
                                const struct net_device *br_dev,
-                               const struct switchdev_obj_port_vlan *vlan)
+                               const struct switchdev_obj_port_vlan *vlan,
+                               struct netlink_ext_ack *extack)
 {
        u16 pvid;
 
@@ -1491,12 +1492,12 @@ mlxsw_sp_br_ban_rif_pvid_change(struct mlxsw_sp *mlxsw_sp,
 
        if (vlan->flags & BRIDGE_VLAN_INFO_PVID) {
                if (vlan->vid != pvid) {
-                       netdev_err(br_dev, "Can't change PVID, it's used by router interface\n");
+                       NL_SET_ERR_MSG_MOD(extack, "Can't change PVID, it's used by router interface");
                        return -EBUSY;
                }
        } else {
                if (vlan->vid == pvid) {
-                       netdev_err(br_dev, "Can't remove PVID, it's used by router interface\n");
+                       NL_SET_ERR_MSG_MOD(extack, "Can't remove PVID, it's used by router interface");
                        return -EBUSY;
                }
        }
@@ -1519,7 +1520,8 @@ static int mlxsw_sp_port_vlans_add(struct mlxsw_sp_port *mlxsw_sp_port,
 
                if (br_vlan_enabled(orig_dev))
                        err = mlxsw_sp_br_ban_rif_pvid_change(mlxsw_sp,
-                                                             orig_dev, vlan);
+                                                             orig_dev, vlan,
+                                                             extack);
                if (!err)
                        err = -EOPNOTSUPP;
                return err;