Date: Mon, 31 May 2021 20:48:32 -0400 From: Mark Johnston <markj@freebsd.org> To: Jessica Clarke <jrtc27@freebsd.org> Cc: "src-committers@freebsd.org" <src-committers@freebsd.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@freebsd.org>, "dev-commits-src-main@freebsd.org" <dev-commits-src-main@freebsd.org> Subject: Re: git: 18f55c67f746 - main - x86: Fix lapic_ipi_alloc() on i386 Message-ID: <YLWD4L0UpKsocEKr@nuc> In-Reply-To: <D8BF2E9F-5309-42FB-8A21-11F298DE36EC@freebsd.org> References: <202105312309.14VN9nWf040168@gitrepo.freebsd.org> <D8BF2E9F-5309-42FB-8A21-11F298DE36EC@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jun 01, 2021 at 12:48:52AM +0100, Jessica Clarke wrote: > On 1 Jun 2021, at 00:09, Mark Johnston <markj@FreeBSD.org> wrote: > > > > The branch main has been updated by markj: > > > > URL: https://cgit.FreeBSD.org/src/commit/?id=18f55c67f746f0ad12fe972328234d340a621df9 > > > > commit 18f55c67f746f0ad12fe972328234d340a621df9 > > Author: Mark Johnston <markj@FreeBSD.org> > > AuthorDate: 2021-05-31 22:51:14 +0000 > > Commit: Mark Johnston <markj@FreeBSD.org> > > 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 <freebsd@omnilan.de> > > Tested by: Harry Schmalzbauer <freebsd@omnilan.de> > > 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? Yes, either of those seems more correct. See https://reviews.freebsd.org/D30590
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YLWD4L0UpKsocEKr>