Skip to content

Commit

Permalink
macsec: let the administrator set UP state even if lowerdev is down
Browse files Browse the repository at this point in the history
Currently, the kernel doesn't let the administrator set a macsec device
up unless its lower device is currently up. This is inconsistent, as a
macsec device that is up won't automatically go down when its lower
device goes down.

Now that linkstate propagation works, there's really no reason for this
limitation, so let's remove it.

Fixes: c09440f ("macsec: introduce IEEE 802.1AE driver")
Reported-by: Radu Rendec <[email protected]>
Signed-off-by: Sabrina Dubroca <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
qsn authored and davem330 committed Oct 29, 2018
1 parent e6ac075 commit 07bddef
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions drivers/net/macsec.c
Original file line number Diff line number Diff line change
Expand Up @@ -2812,9 +2812,6 @@ static int macsec_dev_open(struct net_device *dev)
struct net_device *real_dev = macsec->real_dev;
int err;

if (!(real_dev->flags & IFF_UP))
return -ENETDOWN;

err = dev_uc_add(real_dev, dev->dev_addr);
if (err < 0)
return err;
Expand Down

0 comments on commit 07bddef

Please sign in to comment.