d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Partially revert r224661:

Browse Source 
octeon_ap_boot is not a bitmask. It holds the CPU Id of the
AP currently being started. As such there's no need to make
it a 64-bit integral and we're not limited to 64 CPUs.

Reported by: jmallet
Obtained from:	Andre Duane
This commit is contained in:
Marcel Moolenaar 2011-10-04 17:49:19 +00:00
parent e53211de04
commit 2873abdc12
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=226018
2 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sys/mips/cavium/asm_octeon.S
View File

@ -50,12 +50,12 @@ LEAF(octeon_ap_wait)
jal platform_processor_id
nop
1: lld t0, octeon_ap_boot
1: ll t0, octeon_ap_boot
bne v0, t0, 1b
nop
move t0, zero
scd t0, octeon_ap_boot
sc t0, octeon_ap_boot
beqz t0, 1b
nop

7
sys/mips/cavium/octeon_mp.c
View File

@ -46,8 +46,7 @@ __FBSDID("$FreeBSD$");
/* XXX */
extern cvmx_bootinfo_t *octeon_bootinfo;
/* NOTE: this 64-bit mask (and many others) limits MAXCPU to 64 */
uint64_t octeon_ap_boot = ~0ULL;
unsigned octeon_ap_boot = ~0;
void
platform_ipi_send(int cpuid)
@ -139,11 +138,11 @@ platform_start_ap(int cpuid)
DELAY(2000); /* Give it a moment to start */
}
if (atomic_cmpset_64(&octeon_ap_boot, ~0, cpuid) == 0)
if (atomic_cmpset_32(&octeon_ap_boot, ~0, cpuid) == 0)
return (-1);
for (;;) {
DELAY(1000);
if (atomic_cmpset_64(&octeon_ap_boot, 0, ~0) != 0)
if (atomic_cmpset_32(&octeon_ap_boot, 0, ~0) != 0)
return (0);
printf("Waiting for cpu%d to start\n", cpuid);
}