diff options
author | H. Peter Anvin <hpa@zytor.com> | 2009-08-31 14:23:29 -0700 |
---|---|---|
committer | H. Peter Anvin <hpa@zytor.com> | 2009-08-31 15:15:23 -0700 |
commit | 0cc0213e73af5963eca259c84876937c20689dbd (patch) | |
tree | 2e3db62cc241269fbb3ee3313e568239cf4c9bd0 /arch/x86/xen | |
parent | 79c5dca3619d6ae15815eec14cd7a43db5f38b47 (diff) | |
download | op-kernel-dev-0cc0213e73af5963eca259c84876937c20689dbd.zip op-kernel-dev-0cc0213e73af5963eca259c84876937c20689dbd.tar.gz |
x86, msr: Have the _safe MSR functions return -EIO, not -EFAULT
For some reason, the _safe MSR functions returned -EFAULT, not -EIO.
However, the only user which cares about the return code as anything
other than a boolean is the MSR driver, which wants -EIO. Change it
to -EIO across the board.
Signed-off-by: H. Peter Anvin <hpa@zytor.com>
Cc: Jeremy Fitzhardinge <jeremy@xensource.com>
Cc: Chris Wright <chrisw@sous-sol.org>
Cc: Alok Kataria <akataria@vmware.com>
Cc: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'arch/x86/xen')
-rw-r--r-- | arch/x86/xen/enlighten.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c index 0a1700a..a8432d8 100644 --- a/arch/x86/xen/enlighten.c +++ b/arch/x86/xen/enlighten.c @@ -713,7 +713,7 @@ static int xen_write_msr_safe(unsigned int msr, unsigned low, unsigned high) set: base = ((u64)high << 32) | low; if (HYPERVISOR_set_segment_base(which, base) != 0) - ret = -EFAULT; + ret = -EIO; break; #endif |