xen/intr: merge parts of resume functionality into new function

[ehem_freebsd_m5p.com]

Summary

The portions of xen_rebind_ipi() and xen_rebind_virq() were already
near-identical. While xen_rebind_ipi() should panic() on
single-processor, still having the functionality to invoke seems
harmless.

Meanwhile much of the loop from xen_intr_resume() seemed to want to be
closer to this same code. This pushes related bits closer together.

[ehem_freebsd_m5p.com]

Fixing a bug which was found while glancing at this delta.

[ehem_freebsd_m5p.com]

Removing one line which really should have been removed before.

[ehem_freebsd_m5p.com]

Move the isrc->xi_cpu = 0; line lower, hopefully hinting this is part of the xen_intr_assign_cpu() call.

[ehem_freebsd_m5p.com]

Small update, keep panic strings closer to original, but add function name. Should make diagnosis easier if this ever gets triggered.

[ehem_freebsd_m5p.com]

Fix a potential unused variable warning. Single-processor systems almost seem archaic, but we should still function on such.

[ehem_freebsd_m5p.com]

Remove a bit more unused in case of uniprocessor code from uniprocessor builds. Such are actually rather more sensible in case of virtual machines...

[ehem_freebsd_m5p.com]

Ugh! Hopefully finish the hunt for bits which disappear if uniprocessor.

[ehem_freebsd_m5p.com]

Fixing issue discovered during review. The blanket wipe of ->xi_port in xen_intr_resume() can't be removed until D30599.

[ehem_freebsd_m5p.com]

I'm wondering if I should merge D30599 into D30598. D30598 is simply a precursor to implementing D30599.

[ehem_freebsd_m5p.com]

Another signed/unsigned update, sorry.

[ehem_freebsd_m5p.com]

Another small update for the sign auditing.

[ehem_freebsd_m5p.com]

sys/x86/xen/xen_intr.c

744–745

Based on what I saw in D32866, I suspect there should be a intr_event_bind() here.

[ehem_freebsd_m5p.com]

Updating to ensure consistency with what I've got. Should still apply, but updates on D30743 have made application unclean.

sys/x86/xen/xen_intr.c

#ifdef KDB
         if (debugger_on_trap) {
                 kdb_why = KDB_WHY_TRAP;
                 handled = kdb_trap(frame->tf_scause & SCAUSE_CODE, 0, frame);
                 kdb_why = KDB_WHY_UNSET;
                 if (handled)
                         return;
         }
 #endif
-        panic("Fatal page fault at %#lx: %#016lx", frame->tf_sepc, stval);
+        panic("Fatal page fault at %#lx: %#lx", frame->tf_sepc, stval);
 }
 
 void
 do_trap_supervisor(struct trapframe *frame)
 {
         uint64_t exception;
 
         /* Ensure we came from supervisor mode, interrupts disabled */
if ((frame->tf_scause & SCAUSE_INTR) != 0) {
                 return;
         }
 
 #ifdef KDTRACE_HOOKS
         if (dtrace_trap_func != NULL && (*dtrace_trap_func)(frame, exception))
                 return;
 #endif
 
-        CTR4(KTR_TRAP, "%s: exception=%lu, sepc=%lx, stval=%lx", __func__,
+        CTR4(KTR_TRAP, "%s: exception=%lu, sepc=%#lx, stval=%#lx", __func__,
             exception, frame->tf_sepc, frame->tf_stval);
 
         switch (exception) {
         case SCAUSE_LOAD_ACCESS_FAULT:
         case SCAUSE_STORE_ACCESS_FAULT:
         case SCAUSE_INST_ACCESS_FAULT:
                 dump_regs(frame);
-                panic("Memory access exception at 0x%016lx\n", frame->tf_sepc);
+                panic("Memory access exception at %#lx: %#lx",
+                    frame->tf_sepc, frame->tf_stval);
                 break;
         case SCAUSE_LOAD_MISALIGNED:
         case SCAUSE_STORE_MISALIGNED:
         case SCAUSE_INST_MISALIGNED:
                 dump_regs(frame);
-                panic("Misaligned address exception at %#016lx: %#016lx\n",
+                panic("Misaligned address exception at %#lx: %#lx",
                     frame->tf_sepc, frame->tf_stval);
                 break;
         case SCAUSE_STORE_PAGE_FAULT:
         case SCAUSE_LOAD_PAGE_FAULT:
         case SCAUSE_INST_PAGE_FAULT:
                 page_fault_handler(frame, 0);
                 break;
         case SCAUSE_BREAKPOINT:
 #ifdef KDTRACE_HOOKS
                 if (dtrace_invop_jump_addr != NULL &&
                     dtrace_invop_jump_addr(frame) == 0)
                                 break;
 #endif
 #ifdef KDB
                 kdb_trap(exception, 0, frame);
 #else
                 dump_regs(frame);
-                panic("No debugger in kernel.\n");
+                panic("No debugger in kernel.");
 #endif
                 break;
         case SCAUSE_ILLEGAL_INSTRUCTION:
                 dump_regs(frame);
-                panic("Illegal instruction at 0x%016lx\n", frame->tf_sepc);
+                panic("Illegal instruction %#lx at %#lx", frame->tf_stval,
+                    frame->tf_sepc);
                 break;
         default:
                 dump_regs(frame);
-                panic("Unknown kernel exception %lx trap value %lx\n",
+                panic("Unknown kernel exception %#lx trap value %#lx",
                     exception, frame->tf_stval);
         }
 }
 
 void
 do_trap_user(struct trapframe *frame)
 {
         uint64_t exception;
do_trap_user(struct trapframe *frame)
         exception = frame->tf_scause & SCAUSE_CODE;
         if ((frame->tf_scause & SCAUSE_INTR) != 0) {
                 /* Interrupt */
                 riscv_cpu_intr(frame);
                 return;
         }
         intr_enable();
 
-        CTR4(KTR_TRAP, "%s: exception=%lu, sepc=%lx, stval=%lx", __func__,
+        CTR4(KTR_TRAP, "%s: exception=%lu, sepc=%#lx, stval=%#lx", __func__,
             exception, frame->tf_sepc, frame->tf_stval);
 
         switch (exception) {
         case SCAUSE_LOAD_ACCESS_FAULT:
         case SCAUSE_STORE_ACCESS_FAULT:
         case SCAUSE_INST_ACCESS_FAULT:
                 call_trapsignal(td, SIGBUS, BUS_ADRERR, (void *)frame->tf_sepc,
                     exception);
case SCAUSE_ILLEGAL_INSTRUCTION:
                 break;
         case SCAUSE_BREAKPOINT:
                 call_trapsignal(td, SIGTRAP, TRAP_BRKPT, (void *)frame->tf_sepc,
                     exception);
                 userret(td, frame);
                 break;
         default:
                 dump_regs(frame);
-                panic("Unknown userland exception %lx, trap value %lx\n",
+                panic("Unknown userland exception %#lx, trap value %#lx",
                     exception, frame->tf_stval);
         }
 }