powerpc: Allow non-hcall return values for lparcfg writes
authorNathan Fontenot <nfont@austin.ibm.com>
Fri, 25 Jul 2008 18:27:06 +0000 (04:27 +1000)
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>
Wed, 30 Jul 2008 05:26:53 +0000 (15:26 +1000)
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>
arch/powerpc/kernel/lparcfg.c

index 9f856a0c3e3876eb1bd6bbe1e2e2b41dba70bf16..1a09719c7628219b5f75ededaf848ec8158fa227 100644 (file)
@@ -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;