diff options
author | Andrew Cooper <andrew.cooper3@citrix.com> | 2013-10-09 12:10:46 +0200 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2013-10-09 12:10:46 +0200 |
commit | 839b966e3f587bbb1a0d954230fb3904330dccb6 (patch) | |
tree | 8e6fbbba127c1624d27f2be469a78584e37f6e99 /xen/common | |
parent | c4d3ccbfb50fa7603595b9bc16b1bba4647d21f1 (diff) | |
download | xen-839b966e3f587bbb1a0d954230fb3904330dccb6.tar.gz xen-839b966e3f587bbb1a0d954230fb3904330dccb6.tar.bz2 xen-839b966e3f587bbb1a0d954230fb3904330dccb6.zip |
x86/traps: improvements to {rd,wr}msr_hypervisor_regs()
Coverity ID: 1055249 1055250
Coverity was complaining that the switch statments contained dead code in
their default statements. While this is quite minor, the code flow in
wrmsr_hypervisor_regs() was sufficiently opaque that I felt it approprate to
fix.
Other improvements include:
* not shadowing the function parameter 'idx'.
* use of PAGE_{SHIFT,SIZE} instead of opencoded numbers.
* a more descriptive error message for attempting to write invalid indicies
for hypercall pages.
There is no behavioural change as a result.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Diffstat (limited to 'xen/common')
0 files changed, 0 insertions, 0 deletions