Regression with 5dcd8400884c ("macsec: missing dev_put() on error in macsec_newlink()")

[Laura Abbott]

Hi,

Fedora got a bug report of a regression when trying to remove the
the macsec module (https://bugzilla.redhat.com/show_bug.cgi?id=1566410).
I did a bisect and found

commit 5dcd8400884cc4a043a6d4617e042489e5d566a9
Author: Dan Carpenter <dan.carpen...@oracle.com>
Date:   Wed Mar 21 11:09:01 2018 +0300

    macsec: missing dev_put() on error in macsec_newlink()

    to update the error paths.

    Signed-off-by: Dan Carpenter <dan.carpen...@oracle.com>
    Signed-off-by: David S. Miller <da...@davemloft.net>

The script I used for testing based on the reporter is attached. It
looks like modprobe is stuck in the D state. Any idea?

Thanks,
Laura

We moved the dev_hold(real_dev); call earlier in the function but forgot Fixes: 0759e552bce7 ("macsec: fix negative refcnt on parent link")

mac-sec-setup.sh
Description: application/shellscript