On 1 Jun 2021, at 00:09, Mark Johnston <[email protected]> wrote: > > The branch main has been updated by markj: > > URL: > https://cgit.FreeBSD.org/src/commit/?id=18f55c67f746f0ad12fe972328234d340a621df9 > > commit 18f55c67f746f0ad12fe972328234d340a621df9 > Author: Mark Johnston <[email protected]> > AuthorDate: 2021-05-31 22:51:14 +0000 > Commit: Mark Johnston <[email protected]> > CommitDate: 2021-05-31 22:51:14 +0000 > > x86: Fix lapic_ipi_alloc() on i386 > > The loop which checks to see if "dynamic" IDT entries are allocated > needs to compare with the trampoline address of the reserved ISR. > Otherwise it will never succeed. > > Reported by: Harry Schmalzbauer <[email protected]> > Tested by: Harry Schmalzbauer <[email protected]> > Reviewed by: kib > MFC after: 1 week > Sponsored by: The FreeBSD Foundation > Differential Revision: https://reviews.freebsd.org/D30576 > --- > sys/x86/x86/local_apic.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/sys/x86/x86/local_apic.c b/sys/x86/x86/local_apic.c > index 5b4910d4b891..bb575d0c601d 100644 > --- a/sys/x86/x86/local_apic.c > +++ b/sys/x86/x86/local_apic.c > @@ -2127,6 +2127,10 @@ native_lapic_ipi_vectored(u_int vector, int dest) > > #endif /* SMP */ > > +#ifdef __i386__ > +extern uintptr_t setidt_disp;
uintptr_t here is fishy. Should it not be size_t or ptrdiff_t? Jess _______________________________________________ [email protected] mailing list https://lists.freebsd.org/mailman/listinfo/dev-commits-src-all To unsubscribe, send any mail to "[email protected]"
