summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNathan Fontenot <nfont@austin.ibm.com>2008-07-26 04:27:06 +1000
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>2008-07-30 15:26:53 +1000
commit9ee07f91a1fab61ff0d8d25be43351a049c0a821 (patch)
tree625807bcb9bb3f24e4dd50c272d43aa658635c54
parent11c675cef2fbe471dc6103a89b156e65c3630f3a (diff)
downloadop-kernel-dev-9ee07f91a1fab61ff0d8d25be43351a049c0a821.zip
op-kernel-dev-9ee07f91a1fab61ff0d8d25be43351a049c0a821.tar.gz
powerpc: Allow non-hcall return values for lparcfg writes
The code to handle writes to /proc/ppc64/lparcfg incorrectly assumes that the return code from the helper routines to update processor or memory entitlement return a hcall return value. It then assumes any non-hcall return value is bad and sets the return code for the write to be -EIO. The update_[mp]pp routines can return values other than a hcall return value. This patch removes the automatic setting of any return code that is not an hcall return value from these routines to -EIO. Signed-off-by: Nathan Fontenot <nfont@austin.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
-rw-r--r--arch/powerpc/kernel/lparcfg.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/arch/powerpc/kernel/lparcfg.c b/arch/powerpc/kernel/lparcfg.c
index 9f856a0..1a09719 100644
--- a/arch/powerpc/kernel/lparcfg.c
+++ b/arch/powerpc/kernel/lparcfg.c
@@ -636,10 +636,6 @@ static ssize_t lparcfg_write(struct file *file, const char __user * buf,
retval = -EIO;
} else if (retval == H_PARAMETER) {
retval = -EINVAL;
- } else {
- printk(KERN_WARNING "%s: received unknown hv return code %ld",
- __func__, retval);
- retval = -EIO;
}
return retval;
OpenPOWER on IntegriCloud