diff options
author | Phil Oester <kernel@linuxace.com> | 2011-03-14 06:22:06 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-03-15 19:29:39 -0700 |
commit | 5f86cad1e8224af9e3b9b43dd84b146a9ff0df87 (patch) | |
tree | 45c7140cd8651fa51a285a4596539a3d56688af1 | |
parent | e826eafa65c6f1f7c8db5a237556cebac57ebcc5 (diff) | |
download | op-kernel-dev-5f86cad1e8224af9e3b9b43dd84b146a9ff0df87.zip op-kernel-dev-5f86cad1e8224af9e3b9b43dd84b146a9ff0df87.tar.gz |
bonding: Improve syslog message at device creation time
When the bonding module is loaded, it creates bond0 by default.
Then, when attempting to create bond0, the following messages
are printed to syslog:
kernel: bonding: bond0 is being created...
kernel: bonding: Bond creation failed.
Which seems to indicate a problem, when in reality there is no
problem. Since the actual error code is passed down from bond_create,
make use of it to print a bit less ominous message:
kernel: bonding: bond0 is being created...
kernel: bond0 already exists.
Signed-off-by: Phil Oester <kernel@linuxace.com>
Signed-off-by: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/bonding/bond_sysfs.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_sysfs.c b/drivers/net/bonding/bond_sysfs.c index 72bb0f6..e718144 100644 --- a/drivers/net/bonding/bond_sysfs.c +++ b/drivers/net/bonding/bond_sysfs.c @@ -118,7 +118,10 @@ static ssize_t bonding_store_bonds(struct class *cls, pr_info("%s is being created...\n", ifname); rv = bond_create(net, ifname); if (rv) { - pr_info("Bond creation failed.\n"); + if (rv == -EEXIST) + pr_info("%s already exists.\n", ifname); + else + pr_info("%s creation failed.\n", ifname); res = rv; } } else if (command[0] == '-') { |