RE: [Linux-ia64] Re: volatile unsigned long cpu_online_map vs uns

["Seth, Rohit" <rohit.seth@intel.com>]

cpu_online_map gets used and defined only for SMP configurations.  We will
need to remove the definition of cpu_online_map from setup.c as it is
defined in smpboot.c.  For setup.c, cpu_online_map is done extern in smp.h
(...change to volatile there..)

Below is the patch for setup.c


--- kernel/setup.c      Fri Jun  1 15:15:53 2001
+++ /tmp/setup.c        Tue Jun 26 09:46:40 2001
@@ -60,10 +60,6 @@
 /* This tells _start which CPU is booting.  */
 int cpu_now_booting;

-#ifdef CONFIG_SMP
-volatile unsigned long cpu_online_map;
-#endif
-
 unsigned long ia64_iobase;     /* virtual address for I/O accesses */

 #define COMMAND_LINE_SIZE      512


-----Original Message-----
From: Andreas Schwab [mailto:schwab@suse.de]
Sent: Tuesday, June 26, 2001 9:57 AM
To: David Mosberger
Cc: linux-ia64@linuxia64.org
Subject: [Linux-ia64] Re: volatile unsigned long cpu_online_map vs
unsigned long cpu_online_map


Bill Nottingham <notting@redhat.com> writes:

|> David Mosberger (davidm@hpl.hp.com) said: 
|> >   Andreas> include/asm-ia64/smp.h has 'extern unsigned long
|> >   Andreas> cpu_online_map', but arch/ia64/kernel/setup.c and
|> >   Andreas> arch/ia64/kernel/smpboot.c have 'volatile unsigned long
|> >   Andreas> cpu_online_map'.  Which one is correct?
|> > 
|> > It should be volatile.
|> 
|> Also, shoudn't it be extern in at least one of setup.c/smpboot.c?

No.  The definition in setup.c is guarded by CONFIG_SMP, and smpboot.c is
not use on non-SMP.

Andreas.

-- 
Andreas Schwab                                  "And now for something
SuSE Labs                                        completely different."
Andreas.Schwab@suse.de
SuSE GmbH, Schanzäckerstr. 10, D-90443 Nürnberg
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5

_______________________________________________
Linux-IA64 mailing list
Linux-IA64@linuxia64.org
http://lists.linuxia64.org/lists/listinfo/linux-ia64