diff options
author | Nathan Fontenot <nfont@austin.ibm.com> | 2008-07-25 14:27:06 -0400 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2008-07-30 01:26:53 -0400 |
commit | 9ee07f91a1fab61ff0d8d25be43351a049c0a821 (patch) | |
tree | 625807bcb9bb3f24e4dd50c272d43aa658635c54 | |
parent | 11c675cef2fbe471dc6103a89b156e65c3630f3a (diff) |
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.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/arch/powerpc/kernel/lparcfg.c b/arch/powerpc/kernel/lparcfg.c index 9f856a0c3e38..1a09719c7628 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, | |||
636 | retval = -EIO; | 636 | retval = -EIO; |
637 | } else if (retval == H_PARAMETER) { | 637 | } else if (retval == H_PARAMETER) { |
638 | retval = -EINVAL; | 638 | retval = -EINVAL; |
639 | } else { | ||
640 | printk(KERN_WARNING "%s: received unknown hv return code %ld", | ||
641 | __func__, retval); | ||
642 | retval = -EIO; | ||
643 | } | 639 | } |
644 | 640 | ||
645 | return retval; | 641 | return retval; |