linux-kernel - RE: [patch x86/hyperv] x86, hyperv: Fix build error

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <05744953b2404341866fad7bb08c5506@SN2PR03MB061.namprd03.prod.outlook.com>
Date:	Fri, 11 Oct 2013 23:31:45 +0000
From:	KY Srinivasan <kys@...rosoft.com>
To:	David Rientjes <rientjes@...gle.com>
CC:	Ingo Molnar <mingo@...nel.org>, "H. Peter Anvin" <hpa@...or.com>,
	"tglx@...utronix.de" <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...ux.intel.com>,
	Olaf Hering <olaf@...fle.de>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-tip-commits@...r.kernel.org" 
	<linux-tip-commits@...r.kernel.org>
Subject: RE: [patch x86/hyperv] x86, hyperv: Fix build error



> -----Original Message-----
> From: David Rientjes [mailto:rientjes@...gle.com]
> Sent: Friday, October 11, 2013 4:29 PM
> To: KY Srinivasan
> Cc: Ingo Molnar; H. Peter Anvin; tglx@...utronix.de; H. Peter Anvin; Olaf Hering;
> linux-kernel@...r.kernel.org; linux-tip-commits@...r.kernel.org
> Subject: RE: [patch x86/hyperv] x86, hyperv: Fix build error
> 
> On Fri, 11 Oct 2013, KY Srinivasan wrote:
> 
> > > From: David Rientjes [mailto:rientjes@...gle.com]
> > > Sent: Friday, October 11, 2013 4:08 PM
> > > To: Ingo Molnar; H. Peter Anvin; tglx@...utronix.de; H. Peter Anvin
> > > Cc: KY Srinivasan; Olaf Hering; linux-kernel@...r.kernel.org; linux-tip-
> > > commits@...r.kernel.org
> > > Subject: [patch x86/hyperv] x86, hyperv: Fix build error
> > >
> > > 9e7827b5ea4c ("x86, hyperv: Get the local APIC timer frequency from the
> > > hypervisor") breaks the build with some configs because apic.h isn't
> > > directly included:
> > >
> > > arch/x86/kernel/cpu/mshyperv.c: In function 'ms_hyperv_init_platform':
> > > arch/x86/kernel/cpu/mshyperv.c:90:3: error: 'lapic_timer_frequency'
> undeclared
> > > (first use in this function)
> > > arch/x86/kernel/cpu/mshyperv.c:90:3: note: each undeclared identifier is
> > > reported only once for each function it appears in
> > >
> > > Fix it by including asm/apic.h.
> >
> > Thank you. The issue was configuration related - local APIC was not configured.
> This has already been fixed.
> >
> 
> Yeah, it's config related as stated in the changelog.  If you're referring
> to 90ab9d551093 ("x86, hyperv: Correctly guard the local APIC calibration
> code ")as "fixing" it, it does not.  (I'm left to wonder what you mean by
> it being fixed since you didn't elaborate.)  You can trigger this build
> breakage even with CONFIG_X86_LOCAL_APIC=y.
> 
> Let me be explicit: this file includes desc.h, which includes linux/smp.h,
> which includes asm/smp.h iff CONFIG_SMP is enabled.  That's what you're
> relying upon to get asm/apic.h when CONFIG_X86_LOCAL_APIC is enabled.  It
> will break when CONFIG_SMP is disabled.
> 
> So you need to include asm/apic.h directly for such a configuration.

Thank you for the details.

Signed-off-by: K. Y. Srinivasan <kys@...rosoft.com>

Regards,

K. Y


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/