[Xen-changelog] [xen master] x86/traps: Misc non-functional cleanup

[p***@xen.org]

commit 9ed942e23e11dffea1ec2a63662d6f13a61a5778
Author: Andrew Cooper <***@citrix.com>
AuthorDate: Mon Nov 5 16:03:03 2018 +0000
Commit: Andrew Cooper <***@citrix.com>
CommitDate: Mon Nov 5 17:07:11 2018 +0000

x86/traps: Misc non-functional cleanup

* s/unsigned char/uint8_t/ for clarity
* Drop redundant return at the end of a void function

Signed-off-by: Andrew Cooper <***@citrix.com>
Reviewed-by: Wei Liu <***@citrix.com>
Acked-by: Jan Beulich <***@suse.com>
---
xen/arch/x86/traps.c | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/xen/arch/x86/traps.c b/xen/arch/x86/traps.c
index 7765f9a5d7..9471d89022 100644
--- a/xen/arch/x86/traps.c
+++ b/xen/arch/x86/traps.c
@@ -1537,9 +1537,9 @@ void do_general_protection(struct cpu_user_regs *regs)
if ( regs->error_code & X86_XEC_IDT )
{
/* This fault must be due to <INT n> instruction. */
- const struct trap_info *ti;
- unsigned char vector = regs->error_code >> 3;
- ti = &v->arch.pv.trap_ctxt[vector];
+ uint8_t vector = regs->error_code >> 3;
+ const struct trap_info *ti = &v->arch.pv.trap_ctxt[vector];
+
if ( permit_softint(TI_GET_DPL(ti), v, regs) )
{
regs->rip += 2;
@@ -1785,8 +1785,6 @@ void do_device_not_available(struct cpu_user_regs *regs)
#else
ASSERT_UNREACHABLE();
#endif
-
- return;
}

void do_debug(struct cpu_user_regs *regs)
--
generated by git-patchbot for /home/xen/git/xen.git#master