diff options
author | Thomas Klein <osstklei@de.ibm.com> | 2007-01-22 06:54:20 -0500 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-01-23 00:59:21 -0500 |
commit | 7674a588e93c6fa1fde8e452a4c025c49037cb96 (patch) | |
tree | ec753e5bb666e5c856b6cccc9fc8845584eacc36 /drivers/net/ehea/ehea_main.c | |
parent | 4e996b32e0585756c335b35980dc68852c33f297 (diff) |
ehea: Improved logging of permission issues
Disabled dump of hcall regs on some permission issues and
fixed appropriate misleading logmessages
Signed-off-by: Thomas Klein <tklein@de.ibm.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net/ehea/ehea_main.c')
-rw-r--r-- | drivers/net/ehea/ehea_main.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/drivers/net/ehea/ehea_main.c b/drivers/net/ehea/ehea_main.c index 9486e0a8ec8b..8281b30da32d 100644 --- a/drivers/net/ehea/ehea_main.c +++ b/drivers/net/ehea/ehea_main.c | |||
@@ -730,10 +730,7 @@ int ehea_set_portspeed(struct ehea_port *port, u32 port_speed) | |||
730 | } | 730 | } |
731 | } else { | 731 | } else { |
732 | if (hret == H_AUTHORITY) { | 732 | if (hret == H_AUTHORITY) { |
733 | ehea_info("Hypervisor denied setting port speed. Either" | 733 | ehea_info("Hypervisor denied setting port speed"); |
734 | " this partition is not authorized to set " | ||
735 | "port speed or another partition has modified" | ||
736 | " port speed first."); | ||
737 | ret = -EPERM; | 734 | ret = -EPERM; |
738 | } else { | 735 | } else { |
739 | ret = -EIO; | 736 | ret = -EIO; |
@@ -1487,11 +1484,12 @@ out: | |||
1487 | 1484 | ||
1488 | static void ehea_promiscuous_error(u64 hret, int enable) | 1485 | static void ehea_promiscuous_error(u64 hret, int enable) |
1489 | { | 1486 | { |
1490 | ehea_info("Hypervisor denied %sabling promiscuous mode.%s", | 1487 | if (hret == H_AUTHORITY) |
1491 | enable == 1 ? "en" : "dis", | 1488 | ehea_info("Hypervisor denied %sabling promiscuous mode", |
1492 | hret != H_AUTHORITY ? "" : " Another partition owning a " | 1489 | enable == 1 ? "en" : "dis"); |
1493 | "logical port on the same physical port might have altered " | 1490 | else |
1494 | "promiscuous mode first."); | 1491 | ehea_error("failed %sabling promiscuous mode", |
1492 | enable == 1 ? "en" : "dis"); | ||
1495 | } | 1493 | } |
1496 | 1494 | ||
1497 | static void ehea_promiscuous(struct net_device *dev, int enable) | 1495 | static void ehea_promiscuous(struct net_device *dev, int enable) |