mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Re: 2.4.19 compile warning for AMD Athlon config
@ 2002-08-06 17:09 John Kinsella
  0 siblings, 0 replies; 2+ messages in thread
From: John Kinsella @ 2002-08-06 17:09 UTC (permalink / raw)
  To: linux-kernel

On Sunday 04 August 2002 21:56, you wrote:
> amd_adv_spec_cache_feature() tries to use have_cpuid_p(), which does not
> exist in 2.4.x tree.
>

Doh,
	it does of course exist, but it might benefit from a forward declaration to
suppress compiler warning.

--- 2.4.19/arch/i386/kernel/setup.c     Sun Aug  4 14:17:35 2002
+++ 2.4.19/arch/i386/kernel/setup.c.tmp Tue Aug  6 14:39:58 2002
@@ -171,6 +171,8 @@
 extern int root_mountflags;
 extern char _text, _etext, _edata, _end;

+static int have_cpuid_p(void);
+
 static int disable_x86_serial_nr __initdata = 1;
 static int disable_x86_fxsr __initdata = 0;
 static int disable_x86_ht __initdata = 0;

^ permalink raw reply	[flat|nested] 2+ messages in thread
* 2.4.19 compile warning for AMD Athlon config
@ 2002-08-04 20:56 John Kinsella
  0 siblings, 0 replies; 2+ messages in thread
From: John Kinsella @ 2002-08-04 20:56 UTC (permalink / raw)
  To: linux-kernel


amd_adv_spec_cache_feature() tries to use have_cpuid_p(), which does not
exist in 2.4.x tree. 

This seems to have been introduced in 2.4.19-rc1, I guess I should have noticed it before.
A case of 2.4 being too stable perhaps, as it works fine so I didn't check compile log :)

/home/compo/2.4/2.4.19/arch/i386/kernel/setup.c: In function `amd_adv_spec_cache_feature':
/home/compo/2.4/2.4.19/arch/i386/kernel/setup.c:751: warning: implicit declaration of func
tion `have_cpuid_p'
/home/compo/2.4/2.4.19/arch/i386/kernel/setup.c: At top level:
/home/compo/2.4/2.4.19/arch/i386/kernel/setup.c:2629: warning: `have_cpuid_p' was declared
 implicitly `extern' and later `static'
/home/compo/2.4/2.4.19/arch/i386/kernel/setup.c:751: warning: previous declaration of `hav
 e_cpuid_p'

$ grep -nr have_cpuid_p *h
 arch/i386/kernel/setup.c:751:   if(!have_cpuid_p())
 arch/i386/kernel/setup.c:2628:static int __init have_cpuid_p(void)
 arch/i386/kernel/setup.c:2708:  return have_cpuid_p();  /* Check to see if CPUID now enabled? */
 arch/i386/kernel/setup.c:2728:  if ( !have_cpuid_p() && !id_and_try_enable_cpuid(c) ) {
 arch/i386/kernel/setup.c:3164:  if(!have_cpuid_p())

#
# Processor type and features
#
# CONFIG_M386 is not set
# CONFIG_M486 is not set
# CONFIG_M586 is not set
# CONFIG_M586TSC is not set
# CONFIG_M586MMX is not set
# CONFIG_M686 is not set
# CONFIG_MPENTIUMIII is not set
# CONFIG_MPENTIUM4 is not set
# CONFIG_MK6 is not set
CONFIG_MK7=y
# CONFIG_MELAN is not set
# CONFIG_MCRUSOE is not set
# CONFIG_MWINCHIPC6 is not set
# CONFIG_MWINCHIP2 is not set
# CONFIG_MWINCHIP3D is not set
# CONFIG_MCYRIXIII is not set
CONFIG_X86_WP_WORKS_OK=y
CONFIG_X86_INVLPG=y
CONFIG_X86_CMPXCHG=y
CONFIG_X86_XADD=y
CONFIG_X86_BSWAP=y
CONFIG_X86_POPAD_OK=y
# CONFIG_RWSEM_GENERIC_SPINLOCK is not set
CONFIG_RWSEM_XCHGADD_ALGORITHM=y
CONFIG_X86_L1_CACHE_SHIFT=6
CONFIG_X86_TSC=y
CONFIG_X86_GOOD_APIC=y
CONFIG_X86_USE_3DNOW=y
CONFIG_X86_PGE=y
CONFIG_X86_USE_PPRO_CHECKSUM=y
CONFIG_X86_MCE=y
# CONFIG_TOSHIBA is not set
# CONFIG_I8K is not set
# CONFIG_MICROCODE is not set
# CONFIG_X86_MSR is not set
# CONFIG_X86_CPUID is not set
CONFIG_NOHIGHMEM=y
# CONFIG_HIGHMEM4G is not set
# CONFIG_HIGHMEM64G is not set
# CONFIG_MATH_EMULATION is not set
CONFIG_MTRR=y
# CONFIG_SMP is not set
CONFIG_X86_UP_APIC=y
CONFIG_X86_UP_IOAPIC=y
CONFIG_X86_LOCAL_APIC=y
CONFIG_X86_IO_APIC=y


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2002-08-06 17:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-08-06 17:09 2.4.19 compile warning for AMD Athlon config John Kinsella
  -- strict thread matches above, loose matches on Subject: below --
2002-08-04 20:56 John Kinsella

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®