diff options
| author | Elad Raz | 2016-10-23 17:43:05 +0200 |
|---|---|---|
| committer | David S. Miller | 2016-10-26 23:30:32 +0200 |
| commit | 6edf10173a1feb1078f2fc8c655baf9614e83493 (patch) | |
| tree | b8b023d22324234adc9827335650404d79995774 /net/core/devlink.c | |
| parent | firewire: net: set initial MTU = 1500 unconditionally, fix IPv6 on some CardB... (diff) | |
| download | kernel-qcow2-linux-6edf10173a1feb1078f2fc8c655baf9614e83493.tar.gz kernel-qcow2-linux-6edf10173a1feb1078f2fc8c655baf9614e83493.tar.xz kernel-qcow2-linux-6edf10173a1feb1078f2fc8c655baf9614e83493.zip | |
devlink: Prevent port_type_set() callback when it's not needed
When a port_type_set() is been called and the new port type set is the same
as the old one, just return success.
Signed-off-by: Elad Raz <eladr@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/devlink.c')
| -rw-r--r-- | net/core/devlink.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/core/devlink.c b/net/core/devlink.c index 1b5063088f1a..d2fd736de6a2 100644 --- a/net/core/devlink.c +++ b/net/core/devlink.c @@ -608,6 +608,8 @@ static int devlink_port_type_set(struct devlink *devlink, if (devlink->ops && devlink->ops->port_type_set) { if (port_type == DEVLINK_PORT_TYPE_NOTSET) return -EINVAL; + if (port_type == devlink_port->type) + return 0; err = devlink->ops->port_type_set(devlink_port, port_type); if (err) return err; |