[PATCH 1/1] x86/hyper-V: Allocate the IDT entry early in boot

Ingo Molnar mingo at kernel.org
Wed Sep 13 09:00:56 UTC 2017


* KY Srinivasan <kys at microsoft.com> wrote:

> 
> 
> > -----Original Message-----
> > From: Dan Carpenter [mailto:dan.carpenter at oracle.com]
> > Sent: Saturday, September 9, 2017 4:04 AM
> > To: KY Srinivasan <kys at microsoft.com>
> > Cc: x86 at kernel.org; gregkh at linuxfoundation.org; linux-
> > kernel at vger.kernel.org; devel at linuxdriverproject.org; olaf at aepfle.de;
> > apw at canonical.com; jasowang at redhat.com; tglx at linutronix.de;
> > hpa at zytor.com; mingo at kernel.org
> > Subject: Re: [PATCH 1/1] x86/hyper-V: Allocate the IDT entry early in boot
> > 
> > On Fri, Sep 08, 2017 at 04:15:57PM -0700, kys at exchange.microsoft.com
> > wrote:
> > > From: "K. Y. Srinivasan" <kys at microsoft.com>
> > >
> > > Allocate the hypervisor callback IDT entry early in the boot sequence.
> > >
> > 
> > I'm guessing this fixes a NULL dereference or something?  The changelog
> > doesn't really say why we are doing this.
> 
> The changelog does say what we are doing - allocating the IDT entry early in the boot sequence.

But the question was the 'why', not the 'what' - so Dan's question is
fully justified ...

> The current code would allocate the entry as part of registering the handler 
> when vmbus driver loaded and this caused a problem for the cleanup Thomas had 
> implemented.

I've put this explanation into the changelog.

Thanks,

	Ingo


More information about the devel mailing list